From: Paul Bakaus <paul.bakaus@googlemail.com>
Date: Sat, 22 Aug 2009 11:48:48 +0000 (+0000)
Subject: draggable: scope option failed when no droppables within that scope exist, fixes... 
X-Git-Tag: 1.8a2~132
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=fefa5bd34208de9b80c6275515a00ba551d76040;p=jquery-ui.git

draggable: scope option failed when no droppables within that scope exist, fixes #4289
---

diff --git a/ui/ui.droppable.js b/ui/ui.droppable.js
index 8d347876a..7966ae3b0 100644
--- a/ui/ui.droppable.js
+++ b/ui/ui.droppable.js
@@ -197,7 +197,7 @@ $.ui.ddmanager = {
 	droppables: { 'default': [] },
 	prepareOffsets: function(t, event) {
 
-		var m = $.ui.ddmanager.droppables[t.options.scope];
+		var m = $.ui.ddmanager.droppables[t.options.scope] || [];
 		var type = event ? event.type : null; // workaround for #2317
 		var list = (t.currentItem || t.element).find(":data(droppable)").andSelf();
 
@@ -218,7 +218,7 @@ $.ui.ddmanager = {
 	drop: function(draggable, event) {
 
 		var dropped = false;
-		$.each($.ui.ddmanager.droppables[draggable.options.scope], function() {
+		$.each($.ui.ddmanager.droppables[draggable.options.scope] || [], function() {
 
 			if(!this.options) return;
 			if (!this.options.disabled && this.visible && $.ui.intersect(draggable, this, this.options.tolerance))
@@ -240,7 +240,7 @@ $.ui.ddmanager = {
 
 		//Run through all droppables and check their positions based on specific tolerance options
 
-		$.each($.ui.ddmanager.droppables[draggable.options.scope], function() {
+		$.each($.ui.ddmanager.droppables[draggable.options.scope] || [], function() {
 
 			if(this.options.disabled || this.greedyChild || !this.visible) return;
 			var intersects = $.ui.intersect(draggable, this, this.options.tolerance);