Draggable: Calling refreshPositions instead of _refreshItems when dragging into a connected sortable. Fixed: #5563 - Draggable li with connectToSortable option cannot be dropped on the connected sortable ol

nathansobo · nathansobo · commit 3eadf2bcab9a · 2010-07-08T19:51:15.000-03:00
diff --git a/ui/jquery.ui.draggable.js b/ui/jquery.ui.draggable.js
@@ -475,7 +475,7 @@ $.ui.plugin.add("draggable", "connectToSortable", {
 					instance: sortable,
 					shouldRevert: sortable.options.revert
 				});
-				sortable._refreshItems();	//Do a one-time refresh at start to refresh the containerCache
+				sortable.refreshPositions();	//Do a one-time refreshPositions at start to initialize the containerCache
 				sortable._trigger("activate", event, uiSortable);
 			}
 		});

Original file line number	Diff line number	Diff line change
`@@ -475,7 +475,7 @@ $.ui.plugin.add("draggable", "connectToSortable", {`
`475`	`475`	`instance: sortable,`
`476`	`476`	`shouldRevert: sortable.options.revert`
`477`	`477`	`});`
`478`		`- sortable._refreshItems(); //Do a one-time refresh at start to refresh the containerCache`
	`478`	`+ sortable.refreshPositions(); //Do a one-time refreshPositions at start to initialize the containerCache`
`479`	`479`	`sortable._trigger("activate", event, uiSortable);`
`480`	`480`	`}`
`481`	`481`	`});`