@builttocreate/engine-utils 2.10.1-beta.v593.23 → 2.10.1-beta.v593.25

package/dist/generateReadableId.js

@@ -164,7 +164,6 @@ function generateReadableTableColumnId(tableId, tableColumns, columnId) {
       var type = (col.type || 'unknown').toLowerCase();
       counts[type] = (counts[type] || 0) + 1;
       var readableId = "".concat(type).concat(counts[type]);
-      counts[type]++;
       if (col._id === columnId) {
         /**
          * If the generatedReadableId is already present in the tableColumns,

package/dist/validateReadableId.js

@@ -146,33 +146,34 @@ var _default = exports["default"] = function _default(readableId, fields, tableC
   }
 
   //6. Check for duplicates in readableIds
-  if (!Array.isArray(fields)) return;
-
-  //DuplicateId check for fieldIds
-  var duplicateField = fields.find(function (field) {
-    return field._id === readableId;
-  });
-  if (duplicateField) {
-    console.warn("validateReadableId: \"".concat(readableId, "\" is invalid - it already exists in fields"));
-    return false;
-  }
 
-  /**
-   * DuplicateId check for nested tables of collection fields
-   * When readableIds for nestedTables are passed in, we need the fieldId to retrieve the collection fields that houses the nestedTable
-   */
-  if (fieldId) {
-    var targetField = fields.find(function (field) {
-      return field._id === fieldId;
+  if (Array.isArray(fields)) {
+    //DuplicateId check for fieldIds
+    var duplicateField = fields.find(function (field) {
+      return field._id === readableId;
     });
-    if ((targetField === null || targetField === void 0 ? void 0 : targetField.type) === _FieldTypes["default"].collection) {
-      var flattenedSchema = (0, _fieldHelper.flattenSchemaTree)(targetField.schema);
-      var readableIdExists = flattenedSchema.some(function (schema) {
-        return schema.id === readableId;
+    if (duplicateField) {
+      console.warn("validateReadableId: \"".concat(readableId, "\" is invalid - it already exists in fields"));
+      return false;
+    }
+
+    /**
+     * DuplicateId check for nested tables of collection fields
+     * When readableIds for nestedTables are passed in, we need the fieldId to retrieve the collection fields that houses the nestedTable
+     */
+    if (fieldId) {
+      var targetField = fields.find(function (field) {
+        return field._id === fieldId;
       });
-      if (readableIdExists) {
-        console.warn("validateReadableId: \"".concat(readableId, "\" is invalid - it already exists in fields"));
-        return false;
+      if ((targetField === null || targetField === void 0 ? void 0 : targetField.type) === _FieldTypes["default"].collection) {
+        var flattenedSchema = (0, _fieldHelper.flattenSchemaTree)(targetField.schema);
+        var readableIdExists = flattenedSchema.some(function (schema) {
+          return schema.id === readableId;
+        });
+        if (readableIdExists) {
+          console.warn("validateReadableId: \"".concat(readableId, "\" is invalid - it already exists in fields"));
+          return false;
+        }
       }
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@builttocreate/engine-utils",
-  "version": "2.10.1-beta.v593.23",
+  "version": "2.10.1-beta.v593.25",
   "description": "Utility library for common logic shared across web and mobile",
   "main": "dist/index.js",
   "files": [