Changeset 574

Show
Ignore:
Timestamp:
01/26/06 12:30:17 (3 years ago)
Author:
therve@gmail.com
Message:

Fix some lazy ifilters, correct a destroy.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • mochikit/branches/scriptaculous/MochiKit/DragAndDrop.js

    r563 r574  
    277277 
    278278    unregister: function (draggable) { 
    279         this.drags = MochiKit.Iter.ifilter(function (d) { 
     279        this.drags = MochiKit.Base.filter(function (d) { 
    280280            return d != draggable 
    281281        }, this.drags); 
    …… 
    338338    removeObserver: function (element) { 
    339339        // element instead of observer fixes mem leaks 
    340         this.observers = MochiKit.Iter.ifilter(function (o) { 
     340        this.observers = MochiKit.Base.filter(function (o) { 
    341341            return o.element != element; 
    342342        }, this.observers); 

  • mochikit/branches/scriptaculous/MochiKit/Sortable.js

    r558 r574  
    5656    destroy: function (element){ 
    5757        element = MochiKit.DOM.getElement(element); 
    58         MochiKit.Iter.forEach(MochiKit.Iter.ifilter(function (s) { 
    59                 return s.element == element; 
    60             }, this.sortables), function (s) { 
    61                 MochiKit.DragAndDrop.Draggables.removeObserver(s.element); 
    62                 MochiKit.Iter.forEach(s.droppables, function (d) { 
    63                     MochiKit.DragAndDrop.Droppables.remove(d); 
    64                 }); 
    65             s.draggables.invoke('destroy'); 
     58        var toDestroy = MochiKit.Base.filter(function (s) { 
     59            return s.element == element; 
     60        }, this.sortables); 
     61        MochiKit.Iter.forEach(toDestroy, function (s) { 
     62            MochiKit.DragAndDrop.Draggables.removeObserver(s.element); 
     63            MochiKit.Iter.forEach(s.droppables, function (d) { 
     64                MochiKit.DragAndDrop.Droppables.remove(d); 
     65            }); 
     66            MochiKit.Iter.forEach(s.draggables, function (d) { 
     67                d.destroy(); 
     68            }); 
    6669        }); 
    6770        this.sortables = MochiKit.Base.filter(function (s) { 
    …… 
    271274        element = MochiKit.DOM.getElement(element); 
    272275        var sortableOptions = this.options(element); 
    273         options = MochiKit.Base.update( 
    274         { 
     276        options = MochiKit.Base.update({ 
    275277            tag: sortableOptions.tag, 
    276278            only: sortableOptions.only,