@budibase/server 1.0.152 → 1.0.153

package/coverage/lcov.info

@@ -5064,9 +5064,10 @@ DA:190,48
 DA:196,48
 DA:201,48
 DA:205,48
-DA:212,48
-LF:28
-LH:28
+DA:211,48
+DA:217,48
+LF:29
+LH:29
 BRDA:59,0,0,96
 BRDA:59,0,1,0
 BRF:2
@@ -6505,30 +6506,34 @@ BRH:0
 end_of_record
 TN:
 SF:src/threads/automation.js
-FN:26,(anonymous_0)
-FN:48,(anonymous_1)
-FN:55,(anonymous_2)
-FN:63,(anonymous_3)
-FN:72,(anonymous_4)
-FN:81,(anonymous_5)
-FN:99,(anonymous_6)
-FN:213,(anonymous_7)
-FN:249,(anonymous_8)
-FN:314,(anonymous_9)
-FN:316,(anonymous_10)
-FNF:11
+FN:20,typecastForLooping
+FN:45,getLoopIterations
+FN:61,(anonymous_2)
+FN:83,(anonymous_3)
+FN:90,(anonymous_4)
+FN:98,(anonymous_5)
+FN:107,(anonymous_6)
+FN:116,(anonymous_7)
+FN:134,(anonymous_8)
+FN:236,(anonymous_9)
+FN:267,(anonymous_10)
+FN:332,(anonymous_11)
+FN:334,(anonymous_12)
+FNF:13
 FNH:9
-FNDA:6,(anonymous_0)
-FNDA:6,(anonymous_1)
-FNDA:5,(anonymous_2)
+FNDA:0,typecastForLooping
+FNDA:0,getLoopIterations
+FNDA:6,(anonymous_2)
 FNDA:6,(anonymous_3)
-FNDA:11,(anonymous_4)
-FNDA:0,(anonymous_5)
-FNDA:6,(anonymous_6)
+FNDA:5,(anonymous_4)
+FNDA:6,(anonymous_5)
+FNDA:11,(anonymous_6)
 FNDA:0,(anonymous_7)
-FNDA:5,(anonymous_8)
-FNDA:6,(anonymous_9)
-FNDA:6,(anonymous_10)
+FNDA:6,(anonymous_8)
+FNDA:0,(anonymous_9)
+FNDA:5,(anonymous_10)
+FNDA:6,(anonymous_11)
+FNDA:6,(anonymous_12)
 DA:1,3
 DA:2,3
 DA:3,3
@@ -6546,201 +6551,231 @@ DA:15,3
 DA:16,3
 DA:17,3
 DA:18,3
-DA:27,6
-DA:28,6
-DA:29,6
-DA:30,6
-DA:31,6
-DA:32,6
-DA:34,6
-DA:35,6
-DA:37,6
-DA:38,6
-DA:41,6
-DA:42,6
-DA:44,6
-DA:45,6
-DA:49,6
+DA:21,0
+DA:22,0
+DA:24,0
+DA:25,0
+DA:26,0
+DA:27,0
+DA:29,0
+DA:30,0
+DA:32,0
+DA:34,0
+DA:35,0
+DA:37,0
+DA:40,0
+DA:42,0
+DA:46,0
+DA:47,0
+DA:48,0
 DA:50,0
-DA:52,6
-DA:56,5
-DA:57,5
-DA:58,0
-DA:60,5
+DA:62,6
+DA:63,6
 DA:64,6
-DA:65,0
+DA:65,6
+DA:66,6
 DA:67,6
-DA:68,6
 DA:69,6
-DA:73,11
-DA:75,11
+DA:70,6
+DA:72,6
+DA:73,6
 DA:76,6
-DA:78,11
-DA:82,0
-DA:92,0
-DA:100,6
-DA:101,6
+DA:77,6
+DA:79,6
+DA:80,6
+DA:84,6
+DA:85,0
+DA:87,6
+DA:91,5
+DA:92,5
+DA:93,0
+DA:95,5
+DA:99,6
+DA:100,0
 DA:102,6
 DA:103,6
-DA:105,6
-DA:106,6
-DA:107,6
-DA:108,6
-DA:109,5
-DA:111,5
-DA:112,0
-DA:113,0
-DA:114,0
-DA:117,5
-DA:118,0
-DA:120,5
-DA:125,5
-DA:126,5
-DA:127,5
-DA:130,5
-DA:131,0
-DA:135,0
-DA:140,0
-DA:141,0
-DA:147,0
+DA:104,6
+DA:108,11
+DA:110,11
+DA:111,6
+DA:113,11
+DA:117,0
+DA:127,0
+DA:135,6
+DA:136,6
+DA:137,6
+DA:138,6
+DA:140,6
+DA:141,6
+DA:142,6
+DA:143,6
+DA:144,5
+DA:146,5
+DA:147,5
+DA:148,5
+DA:149,0
+DA:150,0
 DA:151,0
-DA:152,0
-DA:153,0
-DA:157,0
-DA:161,0
-DA:163,0
-DA:166,0
-DA:172,0
+DA:154,5
+DA:155,0
+DA:156,0
+DA:159,5
+DA:160,5
+DA:163,5
+DA:164,0
+DA:169,0
+DA:170,0
+DA:171,0
 DA:173,0
-DA:174,0
 DA:177,0
 DA:178,0
-DA:186,0
+DA:179,0
+DA:183,0
 DA:187,0
-DA:194,0
+DA:189,0
+DA:192,0
 DA:198,0
-DA:202,0
+DA:199,0
+DA:200,0
 DA:203,0
 DA:204,0
-DA:207,0
-DA:208,0
-DA:211,0
-DA:214,0
+DA:212,0
+DA:213,0
 DA:220,0
-DA:225,0
-DA:226,0
+DA:224,0
+DA:228,0
+DA:229,0
 DA:230,0
-DA:231,0
-DA:232,0
-DA:237,5
-DA:238,0
-DA:239,0
-DA:243,5
-DA:244,5
-DA:245,5
-DA:246,5
-DA:248,5
-DA:249,5
-DA:250,5
-DA:257,5
-DA:260,5
-DA:261,0
-DA:262,0
-DA:266,0
+DA:233,0
+DA:234,0
+DA:235,0
+DA:236,0
+DA:237,0
+DA:240,0
+DA:243,0
+DA:244,0
+DA:248,0
+DA:249,0
+DA:250,0
+DA:255,5
+DA:256,0
+DA:257,0
+DA:261,5
+DA:262,5
+DA:263,5
+DA:264,5
+DA:266,5
+DA:267,5
 DA:268,5
-DA:269,0
-DA:271,5
+DA:275,5
+DA:278,5
 DA:279,0
 DA:280,0
-DA:282,5
-DA:283,0
 DA:284,0
-DA:285,0
-DA:286,0
+DA:286,5
 DA:287,0
-DA:292,5
-DA:293,0
+DA:289,5
+DA:297,0
 DA:298,0
+DA:300,5
+DA:301,0
+DA:302,0
+DA:303,0
+DA:304,0
 DA:305,0
-DA:306,0
-DA:310,6
-DA:314,3
-DA:315,6
-DA:316,6
-DA:317,6
-DA:321,6
-DA:322,6
-DA:323,6
-DA:325,0
-LF:143
+DA:310,5
+DA:311,0
+DA:316,0
+DA:323,0
+DA:324,0
+DA:328,6
+DA:332,3
+DA:333,6
+DA:334,6
+DA:335,6
+DA:339,6
+DA:340,6
+DA:341,6
+DA:343,0
+LF:163
 LH:82
-BRDA:26,0,0,0
-BRDA:28,1,0,4
-BRDA:28,1,1,2
-BRDA:49,2,0,0
-BRDA:49,2,1,6
-BRDA:57,3,0,0
-BRDA:57,3,1,5
-BRDA:64,4,0,0
-BRDA:64,4,1,6
-BRDA:75,5,0,6
-BRDA:75,5,1,5
-BRDA:111,6,0,0
-BRDA:111,6,1,5
-BRDA:117,7,0,0
-BRDA:117,7,1,5
-BRDA:120,8,0,0
-BRDA:120,8,1,5
-BRDA:121,9,0,0
-BRDA:121,9,1,0
-BRDA:130,10,0,0
-BRDA:130,10,1,5
-BRDA:141,11,0,0
-BRDA:141,11,1,0
-BRDA:142,12,0,0
-BRDA:142,12,1,0
-BRDA:142,12,2,0
-BRDA:142,12,3,0
-BRDA:157,13,0,0
-BRDA:157,13,1,0
-BRDA:158,14,0,0
-BRDA:158,14,1,0
-BRDA:173,15,0,0
-BRDA:173,15,1,0
-BRDA:177,16,0,0
-BRDA:177,16,1,0
-BRDA:186,17,0,0
-BRDA:186,17,1,0
-BRDA:194,18,0,0
-BRDA:194,18,1,0
-BRDA:195,19,0,0
-BRDA:195,19,1,0
-BRDA:208,20,0,0
-BRDA:208,20,1,0
-BRDA:225,21,0,0
-BRDA:225,21,1,0
-BRDA:237,22,0,0
-BRDA:237,22,1,5
-BRDA:248,23,0,5
-BRDA:248,23,1,0
-BRDA:260,24,0,0
-BRDA:260,24,1,5
-BRDA:260,25,0,5
-BRDA:260,25,1,0
-BRDA:268,26,0,0
-BRDA:268,26,1,5
-BRDA:268,27,0,5
-BRDA:268,27,1,0
-BRDA:282,28,0,0
-BRDA:282,28,1,5
-BRDA:284,29,0,0
-BRDA:284,29,1,0
-BRDA:292,30,0,0
-BRDA:292,30,1,5
-BRDA:292,31,0,5
-BRDA:292,31,1,5
-BRF:65
-BRH:21
+BRDA:21,0,0,0
+BRDA:21,0,1,0
+BRDA:21,1,0,0
+BRDA:21,1,1,0
+BRDA:27,2,0,0
+BRDA:27,2,1,0
+BRDA:29,3,0,0
+BRDA:29,3,1,0
+BRDA:34,4,0,0
+BRDA:34,4,1,0
+BRDA:47,5,0,0
+BRDA:47,5,1,0
+BRDA:47,6,0,0
+BRDA:47,6,1,0
+BRDA:50,7,0,0
+BRDA:50,7,1,0
+BRDA:61,8,0,0
+BRDA:63,9,0,4
+BRDA:63,9,1,2
+BRDA:84,10,0,0
+BRDA:84,10,1,6
+BRDA:92,11,0,0
+BRDA:92,11,1,5
+BRDA:99,12,0,0
+BRDA:99,12,1,6
+BRDA:110,13,0,6
+BRDA:110,13,1,5
+BRDA:148,14,0,0
+BRDA:148,14,1,5
+BRDA:154,15,0,0
+BRDA:154,15,1,5
+BRDA:163,16,0,0
+BRDA:163,16,1,5
+BRDA:183,17,0,0
+BRDA:183,17,1,0
+BRDA:184,18,0,0
+BRDA:184,18,1,0
+BRDA:199,19,0,0
+BRDA:199,19,1,0
+BRDA:203,20,0,0
+BRDA:203,20,1,0
+BRDA:212,21,0,0
+BRDA:212,21,1,0
+BRDA:220,22,0,0
+BRDA:220,22,1,0
+BRDA:221,23,0,0
+BRDA:221,23,1,0
+BRDA:235,24,0,0
+BRDA:235,24,1,0
+BRDA:235,25,0,0
+BRDA:235,25,1,0
+BRDA:240,26,0,0
+BRDA:240,26,1,0
+BRDA:243,27,0,0
+BRDA:243,27,1,0
+BRDA:255,28,0,0
+BRDA:255,28,1,5
+BRDA:266,29,0,5
+BRDA:266,29,1,0
+BRDA:278,30,0,0
+BRDA:278,30,1,5
+BRDA:278,31,0,5
+BRDA:278,31,1,0
+BRDA:286,32,0,0
+BRDA:286,32,1,5
+BRDA:286,33,0,5
+BRDA:286,33,1,0
+BRDA:300,34,0,0
+BRDA:300,34,1,5
+BRDA:302,35,0,0
+BRDA:302,35,1,0
+BRDA:310,36,0,0
+BRDA:310,36,1,5
+BRDA:310,37,0,5
+BRDA:310,37,1,5
+BRF:75
+BRH:20
 end_of_record
 TN:
 SF:src/threads/query.js

package/dist/constants/index.js

@@ -185,5 +185,9 @@ exports.AutomationErrors = {
     MAX_ITERATIONS: "MAX_ITERATIONS_REACHED",
     FAILURE_CONDITION: "FAILURE_CONDITION_MET",
 };
+exports.LoopStepTypes = {
+    ARRAY: "Array",
+    STRING: "String",
+};
 // pass through the list from the auth/core lib
 exports.ObjectStoreBuckets = ObjectStoreBuckets;

package/dist/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@budibase/server",
     "email": "hi@budibase.com",
-    "version": "1.0.151",
+    "version": "1.0.152",
     "description": "Budibase Web Server",
     "main": "src/index.ts",
     "repository": {
@@ -69,10 +69,10 @@
     "license": "GPL-3.0",
     "dependencies": {
         "@apidevtools/swagger-parser": "^10.0.3",
-        "@budibase/backend-core": "^1.0.151",
-        "@budibase/client": "^1.0.151",
-        "@budibase/pro": "1.0.150",
-        "@budibase/string-templates": "^1.0.151",
+        "@budibase/backend-core": "^1.0.152",
+        "@budibase/client": "^1.0.152",
+        "@budibase/pro": "1.0.152",
+        "@budibase/string-templates": "^1.0.152",
         "@bull-board/api": "^3.7.0",
         "@bull-board/koa": "^3.7.0",
         "@elastic/elasticsearch": "7.10.0",

package/dist/threads/automation.js

@@ -18,13 +18,46 @@ const { DocumentTypes } = require("../db/utils");
 const { doInTenant } = require("@budibase/backend-core/tenancy");
 const { definitions: triggerDefs } = require("../automations/triggerInfo");
 const { doInAppContext, getAppDB } = require("@budibase/backend-core/context");
-const { AutomationErrors } = require("../constants");
+const { AutomationErrors, LoopStepTypes } = require("../constants");
 const FILTER_STEP_ID = actions.ACTION_DEFINITIONS.FILTER.stepId;
 const LOOP_STEP_ID = actions.ACTION_DEFINITIONS.LOOP.stepId;
 const CRON_STEP_ID = triggerDefs.CRON.stepId;
 const STOPPED_STATUS = { success: false, status: "STOPPED" };
 const { cloneDeep } = require("lodash/fp");
 const env = require("../environment");
+function typecastForLooping(loopStep, input) {
+    if (!input || !input.binding) {
+        return null;
+    }
+    const isArray = Array.isArray(input.binding), isString = typeof input.binding === "string";
+    try {
+        switch (loopStep.inputs.option) {
+            case LoopStepTypes.ARRAY:
+                if (isString) {
+                    return JSON.parse(input.binding);
+                }
+                break;
+            case LoopStepTypes.STRING:
+                if (isArray) {
+                    return input.binding.join(",");
+                }
+                break;
+        }
+    }
+    catch (err) {
+        throw new Error("Unable to cast to correct type");
+    }
+    return input.binding;
+}
+function getLoopIterations(loopStep, input) {
+    const binding = typecastForLooping(loopStep, input);
+    if (!loopStep || !binding) {
+        return 1;
+    }
+    return Array.isArray(binding)
+        ? binding.length
+        : automationUtils.stringSplit(binding).length;
+}
 /**
  * The automation orchestrator is a class responsible for executing automations.
  * It handles the context of the automation and makes sure each step gets the correct
@@ -95,7 +128,7 @@ class Orchestrator {
         this._context.steps.splice(loopStepNumber, 0, Object.assign(Object.assign({}, output), { success: result.success, status: result.status }));
     }
     execute() {
-        var _a, _b;
+        var _a;
         return __awaiter(this, void 0, void 0, function* () {
             let automation = this._automation;
             const app = yield this.getApp();
@@ -106,7 +139,7 @@ class Orchestrator {
             let loopSteps = [];
             for (let step of automation.definition.steps) {
                 stepCount++;
-                let input;
+                let input, iterations = 1, iterationCount = 0;
                 if (step.stepId === LOOP_STEP_ID) {
                     loopStep = step;
                     loopStepNumber = stepCount;
@@ -114,24 +147,18 @@ class Orchestrator {
                 }
                 if (loopStep) {
                     input = yield processObject(loopStep.inputs, this._context);
+                    iterations = getLoopIterations(loopStep, input);
                 }
-                let iterations = loopStep
-                    ? Array.isArray(input.binding)
-                        ? input.binding.length
-                        : automationUtils.stringSplit(input.binding).length
-                    : 1;
-                let iterationCount = 0;
                 for (let index = 0; index < iterations; index++) {
                     let originalStepInput = cloneDeep(step.inputs);
                     // Handle if the user has set a max iteration count or if it reaches the max limit set by us
                     if (loopStep) {
                         let newInput = yield processObject(loopStep.inputs, cloneDeep(this._context));
-                        newInput = automationUtils.cleanInputValues(newInput, loopStep.schema.inputs);
                         let tempOutput = { items: loopSteps, iterations: iterationCount };
-                        if ((loopStep.inputs.option === "Array" &&
-                            !Array.isArray(newInput.binding)) ||
-                            (loopStep.inputs.option === "String" &&
-                                typeof newInput.binding !== "string")) {
+                        try {
+                            newInput.binding = typecastForLooping(loopStep, newInput);
+                        }
+                        catch (err) {
                             this.updateContextAndOutput(loopStepNumber, step, tempOutput, {
                                 status: AutomationErrors.INCORRECT_TYPE,
                                 success: false,
@@ -179,16 +206,14 @@ class Orchestrator {
                             break;
                         }
                         let isFailure = false;
-                        if (typeof ((_a = this._context.steps[loopStepNumber]) === null || _a === void 0 ? void 0 : _a.currentItem) === "object") {
-                            isFailure = Object.keys(this._context.steps[loopStepNumber].currentItem).some(value => {
-                                return (this._context.steps[loopStepNumber].currentItem[value] ===
-                                    loopStep.inputs.failure);
+                        const currentItem = (_a = this._context.steps[loopStepNumber]) === null || _a === void 0 ? void 0 : _a.currentItem;
+                        if (currentItem && typeof currentItem === "object") {
+                            isFailure = Object.keys(currentItem).some(value => {
+                                return currentItem[value] === loopStep.inputs.failure;
                             });
                         }
                         else {
-                            isFailure =
-                                ((_b = this._context.steps[loopStepNumber]) === null || _b === void 0 ? void 0 : _b.currentItem) ===
-                                    loopStep.inputs.failure;
+                            isFailure = currentItem && currentItem === loopStep.inputs.failure;
                         }
                         if (isFailure) {
                             this.updateContextAndOutput(loopStepNumber, step, tempOutput, {