apostrophe 4.3.2 → 4.3.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.3.3 (2024-06-04)
+
+* Removes `$nextTick` use to re render schema in `AposArrayEditor` because it was triggering weird vue error in production.
+Instead, makes the AposSchema for loop keys more unique using `modelValue.data._id`, 
+if document changes it re-renders schema fields.
+
 ## 4.3.2 (2024-05-18)
 
 ### Fixes

package/modules/@apostrophecms/schema/ui/apos/components/AposSchema.vue

@@ -32,7 +32,7 @@
     <component
       :is="fieldStyle === 'table' ? 'td' : 'div'"
       v-for="field in schema"
-      :key="field.name.concat(field._id ?? '')"
+      :key="generateItemUniqueKey(field)"
       :data-apos-field="field.name"
       :style="(fieldStyle === 'table' && field.columnStyle) || {}"
       :class="{'apos-field--hidden': !displayComponent(field)}"

package/modules/@apostrophecms/schema/ui/apos/logic/AposArrayEditor.js

@@ -149,7 +149,7 @@ export default {
       const [ _id, name ] = first.path.split('.');
       await this.select(_id);
       const aposSchema = this.$refs.schema;
-      await this.nextTick();
+      await this.$nextTick();
       aposSchema.scrollFieldIntoView(name);
     }
     this.titleFieldChoices = await this.getTitleFieldChoices();
@@ -160,11 +160,7 @@ export default {
         return;
       }
       if (await this.validate(true, false)) {
-        // Force the array editor to totally reset to avoid in-schema
-        // animations when switching (e.g., the relationship input).
         this.currentDocToCurrentItem();
-        this.currentId = null;
-        await this.nextTick();
         this.currentId = _id;
         this.currentDoc = {
           hasErrors: false,
@@ -261,7 +257,7 @@ export default {
       if (validateItem) {
         this.triggerValidation = true;
       }
-      await this.nextTick();
+      await this.$nextTick();
       if (validateLength) {
         this.updateMinMax();
       }
@@ -279,14 +275,6 @@ export default {
         return true;
       }
     },
-    // Awaitable nextTick
-    nextTick() {
-      return new Promise((resolve, reject) => {
-        this.$nextTick(() => {
-          return resolve();
-        });
-      });
-    },
     newInstance() {
       const instance = {};
       for (const field of this.schema) {

package/modules/@apostrophecms/schema/ui/apos/logic/AposSchema.js

@@ -181,7 +181,7 @@ export default {
         this.populateDocData();
       }
     },
-    generation() {
+    generation(generated) {
       // repopulate the schema.
       this.populateDocData();
     },
@@ -331,6 +331,10 @@ export default {
     },
     highlight(fieldName) {
       return this.meta[fieldName]?.['@apostrophecms/schema:highlight'];
+    },
+    generateItemUniqueKey(field) {
+      return `${field.name}:${field._id ?? ''}:${this.modelValue?.data?._id ?? ''}`;
+
     }
   }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "apostrophe",
-  "version": "4.3.2",
+  "version": "4.3.3",
   "description": "The Apostrophe Content Management System.",
   "main": "index.js",
   "scripts": {