@mikro-orm/knex 6.4.7-dev.17 → 6.4.7-dev.19

package/dialects/mssql/MsSqlQueryCompiler.js

@@ -59,12 +59,15 @@ class MsSqlQueryCompiler extends MonkeyPatchable_1.MonkeyPatchable.MsSqlQueryCom
             || (typeof insert === 'object' && core_1.Utils.isEmpty(insert));
     }
     _mergeOn(conflict) {
-        let sql = 'on 1=1';
-        if (Array.isArray(conflict)) {
-            const conflictColumn = this.formatter.columnize(conflict[0]);
-            sql = `on ${this.tableName}.${conflictColumn} = tsource.${conflictColumn}`;
+        if (!Array.isArray(conflict)) {
+            return 'on 1=1';
         }
-        return sql;
+        const parts = [];
+        for (const col of conflict) {
+            const conflictColumn = this.formatter.columnize(col);
+            parts.push(`${this.tableName}.${conflictColumn} = tsource.${conflictColumn}`);
+        }
+        return `on ${parts.join(' and ')}`;
     }
     _insertWithMerge() {
         const { insert = [], onConflict, ignore, merge, returning, options = {} } = this.single;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/knex",
-  "version": "6.4.7-dev.17",
+  "version": "6.4.7-dev.19",
   "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",
@@ -66,7 +66,7 @@
     "@mikro-orm/core": "^6.4.6"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "6.4.7-dev.17",
+    "@mikro-orm/core": "6.4.7-dev.19",
     "better-sqlite3": "*",
     "libsql": "*",
     "mariadb": "*"

package/query/CriteriaNode.js

@@ -72,16 +72,12 @@ class CriteriaNode {
         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}`));
+            return core_1.Utils.getPrimaryKeyHash(this.prop.ownColumns.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}`));
         }
-        // if we found a matching join, we need to use the target table column names, as we use that alias instead of the root
-        if (!joinAlias && this.prop.owner && this.prop.joinColumns.length > 1) {
-            return core_1.Utils.getPrimaryKeyHash(this.prop.joinColumns.map(col => `${alias}.${col}`));
-        }
         return core_1.Utils.getPrimaryKeyHash(this.prop.referencedColumnNames.map(col => `${alias}.${col}`));
     }
     getPath(addIndex = false) {

package/query/QueryBuilderHelper.js

@@ -43,9 +43,6 @@ class QueryBuilderHelper {
                 const prop = this.getProperty(f, a);
                 const fkIdx2 = prop?.fieldNames.findIndex(name => name === f) ?? -1;
                 if (fkIdx2 !== -1) {
-                    if (prop?.ownColumns && !prop.ownColumns.includes(f)) {
-                        continue;
-                    }
                     parts.push(this.mapper(a !== this.alias ? `${a}.${prop.fieldNames[fkIdx2]}` : prop.fieldNames[fkIdx2], type, value, alias));
                 }
                 else if (prop) {
@@ -66,9 +63,6 @@ class QueryBuilderHelper {
                     }
                 });
             }
-            if (parts.length === 1) {
-                return parts[0];
-            }
             return this.knex.raw('(' + parts.map(part => this.knex.ref(part)).join(', ') + ')');
         }
         const rawField = core_1.RawQueryFragment.getKnownFragment(field);