ACI0105889 support option allowInvalidFormula and set it by default to true

When even with option the formula could not be set (same in excel) then set it as text and have a better log in js console

Author: e-marchand

Resources/schemas/SJS.json

@@ -187,6 +187,10 @@
       "type": "boolean",
       "default": true
     },
+    "allowInvalidFormula": {
+      "type": "boolean",
+      "default": true
+    },
     "referenceStyle": {
       "$ref": "#/definitions/ReferenceStyle",
       "default": 0

Resources/scripts/commands/values.js

@@ -375,9 +375,17 @@
                 Utils.getRanges(range, instancesArray);
 
                 instancesArray.forEach(i => {
-                    let instance = i.sheet.getRange(i.row, i.column, i.rowCount, i.columnCount);
-
-                    instance.formula(params.formula);
+                    const instance = i.sheet.getRange(i.row, i.column, i.rowCount, i.columnCount);
+                    try {
+                        instance.formula(params.formula);
+                    }
+                    catch(e) {
+                        Utils.logEvent({ type: 'error-catched', data: e, formula: params.formula});
+                        if (Utils.spread.options.allowInvalidFormula == true) {
+                            instance.formula(null);
+                            instance.value("=" + params.formula);
+                        }
+                    }
 
                     if (format != null) {
                         instance.formatter(format);

Resources/scripts/commands/workbook.js

@@ -23,6 +23,7 @@
         'allowCopyPasteExcelStyle',
         'allowDynamicArray',
         'allowExtendPasteRange',
+        'allowInvalidFormula',
         'allowSheetReorder',
         'allowUndo',
         'allowUserDeselect',

Resources/scripts/init.js

@@ -717,6 +717,7 @@ document.addEventListener('DOMContentLoaded', function () {
         // Allow users to enter mathematical formula.
         Utils.spread.options.allowUserEditFormula = true;
         Utils.spread.options.enableFormulaTextbox = true;
+        Utils.spread.options.allowInvalidFormula = true;
 
         if ((vp_localizedFolder !== 'ja.lproj') && (vp_localizedFolder !== 'en.lproj')) {
             let sheetName = vp_spreadJsResources.Sheets.SHEET_NAME;

Resources/scripts/utils.js

@@ -93,17 +93,18 @@ Utils.diagnostics = {
     events: []
 };
 
-Utils.logEvent = function ({ type, data }) {
+Utils.logEvent = function (eventData) {
     if (Utils.diagnostics.events.length >= 500) {
         Utils.diagnostics.events = [];
     }
-
-    Utils.diagnostics.events.push({
+    const event = {
         time: new Date(),
-        type,
-        data
-    });
-}
+        ...eventData
+    };
+
+    Utils.diagnostics.events.push(event);
+    if (Utils.debug || (eventData.type && eventData.type.includes("error"))) console.log(event);
+};
 
 Utils.adjustFormat = function (format) {
     let propertyName = '';