Why a JavaScript function with some specific name is not working while others are?

Question

A very interesting problem I am facing these days is regarding one of my JavaScript function. My JavaScript function with some specific name is not working but if I change its name to anything else then it is working. Have a look -

// function to retain the jquery ui css for toolbar
function retain_css() {
    alert('hi');
    $( "#new_sort_options" ).buttonset();
}

// new sort
$(document).on("click", ".new_sort_button", function() { 
    var order       = $(this).val();
    var make_id     = $('#new_make_id').val();
    $.ajax({
        beforeSend  : start_loader(),
        type        : 'POST',
        url         : '/ajax/new-sort.php',
        data        : 'order='+order+'&make_id='+make_id,
        dataType    : 'json',
        success     : function(data) {
                        $("#new_results_toolbar").html(data.toolbar);
                        $("#new_results").html(data.models);
                        retain_css();
                        end_loader();
        }
    }); 
});

But retain_css() is not working at all. Even alert() is not firing. But if i change its name to anything such as my_fun() then the code works. I don't understand why it is happening so? Any idea? Don't worry about end_loader() function as it has nothing to deal with my problem. I also changed the order of code when retain_css() was being used but didn't work.

Wow, how is that possible? Any chance you can replicate this in a live example (a snippet here or jsfiddle, etc.)? — Shomz
– Shomz, Commented Oct 17, 2014 at 11:47
"Don't worry about end_loader() function as it has nothing to deal with my problem" Then remove it from the code that you show us. — Lightness Races in Orbit
– Lightness Races in Orbit, Commented Oct 17, 2014 at 11:48
Any errors in console? Can you abstract away the AJAX? Make a testcase and show us the results of your debugging. — Lightness Races in Orbit
– Lightness Races in Orbit, Commented Oct 17, 2014 at 11:49
The only way this is possible is if retain_css is declared again further down or somewhere else and is overwriting the function. — Rhumborl
– Rhumborl, Commented Oct 17, 2014 at 11:51
retain_css works perfect for me, so your error may be above the code that you´re showing us. Please paste part of your code that is above the retain_css(), or just a fiddle. — Guillermo
– Guillermo, Commented Oct 17, 2014 at 11:51

Radek Pech · Accepted Answer · 2014-10-17 11:52:02Z

1

Try not to create global functions because it may collide with other frameworks or libraries.

//define private namespace
window.user3779493Functions = {};
//define method
user3779493Functions.retain_css = function() { ... }
//call method
user3779493Functions.retain_css();

answered Oct 17, 2014 at 11:52

Radek Pech

3,1081 gold badge27 silver badges31 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

user4152928 · Accepted Answer · 2014-10-17 11:59:08Z

0

Some functions are already programmed like 'alert('hi');', that is a function called alert:

function alert() {
    /* do something */
}

That function also doesn't work.

answered Oct 17, 2014 at 11:59

user4152928

Collectives™ on Stack Overflow

Why a JavaScript function with some specific name is not working while others are?

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related