Sortable: Fix a bug of removing an item while iterating an array. Fix…

…es #8571 - Out of range problem in when dragging a nested sortable.
jquery · Oct 10, 2012 · 77a4aaf · 77a4aaf
1 parent a3f1a34
commit 77a4aaf
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/ui/jquery.ui.sortable.js b/ui/jquery.ui.sortable.js
@@ -562,14 +562,13 @@ $.widget("ui.sortable", $.ui.mouse, {
 
 		var list = this.currentItem.find(":data(" + this.widgetName + "-item)");
 
-		for (var i=0; i < this.items.length; i++) {
-
+		this.items = $.grep(this.items, function (item) {
 			for (var j=0; j < list.length; j++) {
-				if(list[j] == this.items[i].item[0])
-					this.items.splice(i,1);
+				if(list[j] == item.item[0])
+					return false;
 			};
-
-		};
+			return true;
+		});
 
 	},