j-templates 8.0.2 → 8.0.4

package/Store/Tree/observableNode.js

@@ -333,7 +333,7 @@ var ObservableNode;
     function Update(value, prop = OBJECT_SCOPE) {
         const wrapper = wrapperCache.get(value);
         if (wrapper) {
-            const scope = wrapper[prop] ?? wrapper[OBJECT_SCOPE];
+            const scope = Array.isArray(wrapper) ? wrapper[OBJECT_SCOPE] : wrapper[prop] ?? wrapper[OBJECT_SCOPE];
             observableScope_1.ObservableScope.Update(scope);
         }
     }
@@ -399,6 +399,10 @@ var ObservableNode;
             }
             return;
         }
+        // Assign every change before emitting any, so the diff lands atomically. Emitting per
+        // assignment let synchronous readers see partial states, e.g. an array's new length
+        // before its new elements were written.
+        // const updates = new Map<any, Set<string | number>>();
         const pathTuples = [["", root]];
         for (let x = 0; x < diffResult.length; x++) {
             const { path, value } = diffResult[x];
@@ -420,7 +424,12 @@ var ObservableNode;
             const assignValue = pathTuples[y][1];
             assignValue[path[y]] = value;
             ObservableNode.Update(assignValue, path[y]);
+            /* let props = updates.get(assignValue);
+            if (props === undefined) updates.set(assignValue, (props = new Set()));
+            props.add(path[y]); */
         }
+        /* for (const [assignValue, props] of updates)
+          for (const prop of props) ObservableNode.Update(assignValue, prop as any); */
     }
     ObservableNode.ApplyDiff = ApplyDiff;
     /**

package/Utils/json.js

@@ -171,11 +171,6 @@ function JsonDiffFactory() {
             return oldValue.length !== newValue.length;
         }
         let allChildrenChanged = true;
-        if (newValue.length !== oldValue.length)
-            resp.push({
-                path: path.concat("length"),
-                value: newValue.length,
-            });
         if (newValue.length > 0 || oldValue.length > 0) {
             for (let y = 0; y < newValue.length; y++) {
                 const arrayPath = path.concat(y);
@@ -187,6 +182,11 @@ function JsonDiffFactory() {
         }
         else
             allChildrenChanged = false;
+        if (newValue.length !== oldValue.length)
+            resp.push({
+                path: path.concat("length"),
+                value: newValue.length,
+            });
         return allChildrenChanged;
     }
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "j-templates",
-  "version": "8.0.2",
+  "version": "8.0.4",
   "description": "j-templates",
   "license": "MIT",
   "repository": "https://github.com/TypesInCode/jTemplates",