Merge pull request #1193 from Budibase/bug/0.8.1-bugs

Bug/0.8.1 bugs

packages/builder/src/components/backend/DataTable/modals/CreateEditRow.svelte

@@ -28,10 +28,8 @@
         .flat()
       // Prevent modal closing if there were errors
       return false
-    }
+    } else if (rowResponse.status === 400 || rowResponse.status === 500) {
-
+      errors = [{ message: rowResponse.message }]
-    if (rowResponse.status === 500) {
-      notifier.danger(rowResponse.message)
       return false
     }
 

packages/builder/src/components/backend/DataTable/modals/CreateEditUser.svelte

@@ -60,7 +60,7 @@
           .flat()
       }
       return false
-    } else if (rowResponse.status === 400 && rowResponse.message) {
+    } else if (rowResponse.status === 400 || rowResponse.status === 500) {
       errors = [{ message: rowResponse.message }]
       return false
     }

packages/builder/src/components/common/LinkedRowSelector.svelte

@@ -43,16 +43,6 @@
   </Label>
 {:else}
   {#if schema.relationshipType === 'one-to-many'}
-    <Multiselect
-      secondary
-      bind:value={linkedIds}
-      {label}
-      placeholder="Choose some options">
-      {#each rows as row}
-        <option value={row._id}>{getPrettyName(row)}</option>
-      {/each}
-    </Multiselect>
-  {:else}
     <Select
       thin
       secondary
@@ -66,5 +56,15 @@
         </option>
       {/each}
     </Select>
+  {:else}
+    <Multiselect
+      secondary
+      bind:value={linkedIds}
+      {label}
+      placeholder="Choose some options">
+      {#each rows as row}
+        <option value={row._id}>{getPrettyName(row)}</option>
+      {/each}
+    </Multiselect>
   {/if}
 {/if}

packages/server/src/api/controllers/row.js

@@ -7,6 +7,7 @@ const {
   DocumentTypes,
   SEPARATOR,
   ViewNames,
+  generateUserID,
 } = require("../../db/utils")
 const usersController = require("./user")
 const {
@@ -140,7 +141,11 @@ exports.save = async function(ctx) {
   }
 
   if (!inputs._rev && !inputs._id) {
-    inputs._id = generateRowID(inputs.tableId)
+    if (inputs.tableId === ViewNames.USERS) {
+      inputs._id = generateUserID(inputs.email)
+    } else {
+      inputs._id = generateRowID(inputs.tableId)
+    }
   }
 
   // this returns the table and row incase they have been updated

packages/server/src/db/linkedRows/LinkController.js

@@ -3,7 +3,6 @@ const { IncludeDocs, getLinkDocuments } = require("./linkUtils")
 const { generateLinkID } = require("../utils")
 const Sentry = require("@sentry/node")
 const { FieldTypes, RelationshipTypes } = require("../../constants")
-const { isEqual } = require("lodash")
 
 /**
  * Creates a new link document structure which can be put to the database. It is important to
@@ -133,6 +132,19 @@ class LinkController {
     }
   }
 
+  /**
+   * Returns whether the two schemas are equal (in the important parts, not a pure equality check)
+   */
+  areSchemasEqual(schema1, schema2) {
+    const compareFields = ["name", "type", "tableId", "fieldName", "autocolumn"]
+    for (let field of compareFields) {
+      if (schema1[field] !== schema2[field]) {
+        return false
+      }
+    }
+    return true
+  }
+
   // all operations here will assume that the table
   // this operation is related to has linked rows
   /**
@@ -310,12 +322,28 @@ class LinkController {
           tableId: table._id,
           fieldName: fieldName,
         }
+
         if (field.autocolumn) {
           linkConfig.autocolumn = field.autocolumn
         }
+
+        if (field.relationshipType) {
+          // Ensure that the other side of the relationship is locked to one record
+          linkConfig.relationshipType = field.relationshipType
+
+          // Update this table to be the many
+          table.schema[field.name].relationshipType =
+            RelationshipTypes.MANY_TO_MANY
+          const response = await this._db.put(table)
+          table._rev = response.rev
+        }
+
         // check the linked table to make sure we aren't overwriting an existing column
         const existingSchema = linkedTable.schema[field.fieldName]
-        if (existingSchema != null && !isEqual(existingSchema, linkConfig)) {
+        if (
+          existingSchema != null &&
+          !this.areSchemasEqual(existingSchema, linkConfig)
+        ) {
           throw new Error("Cannot overwrite existing column.")
         }
         // create the link field in the other table