Merge pull request #2217 from decden/2216-observable-array-leaks-last…

…-value Fixed memory leak of previousContents when tracking array changes and disposing the subscription
knockout · Apr 8, 2017 · 0ee72ac · 0ee72ac
2 parents 5f0b9a7 + 797e002
commit 0ee72ac
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/subscribables/observableArray.changeTracking.js b/src/subscribables/observableArray.changeTracking.js
@@ -36,7 +36,10 @@ ko.extenders['trackArrayChanges'] = function(target, options) {
                 target['notifySubscribers'] = underlyingNotifySubscribersFunction;
                 underlyingNotifySubscribersFunction = undefined;
             }
-            arrayChangeSubscription.dispose();
+            if (arrayChangeSubscription) {
+                arrayChangeSubscription.dispose();
+            }
+            arrayChangeSubscription = null;
             trackingChanges = false;
         }
     };