@odoo/o-spreadsheet 17.4.0 → 17.4.1

package/dist/o-spreadsheet.cjs.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.4.0
- * @date 2024-07-12T04:41:51.675Z
- * @hash f8e8543
+ * @version 17.4.1
+ * @date 2024-07-18T14:25:54.421Z
+ * @hash fd7deaa
  */
 
 'use strict';
@@ -2724,7 +2724,7 @@ function getPredicate(descr, locale) {
             operand = descr;
         }
     }
-    if (isNumber(operand, locale)) {
+    if (isNumber(operand, locale) || isDateTime(operand, locale)) {
         operand = toNumber(operand, locale);
     }
     else if (operand === "TRUE" || operand === "FALSE") {
@@ -35974,9 +35974,15 @@ class PivotLayoutConfigurator extends owl.Component {
     addMeasureDimension(fieldName) {
         const { measures } = this.props.definition;
         this.props.onDimensionsUpdated({
-            measures: measures.concat([{ name: fieldName }]),
+            measures: measures.concat([
+                { name: fieldName, aggregator: this.getDefaultMeasureAggregator(fieldName) },
+            ]),
         });
     }
+    getDefaultMeasureAggregator(fieldName) {
+        const field = this.props.unusedMeasureFields.find((f) => f.name === fieldName);
+        return field?.aggregator ? field.aggregator : "count";
+    }
     updateAggregator(updatedMeasure, aggregator) {
         const { measures } = this.props.definition;
         this.props.onDimensionsUpdated({
@@ -36200,7 +36206,7 @@ function createMeasure(fields, measure) {
     const field = name === "__count"
         ? { name: "__count", string: _t("Count"), type: "integer", aggregator: "sum" }
         : fields[name];
-    const aggregator = measure.aggregator || field?.aggregator;
+    const aggregator = measure.aggregator;
     return {
         nameWithAggregator: name + (aggregator ? `:${aggregator}` : ""),
         /**
@@ -36936,7 +36942,7 @@ class SpreadsheetPivot {
             .map((value) => value[measure])
             .filter((cell) => cell && cell.type !== CellValueType.empty)
             .filter(isDefined);
-        const aggregator = this.getMeasure(measure).aggregator || "count";
+        const aggregator = this.getMeasure(measure).aggregator;
         const operator = AGGREGATORS_FN[aggregator];
         if (!operator) {
             throw new Error(`Aggregator ${aggregator} does not exist`);
@@ -54204,9 +54210,9 @@ class Evaluator {
     assertNoMergedCellsInSpreadZone({ sheetId, col, row }, matrixResult) {
         const mergedCells = this.getters.getMergesInZone(sheetId, {
             top: row,
-            bottom: row + matrixResult.length,
+            bottom: row + matrixResult[0].length - 1,
             left: col,
-            right: col + matrixResult[0].length,
+            right: col + matrixResult.length - 1,
         });
         if (mergedCells.length === 0) {
             return;
@@ -68393,6 +68399,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.4.0";
-__info__.date = "2024-07-12T04:41:51.675Z";
-__info__.hash = "f8e8543";
+__info__.version = "17.4.1";
+__info__.date = "2024-07-18T14:25:54.421Z";
+__info__.hash = "fd7deaa";

package/dist/o-spreadsheet.d.ts

@@ -531,7 +531,7 @@ interface PivotCoreDimension {
 }
 interface PivotCoreMeasure {
     name: string;
-    aggregator?: Aggregator | string;
+    aggregator: Aggregator | string;
 }
 interface CommonPivotCoreDefinition {
     columns: PivotCoreDimension[];
@@ -9354,6 +9354,7 @@ declare class PivotLayoutConfigurator extends Component<Props$d, SpreadsheetChil
     addColumnDimension(fieldName: string): void;
     addRowDimension(fieldName: string): void;
     addMeasureDimension(fieldName: string): void;
+    private getDefaultMeasureAggregator;
     updateAggregator(updatedMeasure: PivotMeasure, aggregator: string): void;
     updateOrder(updateDimension: PivotDimension$1, order?: "asc" | "desc"): void;
     updateGranularity(dimension: PivotDimension$1, granularity: Granularity): void;

package/dist/o-spreadsheet.esm.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.4.0
- * @date 2024-07-12T04:41:51.675Z
- * @hash f8e8543
+ * @version 17.4.1
+ * @date 2024-07-18T14:25:54.421Z
+ * @hash fd7deaa
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -2722,7 +2722,7 @@ function getPredicate(descr, locale) {
             operand = descr;
         }
     }
-    if (isNumber(operand, locale)) {
+    if (isNumber(operand, locale) || isDateTime(operand, locale)) {
         operand = toNumber(operand, locale);
     }
     else if (operand === "TRUE" || operand === "FALSE") {
@@ -35972,9 +35972,15 @@ class PivotLayoutConfigurator extends Component {
     addMeasureDimension(fieldName) {
         const { measures } = this.props.definition;
         this.props.onDimensionsUpdated({
-            measures: measures.concat([{ name: fieldName }]),
+            measures: measures.concat([
+                { name: fieldName, aggregator: this.getDefaultMeasureAggregator(fieldName) },
+            ]),
         });
     }
+    getDefaultMeasureAggregator(fieldName) {
+        const field = this.props.unusedMeasureFields.find((f) => f.name === fieldName);
+        return field?.aggregator ? field.aggregator : "count";
+    }
     updateAggregator(updatedMeasure, aggregator) {
         const { measures } = this.props.definition;
         this.props.onDimensionsUpdated({
@@ -36198,7 +36204,7 @@ function createMeasure(fields, measure) {
     const field = name === "__count"
         ? { name: "__count", string: _t("Count"), type: "integer", aggregator: "sum" }
         : fields[name];
-    const aggregator = measure.aggregator || field?.aggregator;
+    const aggregator = measure.aggregator;
     return {
         nameWithAggregator: name + (aggregator ? `:${aggregator}` : ""),
         /**
@@ -36934,7 +36940,7 @@ class SpreadsheetPivot {
             .map((value) => value[measure])
             .filter((cell) => cell && cell.type !== CellValueType.empty)
             .filter(isDefined);
-        const aggregator = this.getMeasure(measure).aggregator || "count";
+        const aggregator = this.getMeasure(measure).aggregator;
         const operator = AGGREGATORS_FN[aggregator];
         if (!operator) {
             throw new Error(`Aggregator ${aggregator} does not exist`);
@@ -54202,9 +54208,9 @@ class Evaluator {
     assertNoMergedCellsInSpreadZone({ sheetId, col, row }, matrixResult) {
         const mergedCells = this.getters.getMergesInZone(sheetId, {
             top: row,
-            bottom: row + matrixResult.length,
+            bottom: row + matrixResult[0].length - 1,
             left: col,
-            right: col + matrixResult[0].length,
+            right: col + matrixResult.length - 1,
         });
         if (mergedCells.length === 0) {
             return;
@@ -68348,6 +68354,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.4.0";
-__info__.date = "2024-07-12T04:41:51.675Z";
-__info__.hash = "f8e8543";
+__info__.version = "17.4.1";
+__info__.date = "2024-07-18T14:25:54.421Z";
+__info__.hash = "fd7deaa";

package/dist/o-spreadsheet.iife.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.4.0
- * @date 2024-07-12T04:41:51.675Z
- * @hash f8e8543
+ * @version 17.4.1
+ * @date 2024-07-18T14:25:54.421Z
+ * @hash fd7deaa
  */
 
 (function (exports, owl) {
@@ -2723,7 +2723,7 @@
               operand = descr;
           }
       }
-      if (isNumber(operand, locale)) {
+      if (isNumber(operand, locale) || isDateTime(operand, locale)) {
           operand = toNumber(operand, locale);
       }
       else if (operand === "TRUE" || operand === "FALSE") {
@@ -35973,9 +35973,15 @@ stores.inject(MyMetaStore, storeInstance);
       addMeasureDimension(fieldName) {
           const { measures } = this.props.definition;
           this.props.onDimensionsUpdated({
-              measures: measures.concat([{ name: fieldName }]),
+              measures: measures.concat([
+                  { name: fieldName, aggregator: this.getDefaultMeasureAggregator(fieldName) },
+              ]),
           });
       }
+      getDefaultMeasureAggregator(fieldName) {
+          const field = this.props.unusedMeasureFields.find((f) => f.name === fieldName);
+          return field?.aggregator ? field.aggregator : "count";
+      }
       updateAggregator(updatedMeasure, aggregator) {
           const { measures } = this.props.definition;
           this.props.onDimensionsUpdated({
@@ -36199,7 +36205,7 @@ stores.inject(MyMetaStore, storeInstance);
       const field = name === "__count"
           ? { name: "__count", string: _t("Count"), type: "integer", aggregator: "sum" }
           : fields[name];
-      const aggregator = measure.aggregator || field?.aggregator;
+      const aggregator = measure.aggregator;
       return {
           nameWithAggregator: name + (aggregator ? `:${aggregator}` : ""),
           /**
@@ -36935,7 +36941,7 @@ stores.inject(MyMetaStore, storeInstance);
               .map((value) => value[measure])
               .filter((cell) => cell && cell.type !== CellValueType.empty)
               .filter(isDefined);
-          const aggregator = this.getMeasure(measure).aggregator || "count";
+          const aggregator = this.getMeasure(measure).aggregator;
           const operator = AGGREGATORS_FN[aggregator];
           if (!operator) {
               throw new Error(`Aggregator ${aggregator} does not exist`);
@@ -54203,9 +54209,9 @@ stores.inject(MyMetaStore, storeInstance);
       assertNoMergedCellsInSpreadZone({ sheetId, col, row }, matrixResult) {
           const mergedCells = this.getters.getMergesInZone(sheetId, {
               top: row,
-              bottom: row + matrixResult.length,
+              bottom: row + matrixResult[0].length - 1,
               left: col,
-              right: col + matrixResult[0].length,
+              right: col + matrixResult.length - 1,
           });
           if (mergedCells.length === 0) {
               return;
@@ -68392,9 +68398,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.4.0";
-  __info__.date = "2024-07-12T04:41:51.675Z";
-  __info__.hash = "f8e8543";
+  __info__.version = "17.4.1";
+  __info__.date = "2024-07-18T14:25:54.421Z";
+  __info__.hash = "fd7deaa";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);