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

package/dist/entity-manager/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/entity-manager/index.ts"],"names":[],"mappings":"AA0CA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,IAAI,CAAC;AAGnC,OAAO,EAAE,aAAa,EAAsB,MAAM,SAAS,CAAC;AAE5D,cAAc,SAAS,CAAC;AAuLxB,eAAO,MAAM,mBAAmB,OAAQ,QAAQ,KAAG,aA2kClD,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/entity-manager/index.ts"],"names":[],"mappings":"AA0CA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,IAAI,CAAC;AAGnC,OAAO,EAAE,aAAa,EAAsB,MAAM,SAAS,CAAC;AAE5D,cAAc,SAAS,CAAC;AAyLxB,eAAO,MAAM,mBAAmB,OAAQ,QAAQ,KAAG,aA2kClD,CAAC"}

package/dist/fields/json.d.ts

@@ -1,6 +1,6 @@
 import Field from './field';
 export default class JSONField extends Field {
-    toDB(value: unknown): string;
+    toDB(value: unknown): {} | null;
     fromDB(value: unknown): any;
 }
 //# sourceMappingURL=json.d.ts.map

package/dist/fields/json.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"json.d.ts","sourceRoot":"","sources":["../../src/fields/json.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,MAAM,SAAS,CAAC;AAE5B,MAAM,CAAC,OAAO,OAAO,SAAU,SAAQ,KAAK;IAC1C,IAAI,CAAC,KAAK,EAAE,OAAO;IAInB,MAAM,CAAC,KAAK,EAAE,OAAO;CAWtB"}
+{"version":3,"file":"json.d.ts","sourceRoot":"","sources":["../../src/fields/json.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,MAAM,SAAS,CAAC;AAE5B,MAAM,CAAC,OAAO,OAAO,SAAU,SAAQ,KAAK;IAC1C,IAAI,CAAC,KAAK,EAAE,OAAO;IAYnB,MAAM,CAAC,KAAK,EAAE,OAAO;CAsBtB"}

package/dist/index.js

@@ -2293,7 +2293,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,
@@ -2322,7 +2322,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,
@@ -2347,7 +2347,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);
@@ -2417,6 +2417,7 @@ const createMorphToMany = (attributeName, attribute, meta, metadata) => {
     columnToAttribute: {}
   });
   const joinTable = {
+    __internal__: true,
     name: joinTableName,
     joinColumn: {
       name: joinColumnName,
@@ -2482,11 +2483,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(
@@ -2570,6 +2574,7 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
     columnToAttribute: {}
   };
   const joinTable = {
+    __internal__: true,
     name: joinTableName,
     joinColumn: {
       name: joinColumnName,
@@ -2589,7 +2594,8 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
       column: {
         unsigned: true,
         defaultTo: null
-      }
+      },
+      columnName: orderColumnName
     };
     metadataSchema.indexes.push({
       name: identifiers.getOrderFkIndexName(joinTableName),
@@ -2604,7 +2610,8 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
       column: {
         unsigned: true,
         defaultTo: null
-      }
+      },
+      columnName: inverseOrderColumnName
     };
     metadataSchema.indexes.push({
       name: identifiers.getOrderInverseFkIndexName(joinTableName),
@@ -2627,6 +2634,7 @@ const createJoinTable = (metadata, { attributeName, attribute, meta }) => {
       );
     }
     inverseAttribute.joinTable = {
+      __internal__: true,
       name: joinTableName,
       joinColumn: joinTable.inverseJoinColumn,
       inverseJoinColumn: joinTable.joinColumn,
@@ -2712,6 +2720,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;
@@ -2775,12 +2786,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;
@@ -3458,7 +3479,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 = ___default.default.uniq(
       results.map((r) => r[joinColumnName]).filter((value) => !___default.default.isNil(value))
     );
@@ -3468,7 +3493,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 map = ___default.default.groupBy(referencedColumnName)(rows);
     results.forEach((result) => {
       result[attributeName] = fromTargetRow(map[result[joinColumnName]] || []);
@@ -5275,8 +5303,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;
       }
@@ -6164,15 +6194,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: cuid2.createId() }).whereIn("id", ids);
@@ -6194,7 +6224,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 = {
@@ -6266,21 +6296,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) {