From: Paul Bakaus <paul.bakaus@googlemail.com>
Date: Thu, 11 Dec 2008 15:02:21 +0000 (+0000)
Subject: sortable: fixed connectToSortable option, wasn't working with Arrays as described... 
X-Git-Tag: 1.6rc3~316
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=a7dae90eff7c96c052e557befc1e56667f9ab331;p=jquery-ui.git

sortable: fixed connectToSortable option, wasn't working with Arrays as described in the documentation
---

diff --git a/ui/ui.draggable.js b/ui/ui.draggable.js
index d1eeef659..f667c3b36 100644
--- a/ui/ui.draggable.js
+++ b/ui/ui.draggable.js
@@ -395,15 +395,19 @@ $.ui.plugin.add("draggable", "connectToSortable", {
 		var inst = $(this).data("draggable");
 		inst.sortables = [];
 		$(ui.options.connectToSortable).each(function() {
-			if($.data(this, 'sortable')) {
-				var sortable = $.data(this, 'sortable');
-				inst.sortables.push({
-					instance: sortable,
-					shouldRevert: sortable.options.revert
-				});
-				sortable._refreshItems();	//Do a one-time refresh at start to refresh the containerCache
-				sortable._propagate("activate", event, inst);
-			}
+			// 'this' points to a string, and should therefore resolved as query, but instead, if the string is assigned to a variable, it loops through the strings properties,
+			// so we have to append '' to make it anonymous again
+			$(this+'').each(function() {
+				if($.data(this, 'sortable')) {
+					var sortable = $.data(this, 'sortable');
+					inst.sortables.push({
+						instance: sortable,
+						shouldRevert: sortable.options.revert
+					});
+					sortable._refreshItems();	//Do a one-time refresh at start to refresh the containerCache
+					sortable._propagate("activate", event, inst);
+				}
+			});
 		});
 
 	},