jquery fx & event: replaced the use of .apply(), where .call() could had been used.
authorAriel Flesler <aflesler@gmail.com>
Mon, 12 May 2008 19:45:02 +0000 (19:45 +0000)
committerAriel Flesler <aflesler@gmail.com>
Mon, 12 May 2008 19:45:02 +0000 (19:45 +0000)
src/event.js
src/fx.js

index a09bcb8..a255f55 100644 (file)
@@ -495,7 +495,7 @@ jQuery.extend({
                        if ( jQuery.readyList ) {
                                // Execute all of them
                                jQuery.each( jQuery.readyList, function(){
-                                       this.apply( document );
+                                       this.call( document );
                                });
                                
                                // Reset the list of functions
index 5f3a492..1a4e8c9 100644 (file)
--- a/src/fx.js
+++ b/src/fx.js
@@ -81,7 +81,7 @@ jQuery.fn.extend({
                        
                        for ( p in prop ) {
                                if ( prop[p] == "hide" && hidden || prop[p] == "show" && !hidden )
-                                       return jQuery.isFunction(opt.complete) && opt.complete.apply(this);
+                                       return jQuery.isFunction(opt.complete) && opt.complete.call(this);
 
                                if ( p == "height" || p == "width" ) {
                                        // Store display property
@@ -148,7 +148,7 @@ jQuery.fn.extend({
                                queue(this, type).push( fn );
                        
                                if ( queue(this, type).length == 1 )
-                                       fn.apply(this);
+                                       fn.call(this);
                        }
                });
        },
@@ -202,7 +202,7 @@ jQuery.fn.dequeue = function(type){
                q.shift();
 
                if ( q.length )
-                       q[0].apply( this );
+                       q[0].call( this );
        });
 };
 
@@ -226,7 +226,7 @@ jQuery.extend({
                        if ( opt.queue !== false )
                                jQuery(this).dequeue();
                        if ( jQuery.isFunction( opt.old ) )
-                               opt.old.apply( this );
+                               opt.old.call( this );
                };
        
                return opt;
@@ -260,7 +260,7 @@ jQuery.fx.prototype = {
        // Simple function for setting a style value
        update: function(){
                if ( this.options.step )
-                       this.options.step.apply( this.elem, [ this.now, this ] );
+                       this.options.step.call( this.elem, this.now, this );
 
                (jQuery.fx.step[this.prop] || jQuery.fx.step._default)( this );
 
@@ -381,7 +381,7 @@ jQuery.fx.prototype = {
                        // If a callback was provided, execute it
                        if ( done && jQuery.isFunction( this.options.complete ) )
                                // Execute the complete function
-                               this.options.complete.apply( this.elem );
+                               this.options.complete.call( this.elem );
 
                        return false;
                } else {