Merge pull request #7 from jderusse/fix-collection-purge

sroze · sroze · commit 9d72e5900904 · 2015-05-05T16:24:25.000+01:00
Revert remove order to avoid to skip indexes
diff --git a/src/model/collection.js b/src/model/collection.js
@@ -37,7 +37,7 @@ function $CollectionFactory (BaseModel) {
         },
         set: function (models, options) {
             var i;
-            for (i = 0; i < this.models.length; i++) {
+            for (i=this.models.length-1; i>=0; i--) {
                 this.removeIndex(i);
             }
 

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ function $CollectionFactory (BaseModel) {`
`37`	`37`	`},`
`38`	`38`	`set: function (models, options) {`
`39`	`39`	`var i;`
`40`		`- for (i = 0; i < this.models.length; i++) {`
	`40`	`+ for (i=this.models.length-1; i>=0; i--) {`
`41`	`41`	`this.removeIndex(i);`
`42`	`42`	`}`
`43`	`43`