@odoo/o-spreadsheet 16.2.7 → 16.2.9

package/dist/o-spreadsheet.iife.js

@@ -1723,27 +1723,36 @@
       }
       return numberString.slice(0, i + 1) || undefined;
   }
+  const leadingZeroesRegexp = /^0+/;
   /**
    * Limit the size of the decimal part of a number to the given number of digits.
    */
   function limitDecimalDigits(decimalDigits, maxDecimals) {
+      var _a;
       let integerDigits = "0";
       let resultDecimalDigits = decimalDigits;
       // Note : we'd want to simply use number.toFixed() to handle the max digits & rounding,
       // but it has very strange behaviour. Ex: 12.345.toFixed(2) => "12.35", but 1.345.toFixed(2) => "1.34"
       let slicedDecimalDigits = decimalDigits.slice(0, maxDecimals);
       const i = maxDecimals;
-      if (Number(Number(decimalDigits[i]) < 5)) {
+      if (Number(decimalDigits[i]) < 5) {
           return { integerDigits, decimalDigits: slicedDecimalDigits };
       }
       // round up
+      const leadingZeroes = ((_a = slicedDecimalDigits.match(leadingZeroesRegexp)) === null || _a === void 0 ? void 0 : _a[0]) || "";
       const slicedRoundedUp = (Number(slicedDecimalDigits) + 1).toString();
-      if (slicedRoundedUp.length > slicedDecimalDigits.length) {
-          integerDigits = (Number(integerDigits) + 1).toString();
+      const withoutLeadingZeroes = slicedDecimalDigits.slice(leadingZeroes.length);
+      // e.g. carry over from 99 to 100
+      const carryOver = slicedRoundedUp.length > withoutLeadingZeroes.length;
+      if (carryOver && !leadingZeroes) {
+          integerDigits = "1";
           resultDecimalDigits = undefined;
       }
+      else if (carryOver) {
+          resultDecimalDigits = leadingZeroes.slice(0, -1) + slicedRoundedUp;
+      }
       else {
-          resultDecimalDigits = slicedRoundedUp;
+          resultDecimalDigits = leadingZeroes + slicedRoundedUp;
       }
       return { integerDigits, decimalDigits: resultDecimalDigits };
   }
@@ -16860,7 +16869,7 @@
           return !!((_a = this.state.labelsDispatchResult) === null || _a === void 0 ? void 0 : _a.isCancelledBecause(33 /* CommandResult.InvalidLabelRange */));
       }
       onUpdateDataSetsHaveTitle(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               dataSetsHaveTitle: ev.target.checked,
           });
       }
@@ -16872,7 +16881,7 @@
           this.dataSeriesRanges = ranges;
       }
       onDataSeriesConfirmed() {
-          this.state.datasetDispatchResult = this.props.updateChart({
+          this.state.datasetDispatchResult = this.props.updateChart(this.props.figureId, {
               dataSets: this.dataSeriesRanges,
           });
       }
@@ -16884,12 +16893,12 @@
           this.labelRange = ranges[0];
       }
       onLabelRangeConfirmed() {
-          this.state.labelsDispatchResult = this.props.updateChart({
+          this.state.labelsDispatchResult = this.props.updateChart(this.props.figureId, {
               labelRange: this.labelRange,
           });
       }
       onUpdateAggregated(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               aggregated: ev.target.checked,
           });
       }
@@ -16904,12 +16913,12 @@
 
   class BarConfigPanel extends LineBarPieConfigPanel {
       onUpdateStacked(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               stacked: ev.target.checked,
           });
       }
       onUpdateAggregated(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               aggregated: ev.target.checked,
           });
       }
@@ -16950,6 +16959,7 @@
     .o-color-picker-toggler {
       display: flex;
       .o-color-picker-toggler-sign {
+        display: flex;
         margin: auto auto;
         width: 55%;
         height: 55%;
@@ -17252,17 +17262,17 @@
           this.state.fillColorTool = !this.state.fillColorTool;
       }
       updateBackgroundColor(color) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               background: color,
           });
       }
       updateTitle(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               title: ev.target.value,
           });
       }
       updateSelect(attr, ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               [attr]: ev.target.value,
           });
       }
@@ -17300,7 +17310,7 @@
           this.dataRange = ranges[0];
       }
       updateDataRange() {
-          this.state.dataRangeDispatchResult = this.props.updateChart({
+          this.state.dataRangeDispatchResult = this.props.updateChart(this.props.figureId, {
               dataRange: this.dataRange,
           });
       }
@@ -17362,12 +17372,12 @@
       }
       updateBackgroundColor(color) {
           this.state.openedMenu = undefined;
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               background: color,
           });
       }
       updateTitle(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               title: ev.target.value,
           });
       }
@@ -17436,7 +17446,7 @@
           }
       }
       updateSectionRule(sectionRule) {
-          this.state.sectionRuleDispatchResult = this.props.updateChart({
+          this.state.sectionRuleDispatchResult = this.props.updateChart(this.props.figureId, {
               sectionRule,
           });
       }
@@ -17461,17 +17471,17 @@
           return false;
       }
       onUpdateLabelsAsText(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               labelsAsText: ev.target.checked,
           });
       }
       onUpdateStacked(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               stacked: ev.target.checked,
           });
       }
       onUpdateAggregated(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               aggregated: ev.target.checked,
           });
       }
@@ -17512,7 +17522,7 @@
           this.keyValue = ranges[0];
       }
       updateKeyValueRange() {
-          this.state.keyValueDispatchResult = this.props.updateChart({
+          this.state.keyValueDispatchResult = this.props.updateChart(this.props.figureId, {
               keyValue: this.keyValue,
           });
       }
@@ -17520,12 +17530,12 @@
           this.baseline = ranges[0];
       }
       updateBaselineRange() {
-          this.state.baselineDispatchResult = this.props.updateChart({
+          this.state.baselineDispatchResult = this.props.updateChart(this.props.figureId, {
               baseline: this.baseline,
           });
       }
       updateBaselineMode(ev) {
-          this.props.updateChart({ baselineMode: ev.target.value });
+          this.props.updateChart(this.props.figureId, { baselineMode: ev.target.value });
       }
   }
   ScorecardChartConfigPanel.template = "o-spreadsheet-ScorecardChartConfigPanel";
@@ -17547,12 +17557,12 @@
           owl.useExternalListener(window, "click", this.closeMenus);
       }
       updateTitle(ev) {
-          this.props.updateChart({
+          this.props.updateChart(this.props.figureId, {
               title: ev.target.value,
           });
       }
       updateBaselineDescr(ev) {
-          this.props.updateChart({ baselineDescr: ev.target.value });
+          this.props.updateChart(this.props.figureId, { baselineDescr: ev.target.value });
       }
       toggleColorPicker(colorPickerId) {
           if (this.state.openedColorPicker === colorPickerId) {
@@ -17565,13 +17575,13 @@
       setColor(color, colorPickerId) {
           switch (colorPickerId) {
               case "backgroundColor":
-                  this.props.updateChart({ background: color });
+                  this.props.updateChart(this.props.figureId, { background: color });
                   break;
               case "baselineColorDown":
-                  this.props.updateChart({ baselineColorDown: color });
+                  this.props.updateChart(this.props.figureId, { baselineColorDown: color });
                   break;
               case "baselineColorUp":
-                  this.props.updateChart({ baselineColorUp: color });
+                  this.props.updateChart(this.props.figureId, { baselineColorUp: color });
                   break;
           }
           this.closeMenus();
@@ -17636,16 +17646,9 @@
   }
 `;
   class ChartPanel extends owl.Component {
-      constructor() {
-          super(...arguments);
-          this.shouldUpdateChart = true;
-      }
       get figureId() {
           return this.state.figureId;
       }
-      get sheetId() {
-          return this.state.sheetId;
-      }
       setup() {
           const selectedFigureId = this.env.model.getters.getSelectedFigureId();
           if (!selectedFigureId) {
@@ -17654,17 +17657,11 @@
           this.state = owl.useState({
               panel: "configuration",
               figureId: selectedFigureId,
-              sheetId: this.env.model.getters.getActiveSheetId(),
           });
           owl.onWillUpdateProps(() => {
               const selectedFigureId = this.env.model.getters.getSelectedFigureId();
               if (selectedFigureId && selectedFigureId !== this.state.figureId) {
                   this.state.figureId = selectedFigureId;
-                  this.state.sheetId = this.env.model.getters.getActiveSheetId();
-                  this.shouldUpdateChart = false;
-              }
-              else {
-                  this.shouldUpdateChart = true;
               }
               if (!this.env.model.getters.isChartDefined(this.figureId)) {
                   this.props.onCloseSidePanel();
@@ -17672,8 +17669,8 @@
               }
           });
       }
-      updateChart(updateDefinition) {
-          if (!this.shouldUpdateChart) {
+      updateChart(figureId, updateDefinition) {
+          if (figureId !== this.figureId) {
               return;
           }
           const definition = {
@@ -17682,8 +17679,8 @@
           };
           return this.env.model.dispatch("UPDATE_CHART", {
               definition,
-              id: this.figureId,
-              sheetId: this.sheetId,
+              id: figureId,
+              sheetId: this.env.model.getters.getFigureSheetId(figureId),
           });
       }
       onTypeChange(type) {
@@ -17695,7 +17692,7 @@
           this.env.model.dispatch("UPDATE_CHART", {
               definition,
               id: this.figureId,
-              sheetId: this.sheetId,
+              sheetId: this.env.model.getters.getFigureSheetId(this.figureId),
           });
       }
       get chartPanel() {
@@ -19857,6 +19854,7 @@
                       (this.autoCompleteState.selectedIndex + 1) % this.autoCompleteState.values.length;
               }
           }
+          this.updateCursorIfNeeded();
       }
       processTabKey(ev) {
           var _a;
@@ -19927,10 +19925,12 @@
           }
           else {
               ev.stopPropagation();
+              this.updateCursorIfNeeded();
           }
-          const { start, end } = this.contentHelper.getCurrentSelection();
-          if (!this.env.model.getters.isSelectingForComposer() &&
-              !(ev.key === "Enter" && (ev.altKey || ev.ctrlKey))) {
+      }
+      updateCursorIfNeeded() {
+          if (!this.env.model.getters.isSelectingForComposer()) {
+              const { start, end } = this.contentHelper.getCurrentSelection();
               this.env.model.dispatch("CHANGE_COMPOSER_CURSOR_SELECTION", { start, end });
               this.isKeyStillDown = true;
           }
@@ -19988,7 +19988,12 @@
               };
           });
           if (searchTerm) {
-              values = fuzzyLookup(searchTerm, values, (t) => t.text);
+              if (searchTerm.toLowerCase() === "true" || searchTerm.toLowerCase() === "false") {
+                  values = [];
+              }
+              else {
+                  values = fuzzyLookup(searchTerm, values, (t) => t.text);
+              }
           }
           else {
               // alphabetical order
@@ -23065,6 +23070,16 @@
       63: "333333",
       64: "000000",
       65: "FFFFFF", // system background
+  };
+  const IMAGE_MIMETYPE_EXTENSION_MAPPING = {
+      "image/avif": "avif",
+      "image/bmp": "bmp",
+      "image/gif": "gif",
+      "image/vnd.microsoft.icon": "ico",
+      "image/jpeg": "jpeg",
+      "image/png": "png",
+      "image/tiff": "tiff",
+      "image/webp": "webp",
   };
 
   /**
@@ -25501,6 +25516,11 @@
     <Override ContentType="${contentType}" PartName="${partName}" />
   `;
   }
+  function createDefaultXMLElement(extension, contentType) {
+      return escapeXml /*xml*/ `
+    <Default Extension="${extension}" ContentType="${contentType}" />
+  `;
+  }
   function joinXmlNodes(xmlNodes) {
       return new XMLString(xmlNodes.join("\n"));
   }
@@ -39562,7 +39582,7 @@
           const file = await this.getImageFromUser();
           const path = await this.fileStore.upload(file);
           const size = await this.getImageSize(path);
-          return { path, size };
+          return { path, size, mimetype: file.type };
       }
       getImageFromUser() {
           return new Promise((resolve, reject) => {
@@ -44628,8 +44648,7 @@
           // Figures and Charts
           let drawingNode = escapeXml ``;
           const drawingRelIds = [];
-          const charts = sheet.charts;
-          for (const chart of charts) {
+          for (const chart of sheet.charts) {
               const xlsxChartId = convertChartId(chart.id);
               const chartRelId = addRelsToFile(construct.relsFiles, `xl/drawings/_rels/drawing${sheetIndex}.xml.rels`, {
                   target: `../charts/chart${xlsxChartId}.xml`,
@@ -44638,20 +44657,27 @@
               drawingRelIds.push(chartRelId);
               files.push(createXMLFile(createChart(chart, sheetIndex, data), `xl/charts/chart${xlsxChartId}.xml`, "chart"));
           }
-          const images = sheet.images;
-          for (const image of images) {
+          for (const image of sheet.images) {
+              const mimeType = image.data.mimetype;
+              if (mimeType === undefined)
+                  continue;
+              const extension = IMAGE_MIMETYPE_EXTENSION_MAPPING[mimeType];
+              // only support exporting images with mimetypes specified in the mapping
+              if (extension === undefined)
+                  continue;
               const xlsxImageId = convertImageId(image.id);
+              let imageFileName = `image${xlsxImageId}.${extension}`;
               const imageRelId = addRelsToFile(construct.relsFiles, `xl/drawings/_rels/drawing${sheetIndex}.xml.rels`, {
-                  target: `../media/image${xlsxImageId}`,
+                  target: `../media/${imageFileName}`,
                   type: XLSX_RELATION_TYPE.image,
               });
               drawingRelIds.push(imageRelId);
               files.push({
-                  path: `xl/media/image${xlsxImageId}`,
-                  imagePath: image.data.path,
+                  path: `xl/media/${imageFileName}`,
+                  imageSrc: image.data.path,
               });
           }
-          const drawings = [...charts, ...images];
+          const drawings = [...sheet.charts, ...sheet.images];
           if (drawings.length) {
               const drawingRelId = addRelsToFile(construct.relsFiles, `xl/worksheets/_rels/sheet${sheetIndex}.xml.rels`, {
                   target: `../drawings/drawing${sheetIndex}.xml`,
@@ -44770,6 +44796,8 @@
   }
   function createContentTypes(files) {
       const overrideNodes = [];
+      // hard-code supported image mimetypes
+      const imageDefaultNodes = Object.entries(IMAGE_MIMETYPE_EXTENSION_MAPPING).map(([mimetype, extension]) => createDefaultXMLElement(extension, mimetype));
       for (const file of files) {
           if ("contentType" in file && file.contentType) {
               overrideNodes.push(createOverride("/" + file.path, CONTENT_TYPES[file.contentType]));
@@ -44777,6 +44805,7 @@
       }
       const xml = escapeXml /*xml*/ `
     <Types xmlns="${NAMESPACE["Types"]}">
+      ${joinXmlNodes(Object.values(imageDefaultNodes))}
       <Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml" />
       <Default Extension="xml" ContentType="application/xml" />
       ${joinXmlNodes(overrideNodes)}
@@ -45338,9 +45367,9 @@
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.2.7';
-  __info__.date = '2023-05-25T12:56:32.163Z';
-  __info__.hash = '047a6d5';
+  __info__.version = '16.2.9';
+  __info__.date = '2023-06-12T13:49:57.216Z';
+  __info__.hash = '37e2719';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);