@mikro-orm/sql 7.0.7-dev.21 → 7.0.7-dev.22

package/AbstractSqlDriver.js

@@ -900,10 +900,10 @@ export class AbstractSqlDriver extends DatabaseDriver {
         sql = sql.substring(0, sql.length - 2) + ' where ';
         const pkProps = meta.primaryKeys.concat(...meta.concurrencyCheckKeys);
         const pks = Utils.flatten(pkProps.map(pk => meta.properties[pk].fieldNames));
-        sql +=
-            pks.length > 1
-                ? `(${pks.map(pk => this.platform.quoteIdentifier(pk)).join(', ')})`
-                : this.platform.quoteIdentifier(pks[0]);
+        const useTupleIn = pks.length <= 1 || this.platform.allowsComparingTuples();
+        const condTemplate = useTupleIn
+            ? `(${pks.map(() => '?').join(', ')})`
+            : `(${pks.map(pk => `${this.platform.quoteIdentifier(pk)} = ?`).join(' and ')})`;
         const conds = where.map(cond => {
             if (Utils.isPlainObject(cond) && Utils.getObjectKeysSize(cond) === 1) {
                 cond = Object.values(cond)[0];
@@ -917,12 +917,21 @@ export class AbstractSqlDriver extends DatabaseDriver {
                         params.push(cond[pk]);
                     }
                 });
-                return `(${Array.from({ length: pks.length }).fill('?').join(', ')})`;
+                return condTemplate;
             }
             params.push(cond);
             return '?';
         });
-        sql += ` in (${conds.join(', ')})`;
+        if (useTupleIn) {
+            sql +=
+                pks.length > 1
+                    ? `(${pks.map(pk => this.platform.quoteIdentifier(pk)).join(', ')})`
+                    : this.platform.quoteIdentifier(pks[0]);
+            sql += ` in (${conds.join(', ')})`;
+        }
+        else {
+            sql += conds.join(' or ');
+        }
         if (this.platform.usesReturningStatement() && returning.size > 0) {
             const returningFields = Utils.flatten([...returning].map(prop => meta.properties[prop].fieldNames));
             /* v8 ignore next */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/sql",
-  "version": "7.0.7-dev.21",
+  "version": "7.0.7-dev.22",
   "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.",
   "keywords": [
     "data-mapper",
@@ -53,7 +53,7 @@
     "@mikro-orm/core": "^7.0.6"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.0.7-dev.21"
+    "@mikro-orm/core": "7.0.7-dev.22"
   },
   "engines": {
     "node": ">= 22.17.0"