@mikro-orm/knex 6.1.10-dev.9 → 6.1.11-dev.0

package/AbstractSqlDriver.js

@@ -826,7 +826,7 @@ class AbstractSqlDriver extends core_1.DatabaseDriver {
                     if (!item[propName]) {
                         continue;
                     }
-                    if (prop.kind === core_1.ReferenceKind.MANY_TO_MANY && ref) {
+                    if ([core_1.ReferenceKind.ONE_TO_MANY, core_1.ReferenceKind.MANY_TO_MANY].includes(prop.kind) && ref) {
                         map[pk][propName] = [...map[pk][propName], ...item[propName]];
                         continue;
                     }

package/AbstractSqlPlatform.js

@@ -55,8 +55,11 @@ class AbstractSqlPlatform extends core_1.Platform {
     getJsonIndexDefinition(index) {
         return index.columnNames
             .map(column => {
+            if (!column.includes('.')) {
+                return column;
+            }
             const [root, ...path] = column.split('.');
-            return `json_extract(${root}, '$.${path.join('.')}')`;
+            return `(json_extract(${root}, '$.${path.join('.')}'))`;
         });
     }
     isRaw(value) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/knex",
-  "version": "6.1.10-dev.9",
+  "version": "6.1.11-dev.0",
   "description": "TypeScript ORM for Node.js based on Data Mapper, Unit of Work and Identity Map patterns. Supports MongoDB, MySQL, PostgreSQL and SQLite databases as well as usage with vanilla JavaScript.",
   "main": "index.js",
   "module": "index.mjs",
@@ -63,9 +63,9 @@
     "sqlstring": "2.3.3"
   },
   "devDependencies": {
-    "@mikro-orm/core": "^6.1.9"
+    "@mikro-orm/core": "^6.1.10"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "6.1.10-dev.9"
+    "@mikro-orm/core": "6.1.11-dev.0"
   }
 }

package/query/CriteriaNode.js

@@ -69,7 +69,11 @@ class CriteriaNode {
         }
     }
     renameFieldToPK(qb) {
-        const joinAlias = qb.getAliasForJoinPath(this.getPath());
+        let joinAlias = qb.getAliasForJoinPath(this.getPath());
+        if (!joinAlias && this.parent && [core_1.ReferenceKind.MANY_TO_ONE, core_1.ReferenceKind.ONE_TO_ONE].includes(this.prop.kind) && this.prop.owner) {
+            joinAlias = qb.getAliasForJoinPath(this.parent.getPath());
+            return core_1.Utils.getPrimaryKeyHash(this.prop.joinColumns.map(col => `${joinAlias ?? qb.alias}.${col}`));
+        }
         const alias = joinAlias ?? qb.alias;
         if (this.prop.kind === core_1.ReferenceKind.MANY_TO_MANY) {
             return core_1.Utils.getPrimaryKeyHash(this.prop.inverseJoinColumns.map(col => `${alias}.${col}`));

package/query/QueryBuilderHelper.js

@@ -86,7 +86,11 @@ class QueryBuilderHelper {
             const alias2 = this.knex.ref(a).toString();
             const aliased = this.knex.ref(prop.fieldNames[0]).toString();
             const as = alias === null ? '' : ` as ${aliased}`;
-            return this.knex.raw(`${prop.formula(alias2)}${as}`);
+            let value = prop.formula(alias2);
+            if (!this.isTableNameAliasRequired(type)) {
+                value = value.replaceAll(alias2 + '.', '');
+            }
+            return this.knex.raw(`${value}${as}`);
         }
         if (prop?.hasConvertToJSValueSQL) {
             let valueSQL;

package/schema/SqlSchemaGenerator.js

@@ -498,7 +498,7 @@ class SqlSchemaGenerator extends core_1.AbstractSchemaGenerator {
             // JSON columns can have unique index but not unique constraint, and we need to distinguish those, so we can properly drop them
             if (index.columnNames.some(column => column.includes('.'))) {
                 const columns = this.platform.getJsonIndexDefinition(index);
-                table.index(columns.map(column => this.knex.raw(`(${column})`)), index.keyName, { indexType: 'unique' });
+                table.index(columns.map(column => this.knex.raw(column)), index.keyName, { indexType: 'unique' });
             }
             else {
                 table.unique(index.columnNames, { indexName: index.keyName });
@@ -517,7 +517,7 @@ class SqlSchemaGenerator extends core_1.AbstractSchemaGenerator {
             // JSON columns can have unique index but not unique constraint, and we need to distinguish those, so we can properly drop them
             if (index.columnNames.some(column => column.includes('.'))) {
                 const columns = this.platform.getJsonIndexDefinition(index);
-                table.index(columns.map(column => this.knex.raw(`(${column})`)), index.keyName, index.type);
+                table.index(columns.map(column => this.knex.raw(column)), index.keyName, index.type);
             }
             else {
                 table.index(index.columnNames, index.keyName, index.type);