@odoo/o-spreadsheet 18.3.13 → 18.3.14

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 18.3.13
- * @date 2025-07-28T13:39:23.645Z
- * @hash d30327c
+ * @version 18.3.14
+ * @date 2025-07-30T11:18:53.973Z
+ * @hash 2a7178a
  */
 
 'use strict';
@@ -8669,12 +8669,12 @@ const AGGREGATOR_NAMES = {
     avg: _t("Average"),
     sum: _t("Sum"),
 };
-const NUMBER_CHAR_AGGREGATORS = ["max", "min", "avg", "sum", "count_distinct", "count"];
+const DEFAULT_AGGREGATORS = ["max", "min", "avg", "sum", "count_distinct", "count"];
 const AGGREGATORS_BY_FIELD_TYPE = {
-    integer: NUMBER_CHAR_AGGREGATORS,
-    char: NUMBER_CHAR_AGGREGATORS,
+    integer: DEFAULT_AGGREGATORS,
+    char: DEFAULT_AGGREGATORS,
+    datetime: DEFAULT_AGGREGATORS,
     boolean: ["count_distinct", "count", "bool_and", "bool_or"],
-    datetime: ["max", "min", "count_distinct", "count"],
 };
 const AGGREGATORS = {};
 for (const type in AGGREGATORS_BY_FIELD_TYPE) {
@@ -22635,31 +22635,40 @@ autoCompleteProviders.add("dataValidation", {
         if (!this.composer.currentEditedCell) {
             return [];
         }
-        const position = this.composer.currentEditedCell;
-        const rule = this.getters.getValidationRuleForCell(position);
-        if (!rule ||
-            (rule.criterion.type !== "isValueInList" && rule.criterion.type !== "isValueInRange")) {
-            return [];
-        }
-        let values;
-        if (rule.criterion.type === "isValueInList") {
-            values = rule.criterion.values;
-        }
-        else {
-            const range = this.getters.getRangeFromSheetXC(position.sheetId, rule.criterion.values[0]);
-            values = Array.from(new Set(this.getters
-                .getRangeValues(range)
-                .filter(isNotNull)
-                .map((value) => value.toString())
-                .filter((val) => val !== "")));
-        }
-        return values.map((value) => ({ text: value }));
+        return getProposedValues(this.getters, this.composer.currentEditedCell).map((value) => ({
+            text: value.value?.toString() || "",
+            htmlContent: [{ value: value.label }],
+            fuzzySearchKey: value.label,
+        }));
     },
     selectProposal(tokenAtCursor, value) {
         this.composer.setCurrentContent(value);
         this.composer.stopEdition();
     },
 });
+function getProposedValues(getters, position) {
+    const rule = getters.getValidationRuleForCell(position);
+    if (!rule ||
+        (rule.criterion.type !== "isValueInList" && rule.criterion.type !== "isValueInRange")) {
+        return [];
+    }
+    let values = [];
+    if (rule.criterion.type === "isValueInList") {
+        values = rule.criterion.values.map((value) => ({ label: value, value }));
+    }
+    else {
+        const labelsSet = new Set();
+        const range = getters.getRangeFromSheetXC(position.sheetId, rule.criterion.values[0]);
+        for (const p of positions(range.zone)) {
+            const cell = getters.getEvaluatedCell({ ...p, sheetId: range.sheetId });
+            if (cell.formattedValue && !labelsSet.has(cell.formattedValue)) {
+                labelsSet.add(cell.formattedValue);
+                values.push({ label: cell.formattedValue, value: cell.value });
+            }
+        }
+    }
+    return values;
+}
 
 function getHtmlContentFromPattern(pattern, value, highlightColor, className) {
     const pendingHtmlContent = [];
@@ -45526,7 +45535,8 @@ class ConditionalFormattingEditor extends owl.Component {
     static props = {
         editedCf: Object,
         onCancel: Function,
-        onSave: Function,
+        onExit: Function,
+        isNewCf: Boolean,
     };
     static components = {
         SelectionInput,
@@ -45545,6 +45555,7 @@ class ConditionalFormattingEditor extends owl.Component {
     getTextDecoration = getTextDecoration;
     colorNumberString = colorNumberString;
     state;
+    hasEditedCf = this.props.isNewCf;
     setup() {
         this.state = owl.useState({
             errors: [],
@@ -45602,6 +45613,9 @@ class ConditionalFormattingEditor extends owl.Component {
             ranges: ranges.map((xc) => this.env.model.getters.getRangeDataFromXc(sheetId, xc)),
             sheetId,
         });
+        if (result.isSuccessful) {
+            this.hasEditedCf = true;
+        }
         const reasons = result.reasons.filter((r) => r !== "NoChanges" /* CommandResult.NoChanges */);
         if (!newCf.suppressErrors) {
             this.state.errors = reasons;
@@ -45623,7 +45637,15 @@ class ConditionalFormattingEditor extends owl.Component {
     onSave() {
         const result = this.updateConditionalFormat({});
         if (result.length === 0) {
-            this.props.onSave();
+            this.props.onExit();
+        }
+    }
+    onCancel() {
+        if (this.hasEditedCf) {
+            this.props.onCancel();
+        }
+        else {
+            this.props.onExit();
         }
     }
     getDefaultRules() {
@@ -80900,6 +80922,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.3.13";
-__info__.date = "2025-07-28T13:39:23.645Z";
-__info__.hash = "d30327c";
+__info__.version = "18.3.14";
+__info__.date = "2025-07-30T11:18:53.973Z";
+__info__.hash = "2a7178a";

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 18.3.13
- * @date 2025-07-28T13:39:23.645Z
- * @hash d30327c
+ * @version 18.3.14
+ * @date 2025-07-30T11:18:53.973Z
+ * @hash 2a7178a
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, App, blockDom, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -8667,12 +8667,12 @@ const AGGREGATOR_NAMES = {
     avg: _t("Average"),
     sum: _t("Sum"),
 };
-const NUMBER_CHAR_AGGREGATORS = ["max", "min", "avg", "sum", "count_distinct", "count"];
+const DEFAULT_AGGREGATORS = ["max", "min", "avg", "sum", "count_distinct", "count"];
 const AGGREGATORS_BY_FIELD_TYPE = {
-    integer: NUMBER_CHAR_AGGREGATORS,
-    char: NUMBER_CHAR_AGGREGATORS,
+    integer: DEFAULT_AGGREGATORS,
+    char: DEFAULT_AGGREGATORS,
+    datetime: DEFAULT_AGGREGATORS,
     boolean: ["count_distinct", "count", "bool_and", "bool_or"],
-    datetime: ["max", "min", "count_distinct", "count"],
 };
 const AGGREGATORS = {};
 for (const type in AGGREGATORS_BY_FIELD_TYPE) {
@@ -22633,31 +22633,40 @@ autoCompleteProviders.add("dataValidation", {
         if (!this.composer.currentEditedCell) {
             return [];
         }
-        const position = this.composer.currentEditedCell;
-        const rule = this.getters.getValidationRuleForCell(position);
-        if (!rule ||
-            (rule.criterion.type !== "isValueInList" && rule.criterion.type !== "isValueInRange")) {
-            return [];
-        }
-        let values;
-        if (rule.criterion.type === "isValueInList") {
-            values = rule.criterion.values;
-        }
-        else {
-            const range = this.getters.getRangeFromSheetXC(position.sheetId, rule.criterion.values[0]);
-            values = Array.from(new Set(this.getters
-                .getRangeValues(range)
-                .filter(isNotNull)
-                .map((value) => value.toString())
-                .filter((val) => val !== "")));
-        }
-        return values.map((value) => ({ text: value }));
+        return getProposedValues(this.getters, this.composer.currentEditedCell).map((value) => ({
+            text: value.value?.toString() || "",
+            htmlContent: [{ value: value.label }],
+            fuzzySearchKey: value.label,
+        }));
     },
     selectProposal(tokenAtCursor, value) {
         this.composer.setCurrentContent(value);
         this.composer.stopEdition();
     },
 });
+function getProposedValues(getters, position) {
+    const rule = getters.getValidationRuleForCell(position);
+    if (!rule ||
+        (rule.criterion.type !== "isValueInList" && rule.criterion.type !== "isValueInRange")) {
+        return [];
+    }
+    let values = [];
+    if (rule.criterion.type === "isValueInList") {
+        values = rule.criterion.values.map((value) => ({ label: value, value }));
+    }
+    else {
+        const labelsSet = new Set();
+        const range = getters.getRangeFromSheetXC(position.sheetId, rule.criterion.values[0]);
+        for (const p of positions(range.zone)) {
+            const cell = getters.getEvaluatedCell({ ...p, sheetId: range.sheetId });
+            if (cell.formattedValue && !labelsSet.has(cell.formattedValue)) {
+                labelsSet.add(cell.formattedValue);
+                values.push({ label: cell.formattedValue, value: cell.value });
+            }
+        }
+    }
+    return values;
+}
 
 function getHtmlContentFromPattern(pattern, value, highlightColor, className) {
     const pendingHtmlContent = [];
@@ -45524,7 +45533,8 @@ class ConditionalFormattingEditor extends Component {
     static props = {
         editedCf: Object,
         onCancel: Function,
-        onSave: Function,
+        onExit: Function,
+        isNewCf: Boolean,
     };
     static components = {
         SelectionInput,
@@ -45543,6 +45553,7 @@ class ConditionalFormattingEditor extends Component {
     getTextDecoration = getTextDecoration;
     colorNumberString = colorNumberString;
     state;
+    hasEditedCf = this.props.isNewCf;
     setup() {
         this.state = useState({
             errors: [],
@@ -45600,6 +45611,9 @@ class ConditionalFormattingEditor extends Component {
             ranges: ranges.map((xc) => this.env.model.getters.getRangeDataFromXc(sheetId, xc)),
             sheetId,
         });
+        if (result.isSuccessful) {
+            this.hasEditedCf = true;
+        }
         const reasons = result.reasons.filter((r) => r !== "NoChanges" /* CommandResult.NoChanges */);
         if (!newCf.suppressErrors) {
             this.state.errors = reasons;
@@ -45621,7 +45635,15 @@ class ConditionalFormattingEditor extends Component {
     onSave() {
         const result = this.updateConditionalFormat({});
         if (result.length === 0) {
-            this.props.onSave();
+            this.props.onExit();
+        }
+    }
+    onCancel() {
+        if (this.hasEditedCf) {
+            this.props.onCancel();
+        }
+        else {
+            this.props.onExit();
         }
     }
     getDefaultRules() {
@@ -80852,6 +80874,6 @@ const chartHelpers = { ...CHART_HELPERS, ...CHART_RUNTIME_HELPERS };
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, CoreViewPlugin, DispatchResult, EvaluationError, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, chartHelpers, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateChartEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "18.3.13";
-__info__.date = "2025-07-28T13:39:23.645Z";
-__info__.hash = "d30327c";
+__info__.version = "18.3.14";
+__info__.date = "2025-07-30T11:18:53.973Z";
+__info__.hash = "2a7178a";

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 18.3.13
- * @date 2025-07-28T13:39:23.645Z
- * @hash d30327c
+ * @version 18.3.14
+ * @date 2025-07-30T11:18:53.973Z
+ * @hash 2a7178a
  */
 
 (function (exports, owl) {
@@ -8668,12 +8668,12 @@
       avg: _t("Average"),
       sum: _t("Sum"),
   };
-  const NUMBER_CHAR_AGGREGATORS = ["max", "min", "avg", "sum", "count_distinct", "count"];
+  const DEFAULT_AGGREGATORS = ["max", "min", "avg", "sum", "count_distinct", "count"];
   const AGGREGATORS_BY_FIELD_TYPE = {
-      integer: NUMBER_CHAR_AGGREGATORS,
-      char: NUMBER_CHAR_AGGREGATORS,
+      integer: DEFAULT_AGGREGATORS,
+      char: DEFAULT_AGGREGATORS,
+      datetime: DEFAULT_AGGREGATORS,
       boolean: ["count_distinct", "count", "bool_and", "bool_or"],
-      datetime: ["max", "min", "count_distinct", "count"],
   };
   const AGGREGATORS = {};
   for (const type in AGGREGATORS_BY_FIELD_TYPE) {
@@ -22634,31 +22634,40 @@ stores.inject(MyMetaStore, storeInstance);
           if (!this.composer.currentEditedCell) {
               return [];
           }
-          const position = this.composer.currentEditedCell;
-          const rule = this.getters.getValidationRuleForCell(position);
-          if (!rule ||
-              (rule.criterion.type !== "isValueInList" && rule.criterion.type !== "isValueInRange")) {
-              return [];
-          }
-          let values;
-          if (rule.criterion.type === "isValueInList") {
-              values = rule.criterion.values;
-          }
-          else {
-              const range = this.getters.getRangeFromSheetXC(position.sheetId, rule.criterion.values[0]);
-              values = Array.from(new Set(this.getters
-                  .getRangeValues(range)
-                  .filter(isNotNull)
-                  .map((value) => value.toString())
-                  .filter((val) => val !== "")));
-          }
-          return values.map((value) => ({ text: value }));
+          return getProposedValues(this.getters, this.composer.currentEditedCell).map((value) => ({
+              text: value.value?.toString() || "",
+              htmlContent: [{ value: value.label }],
+              fuzzySearchKey: value.label,
+          }));
       },
       selectProposal(tokenAtCursor, value) {
           this.composer.setCurrentContent(value);
           this.composer.stopEdition();
       },
   });
+  function getProposedValues(getters, position) {
+      const rule = getters.getValidationRuleForCell(position);
+      if (!rule ||
+          (rule.criterion.type !== "isValueInList" && rule.criterion.type !== "isValueInRange")) {
+          return [];
+      }
+      let values = [];
+      if (rule.criterion.type === "isValueInList") {
+          values = rule.criterion.values.map((value) => ({ label: value, value }));
+      }
+      else {
+          const labelsSet = new Set();
+          const range = getters.getRangeFromSheetXC(position.sheetId, rule.criterion.values[0]);
+          for (const p of positions(range.zone)) {
+              const cell = getters.getEvaluatedCell({ ...p, sheetId: range.sheetId });
+              if (cell.formattedValue && !labelsSet.has(cell.formattedValue)) {
+                  labelsSet.add(cell.formattedValue);
+                  values.push({ label: cell.formattedValue, value: cell.value });
+              }
+          }
+      }
+      return values;
+  }
 
   function getHtmlContentFromPattern(pattern, value, highlightColor, className) {
       const pendingHtmlContent = [];
@@ -45525,7 +45534,8 @@ stores.inject(MyMetaStore, storeInstance);
       static props = {
           editedCf: Object,
           onCancel: Function,
-          onSave: Function,
+          onExit: Function,
+          isNewCf: Boolean,
       };
       static components = {
           SelectionInput,
@@ -45544,6 +45554,7 @@ stores.inject(MyMetaStore, storeInstance);
       getTextDecoration = getTextDecoration;
       colorNumberString = colorNumberString;
       state;
+      hasEditedCf = this.props.isNewCf;
       setup() {
           this.state = owl.useState({
               errors: [],
@@ -45601,6 +45612,9 @@ stores.inject(MyMetaStore, storeInstance);
               ranges: ranges.map((xc) => this.env.model.getters.getRangeDataFromXc(sheetId, xc)),
               sheetId,
           });
+          if (result.isSuccessful) {
+              this.hasEditedCf = true;
+          }
           const reasons = result.reasons.filter((r) => r !== "NoChanges" /* CommandResult.NoChanges */);
           if (!newCf.suppressErrors) {
               this.state.errors = reasons;
@@ -45622,7 +45636,15 @@ stores.inject(MyMetaStore, storeInstance);
       onSave() {
           const result = this.updateConditionalFormat({});
           if (result.length === 0) {
-              this.props.onSave();
+              this.props.onExit();
+          }
+      }
+      onCancel() {
+          if (this.hasEditedCf) {
+              this.props.onCancel();
+          }
+          else {
+              this.props.onExit();
           }
       }
       getDefaultRules() {
@@ -80899,9 +80921,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "18.3.13";
-  __info__.date = "2025-07-28T13:39:23.645Z";
-  __info__.hash = "d30327c";
+  __info__.version = "18.3.14";
+  __info__.date = "2025-07-30T11:18:53.973Z";
+  __info__.hash = "2a7178a";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);