@budibase/server 1.0.49-alpha.8 → 1.0.50

package/dist/api/controllers/user.js

@@ -16,7 +16,7 @@ const { isEqual } = require("lodash");
 const { BUILTIN_ROLE_IDS } = require("@budibase/backend-core/roles");
 const { getDevelopmentAppID, getProdAppIDs, dbExists, } = require("@budibase/backend-core/db");
 const { UserStatus } = require("@budibase/backend-core/constants");
-const { getAppDB } = require("@budibase/backend-core/context");
+const { getAppDB, doInAppContext } = require("@budibase/backend-core/context");
 function rawMetadata() {
     return __awaiter(this, void 0, void 0, function* () {
         const db = getAppDB();
@@ -97,31 +97,33 @@ exports.syncUser = function (ctx) {
                 if (!(yield dbExists(appId))) {
                     continue;
                 }
-                const db = getAppDB();
-                const metadataId = generateUserMetadataID(userId);
-                let metadata;
-                try {
-                    metadata = yield db.get(metadataId);
-                }
-                catch (err) {
-                    if (deleting) {
-                        continue;
+                yield doInAppContext(appId, () => __awaiter(this, void 0, void 0, function* () {
+                    const db = getAppDB();
+                    const metadataId = generateUserMetadataID(userId);
+                    let metadata;
+                    try {
+                        metadata = yield db.get(metadataId);
                     }
-                    metadata = {
-                        tableId: InternalTables.USER_METADATA,
-                    };
-                }
-                // assign the roleId for the metadata doc
-                if (roleId) {
-                    metadata.roleId = roleId;
-                }
-                let combined = !deleting
-                    ? combineMetadataAndUser(user, metadata)
-                    : Object.assign(Object.assign({}, metadata), { status: UserStatus.INACTIVE, metadata: BUILTIN_ROLE_IDS.PUBLIC });
-                // if its null then there was no updates required
-                if (combined) {
-                    yield db.put(combined);
-                }
+                    catch (err) {
+                        if (deleting) {
+                            return;
+                        }
+                        metadata = {
+                            tableId: InternalTables.USER_METADATA,
+                        };
+                    }
+                    // assign the roleId for the metadata doc
+                    if (roleId) {
+                        metadata.roleId = roleId;
+                    }
+                    let combined = !deleting
+                        ? combineMetadataAndUser(user, metadata)
+                        : Object.assign(Object.assign({}, metadata), { status: UserStatus.INACTIVE, metadata: BUILTIN_ROLE_IDS.PUBLIC });
+                    // if its null then there was no updates required
+                    if (combined) {
+                        yield db.put(combined);
+                    }
+                }));
             }
         }
         ctx.body = {

package/dist/integrations/base/sql.js

@@ -159,15 +159,14 @@ class InternalBuilder {
     addSorting(query, json) {
         var _a;
         let { sort, paginate } = json;
-        if (!sort) {
-            return query;
-        }
         const table = (_a = json.meta) === null || _a === void 0 ? void 0 : _a.table;
-        for (let [key, value] of Object.entries(sort)) {
-            const direction = value === datasource_1.SortDirection.ASCENDING ? "asc" : "desc";
-            query = query.orderBy(`${table === null || table === void 0 ? void 0 : table.name}.${key}`, direction);
+        if (sort) {
+            for (let [key, value] of Object.entries(sort)) {
+                const direction = value === datasource_1.SortDirection.ASCENDING ? "asc" : "desc";
+                query = query.orderBy(`${table === null || table === void 0 ? void 0 : table.name}.${key}`, direction);
+            }
         }
-        if (this.client === utils_1.SqlClients.MS_SQL && !sort && (paginate === null || paginate === void 0 ? void 0 : paginate.limit)) {
+        else if (this.client === utils_1.SqlClients.MS_SQL && (paginate === null || paginate === void 0 ? void 0 : paginate.limit)) {
             // @ts-ignore
             query = query.orderBy(`${table === null || table === void 0 ? void 0 : table.name}.${table === null || table === void 0 ? void 0 : table.primary[0]}`);
         }
@@ -177,20 +176,55 @@ class InternalBuilder {
         if (!relationships) {
             return query;
         }
+        const tableSets = {};
+        // aggregate into table sets (all the same to tables)
         for (let relationship of relationships) {
-            const from = relationship.from, to = relationship.to, toTable = relationship.tableName;
-            if (!relationship.through) {
+            const keyObj = {
+                toTable: relationship.tableName,
+                throughTable: undefined,
+            };
+            if (relationship.through) {
+                keyObj.throughTable = relationship.through;
+            }
+            const key = JSON.stringify(keyObj);
+            if (tableSets[key]) {
+                tableSets[key].push(relationship);
+            }
+            else {
+                tableSets[key] = [relationship];
+            }
+        }
+        for (let [key, relationships] of Object.entries(tableSets)) {
+            const { toTable, throughTable } = JSON.parse(key);
+            if (!throughTable) {
                 // @ts-ignore
-                query = query.leftJoin(toTable, `${fromTable}.${from}`, `${toTable}.${to}`);
+                query = query.join(toTable, function () {
+                    for (let relationship of relationships) {
+                        const from = relationship.from, to = relationship.to;
+                        // @ts-ignore
+                        this.orOn(`${fromTable}.${from}`, "=", `${toTable}.${to}`);
+                    }
+                }, "left");
             }
             else {
-                const throughTable = relationship.through;
-                const fromPrimary = relationship.fromPrimary;
-                const toPrimary = relationship.toPrimary;
                 query = query
                     // @ts-ignore
-                    .leftJoin(throughTable, `${fromTable}.${fromPrimary}`, `${throughTable}.${from}`)
-                    .leftJoin(toTable, `${toTable}.${toPrimary}`, `${throughTable}.${to}`);
+                    .join(throughTable, function () {
+                    for (let relationship of relationships) {
+                        const fromPrimary = relationship.fromPrimary;
+                        const from = relationship.from;
+                        // @ts-ignore
+                        this.orOn(`${fromTable}.${fromPrimary}`, "=", `${throughTable}.${from}`);
+                    }
+                }, "left")
+                    .join(toTable, function () {
+                    for (let relationship of relationships) {
+                        const toPrimary = relationship.toPrimary;
+                        const to = relationship.to;
+                        // @ts-ignore
+                        this.orOn(`${toTable}.${toPrimary}`, `${throughTable}.${to}`);
+                    }
+                }, "left");
             }
         }
         return query.limit(BASE_LIMIT);

package/dist/package.json

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

package/dist/threads/query.js

@@ -14,8 +14,6 @@ const ScriptRunner = require("../utilities/scriptRunner");
 const { integrations } = require("../integrations");
 const { processStringSync } = require("@budibase/string-templates");
 const { doInAppContext, getAppDB } = require("@budibase/backend-core/context");
-const IS_TRIPLE_BRACE = new RegExp(/^{{3}.*}{3}$/);
-const IS_HANDLEBARS = new RegExp(/^{{2}.*}{2}$/);
 class QueryRunner {
     constructor(input, flags = { noRecursiveQuery: false }) {
         this.datasource = input.datasource;
@@ -180,12 +178,8 @@ class QueryRunner {
             }
             else if (typeof fields[key] === "string") {
                 // enrich string value as normal
-                let value = fields[key];
-                // add triple brace to avoid escaping e.g. '=' in cookie header
-                if (IS_HANDLEBARS.test(value) && !IS_TRIPLE_BRACE.test(value)) {
-                    value = `{${value}}`;
-                }
-                enrichedQuery[key] = processStringSync(value, parameters, {
+                enrichedQuery[key] = processStringSync(fields[key], parameters, {
+                    noEscaping: true,
                     noHelpers: true,
                 });
             }