From fefa5bd34208de9b80c6275515a00ba551d76040 Mon Sep 17 00:00:00 2001 From: Paul Bakaus Date: Sat, 22 Aug 2009 11:48:48 +0000 Subject: [PATCH] draggable: scope option failed when no droppables within that scope exist, fixes #4289 --- ui/ui.droppable.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) 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); -- 2.39.5