Sortable: Don't create functions inside loops.

diff --git a/ui/jquery.ui.sortable.js b/ui/jquery.ui.sortable.js
index 7b145159ebc31e0e8dcfad6a559ff27153f3479f..c7794f06a40f4511b95cff9587088bc9ed26baa5 100644 (file)
--- a/ui/jquery.ui.sortable.js
+++ b/ui/jquery.ui.sortable.js
@@ -15,8 +15,6 @@
  */
 (function( $, undefined ) {
 
-/*jshint loopfunc: true */
-
 function isOverAxis( x, reference, size ) {
        return ( x > reference ) && ( x < ( reference + size ) );
 }
@@ -629,10 +627,11 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                queries.push([$.isFunction(this.options.items) ? this.options.items.call(this.element, null, { options: this.options, item: this.currentItem }) : $(this.options.items, this.element).not(".ui-sortable-helper").not(".ui-sortable-placeholder"), this]);
 
+               function addItems() {
+                       items.push( this );
+               }
                for (i = queries.length - 1; i >= 0; i--){
-                       queries[i][0].each(function() {
-                               items.push(this);
-                       });
+                       queries[i][0].each( addItems );
                }
 
                return $(items);
@@ -1190,12 +1189,17 @@ $.widget("ui.sortable", $.ui.mouse, {
 
 
                //Post events to containers
+               function delayEvent( type, instance, container ) {
+                       return function( event ) {
+                               container._trigger( type, event, instance._uiHash( instance ) );
+                       };
+               }
                for (i = this.containers.length - 1; i >= 0; i--){
-                       if(!noPropagation) {
-                               delayedTriggers.push((function(c) { return function(event) { c._trigger("deactivate", event, this._uiHash(this)); };  }).call(this, this.containers[i]));
+                       if (!noPropagation) {
+                               delayedTriggers.push( delayEvent( "deactivate", this, this.containers[ i ] ) );
                        }
                        if(this.containers[i].containerCache.over) {
-                               delayedTriggers.push((function(c) { return function(event) { c._trigger("out", event, this._uiHash(this)); };  }).call(this, this.containers[i]));
+                               delayedTriggers.push( delayEvent( "out", this, this.containers[ i ] ) );
                                this.containers[i].containerCache.over = 0;
                        }
                }