@strapi/database 5.0.0-rc.3 → 5.0.0-rc.30

package/dist/index.mjs

@@ -2261,7 +2261,7 @@ const createOneToOne = (attributeName, attribute, meta, metadata) => {
   }
 };
 const createOneToMany = (attributeName, attribute, meta, metadata) => {
-  if (!isBidirectional(attribute)) {
+  if (shouldUseJoinTable(attribute) && !isBidirectional(attribute)) {
     createJoinTable(metadata, {
       attribute,
       attributeName,
@@ -2290,7 +2290,7 @@ const createManyToOne = (attributeName, attribute, meta, metadata) => {
   }
 };
 const createManyToMany = (attributeName, attribute, meta, metadata) => {
-  if (!isBidirectional(attribute) || isOwner(attribute)) {
+  if (shouldUseJoinTable(attribute) && (!isBidirectional(attribute) || isOwner(attribute))) {
     createJoinTable(metadata, {
       attribute,
       attributeName,
@@ -2315,7 +2315,7 @@ const createMorphToOne = (attributeName, attribute) => {
   });
 };
 const createMorphToMany = (attributeName, attribute, meta, metadata) => {
-  if ("joinTable" in attribute && attribute.joinTable) {
+  if ("joinTable" in attribute && attribute.joinTable && !attribute.joinTable.__internal__) {
     return;
   }
   const joinTableName = identifiers.getMorphTableName(meta.tableName, attributeName);
@@ -2385,6 +2385,7 @@ const createMorphToMany = (attributeName, attribute, meta, metadata) => {
     columnToAttribute: {}
   });
   const joinTable = {
+    __internal__: true,
     name: joinTableName,
     joinColumn: {
       name: joinColumnName,
@@ -2450,11 +2451,14 @@ const createJoinColumn = (metadata, { attribute, attributeName }) => {
   }
 };
 const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
+  if (!shouldUseJoinTable(attribute)) {
+    throw new Error("Attempted to create join table when useJoinTable is false");
+  }
   const targetMeta = metadata.get(attribute.target);
   if (!targetMeta) {
     throw new Error(`Unknown target ${attribute.target}`);
   }
-  if ("joinTable" in attribute && attribute.joinTable) {
+  if ("joinTable" in attribute && attribute.joinTable && !attribute.joinTable.__internal__) {
     return;
   }
   const joinTableName = identifiers.getJoinTableName(
@@ -2538,6 +2542,7 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
     columnToAttribute: {}
   };
   const joinTable = {
+    __internal__: true,
     name: joinTableName,
     joinColumn: {
       name: joinColumnName,
@@ -2557,7 +2562,8 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
       column: {
         unsigned: true,
         defaultTo: null
-      }
+      },
+      columnName: orderColumnName
     };
     metadataSchema.indexes.push({
       name: identifiers.getOrderFkIndexName(joinTableName),
@@ -2572,7 +2578,8 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
       column: {
         unsigned: true,
         defaultTo: null
-      }
+      },
+      columnName: inverseOrderColumnName
     };
     metadataSchema.indexes.push({
       name: identifiers.getOrderInverseFkIndexName(joinTableName),
@@ -2595,6 +2602,7 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
       );
     }
     inverseAttribute.joinTable = {
+      __internal__: true,
       name: joinTableName,
       joinColumn: joinTable.inverseJoinColumn,
       inverseJoinColumn: joinTable.joinColumn,
@@ -2680,6 +2688,9 @@ class Metadata extends Map {
     for (const meta of this.values()) {
       for (const [attributeName, attribute] of Object.entries(meta.attributes)) {
         try {
+          if (attribute.unstable_virtual) {
+            continue;
+          }
           if (isRelationalAttribute(attribute)) {
             createRelation(attributeName, attribute, meta, this);
             continue;
@@ -2743,12 +2754,22 @@ class StringField extends Field {
 }
 class JSONField extends Field {
   toDB(value) {
-    return JSON.stringify(value);
+    if (value == null) {
+      return null;
+    }
+    if (typeof value === "object") {
+      return JSON.stringify(value);
+    }
+    return value;
   }
   fromDB(value) {
     try {
       if (typeof value === "string") {
-        return JSON.parse(value);
+        const parsedValue = JSON.parse(value);
+        if (typeof parsedValue === "string") {
+          return JSON.parse(parsedValue);
+        }
+        return parsedValue;
       }
     } catch (error) {
       return value;
@@ -3426,7 +3447,11 @@ const oneToMany = async (input, ctx) => {
   const { db, qb } = ctx;
   const fromTargetRow = (rowOrRows) => fromRow(targetMeta, rowOrRows);
   if ("joinColumn" in attribute && attribute.joinColumn) {
-    const { name: joinColumnName, referencedColumn: referencedColumnName } = attribute.joinColumn;
+    const {
+      name: joinColumnName,
+      referencedColumn: referencedColumnName,
+      on
+    } = attribute.joinColumn;
     const referencedValues = _.uniq(
       results.map((r) => r[joinColumnName]).filter((value) => !_.isNil(value))
     );
@@ -3436,7 +3461,10 @@ const oneToMany = async (input, ctx) => {
       });
       return;
     }
-    const rows = await db.entityManager.createQueryBuilder(targetMeta.uid).init(populateValue).addSelect(`${qb.alias}.${referencedColumnName}`).where({ [referencedColumnName]: referencedValues }).execute({ mapResults: false });
+    const rows = await db.entityManager.createQueryBuilder(targetMeta.uid).init(populateValue).addSelect(`${qb.alias}.${referencedColumnName}`).where({
+      [referencedColumnName]: referencedValues,
+      ...on && typeof on === "function" ? on({ populateValue, results }) : {}
+    }).execute({ mapResults: false });
     const map2 = _.groupBy(referencedColumnName)(rows);
     results.forEach((result) => {
       result[attributeName] = fromTargetRow(map2[result[joinColumnName]] || []);
@@ -5243,8 +5271,10 @@ const processData = (metadata, data = {}, { withDefaults = false } = {}) => {
       if ("joinColumn" in attribute && attribute.joinColumn && attribute.owner) {
         const joinColumnName = attribute.joinColumn.name;
         const attrValue = !isUndefined(data[attributeName]) ? data[attributeName] : data[joinColumnName];
-        if (!isUndefined(attrValue)) {
+        if (isNull(attrValue)) {
           obj[joinColumnName] = attrValue;
+        } else if (!isUndefined(attrValue)) {
+          obj[joinColumnName] = toId(attrValue);
         }
         continue;
       }
@@ -6132,15 +6162,15 @@ const getNextIdsToCreateDocumentId = async (db, knex2, {
 };
 const migrateDocumentIdsWithLocalizations = async (db, knex2, meta) => {
   const singularName = meta.singularName.toLowerCase();
-  const joinColumn = identifiers.getJoinColumnAttributeIdName(singularName);
-  const inverseJoinColumn = identifiers.getInverseJoinColumnAttributeIdName(singularName);
+  const joinColumn = snakeCase(`${singularName}_id`);
+  const inverseJoinColumn = snakeCase(`inv_${singularName}_id`);
   let ids;
   do {
     ids = await getNextIdsToCreateDocumentId(db, knex2, {
       joinColumn,
       inverseJoinColumn,
       tableName: meta.tableName,
-      joinTableName: identifiers.getJoinTableName(meta.tableName, `localizations`)
+      joinTableName: snakeCase(`${meta.tableName}_localizations_links`)
     });
     if (ids.length > 0) {
       await knex2(meta.tableName).update({ document_id: createId() }).whereIn("id", ids);
@@ -6162,7 +6192,7 @@ const createDocumentIdColumn = async (knex2, tableName) => {
   });
 };
 const hasLocalizationsJoinTable = async (knex2, tableName) => {
-  const joinTableName = identifiers.getJoinTableName(tableName, "localizations");
+  const joinTableName = snakeCase(`${tableName}_localizations_links`);
   return knex2.schema.hasTable(joinTableName);
 };
 const createdDocumentId = {
@@ -6234,21 +6264,24 @@ const renameIndex = async (knex2, db, diff) => {
     debug(`not renaming index ${full.indexName} because name hasn't changed`);
     return;
   }
-  if (short.indexName.includes("_lnk_") || full.indexName.includes("_lnk_") || short.indexName.endsWith("fk") || full.indexName.endsWith("fk")) {
+  if (short.indexName.endsWith("fk") || full.indexName.endsWith("fk")) {
     return;
   }
   debug(`renaming index from ${full.indexName} to ${short.indexName}`);
   try {
     await knex2.transaction(async (trx) => {
       if (client === "mysql" || client === "mariadb") {
-        await knex2.raw(
-          `ALTER TABLE \`${full.tableName}\` RENAME INDEX \`${full.indexName}\` TO \`${short.indexName}\``
-        ).transacting(trx);
+        await knex2.raw("ALTER TABLE ?? RENAME INDEX ?? TO ??", [
+          full.tableName,
+          full.indexName,
+          short.indexName
+        ]).transacting(trx);
       } else if (client === "pg" || client === "postgres") {
-        await knex2.raw(`ALTER INDEX "${full.indexName}" RENAME TO "${short.indexName}"`).transacting(trx);
-      } else if (client === "sqlite" || client === "better") {
+        await knex2.raw("ALTER INDEX ?? RENAME TO ??", [full.indexName, short.indexName]).transacting(trx);
+      } else if (["sqlite", "sqlite3", "better-sqlite3"].includes(client)) {
+        debug(`SQLite does not support index renaming, not renaming index ${full.indexName}`);
       } else {
-        debug("No db client name matches, not creating index");
+        debug(`No db client name matches, not renaming index ${full.indexName}`);
       }
     });
   } catch (err) {