@odoo/o-spreadsheet 18.0.0 → 18.0.1

package/dist/o-spreadsheet.cjs.js

@@ -2,15 +2,121 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 18.0.0
- * @date 2024-09-25T12:54:19.974Z
- * @hash cee2e47
+ * @version 18.0.1
+ * @date 2024-10-14T07:54:24.768Z
+ * @hash 1771f68
  */
 
 'use strict';
 
 var owl = require('@odoo/owl');
 
+function createActions(menuItems) {
+    return menuItems.map(createAction).sort((a, b) => a.sequence - b.sequence);
+}
+let nextItemId = 1;
+function createAction(item) {
+    const name = item.name;
+    const children = item.children;
+    const description = item.description;
+    const icon = item.icon;
+    const secondaryIcon = item.secondaryIcon;
+    const itemId = item.id || nextItemId++;
+    return {
+        id: itemId.toString(),
+        name: typeof name === "function" ? name : () => name,
+        isVisible: item.isVisible ? item.isVisible : () => true,
+        isEnabled: item.isEnabled ? item.isEnabled : () => true,
+        isActive: item.isActive,
+        execute: item.execute,
+        children: children
+            ? (env) => {
+                return children
+                    .map((child) => (typeof child === "function" ? child(env) : child))
+                    .flat()
+                    .map(createAction);
+            }
+            : () => [],
+        isReadonlyAllowed: item.isReadonlyAllowed || false,
+        separator: item.separator || false,
+        icon: typeof icon === "function" ? icon : () => icon || "",
+        iconColor: item.iconColor,
+        secondaryIcon: typeof secondaryIcon === "function" ? secondaryIcon : () => secondaryIcon || "",
+        description: typeof description === "function" ? description : () => description || "",
+        textColor: item.textColor,
+        sequence: item.sequence || 0,
+        onStartHover: item.onStartHover,
+        onStopHover: item.onStopHover,
+    };
+}
+
+/**
+ * Registry
+ *
+ * The Registry class is basically just a mapping from a string key to an object.
+ * It is really not much more than an object. It is however useful for the
+ * following reasons:
+ *
+ * 1. it let us react and execute code when someone add something to the registry
+ *   (for example, the FunctionRegistry subclass this for this purpose)
+ * 2. it throws an error when the get operation fails
+ * 3. it provides a chained API to add items to the registry.
+ */
+class Registry {
+    content = {};
+    /**
+     * Add an item to the registry
+     *
+     * Note that this also returns the registry, so another add method call can
+     * be chained
+     */
+    add(key, value) {
+        this.content[key] = value;
+        return this;
+    }
+    /**
+     * Get an item from the registry
+     */
+    get(key) {
+        /**
+         * Note: key in {} is ~12 times slower than {}[key].
+         * So, we check the absence of key only when the direct access returns
+         * a falsy value. It's done to ensure that the registry can contains falsy values
+         */
+        const content = this.content[key];
+        if (!content) {
+            if (!(key in this.content)) {
+                throw new Error(`Cannot find ${key} in this registry!`);
+            }
+        }
+        return content;
+    }
+    /**
+     * Check if the key is already in the registry
+     */
+    contains(key) {
+        return key in this.content;
+    }
+    /**
+     * Get a list of all elements in the registry
+     */
+    getAll() {
+        return Object.values(this.content);
+    }
+    /**
+     * Get a list of all keys in the registry
+     */
+    getKeys() {
+        return Object.keys(this.content);
+    }
+    /**
+     * Remove an item from the registry
+     */
+    remove(key) {
+        delete this.content[key];
+    }
+}
+
 const CANVAS_SHIFT = 0.5;
 // Colors
 const HIGHLIGHT_COLOR = "#37A850";
@@ -5995,111 +6101,6 @@ class UuidGenerator {
     }
 }
 
-function createActions(menuItems) {
-    return menuItems.map(createAction).sort((a, b) => a.sequence - b.sequence);
-}
-const uuidGenerator$1 = new UuidGenerator();
-function createAction(item) {
-    const name = item.name;
-    const children = item.children;
-    const description = item.description;
-    const icon = item.icon;
-    const secondaryIcon = item.secondaryIcon;
-    return {
-        id: item.id || uuidGenerator$1.uuidv4(),
-        name: typeof name === "function" ? name : () => name,
-        isVisible: item.isVisible ? item.isVisible : () => true,
-        isEnabled: item.isEnabled ? item.isEnabled : () => true,
-        isActive: item.isActive,
-        execute: item.execute,
-        children: children
-            ? (env) => {
-                return children
-                    .map((child) => (typeof child === "function" ? child(env) : child))
-                    .flat()
-                    .map(createAction);
-            }
-            : () => [],
-        isReadonlyAllowed: item.isReadonlyAllowed || false,
-        separator: item.separator || false,
-        icon: typeof icon === "function" ? icon : () => icon || "",
-        iconColor: item.iconColor,
-        secondaryIcon: typeof secondaryIcon === "function" ? secondaryIcon : () => secondaryIcon || "",
-        description: typeof description === "function" ? description : () => description || "",
-        textColor: item.textColor,
-        sequence: item.sequence || 0,
-        onStartHover: item.onStartHover,
-        onStopHover: item.onStopHover,
-    };
-}
-
-/**
- * Registry
- *
- * The Registry class is basically just a mapping from a string key to an object.
- * It is really not much more than an object. It is however useful for the
- * following reasons:
- *
- * 1. it let us react and execute code when someone add something to the registry
- *   (for example, the FunctionRegistry subclass this for this purpose)
- * 2. it throws an error when the get operation fails
- * 3. it provides a chained API to add items to the registry.
- */
-class Registry {
-    content = {};
-    /**
-     * Add an item to the registry
-     *
-     * Note that this also returns the registry, so another add method call can
-     * be chained
-     */
-    add(key, value) {
-        this.content[key] = value;
-        return this;
-    }
-    /**
-     * Get an item from the registry
-     */
-    get(key) {
-        /**
-         * Note: key in {} is ~12 times slower than {}[key].
-         * So, we check the absence of key only when the direct access returns
-         * a falsy value. It's done to ensure that the registry can contains falsy values
-         */
-        const content = this.content[key];
-        if (!content) {
-            if (!(key in this.content)) {
-                throw new Error(`Cannot find ${key} in this registry!`);
-            }
-        }
-        return content;
-    }
-    /**
-     * Check if the key is already in the registry
-     */
-    contains(key) {
-        return key in this.content;
-    }
-    /**
-     * Get a list of all elements in the registry
-     */
-    getAll() {
-        return Object.values(this.content);
-    }
-    /**
-     * Get a list of all keys in the registry
-     */
-    getKeys() {
-        return Object.keys(this.content);
-    }
-    /**
-     * Remove an item from the registry
-     */
-    remove(key) {
-        delete this.content[key];
-    }
-}
-
 function getClipboardDataPositions(sheetId, zones) {
     const lefts = new Set(zones.map((z) => z.left));
     const rights = new Set(zones.map((z) => z.right));
@@ -8454,31 +8455,34 @@ class TableClipboardHandler extends AbstractCellClipboardHandler {
             for (let col of columnsIndexes) {
                 const position = { col, row, sheetId };
                 const table = this.getters.getTable(position);
-                if (!table || copiedTablesIds.has(table.id)) {
+                if (!table) {
                     tableCellsInRow.push({});
                     continue;
                 }
                 const coreTable = this.getters.getCoreTable(position);
                 const tableZone = coreTable?.range.zone;
+                let copiedTable = undefined;
                 // Copy whole table
-                if (coreTable && tableZone && zones.some((z) => isZoneInside(tableZone, z))) {
-                    copiedTablesIds.add(coreTable.id);
+                if (!copiedTablesIds.has(table.id) &&
+                    coreTable &&
+                    tableZone &&
+                    zones.some((z) => isZoneInside(tableZone, z))) {
+                    copiedTablesIds.add(table.id);
                     const values = [];
                     for (const col of range(tableZone.left, tableZone.right + 1)) {
                         values.push(this.getters.getFilterHiddenValues({ sheetId, col, row: tableZone.top }));
                     }
-                    tableCellsInRow.push({
-                        table: {
-                            range: coreTable.range.rangeData,
-                            config: coreTable.config,
-                            type: coreTable.type,
-                        },
-                    });
-                }
-                // Copy only style of cell
-                else if (table) {
-                    tableCellsInRow.push({ style: this.getTableStyleToCopy(position) });
+                    copiedTable = {
+                        range: coreTable.range.rangeData,
+                        config: coreTable.config,
+                        type: coreTable.type,
+                    };
                 }
+                tableCellsInRow.push({
+                    table: copiedTable,
+                    style: this.getTableStyleToCopy(position),
+                    isWholeTableCopied: copiedTablesIds.has(table.id),
+                });
             }
         }
         return {
@@ -8559,11 +8563,14 @@ class TableClipboardHandler extends AbstractCellClipboardHandler {
                 tableType: tableCell.table.type,
             });
         }
-        // Do not paste table style if we're inside another table
         // We cannot check for dynamic tables, because at this point the paste can have changed the evaluation, and the
         // dynamic tables are not yet computed
-        if (!this.getters.getCoreTable(position)) {
-            if (tableCell.style?.style && options?.pasteOption !== "asValue") {
+        if (this.getters.getCoreTable(position) || options?.pasteOption === "asValue") {
+            return;
+        }
+        if ((!options?.pasteOption && !tableCell.isWholeTableCopied) ||
+            options?.pasteOption === "onlyFormat") {
+            if (tableCell.style?.style) {
                 this.dispatch("UPDATE_CELL", { ...position, style: tableCell.style.style });
             }
             if (tableCell.style?.border) {
@@ -17843,7 +17850,7 @@ function assertDomainLength(domain) {
         throw new EvaluationError(_t("Function PIVOT takes an even number of arguments."));
     }
 }
-function addPivotDependencies(evalContext, coreDefinition) {
+function addPivotDependencies(evalContext, coreDefinition, forMeasures) {
     //TODO This function can be very costly when used with PIVOT.VALUE and PIVOT.HEADER
     const dependencies = [];
     if (coreDefinition.type === "SPREADSHEET" && coreDefinition.dataSet) {
@@ -17855,7 +17862,7 @@ function addPivotDependencies(evalContext, coreDefinition) {
         }
         dependencies.push(range);
     }
-    for (const measure of coreDefinition.measures) {
+    for (const measure of forMeasures) {
         if (measure.computedBy) {
             const formula = evalContext.getters.getMeasureCompiledFormula(measure);
             dependencies.push(...formula.dependencies.filter((range) => !range.invalidXc));
@@ -18288,7 +18295,7 @@ const PIVOT_VALUE = {
         assertDomainLength(domainArgs);
         const pivot = this.getters.getPivot(pivotId);
         const coreDefinition = this.getters.getPivotCoreDefinition(pivotId);
-        addPivotDependencies(this, coreDefinition);
+        addPivotDependencies(this, coreDefinition, coreDefinition.measures.filter((m) => m.id === _measure));
         pivot.init({ reload: pivot.needsReevaluation });
         const error = pivot.assertIsValid({ throwOnError: false });
         if (error) {
@@ -18318,7 +18325,7 @@ const PIVOT_HEADER = {
         assertDomainLength(domainArgs);
         const pivot = this.getters.getPivot(_pivotId);
         const coreDefinition = this.getters.getPivotCoreDefinition(_pivotId);
-        addPivotDependencies(this, coreDefinition);
+        addPivotDependencies(this, coreDefinition, []);
         pivot.init({ reload: pivot.needsReevaluation });
         const error = pivot.assertIsValid({ throwOnError: false });
         if (error) {
@@ -18369,7 +18376,7 @@ const PIVOT = {
         const pivotId = getPivotId(_pivotFormulaId, this.getters);
         const pivot = this.getters.getPivot(pivotId);
         const coreDefinition = this.getters.getPivotCoreDefinition(pivotId);
-        addPivotDependencies(this, coreDefinition);
+        addPivotDependencies(this, coreDefinition, coreDefinition.measures);
         pivot.init({ reload: pivot.needsReevaluation });
         const error = pivot.assertIsValid({ throwOnError: false });
         if (error) {
@@ -21578,6 +21585,7 @@ function insertTokenAfterArgSeparator(tokenAtCursor, value) {
         // replace the whole token
         start = tokenAtCursor.start;
     }
+    this.composer.stopComposerRangeSelection();
     this.composer.changeComposerCursorSelection(start, end);
     this.composer.replaceComposerCursorSelection(value);
 }
@@ -21595,6 +21603,7 @@ function insertTokenAfterLeftParenthesis(tokenAtCursor, value) {
         // replace the whole token
         start = tokenAtCursor.start;
     }
+    this.composer.stopComposerRangeSelection();
     this.composer.changeComposerCursorSelection(start, end);
     this.composer.replaceComposerCursorSelection(value);
 }
@@ -22187,6 +22196,27 @@ autofillModifiersRegistry
             tooltip: content ? { props: { content: tooltipValue } } : undefined,
         };
     },
+})
+    .add("DATE_INCREMENT_MODIFIER", {
+    apply: (rule, data, getters) => {
+        const date = toJsDate(rule.current, getters.getLocale());
+        date.setFullYear(date.getFullYear() + rule.increment.years || 0);
+        date.setMonth(date.getMonth() + rule.increment.months || 0);
+        date.setDate(date.getDate() + rule.increment.days || 0);
+        const value = jsDateToNumber(date);
+        rule.current = value;
+        const locale = getters.getLocale();
+        const tooltipValue = formatValue(value, { format: data.cell?.format, locale });
+        return {
+            cellData: {
+                border: data.border,
+                style: data.cell && data.cell.style,
+                format: data.cell && data.cell.format,
+                content: value.toString(),
+            },
+            tooltip: value ? { props: { content: tooltipValue } } : undefined,
+        };
+    },
 })
     .add("COPY_MODIFIER", {
     apply: (rule, data, getters) => {
@@ -22267,7 +22297,9 @@ function getGroup(cell, cells, filter) {
         if (x === cell) {
             found = true;
         }
-        const cellValue = x === undefined || x.isFormula ? undefined : evaluateLiteral(x, { locale: DEFAULT_LOCALE });
+        const cellValue = x === undefined || x.isFormula
+            ? undefined
+            : evaluateLiteral(x, { locale: DEFAULT_LOCALE, format: x.format });
         if (cellValue && filter(cellValue)) {
             group.push(cellValue);
         }
@@ -22303,6 +22335,72 @@ function calculateIncrementBasedOnGroup(group) {
     }
     return increment;
 }
+/**
+ * Iterates on a list of date intervals.
+ * if every interval is the same, return the interval
+ * Otherwise return undefined
+ *
+ */
+function getEqualInterval(intervals) {
+    if (intervals.length < 2) {
+        return intervals[0] || { years: 0, months: 0, days: 0 };
+    }
+    const equal = intervals.every((interval) => interval.years === intervals[0].years &&
+        interval.months === intervals[0].months &&
+        interval.days === intervals[0].days);
+    return equal ? intervals[0] : undefined;
+}
+/**
+ * Based on a group of dates, calculate the increment that should be applied
+ * to the next date.
+ *
+ * This will compute the date difference in calendar terms (years, months, days)
+ * In order to make abstraction of leap years and months with different number of days.
+ *
+ * In case the dates are not equidistant in calendar terms, no rule can be extrapolated
+ * In case of equidistant dates, we either have in that order:
+ *  - exact date interval (e.g. +n year OR +n month OR +n day) in which case we increment by the same interval
+ *  - exact day interval (e.g. +n days) in which case we increment by the same day interval
+ *  - equidistant dates but not the same interval, in which case we return increment of the same interval
+ *
+ * */
+function calculateDateIncrementBasedOnGroup(group) {
+    if (group.length < 2) {
+        return 1;
+    }
+    const jsDates = group.map((date) => toJsDate(date, DEFAULT_LOCALE));
+    const datesIntervals = getDateIntervals(jsDates);
+    const datesEquidistantInterval = getEqualInterval(datesIntervals);
+    if (datesEquidistantInterval === undefined) {
+        // dates are not equidistant in terms of years, months or days, thus no rule can be extrapolated
+        return undefined;
+    }
+    // The dates are apart by an exact interval of years, months or days
+    // but not a combination of them
+    const exactDateInterval = Object.values(datesEquidistantInterval).filter((value) => value !== 0).length === 1;
+    const isSameDay = Object.values(datesEquidistantInterval).every((el) => el === 0); // handles time values (strict decimals)
+    if (!exactDateInterval || isSameDay) {
+        const timeIntervals = jsDates
+            .map((date, index) => {
+            if (index === 0) {
+                return 0;
+            }
+            const previous = jsDates[index - 1];
+            const days = Math.floor(date.getTime()) - Math.floor(previous.getTime());
+            return days;
+        })
+            .slice(1);
+        const equidistantDates = timeIntervals.every((interval) => interval === timeIntervals[0]);
+        if (equidistantDates) {
+            return group.length * (group[1] - group[0]);
+        }
+    }
+    return {
+        years: datesEquidistantInterval.years * group.length,
+        months: datesEquidistantInterval.months * group.length,
+        days: datesEquidistantInterval.days * group.length,
+    };
+}
 autofillRulesRegistry
     .add("simple_value_copy", {
     condition: (cell, cells) => {
@@ -22350,12 +22448,47 @@ autofillRulesRegistry
         return { type: "FORMULA_MODIFIER", increment: cells.length, current: 0 };
     },
     sequence: 30,
+})
+    .add("increment_dates", {
+    condition: (cell, cells) => {
+        return (!cell.isFormula &&
+            evaluateLiteral(cell, { locale: DEFAULT_LOCALE }).type === CellValueType.number &&
+            !!cell.format &&
+            isDateTimeFormat(cell.format));
+    },
+    generateRule: (cell, cells) => {
+        const group = getGroup(cell, cells, (evaluatedCell) => evaluatedCell.type === CellValueType.number &&
+            !!evaluatedCell.format &&
+            isDateTimeFormat(evaluatedCell.format)).map((cell) => Number(cell.value));
+        const increment = calculateDateIncrementBasedOnGroup(group);
+        if (increment === undefined) {
+            return { type: "COPY_MODIFIER" };
+        }
+        /**  requires to detect the current date (requires to be an integer value  with the right format)
+         * detect  if year  or if month or if day then extrapolate increment required (+1 month, +1 year + 1 day)
+         */
+        const evaluation = evaluateLiteral(cell, { locale: DEFAULT_LOCALE });
+        if (typeof increment === "object") {
+            return {
+                type: "DATE_INCREMENT_MODIFIER",
+                increment,
+                current: evaluation.type === CellValueType.number ? evaluation.value : 0,
+            };
+        }
+        return {
+            type: "INCREMENT_MODIFIER",
+            increment,
+            current: evaluation.type === CellValueType.number ? evaluation.value : 0,
+        };
+    },
+    sequence: 25,
 })
     .add("increment_number", {
     condition: (cell) => !cell.isFormula &&
         evaluateLiteral(cell, { locale: DEFAULT_LOCALE }).type === CellValueType.number,
     generateRule: (cell, cells) => {
-        const group = getGroup(cell, cells, (evaluatedCell) => evaluatedCell.type === CellValueType.number).map((cell) => Number(cell.value));
+        const group = getGroup(cell, cells, (evaluatedCell) => evaluatedCell.type === CellValueType.number &&
+            !isDateTimeFormat(evaluatedCell.format || "")).map((cell) => Number(cell.value));
         const increment = calculateIncrementBasedOnGroup(group);
         const evaluation = evaluateLiteral(cell, { locale: DEFAULT_LOCALE });
         return {
@@ -22366,6 +22499,37 @@ autofillRulesRegistry
     },
     sequence: 40,
 });
+/**
+ * Returns the date intervals between consecutive dates of an array
+ * in the format of { years: number, months: number, days: number }
+ *
+ * The split is necessary to make abstraction of leap years and
+ * months with different number of days.
+ *
+ * @param dates
+ */
+function getDateIntervals(dates) {
+    if (dates.length < 2) {
+        return [{ years: 0, months: 0, days: 0 }];
+    }
+    const res = dates.map((date, index) => {
+        if (index === 0) {
+            return { years: 0, months: 0, days: 0 };
+        }
+        const previous = DateTime.fromTimestamp(dates[index - 1].getTime());
+        const years = getTimeDifferenceInWholeYears(previous, date);
+        const months = getTimeDifferenceInWholeMonths(previous, date) % 12;
+        previous.setFullYear(previous.getFullYear() + years);
+        previous.setMonth(previous.getMonth() + months);
+        const days = getTimeDifferenceInWholeDays(previous, date);
+        return {
+            years,
+            months,
+            days,
+        };
+    });
+    return res.slice(1);
+}
 
 const cellPopoverRegistry = new Registry();
 
@@ -28900,6 +29064,7 @@ class ComboChart extends AbstractChart {
             ranges.push({
                 ...this.dataSetDesign?.[i],
                 dataRange: this.getters.getRangeString(dataSet.dataRange, targetSheetId || this.sheetId),
+                type: this.dataSetDesign?.[i]?.type ?? (i ? "line" : "bar"),
             });
         }
         return {
@@ -28945,9 +29110,13 @@ class ComboChart extends AbstractChart {
         return new ComboChart(definition, this.sheetId, this.getters);
     }
     static getDefinitionFromContextCreation(context) {
+        const dataSets = (context.range ?? []).map((ds, index) => ({
+            ...ds,
+            type: index ? "line" : "bar",
+        }));
         return {
             background: context.background,
-            dataSets: context.range ?? [],
+            dataSets,
             dataSetsHaveTitle: context.dataSetsHaveTitle ?? false,
             aggregated: context.aggregated,
             legendPosition: context.legendPosition ?? "top",
@@ -28992,7 +29161,6 @@ function createComboChartRuntime(chart, getters) {
     const config = getDefaultChartJsRuntime(chart, labels, fontColor, localeFormat);
     const legend = {
         labels: { color: fontColor },
-        reverse: true,
     };
     if (chart.legendPosition === "none") {
         legend.display = false;
@@ -29060,14 +29228,15 @@ function createComboChartRuntime(chart, getters) {
     for (let [index, { label, data }] of dataSetsValues.entries()) {
         const design = definition.dataSets[index];
         const color = colors.next();
+        const type = design?.type ?? "line";
         const dataset = {
             label: design?.label ?? label,
             data,
             borderColor: color,
             backgroundColor: color,
             yAxisID: design?.yAxisId ?? "y",
-            type: index === 0 ? "bar" : "line",
-            order: -index,
+            type,
+            order: type === "bar" ? dataSetsValues.length + index : index,
         };
         config.data.datasets.push(dataset);
         const trend = definition.dataSets?.[index].trend;
@@ -35296,6 +35465,7 @@ const CHECK_SVG = /*xml*/ `
   <path fill='none' stroke='#FFF' stroke-linecap='round' stroke-linejoin='round' stroke-width='3' d='m6 10 3 3 6-6'/>
 </svg>
 `;
+const CHECKBOX_WIDTH = 14;
 css /* scss */ `
   label.o-checkbox {
     input {
@@ -35303,8 +35473,8 @@ css /* scss */ `
       -webkit-appearance: none;
       -moz-appearance: none;
       border-radius: 0;
-      width: 14px;
-      height: 14px;
+      width: ${CHECKBOX_WIDTH}px;
+      height: ${CHECKBOX_WIDTH}px;
       vertical-align: top;
       box-sizing: border-box;
       outline: none;
@@ -37214,6 +37384,32 @@ class ChartWithAxisDesignPanel extends owl.Component {
     }
 }
 
+class ComboChartDesignPanel extends ChartWithAxisDesignPanel {
+    static template = "o-spreadsheet-ComboChartDesignPanel";
+    seriesTypeChoices = [
+        { value: "bar", label: _t("Bar") },
+        { value: "line", label: _t("Line") },
+    ];
+    updateDataSeriesType(type) {
+        const dataSets = [...this.props.definition.dataSets];
+        if (!dataSets?.[this.state.index]) {
+            return;
+        }
+        dataSets[this.state.index] = {
+            ...dataSets[this.state.index],
+            type,
+        };
+        this.props.updateChart(this.props.figureId, { dataSets });
+    }
+    getDataSeriesType() {
+        const dataSets = this.props.definition.dataSets;
+        if (!dataSets?.[this.state.index]) {
+            return "bar";
+        }
+        return dataSets[this.state.index].type ?? "line";
+    }
+}
+
 class GaugeChartConfigPanel extends owl.Component {
     static template = "o-spreadsheet-GaugeChartConfigPanel";
     static components = { ChartErrorSection, ChartDataSeries };
@@ -37640,7 +37836,7 @@ chartSidePanelComponentRegistry
 })
     .add("combo", {
     configuration: GenericChartConfigPanel,
-    design: ChartWithAxisDesignPanel,
+    design: ComboChartDesignPanel,
 })
     .add("pie", {
     configuration: GenericChartConfigPanel,
@@ -42819,8 +43015,9 @@ const EMPTY_PIVOT_CELL = { type: "EMPTY" };
  * This function converts a list of data entry into a spreadsheet pivot table.
  */
 function dataEntriesToSpreadsheetPivotTable(dataEntries, definition) {
+    const measureIds = definition.measures.filter((measure) => !measure.isHidden).map((m) => m.id);
     const columnsTree = dataEntriesToColumnsTree(dataEntries, definition.columns, 0);
-    computeWidthOfColumnsNodes(columnsTree, definition.measures.length);
+    computeWidthOfColumnsNodes(columnsTree, measureIds.length);
     const cols = columnsTreeToColumns(columnsTree, definition);
     const rows = dataEntriesToRows(dataEntries, 0, definition.rows, [], []);
     // Add the total row
@@ -42829,7 +43026,6 @@ function dataEntriesToSpreadsheetPivotTable(dataEntries, definition) {
         values: [],
         indent: 0,
     });
-    const measureIds = definition.measures.filter((measure) => !measure.isHidden).map((m) => m.id);
     const fieldsType = {};
     for (const columns of definition.columns) {
         fieldsType[columns.fieldName] = columns.type;
@@ -43590,7 +43786,7 @@ pivotRegistry.add("SPREADSHEET", {
     onIterationEndEvaluation: (pivot) => pivot.markAsDirtyForEvaluation(),
     dateGranularities: [...dateGranularities],
     datetimeGranularities: [...dateGranularities, "hour_number", "minute_number", "second_number"],
-    isMeasureCandidate: (field) => !["date", "boolean"].includes(field.type),
+    isMeasureCandidate: (field) => !["datetime", "boolean"].includes(field.type),
     isGroupable: () => true,
 });
 
@@ -46119,13 +46315,10 @@ class GridCellIcon extends owl.Component {
     }
 }
 
-const CHECKBOX_WIDTH = 15;
 const MARGIN = (GRID_ICON_EDGE_LENGTH - CHECKBOX_WIDTH) / 2;
 css /* scss */ `
   .o-dv-checkbox {
     box-sizing: border-box !important;
-    width: ${CHECKBOX_WIDTH}px;
-    height: ${CHECKBOX_WIDTH}px;
     accent-color: #808080;
     margin: ${MARGIN}px;
     /** required to prevent the checkbox position to be sensible to the font-size (affects Firefox) */
@@ -46134,13 +46327,15 @@ css /* scss */ `
 `;
 class DataValidationCheckbox extends owl.Component {
     static template = "o-spreadsheet-DataValidationCheckbox";
+    static components = {
+        Checkbox,
+    };
     static props = {
         cellPosition: Object,
     };
-    onCheckboxChange(ev) {
-        const newValue = ev.target.checked;
+    onCheckboxChange(value) {
         const { sheetId, col, row } = this.props.cellPosition;
-        const cellContent = newValue ? "TRUE" : "FALSE";
+        const cellContent = value ? "TRUE" : "FALSE";
         this.env.model.dispatch("UPDATE_CELL", { sheetId, col, row, content: cellContent });
     }
     get checkBoxValue() {
@@ -46954,7 +47149,11 @@ class GridAddRowsFooter extends owl.Component {
 class PaintFormatStore extends SpreadsheetStore {
     mutators = ["activate", "cancel", "pasteFormat"];
     highlightStore = this.get(HighlightStore);
-    cellClipboardHandler = new CellClipboardHandler(this.getters, this.model.dispatch);
+    clipboardHandlers = [
+        new CellClipboardHandler(this.getters, this.model.dispatch),
+        new BorderClipboardHandler(this.getters, this.model.dispatch),
+        new TableClipboardHandler(this.getters, this.model.dispatch),
+    ];
     status = "inactive";
     copiedData;
     constructor(get) {
@@ -46975,10 +47174,12 @@ class PaintFormatStore extends SpreadsheetStore {
     pasteFormat(target) {
         if (this.copiedData) {
             const sheetId = this.getters.getActiveSheetId();
-            this.cellClipboardHandler.paste({ zones: target, sheetId }, this.copiedData, {
-                isCutOperation: false,
-                pasteOption: "onlyFormat",
-            });
+            for (const handler of this.clipboardHandlers) {
+                handler.paste({ zones: target, sheetId }, this.copiedData, {
+                    isCutOperation: false,
+                    pasteOption: "onlyFormat",
+                });
+            }
         }
         if (this.status === "oneOff") {
             this.cancel();
@@ -46990,7 +47191,11 @@ class PaintFormatStore extends SpreadsheetStore {
     copyFormats() {
         const sheetId = this.getters.getActiveSheetId();
         const zones = this.getters.getSelectedZones();
-        return this.cellClipboardHandler.copy(getClipboardDataPositions(sheetId, zones));
+        const copiedData = {};
+        for (const handler of this.clipboardHandlers) {
+            Object.assign(copiedData, handler.copy(getClipboardDataPositions(sheetId, zones)));
+        }
+        return copiedData;
     }
     get highlights() {
         const data = this.copiedData;
@@ -59052,7 +59257,7 @@ function withPivotPresentationLayer (PivotClass) {
             const ranking = {};
             const mainDimension = getFieldDimensionType(this, fieldNameWithGranularity);
             const secondaryDimension = mainDimension === "row" ? "column" : "row";
-            let pivotCells = this.getPivotValueCells();
+            let pivotCells = this.getPivotValueCells(measure.id);
             if (mainDimension === "column") {
                 // Transpose the pivot cells so we can do the same operations on the columns as on the rows
                 // This means that we need to transpose back the ranking at the end
@@ -59096,7 +59301,7 @@ function withPivotPresentationLayer (PivotClass) {
             const cellsRunningTotals = {};
             const mainDimension = getFieldDimensionType(this, fieldNameWithGranularity);
             const secondaryDimension = mainDimension === "row" ? "column" : "row";
-            let pivotCells = this.getPivotValueCells();
+            let pivotCells = this.getPivotValueCells(measure.id);
             if (mainDimension === "column") {
                 // Transpose the pivot cells so we can do the same operations on the columns as on the rows
                 // This means that we need to transpose back the totals at the end
@@ -59175,10 +59380,10 @@ function withPivotPresentationLayer (PivotClass) {
             const comparedValueNumber = this.strictMeasureValueToNumber(comparedValue);
             return comparedValueNumber;
         }
-        getPivotValueCells() {
+        getPivotValueCells(measureId) {
             return this.getTableStructure()
                 .getPivotCells()
-                .map((col) => col.filter((cell) => cell.type === "VALUE"))
+                .map((col) => col.filter((cell) => cell.type === "VALUE" && cell.measure === measureId))
                 .filter((col) => col.length > 0);
         }
         measureValueToNumber(result) {
@@ -60946,6 +61151,7 @@ class Session extends EventBus {
             case "REMOTE_REVISION":
             case "REVISION_REDONE":
             case "REVISION_UNDONE":
+            case "SNAPSHOT_CREATED":
                 return this.processedRevisions.has(message.nextRevisionId);
             default:
                 return false;
@@ -61508,7 +61714,7 @@ class InsertPivotPlugin extends UIPlugin {
         const position = this.getters.getSheetIds().indexOf(activeSheetId) + 1;
         const formulaId = this.getters.getPivotFormulaId(newPivotId);
         const newPivotName = this.getters.getPivotName(newPivotId);
-        this.dispatch("CREATE_SHEET", {
+        const result = this.dispatch("CREATE_SHEET", {
             sheetId: newSheetId,
             name: this.getPivotDuplicateSheetName(_t("%(newPivotName)s (Pivot #%(formulaId)s)", {
                 newPivotName,
@@ -61516,20 +61722,19 @@ class InsertPivotPlugin extends UIPlugin {
             })),
             position,
         });
-        this.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: newSheetId });
-        this.dispatch("UPDATE_CELL", {
-            sheetId: newSheetId,
-            col: 0,
-            row: 0,
-            content: `=PIVOT(${formulaId})`,
-        });
+        if (result.isSuccessful) {
+            this.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: newSheetId });
+            const pivot = this.getters.getPivot(pivotId);
+            this.insertPivotWithTable(newSheetId, 0, 0, newPivotId, pivot.getTableStructure().export(), "dynamic");
+        }
     }
     getPivotDuplicateSheetName(pivotName) {
         let i = 1;
         const names = this.getters.getSheetIds().map((id) => this.getters.getSheetName(id));
-        let name = pivotName;
+        const sanitizedName = pivotName.replace(new RegExp(FORBIDDEN_IN_EXCEL_REGEX, "g"), " ");
+        let name = sanitizedName;
         while (names.includes(name)) {
-            name = `${pivotName} (${i})`;
+            name = `${sanitizedName} (${i})`;
             i++;
         }
         return name;
@@ -70882,6 +71087,15 @@ function addTableColumns(table, sheetData) {
             ["id", i + 1], // id cannot be 0
             ["name", colName],
         ];
+        if (table.config.totalRow) {
+            // Note: To be 100% complete, we could also add a `totalsRowLabel` attribute for total strings, and a tag
+            // `<totalsRowFormula>` for the formula of the total. But those doesn't seem to be mandatory for Excel.
+            const colTotalXc = toXC(tableZone.left + i, tableZone.bottom);
+            const colTotalContent = sheetData.cells[colTotalXc]?.content;
+            if (colTotalContent?.startsWith("=")) {
+                colAttributes.push(["totalsRowFunction", "custom"]);
+            }
+        }
         columns.push(escapeXml /*xml*/ `<tableColumn ${formatAttributes(colAttributes)}/>`);
     }
     return escapeXml /*xml*/ `
@@ -70976,8 +71190,9 @@ function addRows(construct, data, sheet) {
                 }
                 else if (cell.content && cell.content !== "") {
                     const isTableHeader = isCellTableHeader(c, r, sheet);
+                    const isTableTotal = isCellTableTotal(c, r, sheet);
                     const isPlainText = !!(cell.format && isTextFormat(data.formats[cell.format]));
-                    ({ attrs: additionalAttrs, node: cellNode } = addContent(cell.content, construct.sharedStrings, isTableHeader || isPlainText));
+                    ({ attrs: additionalAttrs, node: cellNode } = addContent(cell.content, construct.sharedStrings, isTableHeader || isTableTotal || isPlainText));
                 }
                 attributes.push(...additionalAttrs);
                 // prettier-ignore
@@ -71011,6 +71226,16 @@ function isCellTableHeader(col, row, sheet) {
         return isInside(col, row, headerZone);
     });
 }
+function isCellTableTotal(col, row, sheet) {
+    return sheet.tables.some((table) => {
+        if (!table.config.totalRow) {
+            return false;
+        }
+        const zone = toZone(table.range);
+        const totalZone = { ...zone, top: zone.bottom };
+        return isInside(col, row, totalZone);
+    });
+}
 function addHyperlinks(construct, data, sheetIndex) {
     const sheet = data.sheets[sheetIndex];
     const cells = sheet.cells;
@@ -71501,7 +71726,6 @@ class Model extends EventBus {
             isReadonly: () => this.config.mode === "readonly" || this.config.mode === "dashboard",
             isDashboard: () => this.config.mode === "dashboard",
         };
-        this.uuidGenerator.setIsFastStrategy(true);
         // Initiate stream processor
         this.selection = new SelectionStreamProcessorImpl(this.getters);
         this.coreHandlers.push(this.range);
@@ -72175,6 +72399,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.0.0";
-__info__.date = "2024-09-25T12:54:19.974Z";
-__info__.hash = "cee2e47";
+__info__.version = "18.0.1";
+__info__.date = "2024-10-14T07:54:24.768Z";
+__info__.hash = "1771f68";