@odoo/o-spreadsheet 17.3.3 → 17.3.4

package/dist/o-spreadsheet.cjs.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.3.3
- * @date 2024-06-14T10:02:58.082Z
- * @hash c690e9f
+ * @version 17.3.4
+ * @date 2024-06-21T20:00:08.515Z
+ * @hash 6efbf3b
  */
 
 'use strict';
@@ -9461,10 +9461,8 @@ class ComposerStore extends SpreadsheetStore {
             const refRange = this.getters.getRangeFromSheetXC(activeSheetId, xc);
             return isEqual(this.getters.expandZone(activeSheetId, refRange.zone), oldZone);
         });
-        // this function assumes that the previous range is always found because
-        // it's called when changing a highlight, which exists by definition
         if (!previousRefToken) {
-            throw new Error("Previous range not found");
+            return;
         }
         const previousRange = this.getters.getRangeFromSheetXC(activeSheetId, previousRefToken.value);
         this.selectionStart = previousRefToken.start;
@@ -34926,7 +34924,7 @@ function createMeasure(fields, measure) {
 function createPivotDimension(fields, dimension) {
     const field = fields[dimension.name];
     const type = field?.type ?? "integer";
-    const granularity = field && isDateField(field) ? dimension.granularity ?? "month_number" : undefined;
+    const granularity = field && isDateField(field) ? dimension.granularity : undefined;
     return {
         /**
          * Get the display name of the dimension
@@ -35329,10 +35327,13 @@ function compareDimensionValues(dimension, a, b) {
 }
 
 function createDate(dimension, value, locale) {
-    const granularity = dimension.granularity || "month_number";
-    if (!(granularity in MAP_VALUE_DIMENSION_DATE)) {
+    const granularity = dimension.granularity;
+    if (!granularity || !(granularity in MAP_VALUE_DIMENSION_DATE)) {
         throw new Error(`Unknown date granularity: ${granularity}`);
     }
+    if (value === null) {
+        return null;
+    }
     if (!MAP_VALUE_DIMENSION_DATE[granularity].set.has(value)) {
         MAP_VALUE_DIMENSION_DATE[granularity].set.add(value);
         const date = toJsDate(value, locale);
@@ -35559,7 +35560,8 @@ class SpreadsheetPivot {
         if (!finalCell) {
             return { value: "" };
         }
-        if (finalCell.value === null) {
+        // Value can be null but stringified (e.g. an empty date, as for now every date is stringified)
+        if (finalCell.value === null || finalCell.value === `${null}`) {
             return { value: _t("(Undefined)") };
         }
         if (dimension.type === "date") {
@@ -35595,10 +35597,15 @@ class SpreadsheetPivot {
         if (!operator) {
             throw new Error(`Aggregator ${aggregator} does not exist`);
         }
-        return {
-            value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
-            format: operator.format(values[0]),
-        };
+        try {
+            return {
+                value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
+                format: operator.format(values[0]),
+            };
+        }
+        catch (e) {
+            return handleError(e, aggregator.toUpperCase());
+        }
     }
     getPossibleFieldValues(groupBy) {
         //TODO This method should be implemented for the autocomplete feature
@@ -52840,7 +52847,6 @@ class Evaluator {
         if (!this.blockedArrayFormulas.has(position)) {
             this.invalidateSpreading(position);
         }
-        this.spreadingRelations.removeNode(position);
         const cell = this.getters.getCell(position);
         if (cell === undefined) {
             return EMPTY_CELL;
@@ -52882,6 +52888,7 @@ class Evaluator {
         this.assertSheetHasEnoughSpaceToSpreadFormulaResult(formulaPosition, formulaReturn);
         const nbColumns = formulaReturn.length;
         const nbRows = formulaReturn[0].length;
+        this.spreadingRelations.removeNode(formulaPosition);
         forEachSpreadPositionInMatrix(nbColumns, nbRows, this.updateSpreadRelation(formulaPosition));
         this.assertNoMergedCellsInSpreadZone(formulaPosition, formulaReturn);
         forEachSpreadPositionInMatrix(nbColumns, nbRows, this.checkCollision(formulaPosition));
@@ -66983,6 +66990,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.3.3";
-__info__.date = "2024-06-14T10:02:58.082Z";
-__info__.hash = "c690e9f";
+__info__.version = "17.3.4";
+__info__.date = "2024-06-21T20:00:08.515Z";
+__info__.hash = "6efbf3b";

package/dist/o-spreadsheet.esm.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.3.3
- * @date 2024-06-14T10:02:58.082Z
- * @hash c690e9f
+ * @version 17.3.4
+ * @date 2024-06-21T20:00:08.515Z
+ * @hash 6efbf3b
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -9459,10 +9459,8 @@ class ComposerStore extends SpreadsheetStore {
             const refRange = this.getters.getRangeFromSheetXC(activeSheetId, xc);
             return isEqual(this.getters.expandZone(activeSheetId, refRange.zone), oldZone);
         });
-        // this function assumes that the previous range is always found because
-        // it's called when changing a highlight, which exists by definition
         if (!previousRefToken) {
-            throw new Error("Previous range not found");
+            return;
         }
         const previousRange = this.getters.getRangeFromSheetXC(activeSheetId, previousRefToken.value);
         this.selectionStart = previousRefToken.start;
@@ -34924,7 +34922,7 @@ function createMeasure(fields, measure) {
 function createPivotDimension(fields, dimension) {
     const field = fields[dimension.name];
     const type = field?.type ?? "integer";
-    const granularity = field && isDateField(field) ? dimension.granularity ?? "month_number" : undefined;
+    const granularity = field && isDateField(field) ? dimension.granularity : undefined;
     return {
         /**
          * Get the display name of the dimension
@@ -35327,10 +35325,13 @@ function compareDimensionValues(dimension, a, b) {
 }
 
 function createDate(dimension, value, locale) {
-    const granularity = dimension.granularity || "month_number";
-    if (!(granularity in MAP_VALUE_DIMENSION_DATE)) {
+    const granularity = dimension.granularity;
+    if (!granularity || !(granularity in MAP_VALUE_DIMENSION_DATE)) {
         throw new Error(`Unknown date granularity: ${granularity}`);
     }
+    if (value === null) {
+        return null;
+    }
     if (!MAP_VALUE_DIMENSION_DATE[granularity].set.has(value)) {
         MAP_VALUE_DIMENSION_DATE[granularity].set.add(value);
         const date = toJsDate(value, locale);
@@ -35557,7 +35558,8 @@ class SpreadsheetPivot {
         if (!finalCell) {
             return { value: "" };
         }
-        if (finalCell.value === null) {
+        // Value can be null but stringified (e.g. an empty date, as for now every date is stringified)
+        if (finalCell.value === null || finalCell.value === `${null}`) {
             return { value: _t("(Undefined)") };
         }
         if (dimension.type === "date") {
@@ -35593,10 +35595,15 @@ class SpreadsheetPivot {
         if (!operator) {
             throw new Error(`Aggregator ${aggregator} does not exist`);
         }
-        return {
-            value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
-            format: operator.format(values[0]),
-        };
+        try {
+            return {
+                value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
+                format: operator.format(values[0]),
+            };
+        }
+        catch (e) {
+            return handleError(e, aggregator.toUpperCase());
+        }
     }
     getPossibleFieldValues(groupBy) {
         //TODO This method should be implemented for the autocomplete feature
@@ -52838,7 +52845,6 @@ class Evaluator {
         if (!this.blockedArrayFormulas.has(position)) {
             this.invalidateSpreading(position);
         }
-        this.spreadingRelations.removeNode(position);
         const cell = this.getters.getCell(position);
         if (cell === undefined) {
             return EMPTY_CELL;
@@ -52880,6 +52886,7 @@ class Evaluator {
         this.assertSheetHasEnoughSpaceToSpreadFormulaResult(formulaPosition, formulaReturn);
         const nbColumns = formulaReturn.length;
         const nbRows = formulaReturn[0].length;
+        this.spreadingRelations.removeNode(formulaPosition);
         forEachSpreadPositionInMatrix(nbColumns, nbRows, this.updateSpreadRelation(formulaPosition));
         this.assertNoMergedCellsInSpreadZone(formulaPosition, formulaReturn);
         forEachSpreadPositionInMatrix(nbColumns, nbRows, this.checkCollision(formulaPosition));
@@ -66938,6 +66945,6 @@ const constants = {
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "17.3.3";
-__info__.date = "2024-06-14T10:02:58.082Z";
-__info__.hash = "c690e9f";
+__info__.version = "17.3.4";
+__info__.date = "2024-06-21T20:00:08.515Z";
+__info__.hash = "6efbf3b";

package/dist/o-spreadsheet.iife.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.3.3
- * @date 2024-06-14T10:02:58.082Z
- * @hash c690e9f
+ * @version 17.3.4
+ * @date 2024-06-21T20:00:08.515Z
+ * @hash 6efbf3b
  */
 
 (function (exports, owl) {
@@ -9460,10 +9460,8 @@ stores.inject(MyMetaStore, storeInstance);
               const refRange = this.getters.getRangeFromSheetXC(activeSheetId, xc);
               return isEqual(this.getters.expandZone(activeSheetId, refRange.zone), oldZone);
           });
-          // this function assumes that the previous range is always found because
-          // it's called when changing a highlight, which exists by definition
           if (!previousRefToken) {
-              throw new Error("Previous range not found");
+              return;
           }
           const previousRange = this.getters.getRangeFromSheetXC(activeSheetId, previousRefToken.value);
           this.selectionStart = previousRefToken.start;
@@ -34925,7 +34923,7 @@ stores.inject(MyMetaStore, storeInstance);
   function createPivotDimension(fields, dimension) {
       const field = fields[dimension.name];
       const type = field?.type ?? "integer";
-      const granularity = field && isDateField(field) ? dimension.granularity ?? "month_number" : undefined;
+      const granularity = field && isDateField(field) ? dimension.granularity : undefined;
       return {
           /**
            * Get the display name of the dimension
@@ -35328,10 +35326,13 @@ stores.inject(MyMetaStore, storeInstance);
   }
 
   function createDate(dimension, value, locale) {
-      const granularity = dimension.granularity || "month_number";
-      if (!(granularity in MAP_VALUE_DIMENSION_DATE)) {
+      const granularity = dimension.granularity;
+      if (!granularity || !(granularity in MAP_VALUE_DIMENSION_DATE)) {
           throw new Error(`Unknown date granularity: ${granularity}`);
       }
+      if (value === null) {
+          return null;
+      }
       if (!MAP_VALUE_DIMENSION_DATE[granularity].set.has(value)) {
           MAP_VALUE_DIMENSION_DATE[granularity].set.add(value);
           const date = toJsDate(value, locale);
@@ -35558,7 +35559,8 @@ stores.inject(MyMetaStore, storeInstance);
           if (!finalCell) {
               return { value: "" };
           }
-          if (finalCell.value === null) {
+          // Value can be null but stringified (e.g. an empty date, as for now every date is stringified)
+          if (finalCell.value === null || finalCell.value === `${null}`) {
               return { value: _t("(Undefined)") };
           }
           if (dimension.type === "date") {
@@ -35594,10 +35596,15 @@ stores.inject(MyMetaStore, storeInstance);
           if (!operator) {
               throw new Error(`Aggregator ${aggregator} does not exist`);
           }
-          return {
-              value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
-              format: operator.format(values[0]),
-          };
+          try {
+              return {
+                  value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
+                  format: operator.format(values[0]),
+              };
+          }
+          catch (e) {
+              return handleError(e, aggregator.toUpperCase());
+          }
       }
       getPossibleFieldValues(groupBy) {
           //TODO This method should be implemented for the autocomplete feature
@@ -52839,7 +52846,6 @@ stores.inject(MyMetaStore, storeInstance);
           if (!this.blockedArrayFormulas.has(position)) {
               this.invalidateSpreading(position);
           }
-          this.spreadingRelations.removeNode(position);
           const cell = this.getters.getCell(position);
           if (cell === undefined) {
               return EMPTY_CELL;
@@ -52881,6 +52887,7 @@ stores.inject(MyMetaStore, storeInstance);
           this.assertSheetHasEnoughSpaceToSpreadFormulaResult(formulaPosition, formulaReturn);
           const nbColumns = formulaReturn.length;
           const nbRows = formulaReturn[0].length;
+          this.spreadingRelations.removeNode(formulaPosition);
           forEachSpreadPositionInMatrix(nbColumns, nbRows, this.updateSpreadRelation(formulaPosition));
           this.assertNoMergedCellsInSpreadZone(formulaPosition, formulaReturn);
           forEachSpreadPositionInMatrix(nbColumns, nbRows, this.checkCollision(formulaPosition));
@@ -66982,9 +66989,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.3.3";
-  __info__.date = "2024-06-14T10:02:58.082Z";
-  __info__.hash = "c690e9f";
+  __info__.version = "17.3.4";
+  __info__.date = "2024-06-21T20:00:08.515Z";
+  __info__.hash = "6efbf3b";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);