Added condition to remove invalud use of view.removeAt().

We were passing undefined in some cases to it.
astoilkov · Dec 2, 2016 · 2082eef · 2082eef
1 parent 8c9152e
commit 2082eef
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/query/ExtenderHelper.js b/src/query/ExtenderHelper.js
@@ -225,7 +225,11 @@ define([
               action = Action.EXISTS;
               skip -= 1;
               if (skip >= 0) {
-                action = Action.REMOVE;
+                if (getConnection(index) !== undefined) {
+                  action = Action.REMOVE;
+                } else {
+                  action = Action.NOOP;
+                }
                 return false;
               } else if (skip < 0 && getConnection(index) === undefined) {
                 action = Action.ADD;
@@ -234,7 +238,11 @@ define([
 
             case ExtenderHelper.operations.TAKE:
               if (take <= 0) {
-                action = Action.REMOVE;
+                if (getConnection(index) === undefined) {
+                  action = Action.NOOP;
+                } else {
+                  action = Action.REMOVE;
+                }
                 return false;
               } else {
                 take -= 1;