Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Useless setTimeout call (missing quotes around argument?)

I have this sniplet of code in jQuery

$element.parent().children().last().hide().show('slide', {direction : 'left'}, 700, function () {
    $element.delay(2000, function() {
        $element.parent().children().last().hide('slide', {direction: 'left'}, 700);             
        reload_table(question_number);
        //delay ends here
    });
});

delay is declared as:

jQuery.fn.delay = function(time,func){
    return this.each(function(){
        setTimeout(func,time);
    });
};

Now I get the error:

useless setTimeout call (missing quotes around argument?)

FF3.x, Firefox 6+ is ok. Any ideas on that? Why could this be happening?

like image 436
kosta5 Avatar asked Nov 21 '11 11:11

kosta5


3 Answers

I got same error when I wrote

setTimeout(updateStatus(), 5000);

instead of

setTimeout(updateStatus, 5000);
like image 141
wsbaser Avatar answered Oct 12 '22 14:10

wsbaser


I agree with wsbaser. I had the additional instance of needed to pass information to the function, and for simplicity used:

setTimeout(function(){ updateStatus(myData) } , 5000);

The argument needs to be a function and not a function being called. Firefox caught this error, chrome let it go.

like image 40
mpickell Avatar answered Oct 12 '22 13:10

mpickell


Just for reference if someone stumbles upon this question and is looking for a possible answer. I got the exact same error message as the initial poster because I was mixing up the setTimeout arguments order.

This:

setTimeout(25, function(){
    spotlight.intro.find('#intro').ellipsis();  
});

... gave me the error message. I changed the function to this:

setTimeout(function(){
    spotlight.intro.find('#intro').ellipsis();
}, 25);

And my problem was solved.

like image 43
polyclick Avatar answered Oct 12 '22 13:10

polyclick