Merge pull request knockout#1613 from klinden/ie8-apply

IE8 throws errors when calling apply with second argument undefined
lucksuper · Apr 6, 2015 · 4208880 · 4208880
2 parents 7752dde + 21ed0e7
commit 4208880
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/binding/editDetection/arrayToDomNodeChildren.js b/src/binding/editDetection/arrayToDomNodeChildren.js
@@ -88,7 +88,7 @@
                             mapData.dependentObservable.dispose();
 
                         // Queue these nodes for later removal
-                        nodesToDelete.push.apply(nodesToDelete, ko.utils.fixUpContinuousNodeArray(mapData.mappedNodes, domNode));
+                        nodesToDelete.push.apply(nodesToDelete, ko.utils.fixUpContinuousNodeArray(mapData.mappedNodes, domNode) || []);
                         if (options['beforeRemove']) {
                             itemsForBeforeRemoveCallbacks[i] = mapData;
                             itemsToProcess.push(mapData);