]> source.dussan.org Git - jquery-ui.git/commitdiff
draggable: scope option failed when no droppables within that scope exist, fixes...
authorPaul Bakaus <paul.bakaus@googlemail.com>
Sat, 22 Aug 2009 11:48:48 +0000 (11:48 +0000)
committerPaul Bakaus <paul.bakaus@googlemail.com>
Sat, 22 Aug 2009 11:48:48 +0000 (11:48 +0000)
ui/ui.droppable.js

index 8d347876a147176ebca800f75774506b04412f60..7966ae3b0154948df281e0c3c6b77197655d724e 100644 (file)
@@ -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);