Model.List must test if Model exist when destroying

Test if target exist before splicing it from a Model.List. Otherwise the latest element will be removed.
canjs · Aug 3, 2012 · 44bc720 · 44bc720
1 parent 517acce
commit 44bc720
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/model/model.js b/model/model.js
@@ -1171,7 +1171,9 @@ steal('can/util','can/observe', function( can ) {
 			var self = this;
 			this.bind('change', function(ev, how){
 				if(/\w+\.destroyed/.test(how)){
-					self.splice(self.indexOf(ev.target),1);
+					if (self.indexOf(ev.target)!=-1) {}
+						self.splice(self.indexOf(ev.target),1);
+					}
 				}
 			})
 		}