ensure view is not null when attempting to call __manager__ on it

tbranyen · Oct 30, 2012 · 48cde02 · 48cde02
1 parent 15dc3cc
commit 48cde02
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/backbone.layoutmanager.js b/backbone.layoutmanager.js
@@ -482,10 +482,10 @@ var LayoutManager = Backbone.View.extend({
       // If this is an array of items remove items that are not marked to
       // keep.
       if (_.isArray(parentViews)) {
-        // Remove directly from the Array reference.
-        return _.each(parentViews, function(view, i) {
+        // Remove duplicate Views.
+        return _.each(_.clone(parentViews), function(view, i) {
           // If the managers match, splice off this View.
-          if (view.__manager__ === manager) {
+          if (view && view.__manager__ === manager) {
             aSplice.call(parentViews, i, 1);
           }
         });

diff --git a/dist/backbone.layoutmanager.min.js b/dist/backbone.layoutmanager.min.js