@odoo/o-spreadsheet 17.1.21 → 17.1.22

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.1.21
- * @date 2024-07-02T10:35:57.801Z
- * @hash ec4c09b
+ * @version 17.1.22
+ * @date 2024-07-08T05:41:57.737Z
+ * @hash da27a94
  */
 
 'use strict';
@@ -1704,7 +1704,7 @@ const getFormulaNumberRegex = memoize(function getFormulaNumberRegex(decimalSepa
 });
 const getNumberRegex = memoize(function getNumberRegex(locale) {
     const decimalSeparator = escapeRegExp(locale.decimalSeparator);
-    const thousandsSeparator = escapeRegExp(locale.thousandsSeparator);
+    const thousandsSeparator = escapeRegExp(locale.thousandsSeparator || "");
     const pIntegerAndDecimals = `(\\d+(${thousandsSeparator}\\d{3,})*(${decimalSeparator}\\d*)?)`; // pattern that match integer number with or without decimal digits
     const pOnlyDecimals = `(${decimalSeparator}\\d+)`; // pattern that match only expression with decimal digits
     const pScientificFormat = "(e(\\+|-)?\\d+)?"; // pattern that match scientific format between zero and one time (should be placed before pPercentFormat)
@@ -1731,7 +1731,7 @@ function isNumber(value, locale) {
     return getNumberRegex(locale).test(value.trim());
 }
 const getInvaluableSymbolsRegexp = memoize(function getInvaluableSymbolsRegexp(locale) {
-    return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator)}]`, "g");
+    return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator || "")}]`, "g");
 });
 /**
  * Convert a string into a number. It assumes that the string actually represents
@@ -7718,19 +7718,22 @@ class TokenizingChars {
 }
 
 function isValidLocale(locale) {
-    if (!(locale &&
-        typeof locale === "object" &&
-        typeof locale.name === "string" &&
-        typeof locale.code === "string" &&
-        typeof locale.thousandsSeparator === "string" &&
-        typeof locale.decimalSeparator === "string" &&
-        typeof locale.dateFormat === "string" &&
-        typeof locale.timeFormat === "string" &&
-        typeof locale.formulaArgSeparator === "string")) {
+    if (!locale ||
+        typeof locale !== "object" ||
+        !(!locale.thousandsSeparator || typeof locale.thousandsSeparator === "string")) {
         return false;
     }
-    if (!Object.values(locale).every((v) => v)) {
-        return false;
+    for (const property of [
+        "code",
+        "name",
+        "decimalSeparator",
+        "dateFormat",
+        "timeFormat",
+        "formulaArgSeparator",
+    ]) {
+        if (!locale[property] || typeof locale[property] !== "string") {
+            return false;
+        }
     }
     if (locale.formulaArgSeparator === locale.decimalSeparator) {
         return false;
@@ -7828,7 +7831,10 @@ function canonicalizeNumberLiteral(content, locale) {
     if (locale.decimalSeparator === "." || !isNumber(content, locale)) {
         return content;
     }
-    return content.replace(locale.thousandsSeparator, "").replace(locale.decimalSeparator, ".");
+    if (locale.thousandsSeparator) {
+        content = content.replaceAll(locale.thousandsSeparator, "");
+    }
+    return content.replace(locale.decimalSeparator, ".");
 }
 /**
  * Change a content string from the given locale to its canonical form (en_US locale). Also convert date string.
@@ -9507,15 +9513,6 @@ function createLineChartRuntime(chart, getters) {
     }
     const colors = new ChartColors();
     for (let [index, { label, data }] of dataSetsValues.entries()) {
-        if (["linear", "time"].includes(axisType)) {
-            // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
-            data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
-        }
-        const color = colors.next();
-        let backgroundRGBA = colorToRGBA(color);
-        if (chart.stacked) {
-            backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
-        }
         if (chart.cumulative) {
             let accumulator = 0;
             data = data.map((value) => {
@@ -9526,6 +9523,15 @@ function createLineChartRuntime(chart, getters) {
                 return value;
             });
         }
+        if (["linear", "time"].includes(axisType)) {
+            // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
+            data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
+        }
+        const color = colors.next();
+        let backgroundRGBA = colorToRGBA(color);
+        if (chart.stacked) {
+            backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
+        }
         const backgroundColor = rgbaToHex(backgroundRGBA);
         const dataset = {
             label,
@@ -24781,7 +24787,13 @@ class SettingsPanel extends owl.Component {
     }
     async loadLocales() {
         this.loadedLocales = (await this.env.loadLocales())
-            .filter(isValidLocale)
+            .filter((locale) => {
+            const isValid = isValidLocale(locale);
+            if (!isValid) {
+                console.warn(`Invalid locale: ${locale["code"]} ${locale}`);
+            }
+            return isValid;
+        })
             .sort((a, b) => a.name.localeCompare(b.name));
     }
     get numberFormatPreview() {
@@ -30537,6 +30549,7 @@ class Grid extends owl.Component {
             return;
         }
         if (clipboardData.types.indexOf(ClipboardMIMEType.PlainText) > -1) {
+            ev.preventDefault();
             const content = clipboardData.getData(ClipboardMIMEType.PlainText);
             const target = this.env.model.getters.getSelectedZones();
             const clipboardString = this.env.model.getters.getClipboardTextContent();
@@ -35134,6 +35147,21 @@ class BordersPlugin extends CorePlugin {
             return [];
         return Object.keys(sheetBorders).map((index) => parseInt(index, 10));
     }
+    /**
+     * Get all the rows which contains at least a border
+     */
+    getRowsWithBorders(sheetId) {
+        const sheetBorders = this.borders[sheetId]?.filter(isDefined$1);
+        if (!sheetBorders)
+            return [];
+        const rowsWithBorders = new Set();
+        for (const rowBorders of sheetBorders) {
+            for (const rowBorder in rowBorders) {
+                rowsWithBorders.add(parseInt(rowBorder, 10));
+            }
+        }
+        return Array.from(rowsWithBorders);
+    }
     /**
      * Get the range of all the rows in the sheet
      */
@@ -35183,7 +35211,7 @@ class BordersPlugin extends CorePlugin {
                 destructive: false,
             });
         }
-        this.getRowsRange(sheetId)
+        this.getRowsWithBorders(sheetId)
             .filter((row) => row >= start)
             .sort((a, b) => (delta < 0 ? a - b : b - a)) // start by the end when moving up
             .forEach((row) => {
@@ -35638,7 +35666,7 @@ function parseExpression(tokens, parent_priority = 0) {
  * Parse an expression (as a string) into an AST.
  */
 function parse(str) {
-    return parseTokens(tokenize(str));
+    return parseTokens(rangeTokenize(str));
 }
 function parseTokens(tokens) {
     tokens = tokens.filter((x) => x.type !== "SPACE");
@@ -57839,6 +57867,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.1.21";
-__info__.date = "2024-07-02T10:35:57.801Z";
-__info__.hash = "ec4c09b";
+__info__.version = "17.1.22";
+__info__.date = "2024-07-08T05:41:57.737Z";
+__info__.hash = "da27a94";

package/dist/o-spreadsheet.d.ts

@@ -2736,7 +2736,7 @@ type LocaleCode = string & Alias;
 interface Locale {
     name: string;
     code: LocaleCode;
-    thousandsSeparator: string;
+    thousandsSeparator?: string;
     decimalSeparator: string;
     dateFormat: string;
     timeFormat: string;
@@ -3189,6 +3189,10 @@ declare class BordersPlugin extends CorePlugin<BordersPluginState> implements Bo
      * Get all the columns which contains at least a border
      */
     private getColumnsWithBorders;
+    /**
+     * Get all the rows which contains at least a border
+     */
+    private getRowsWithBorders;
     /**
      * Get the range of all the rows in the sheet
      */

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.1.21
- * @date 2024-07-02T10:35:57.801Z
- * @hash ec4c09b
+ * @version 17.1.22
+ * @date 2024-07-08T05:41:57.737Z
+ * @hash da27a94
  */
 
 import { Component, useRef, onMounted, onWillUnmount, useEffect, onWillPatch, useState, onWillUpdateProps, onPatched, useComponent, useExternalListener, onWillStart, xml, useChildSubEnv, useSubEnv, markRaw } from '@odoo/owl';
@@ -1702,7 +1702,7 @@ const getFormulaNumberRegex = memoize(function getFormulaNumberRegex(decimalSepa
 });
 const getNumberRegex = memoize(function getNumberRegex(locale) {
     const decimalSeparator = escapeRegExp(locale.decimalSeparator);
-    const thousandsSeparator = escapeRegExp(locale.thousandsSeparator);
+    const thousandsSeparator = escapeRegExp(locale.thousandsSeparator || "");
     const pIntegerAndDecimals = `(\\d+(${thousandsSeparator}\\d{3,})*(${decimalSeparator}\\d*)?)`; // pattern that match integer number with or without decimal digits
     const pOnlyDecimals = `(${decimalSeparator}\\d+)`; // pattern that match only expression with decimal digits
     const pScientificFormat = "(e(\\+|-)?\\d+)?"; // pattern that match scientific format between zero and one time (should be placed before pPercentFormat)
@@ -1729,7 +1729,7 @@ function isNumber(value, locale) {
     return getNumberRegex(locale).test(value.trim());
 }
 const getInvaluableSymbolsRegexp = memoize(function getInvaluableSymbolsRegexp(locale) {
-    return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator)}]`, "g");
+    return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator || "")}]`, "g");
 });
 /**
  * Convert a string into a number. It assumes that the string actually represents
@@ -7716,19 +7716,22 @@ class TokenizingChars {
 }
 
 function isValidLocale(locale) {
-    if (!(locale &&
-        typeof locale === "object" &&
-        typeof locale.name === "string" &&
-        typeof locale.code === "string" &&
-        typeof locale.thousandsSeparator === "string" &&
-        typeof locale.decimalSeparator === "string" &&
-        typeof locale.dateFormat === "string" &&
-        typeof locale.timeFormat === "string" &&
-        typeof locale.formulaArgSeparator === "string")) {
+    if (!locale ||
+        typeof locale !== "object" ||
+        !(!locale.thousandsSeparator || typeof locale.thousandsSeparator === "string")) {
         return false;
     }
-    if (!Object.values(locale).every((v) => v)) {
-        return false;
+    for (const property of [
+        "code",
+        "name",
+        "decimalSeparator",
+        "dateFormat",
+        "timeFormat",
+        "formulaArgSeparator",
+    ]) {
+        if (!locale[property] || typeof locale[property] !== "string") {
+            return false;
+        }
     }
     if (locale.formulaArgSeparator === locale.decimalSeparator) {
         return false;
@@ -7826,7 +7829,10 @@ function canonicalizeNumberLiteral(content, locale) {
     if (locale.decimalSeparator === "." || !isNumber(content, locale)) {
         return content;
     }
-    return content.replace(locale.thousandsSeparator, "").replace(locale.decimalSeparator, ".");
+    if (locale.thousandsSeparator) {
+        content = content.replaceAll(locale.thousandsSeparator, "");
+    }
+    return content.replace(locale.decimalSeparator, ".");
 }
 /**
  * Change a content string from the given locale to its canonical form (en_US locale). Also convert date string.
@@ -9505,15 +9511,6 @@ function createLineChartRuntime(chart, getters) {
     }
     const colors = new ChartColors();
     for (let [index, { label, data }] of dataSetsValues.entries()) {
-        if (["linear", "time"].includes(axisType)) {
-            // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
-            data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
-        }
-        const color = colors.next();
-        let backgroundRGBA = colorToRGBA(color);
-        if (chart.stacked) {
-            backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
-        }
         if (chart.cumulative) {
             let accumulator = 0;
             data = data.map((value) => {
@@ -9524,6 +9521,15 @@ function createLineChartRuntime(chart, getters) {
                 return value;
             });
         }
+        if (["linear", "time"].includes(axisType)) {
+            // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
+            data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
+        }
+        const color = colors.next();
+        let backgroundRGBA = colorToRGBA(color);
+        if (chart.stacked) {
+            backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
+        }
         const backgroundColor = rgbaToHex(backgroundRGBA);
         const dataset = {
             label,
@@ -24779,7 +24785,13 @@ class SettingsPanel extends Component {
     }
     async loadLocales() {
         this.loadedLocales = (await this.env.loadLocales())
-            .filter(isValidLocale)
+            .filter((locale) => {
+            const isValid = isValidLocale(locale);
+            if (!isValid) {
+                console.warn(`Invalid locale: ${locale["code"]} ${locale}`);
+            }
+            return isValid;
+        })
             .sort((a, b) => a.name.localeCompare(b.name));
     }
     get numberFormatPreview() {
@@ -30535,6 +30547,7 @@ class Grid extends Component {
             return;
         }
         if (clipboardData.types.indexOf(ClipboardMIMEType.PlainText) > -1) {
+            ev.preventDefault();
             const content = clipboardData.getData(ClipboardMIMEType.PlainText);
             const target = this.env.model.getters.getSelectedZones();
             const clipboardString = this.env.model.getters.getClipboardTextContent();
@@ -35132,6 +35145,21 @@ class BordersPlugin extends CorePlugin {
             return [];
         return Object.keys(sheetBorders).map((index) => parseInt(index, 10));
     }
+    /**
+     * Get all the rows which contains at least a border
+     */
+    getRowsWithBorders(sheetId) {
+        const sheetBorders = this.borders[sheetId]?.filter(isDefined$1);
+        if (!sheetBorders)
+            return [];
+        const rowsWithBorders = new Set();
+        for (const rowBorders of sheetBorders) {
+            for (const rowBorder in rowBorders) {
+                rowsWithBorders.add(parseInt(rowBorder, 10));
+            }
+        }
+        return Array.from(rowsWithBorders);
+    }
     /**
      * Get the range of all the rows in the sheet
      */
@@ -35181,7 +35209,7 @@ class BordersPlugin extends CorePlugin {
                 destructive: false,
             });
         }
-        this.getRowsRange(sheetId)
+        this.getRowsWithBorders(sheetId)
             .filter((row) => row >= start)
             .sort((a, b) => (delta < 0 ? a - b : b - a)) // start by the end when moving up
             .forEach((row) => {
@@ -35636,7 +35664,7 @@ function parseExpression(tokens, parent_priority = 0) {
  * Parse an expression (as a string) into an AST.
  */
 function parse(str) {
-    return parseTokens(tokenize(str));
+    return parseTokens(rangeTokenize(str));
 }
 function parseTokens(tokens) {
     tokens = tokens.filter((x) => x.type !== "SPACE");
@@ -57801,6 +57829,6 @@ const constants = {
 export { AbstractChart, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
 
 
-__info__.version = "17.1.21";
-__info__.date = "2024-07-02T10:35:57.801Z";
-__info__.hash = "ec4c09b";
+__info__.version = "17.1.22";
+__info__.date = "2024-07-08T05:41:57.737Z";
+__info__.hash = "da27a94";

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.1.21
- * @date 2024-07-02T10:35:57.801Z
- * @hash ec4c09b
+ * @version 17.1.22
+ * @date 2024-07-08T05:41:57.737Z
+ * @hash da27a94
  */
 
 (function (exports, owl) {
@@ -1703,7 +1703,7 @@
   });
   const getNumberRegex = memoize(function getNumberRegex(locale) {
       const decimalSeparator = escapeRegExp(locale.decimalSeparator);
-      const thousandsSeparator = escapeRegExp(locale.thousandsSeparator);
+      const thousandsSeparator = escapeRegExp(locale.thousandsSeparator || "");
       const pIntegerAndDecimals = `(\\d+(${thousandsSeparator}\\d{3,})*(${decimalSeparator}\\d*)?)`; // pattern that match integer number with or without decimal digits
       const pOnlyDecimals = `(${decimalSeparator}\\d+)`; // pattern that match only expression with decimal digits
       const pScientificFormat = "(e(\\+|-)?\\d+)?"; // pattern that match scientific format between zero and one time (should be placed before pPercentFormat)
@@ -1730,7 +1730,7 @@
       return getNumberRegex(locale).test(value.trim());
   }
   const getInvaluableSymbolsRegexp = memoize(function getInvaluableSymbolsRegexp(locale) {
-      return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator)}]`, "g");
+      return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator || "")}]`, "g");
   });
   /**
    * Convert a string into a number. It assumes that the string actually represents
@@ -7717,19 +7717,22 @@
   }
 
   function isValidLocale(locale) {
-      if (!(locale &&
-          typeof locale === "object" &&
-          typeof locale.name === "string" &&
-          typeof locale.code === "string" &&
-          typeof locale.thousandsSeparator === "string" &&
-          typeof locale.decimalSeparator === "string" &&
-          typeof locale.dateFormat === "string" &&
-          typeof locale.timeFormat === "string" &&
-          typeof locale.formulaArgSeparator === "string")) {
+      if (!locale ||
+          typeof locale !== "object" ||
+          !(!locale.thousandsSeparator || typeof locale.thousandsSeparator === "string")) {
           return false;
       }
-      if (!Object.values(locale).every((v) => v)) {
-          return false;
+      for (const property of [
+          "code",
+          "name",
+          "decimalSeparator",
+          "dateFormat",
+          "timeFormat",
+          "formulaArgSeparator",
+      ]) {
+          if (!locale[property] || typeof locale[property] !== "string") {
+              return false;
+          }
       }
       if (locale.formulaArgSeparator === locale.decimalSeparator) {
           return false;
@@ -7827,7 +7830,10 @@
       if (locale.decimalSeparator === "." || !isNumber(content, locale)) {
           return content;
       }
-      return content.replace(locale.thousandsSeparator, "").replace(locale.decimalSeparator, ".");
+      if (locale.thousandsSeparator) {
+          content = content.replaceAll(locale.thousandsSeparator, "");
+      }
+      return content.replace(locale.decimalSeparator, ".");
   }
   /**
    * Change a content string from the given locale to its canonical form (en_US locale). Also convert date string.
@@ -9506,15 +9512,6 @@
       }
       const colors = new ChartColors();
       for (let [index, { label, data }] of dataSetsValues.entries()) {
-          if (["linear", "time"].includes(axisType)) {
-              // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
-              data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
-          }
-          const color = colors.next();
-          let backgroundRGBA = colorToRGBA(color);
-          if (chart.stacked) {
-              backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
-          }
           if (chart.cumulative) {
               let accumulator = 0;
               data = data.map((value) => {
@@ -9525,6 +9522,15 @@
                   return value;
               });
           }
+          if (["linear", "time"].includes(axisType)) {
+              // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
+              data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
+          }
+          const color = colors.next();
+          let backgroundRGBA = colorToRGBA(color);
+          if (chart.stacked) {
+              backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
+          }
           const backgroundColor = rgbaToHex(backgroundRGBA);
           const dataset = {
               label,
@@ -24780,7 +24786,13 @@
       }
       async loadLocales() {
           this.loadedLocales = (await this.env.loadLocales())
-              .filter(isValidLocale)
+              .filter((locale) => {
+              const isValid = isValidLocale(locale);
+              if (!isValid) {
+                  console.warn(`Invalid locale: ${locale["code"]} ${locale}`);
+              }
+              return isValid;
+          })
               .sort((a, b) => a.name.localeCompare(b.name));
       }
       get numberFormatPreview() {
@@ -30536,6 +30548,7 @@
               return;
           }
           if (clipboardData.types.indexOf(ClipboardMIMEType.PlainText) > -1) {
+              ev.preventDefault();
               const content = clipboardData.getData(ClipboardMIMEType.PlainText);
               const target = this.env.model.getters.getSelectedZones();
               const clipboardString = this.env.model.getters.getClipboardTextContent();
@@ -35133,6 +35146,21 @@
               return [];
           return Object.keys(sheetBorders).map((index) => parseInt(index, 10));
       }
+      /**
+       * Get all the rows which contains at least a border
+       */
+      getRowsWithBorders(sheetId) {
+          const sheetBorders = this.borders[sheetId]?.filter(isDefined$1);
+          if (!sheetBorders)
+              return [];
+          const rowsWithBorders = new Set();
+          for (const rowBorders of sheetBorders) {
+              for (const rowBorder in rowBorders) {
+                  rowsWithBorders.add(parseInt(rowBorder, 10));
+              }
+          }
+          return Array.from(rowsWithBorders);
+      }
       /**
        * Get the range of all the rows in the sheet
        */
@@ -35182,7 +35210,7 @@
                   destructive: false,
               });
           }
-          this.getRowsRange(sheetId)
+          this.getRowsWithBorders(sheetId)
               .filter((row) => row >= start)
               .sort((a, b) => (delta < 0 ? a - b : b - a)) // start by the end when moving up
               .forEach((row) => {
@@ -35637,7 +35665,7 @@
    * Parse an expression (as a string) into an AST.
    */
   function parse(str) {
-      return parseTokens(tokenize(str));
+      return parseTokens(rangeTokenize(str));
   }
   function parseTokens(tokens) {
       tokens = tokens.filter((x) => x.type !== "SPACE");
@@ -57838,9 +57866,9 @@
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.1.21";
-  __info__.date = "2024-07-02T10:35:57.801Z";
-  __info__.hash = "ec4c09b";
+  __info__.version = "17.1.22";
+  __info__.date = "2024-07-08T05:41:57.737Z";
+  __info__.hash = "da27a94";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);