@strapi/core 5.50.2 → 5.51.1

package/dist/migrations/database/5.0.0-discard-drafts.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"5.0.0-discard-drafts.d.ts","sourceRoot":"","sources":["../../../src/migrations/database/5.0.0-discard-drafts.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AAIH,OAAO,KAAK,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAY5D,KAAK,eAAe,GAAG;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAC9D,KAAK,IAAI,GAAG,UAAU,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AA24G3C;;;;;GAKG;AACH,wBAAuB,iBAAiB,CAAC,EACvC,EAAE,EACF,GAAG,EACH,GAAG,EACH,gBAAuB,GACxB,EAAE;IACD,EAAE,EAAE,QAAQ,CAAC;IACb,GAAG,EAAE,IAAI,CAAC;IACV,GAAG,EAAE,MAAM,CAAC;IACZ,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B,oDAwBA;AAED,eAAO,MAAM,qBAAqB,EAAE,SAQnC,CAAC"}
+{"version":3,"file":"5.0.0-discard-drafts.d.ts","sourceRoot":"","sources":["../../../src/migrations/database/5.0.0-discard-drafts.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AAIH,OAAO,KAAK,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAY5D,KAAK,eAAe,GAAG;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAC9D,KAAK,IAAI,GAAG,UAAU,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AA2+G3C;;;;;GAKG;AACH,wBAAuB,iBAAiB,CAAC,EACvC,EAAE,EACF,GAAG,EACH,GAAG,EACH,gBAAuB,GACxB,EAAE;IACD,EAAE,EAAE,QAAQ,CAAC;IACb,GAAG,EAAE,IAAI,CAAC;IACV,GAAG,EAAE,MAAM,CAAC;IACZ,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B,oDAwBA;AAED,eAAO,MAAM,qBAAqB,EAAE,SAQnC,CAAC"}

package/dist/migrations/database/5.0.0-discard-drafts.js

@@ -483,6 +483,62 @@ const recordClonedComponentPair = (clonedComponentPairsCache, componentUid, orig
     enqueueColumn(joinTable?.orderColumn, 'asc');
     enqueueColumn('id', 'asc');
 };
+/**
+ * v4 join rows often only persisted order on one side of a bidirectional join table.
+ * When cloning relations for draft rows, derive missing order values from the other side
+ * so populate does not fall back to primary-key order.
+ */ const assignMissingOrderColumnFromFallback = (relations, { orderColumn, fallbackOrderColumn, groupByColumn, tieBreakerColumn })=>{
+    if (!orderColumn || !fallbackOrderColumn || relations.length === 0) {
+        return relations;
+    }
+    const byGroup = new Map();
+    for (const relation of relations){
+        const groupId = relation[groupByColumn];
+        const key = groupId ?? 'null';
+        const group = byGroup.get(key);
+        if (group) {
+            group.push(relation);
+        } else {
+            byGroup.set(key, [
+                relation
+            ]);
+        }
+    }
+    for (const group of byGroup.values()){
+        if (!group.some((relation)=>relation[orderColumn] == null)) {
+            continue;
+        }
+        const sorted = [
+            ...group
+        ].sort((left, right)=>{
+            const leftOrder = left[fallbackOrderColumn];
+            const rightOrder = right[fallbackOrderColumn];
+            if (leftOrder != null && rightOrder != null) {
+                if (leftOrder !== rightOrder) {
+                    return leftOrder - rightOrder;
+                }
+            } else if (leftOrder != null) {
+                return -1;
+            } else if (rightOrder != null) {
+                return 1;
+            }
+            if (tieBreakerColumn) {
+                const leftTie = normalizeId(left[tieBreakerColumn]) ?? left[tieBreakerColumn];
+                const rightTie = normalizeId(right[tieBreakerColumn]) ?? right[tieBreakerColumn];
+                if (leftTie != null && rightTie != null && leftTie !== rightTie) {
+                    return leftTie < rightTie ? -1 : 1;
+                }
+            }
+            return 0;
+        });
+        sorted.forEach((relation, index)=>{
+            if (relation[orderColumn] == null) {
+                relation[orderColumn] = index + 1;
+            }
+        });
+    }
+    return relations;
+};
 /**
  * Builds a stable key for join-table relations to detect duplicates.
  * Key format: sourceId::targetId::field::componentType
@@ -494,14 +550,15 @@ const recordClonedComponentPair = (clonedComponentPairsCache, componentUid, orig
 };
 /**
  * Queries existing relations from a join table and builds a set of keys for duplicate detection.
- */ async function getExistingRelationKeys({ trx, joinTable, sourceColumnName, targetColumnName, sourceIds }) {
+ */ async function getExistingRelationKeys({ trx, joinTable, sourceColumnName, targetColumnName, filterIds, filterColumnName }) {
     const existingKeys = new Set();
-    if (sourceIds.length === 0) {
+    const columnName = filterColumnName ?? sourceColumnName;
+    if (filterIds.length === 0) {
         return existingKeys;
     }
-    const idChunks = chunkArray(sourceIds, getBatchSize(trx, 1000));
+    const idChunks = chunkArray(filterIds, getBatchSize(trx, 1000));
     for (const chunk of idChunks){
-        const existingRelationsQuery = trx(joinTable.name).select('*').whereIn(sourceColumnName, chunk);
+        const existingRelationsQuery = trx(joinTable.name).select('*').whereIn(columnName, chunk);
         applyJoinTableOrdering(existingRelationsQuery, joinTable, sourceColumnName);
         const existingRelations = await existingRelationsQuery;
         for (const relation of existingRelations){
@@ -1446,7 +1503,7 @@ const debug = createDebug__default.default('strapi::migration::discard-drafts');
                 joinTable,
                 sourceColumnName,
                 targetColumnName,
-                sourceIds: draftSourceIds
+                filterIds: draftSourceIds
             });
             const relationsToInsert = newRelations.filter((relation)=>{
                 const targetId = normalizeId(relation[targetColumnName]) ?? relation[targetColumnName];
@@ -1518,7 +1575,8 @@ const debug = createDebug__default.default('strapi::migration::discard-drafts');
                 joinTable,
                 sourceColumnName,
                 targetColumnName,
-                sourceIds: draftTargetIds
+                filterIds: draftTargetIds,
+                filterColumnName: targetColumnName
             });
             const publishedIdChunks = chunkArray(publishedTargetIds, getBatchSize(trx, 1000));
             for (const chunk of publishedIdChunks){
@@ -1548,6 +1606,12 @@ const debug = createDebug__default.default('strapi::migration::discard-drafts');
                 if (newRelations.length === 0) {
                     continue;
                 }
+                assignMissingOrderColumnFromFallback(newRelations, {
+                    orderColumn: joinTable.inverseOrderColumnName,
+                    fallbackOrderColumn: joinTable.orderColumnName,
+                    groupByColumn: targetColumnName,
+                    tieBreakerColumn: sourceColumnName
+                });
                 await insertRelationsWithDuplicateHandling({
                     trx,
                     tableName: joinTable.name,
@@ -1663,7 +1727,7 @@ const debug = createDebug__default.default('strapi::migration::discard-drafts');
                 joinTable,
                 sourceColumnName,
                 targetColumnName,
-                sourceIds: draftSourceIds
+                filterIds: draftSourceIds
             });
             // Filter out relations that already exist
             const relationsToInsert = safeRelations.filter((relation)=>{
@@ -1672,6 +1736,12 @@ const debug = createDebug__default.default('strapi::migration::discard-drafts');
                 return !existingKeys.has(key);
             });
             if (relationsToInsert.length > 0) {
+                assignMissingOrderColumnFromFallback(relationsToInsert, {
+                    orderColumn: joinTable.orderColumnName,
+                    fallbackOrderColumn: joinTable.inverseOrderColumnName,
+                    groupByColumn: sourceColumnName,
+                    tieBreakerColumn: targetColumnName
+                });
                 await insertRelationsWithDuplicateHandling({
                     trx,
                     tableName: joinTable.name,