Store data on the node itself (except in old IE), rather than using W…

…eakMap.
knockout · Jul 22, 2017 · 6c02fe1 · 6c02fe1
1 parent 199fdb2
commit 6c02fe1
Showing 1 changed file with 10 additions and 12 deletions.
diff --git a/src/utils.domData.js b/src/utils.domData.js
@@ -5,22 +5,20 @@ ko.utils.domData = new (function () {
     var dataStore = {};
 
     var getDataForNode, clear;
-    if (window['WeakMap']) {
+    if (!ko.utils.ieVersion) {
         getDataForNode = function (node, createIfNotFound) {
-            var ownerDoc = node.ownerDocument,
-                dataStore = ownerDoc[dataStoreKeyExpandoPropertyName] || (ownerDoc[dataStoreKeyExpandoPropertyName] = new ownerDoc.defaultView['WeakMap']());
-            if (dataStore['has'](node)) {
-                return dataStore.get(node);
-            }
-            if (createIfNotFound) {
-                var dataForNode = {};
-                dataStore.set(node, dataForNode);
-                return dataForNode;
+            var dataForNode = node[dataStoreKeyExpandoPropertyName];
+            if (!dataForNode && createIfNotFound) {
+                dataForNode = node[dataStoreKeyExpandoPropertyName] = {};
             }
+            return dataForNode;
         };
         clear = function (node) {
-            var dataStore = node.ownerDocument[dataStoreKeyExpandoPropertyName];
-            return dataStore ? dataStore['delete'](node) : false;
+            if (node[dataStoreKeyExpandoPropertyName]) {
+                delete node[dataStoreKeyExpandoPropertyName];
+                return true;
+            }
+            return false;
         };
     } else {
         getDataForNode = function (node, createIfNotFound) {