@odoo/o-spreadsheet 15.0.12 → 15.0.14

package/dist/o-spreadsheet.cjs.js

@@ -2701,13 +2701,6 @@ for (let font of fontSizes) {
 }
 
 //------------------------------------------------------------------------------
-/**
- * Stringify an object, like JSON.stringify, except that the first level of keys
- * is ordered.
- */
-function stringify(obj) {
-    return JSON.stringify(obj, Object.keys(obj).sort());
-}
 /**
  * Escapes a string to use as a literal string in a RegExp.
  * @url https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions#Escaping
@@ -9426,7 +9419,7 @@ class BordersPlugin extends CorePlugin {
          */
         function getBorderId(border) {
             for (let [key, value] of Object.entries(borders)) {
-                if (stringify(value) === stringify(border)) {
+                if (deepEquals(value, border)) {
                     return parseInt(key, 10);
                 }
             }
@@ -9622,7 +9615,7 @@ class CellPlugin extends CorePlugin {
          */
         function getStyleId(style) {
             for (let [key, value] of Object.entries(styles)) {
-                if (stringify(value) === stringify(style)) {
+                if (deepEquals(value, style)) {
                     return parseInt(key, 10);
                 }
             }
@@ -9676,12 +9669,10 @@ class CellPlugin extends CorePlugin {
      * starting an async evaluation even if it has been moved or re-allocated
      */
     getCellById(cellId) {
-        for (const sheet of Object.values(this.cells)) {
-            if (sheet[cellId]) {
-                return sheet[cellId];
-            }
-        }
-        return undefined;
+        // this must be as fast as possible
+        const position = this.getters.getCellPosition(cellId);
+        const sheet = this.cells[position.sheetId];
+        return sheet[cellId];
     }
     /**
      * Reconstructs the original formula string based on a normalized form and its dependencies
@@ -15850,7 +15841,7 @@ const CSS$i = css$k /* scss */ `
       }
       .o-input-count {
         width: fit-content;
-        padding: 4 0 4 4;
+        padding: 4px 0 4px 4px;
       }
     }
   }
@@ -23502,9 +23493,9 @@ function revertChanges(revisions) {
  * Apply the changes of the given HistoryChange to the state
  */
 function applyChange(change, target) {
-    let val = change.root;
-    let key = change.path[change.path.length - 1];
-    for (let pathIndex = 0; pathIndex < change.path.slice(0, -1).length; pathIndex++) {
+    let val = change.path[0];
+    const key = change.path[change.path.length - 1];
+    for (let pathIndex = 1; pathIndex < change.path.slice(0, -1).length; pathIndex++) {
         const p = change.path[pathIndex];
         if (val[p] === undefined) {
             const nextPath = change.path[pathIndex + 1];
@@ -23960,13 +23951,14 @@ class StateObserver {
     }
     addChange(...args) {
         const val = args.pop();
-        const [root, ...path] = args;
+        const root = args[0];
         let value = root;
-        let key = path[path.length - 1];
-        for (let pathIndex = 0; pathIndex <= path.length - 2; pathIndex++) {
-            const p = path[pathIndex];
+        let key = args[args.length - 1];
+        const pathLength = args.length - 2;
+        for (let pathIndex = 1; pathIndex <= pathLength; pathIndex++) {
+            const p = args[pathIndex];
             if (value[p] === undefined) {
-                const nextPath = path[pathIndex + 1];
+                const nextPath = args[pathIndex + 1];
                 value[p] = createEmptyStructure(nextPath);
             }
             value = value[p];
@@ -23975,8 +23967,7 @@ class StateObserver {
             return;
         }
         this.changes.push({
-            root,
-            path,
+            path: args,
             before: value[key],
             after: val,
         });
@@ -26034,7 +26025,10 @@ class Model extends owl__namespace.core.EventBus {
      */
     checkDispatchAllowed(command) {
         const results = this.handlers.map((handler) => handler.allowDispatch(command));
-        return new DispatchResult(results.flat());
+        if (results.some((r) => r !== 0 /* Success */)) {
+            return new DispatchResult(results.flat());
+        }
+        return DispatchResult.Success;
     }
     finalize() {
         this.status = 3 /* Finalizing */;
@@ -31456,6 +31450,6 @@ exports.registries = registries;
 exports.setTranslationMethod = setTranslationMethod;
 
 
-__info__.version = '15.0.12';
-__info__.date = '2023-11-02T09:28:12.777Z';
-__info__.hash = 'e82bd99';
+__info__.version = '15.0.14';
+__info__.date = '2023-11-24T13:06:57.405Z';
+__info__.hash = 'c9a805f';

package/dist/o-spreadsheet.d.ts

@@ -922,7 +922,7 @@ declare class StateObserver {
         commands: CoreCommand[];
     };
     addCommand(command: CoreCommand): void;
-    addChange(...args: any[]): void;
+    addChange(...args: [...HistoryChange["path"], any]): void;
 }
 
 /**
@@ -3890,8 +3890,7 @@ declare type EvalContext = {
 };
 
 interface HistoryChange {
-    root: any;
-    path: (string | number)[];
+    path: [any, ...(number | string)[]];
     before: any;
     after: any;
 }

package/dist/o-spreadsheet.esm.js

@@ -2678,13 +2678,6 @@ for (let font of fontSizes) {
 }
 
 //------------------------------------------------------------------------------
-/**
- * Stringify an object, like JSON.stringify, except that the first level of keys
- * is ordered.
- */
-function stringify(obj) {
-    return JSON.stringify(obj, Object.keys(obj).sort());
-}
 /**
  * Escapes a string to use as a literal string in a RegExp.
  * @url https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions#Escaping
@@ -9403,7 +9396,7 @@ class BordersPlugin extends CorePlugin {
          */
         function getBorderId(border) {
             for (let [key, value] of Object.entries(borders)) {
-                if (stringify(value) === stringify(border)) {
+                if (deepEquals(value, border)) {
                     return parseInt(key, 10);
                 }
             }
@@ -9599,7 +9592,7 @@ class CellPlugin extends CorePlugin {
          */
         function getStyleId(style) {
             for (let [key, value] of Object.entries(styles)) {
-                if (stringify(value) === stringify(style)) {
+                if (deepEquals(value, style)) {
                     return parseInt(key, 10);
                 }
             }
@@ -9653,12 +9646,10 @@ class CellPlugin extends CorePlugin {
      * starting an async evaluation even if it has been moved or re-allocated
      */
     getCellById(cellId) {
-        for (const sheet of Object.values(this.cells)) {
-            if (sheet[cellId]) {
-                return sheet[cellId];
-            }
-        }
-        return undefined;
+        // this must be as fast as possible
+        const position = this.getters.getCellPosition(cellId);
+        const sheet = this.cells[position.sheetId];
+        return sheet[cellId];
     }
     /**
      * Reconstructs the original formula string based on a normalized form and its dependencies
@@ -15827,7 +15818,7 @@ const CSS$i = css$k /* scss */ `
       }
       .o-input-count {
         width: fit-content;
-        padding: 4 0 4 4;
+        padding: 4px 0 4px 4px;
       }
     }
   }
@@ -23479,9 +23470,9 @@ function revertChanges(revisions) {
  * Apply the changes of the given HistoryChange to the state
  */
 function applyChange(change, target) {
-    let val = change.root;
-    let key = change.path[change.path.length - 1];
-    for (let pathIndex = 0; pathIndex < change.path.slice(0, -1).length; pathIndex++) {
+    let val = change.path[0];
+    const key = change.path[change.path.length - 1];
+    for (let pathIndex = 1; pathIndex < change.path.slice(0, -1).length; pathIndex++) {
         const p = change.path[pathIndex];
         if (val[p] === undefined) {
             const nextPath = change.path[pathIndex + 1];
@@ -23937,13 +23928,14 @@ class StateObserver {
     }
     addChange(...args) {
         const val = args.pop();
-        const [root, ...path] = args;
+        const root = args[0];
         let value = root;
-        let key = path[path.length - 1];
-        for (let pathIndex = 0; pathIndex <= path.length - 2; pathIndex++) {
-            const p = path[pathIndex];
+        let key = args[args.length - 1];
+        const pathLength = args.length - 2;
+        for (let pathIndex = 1; pathIndex <= pathLength; pathIndex++) {
+            const p = args[pathIndex];
             if (value[p] === undefined) {
-                const nextPath = path[pathIndex + 1];
+                const nextPath = args[pathIndex + 1];
                 value[p] = createEmptyStructure(nextPath);
             }
             value = value[p];
@@ -23952,8 +23944,7 @@ class StateObserver {
             return;
         }
         this.changes.push({
-            root,
-            path,
+            path: args,
             before: value[key],
             after: val,
         });
@@ -26011,7 +26002,10 @@ class Model extends owl.core.EventBus {
      */
     checkDispatchAllowed(command) {
         const results = this.handlers.map((handler) => handler.allowDispatch(command));
-        return new DispatchResult(results.flat());
+        if (results.some((r) => r !== 0 /* Success */)) {
+            return new DispatchResult(results.flat());
+        }
+        return DispatchResult.Success;
     }
     finalize() {
         this.status = 3 /* Finalizing */;
@@ -31411,6 +31405,6 @@ const helpers = {
 export { CommandResult, CorePlugin, DATETIME_FORMAT, DataSource, DispatchResult, Model, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, DEBUG as __DEBUG__, __info__, astToFormula, cellTypes, convertAstNodes, coreTypes, functionCache, helpers, invalidateEvaluationCommands, normalize, parse, readonlyAllowedCommands, registries, setTranslationMethod };
 
 
-__info__.version = '15.0.12';
-__info__.date = '2023-11-02T09:28:12.777Z';
-__info__.hash = 'e82bd99';
+__info__.version = '15.0.14';
+__info__.date = '2023-11-24T13:06:57.405Z';
+__info__.hash = 'c9a805f';

package/dist/o-spreadsheet.iife.js

@@ -2698,13 +2698,6 @@
   }
 
   //------------------------------------------------------------------------------
-  /**
-   * Stringify an object, like JSON.stringify, except that the first level of keys
-   * is ordered.
-   */
-  function stringify(obj) {
-      return JSON.stringify(obj, Object.keys(obj).sort());
-  }
   /**
    * Escapes a string to use as a literal string in a RegExp.
    * @url https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions#Escaping
@@ -9423,7 +9416,7 @@
            */
           function getBorderId(border) {
               for (let [key, value] of Object.entries(borders)) {
-                  if (stringify(value) === stringify(border)) {
+                  if (deepEquals(value, border)) {
                       return parseInt(key, 10);
                   }
               }
@@ -9619,7 +9612,7 @@
            */
           function getStyleId(style) {
               for (let [key, value] of Object.entries(styles)) {
-                  if (stringify(value) === stringify(style)) {
+                  if (deepEquals(value, style)) {
                       return parseInt(key, 10);
                   }
               }
@@ -9673,12 +9666,10 @@
        * starting an async evaluation even if it has been moved or re-allocated
        */
       getCellById(cellId) {
-          for (const sheet of Object.values(this.cells)) {
-              if (sheet[cellId]) {
-                  return sheet[cellId];
-              }
-          }
-          return undefined;
+          // this must be as fast as possible
+          const position = this.getters.getCellPosition(cellId);
+          const sheet = this.cells[position.sheetId];
+          return sheet[cellId];
       }
       /**
        * Reconstructs the original formula string based on a normalized form and its dependencies
@@ -15847,7 +15838,7 @@
       }
       .o-input-count {
         width: fit-content;
-        padding: 4 0 4 4;
+        padding: 4px 0 4px 4px;
       }
     }
   }
@@ -23499,9 +23490,9 @@
    * Apply the changes of the given HistoryChange to the state
    */
   function applyChange(change, target) {
-      let val = change.root;
-      let key = change.path[change.path.length - 1];
-      for (let pathIndex = 0; pathIndex < change.path.slice(0, -1).length; pathIndex++) {
+      let val = change.path[0];
+      const key = change.path[change.path.length - 1];
+      for (let pathIndex = 1; pathIndex < change.path.slice(0, -1).length; pathIndex++) {
           const p = change.path[pathIndex];
           if (val[p] === undefined) {
               const nextPath = change.path[pathIndex + 1];
@@ -23957,13 +23948,14 @@
       }
       addChange(...args) {
           const val = args.pop();
-          const [root, ...path] = args;
+          const root = args[0];
           let value = root;
-          let key = path[path.length - 1];
-          for (let pathIndex = 0; pathIndex <= path.length - 2; pathIndex++) {
-              const p = path[pathIndex];
+          let key = args[args.length - 1];
+          const pathLength = args.length - 2;
+          for (let pathIndex = 1; pathIndex <= pathLength; pathIndex++) {
+              const p = args[pathIndex];
               if (value[p] === undefined) {
-                  const nextPath = path[pathIndex + 1];
+                  const nextPath = args[pathIndex + 1];
                   value[p] = createEmptyStructure(nextPath);
               }
               value = value[p];
@@ -23972,8 +23964,7 @@
               return;
           }
           this.changes.push({
-              root,
-              path,
+              path: args,
               before: value[key],
               after: val,
           });
@@ -26031,7 +26022,10 @@
        */
       checkDispatchAllowed(command) {
           const results = this.handlers.map((handler) => handler.allowDispatch(command));
-          return new DispatchResult(results.flat());
+          if (results.some((r) => r !== 0 /* Success */)) {
+              return new DispatchResult(results.flat());
+          }
+          return DispatchResult.Success;
       }
       finalize() {
           this.status = 3 /* Finalizing */;
@@ -31455,9 +31449,9 @@
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '15.0.12';
-  __info__.date = '2023-11-02T09:28:12.777Z';
-  __info__.hash = 'e82bd99';
+  __info__.version = '15.0.14';
+  __info__.date = '2023-11-24T13:06:57.405Z';
+  __info__.hash = 'c9a805f';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);