@budibase/server 1.0.27-alpha.7 → 1.0.27-alpha.8

package/dist/api/controllers/row/internal.js

@@ -23,7 +23,7 @@ const CouchDB = require("../../../db");
 const linkRows = require("../../../db/linkedRows");
 const { getRowParams, generateRowID, DocumentTypes, InternalTables, } = require("../../../db/utils");
 const userController = require("../user");
-const { inputProcessing, outputProcessing, processAutoColumn, } = require("../../../utilities/rowProcessor");
+const { inputProcessing, outputProcessing, processAutoColumn, cleanupAttachments, } = require("../../../utilities/rowProcessor");
 const { FieldTypes } = require("../../../constants");
 const { isEqual } = require("lodash");
 const { validate, findRow } = require("./utils");
@@ -32,6 +32,7 @@ const { getGlobalUsersFromMetadata } = require("../../../utilities/global");
 const inMemoryViews = require("../../../db/inMemoryView");
 const env = require("../../../environment");
 const { migrateToInMemoryView, migrateToDesignView, getFromDesignDoc, getFromMemoryDoc, } = require("../view/utils");
+const { cloneDeep } = require("lodash/fp");
 const CALCULATION_TYPES = {
     SUM: "sum",
     COUNT: "count",
@@ -119,15 +120,15 @@ exports.patch = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
     const inputs = ctx.request.body;
     const tableId = inputs.tableId;
     const isUserTable = tableId === InternalTables.USER_METADATA;
-    let dbRow;
+    let oldRow;
     try {
-        dbRow = yield db.get(inputs._id);
+        oldRow = yield db.get(inputs._id);
     }
     catch (err) {
         if (isUserTable) {
             // don't include the rev, it'll be the global rev
             // this time
-            dbRow = {
+            oldRow = {
                 _id: inputs._id,
             };
         }
@@ -137,13 +138,14 @@ exports.patch = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
     }
     let dbTable = yield db.get(tableId);
     // need to build up full patch fields before coerce
+    let combinedRow = cloneDeep(oldRow);
     for (let key of Object.keys(inputs)) {
         if (!dbTable.schema[key])
             continue;
-        dbRow[key] = inputs[key];
+        combinedRow[key] = inputs[key];
     }
     // this returns the table and row incase they have been updated
-    let { table, row } = inputProcessing(ctx.user, dbTable, dbRow);
+    let { table, row } = inputProcessing(ctx.user, dbTable, combinedRow);
     const validateResult = yield validate({
         row,
         table,
@@ -159,6 +161,8 @@ exports.patch = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
         tableId: row.tableId,
         table,
     });
+    // check if any attachments removed
+    yield cleanupAttachments(appId, table, { oldRow, row });
     if (isUserTable) {
         // the row has been updated, need to put it into the ctx
         ctx.request.body = row;
@@ -284,6 +288,8 @@ exports.destroy = function (ctx) {
             row,
             tableId: row.tableId,
         });
+        // remove any attachments that were on the row from object storage
+        yield cleanupAttachments(appId, table, { row });
         let response;
         if (ctx.params.tableId === InternalTables.USER_METADATA) {
             ctx.params = {
@@ -325,6 +331,8 @@ exports.bulkDestroy = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
     else {
         yield db.bulkDocs(rows.map(row => (Object.assign(Object.assign({}, row), { _deleted: true }))));
     }
+    // remove any attachments that were on the rows from object storage
+    yield cleanupAttachments(appId, table, { rows });
     yield Promise.all(updates);
     return { response: { ok: true }, rows };
 });

package/dist/integrations/base/sqlTable.js

@@ -29,7 +29,7 @@ function generateSchema(schema, table, tables, oldTable = null) {
             case FieldTypes.STRING:
             case FieldTypes.OPTIONS:
             case FieldTypes.LONGFORM:
-                schema.string(key);
+                schema.text(key);
                 break;
             case FieldTypes.NUMBER:
                 // if meta is specified then this is a junction table entry

package/dist/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@budibase/server",
     "email": "hi@budibase.com",
-    "version": "1.0.27-alpha.6",
+    "version": "1.0.27-alpha.7",
     "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/auth": "^1.0.27-alpha.6",
-        "@budibase/client": "^1.0.27-alpha.6",
-        "@budibase/string-templates": "^1.0.27-alpha.6",
+        "@budibase/auth": "^1.0.27-alpha.7",
+        "@budibase/client": "^1.0.27-alpha.7",
+        "@budibase/string-templates": "^1.0.27-alpha.7",
         "@bull-board/api": "^3.7.0",
         "@bull-board/koa": "^3.7.0",
         "@elastic/elasticsearch": "7.10.0",