@mikro-orm/sql 7.0.3-dev.14 → 7.0.3-dev.16

package/AbstractSqlDriver.js

@@ -1412,10 +1412,11 @@ export class AbstractSqlDriver extends DatabaseDriver {
             const [propName, ref] = hint.field.split(':', 2);
             const prop = meta.properties[propName];
             // Polymorphic to-one: create a LEFT JOIN per target type
-            // Skip :ref hints — polymorphic to-one already has FK + discriminator in the row
+            // Skip regular :ref hints — polymorphic to-one already has FK + discriminator in the row
+            // But allow filter :ref hints through to create per-target LEFT JOINs with filter checks
             if (prop.polymorphic &&
                 prop.polymorphTargets?.length &&
-                !ref &&
+                (!ref || hint.filter) &&
                 [ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind)) {
                 const basePath = options.parentJoinPath
                     ? `${options.parentJoinPath}.${prop.name}`
@@ -1426,13 +1427,19 @@ export class AbstractSqlDriver extends DatabaseDriver {
                     const targetPath = `${pathPrefix}${basePath}[${targetMeta.className}]`;
                     const schema = targetMeta.schema === '*' ? (options?.schema ?? this.config.get('schema')) : targetMeta.schema;
                     qb.addPolymorphicJoin(prop, targetMeta, options.parentTableAlias, tableAlias, JoinType.leftJoin, targetPath, schema);
-                    // Select fields from each target table
-                    fields.push(...this.getFieldsForJoinedLoad(qb, targetMeta, {
-                        ...options,
-                        populate: hint.children,
-                        parentTableAlias: tableAlias,
-                        parentJoinPath: targetPath,
-                    }));
+                    if (ref) {
+                        // For filter :ref hints, schedule filter check for each target (no field selection)
+                        qb.scheduleFilterCheck(targetPath);
+                    }
+                    else {
+                        // Select fields from each target table
+                        fields.push(...this.getFieldsForJoinedLoad(qb, targetMeta, {
+                            ...options,
+                            populate: hint.children,
+                            parentTableAlias: tableAlias,
+                            parentJoinPath: targetPath,
+                        }));
+                    }
                 }
                 continue;
             }

package/README.md

@@ -64,7 +64,7 @@ export class Book extends BookSchema.class {}
 BookSchema.setClass(Book);
 ```
 
-You can also define entities using [decorators](https://mikro-orm.io/docs/defining-entities) or [`EntitySchema`](https://mikro-orm.io/docs/entity-schema). See the [defining entities guide](https://mikro-orm.io/docs/defining-entities) for all options.
+You can also define entities using [decorators](https://mikro-orm.io/docs/using-decorators) or [`EntitySchema`](https://mikro-orm.io/docs/define-entity#entityschema-low-level-api). See the [defining entities guide](https://mikro-orm.io/docs/defining-entities) for all options.
 
 ### Initialize and Use
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/sql",
-  "version": "7.0.3-dev.14",
+  "version": "7.0.3-dev.16",
   "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.2"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.0.3-dev.14"
+    "@mikro-orm/core": "7.0.3-dev.16"
   },
   "engines": {
     "node": ">= 22.17.0"

package/query/QueryBuilder.js

@@ -391,6 +391,26 @@ export class QueryBuilder {
                 else {
                     join.cond = { ...cond };
                 }
+                // For polymorphic LEFT JOIN filters, add a WHERE condition to enforce the filter
+                // only for rows matching this target's discriminator value. This ensures rows pointing
+                // to other polymorphic targets are not excluded.
+                if (join.prop.polymorphic &&
+                    join.prop.targetMeta &&
+                    join.type === JoinType.leftJoin &&
+                    join.prop.discriminatorColumn &&
+                    join.prop.discriminatorMap) {
+                    const discriminatorValue = QueryHelper.findDiscriminatorValue(join.prop.discriminatorMap, join.prop.targetMeta.class);
+                    if (discriminatorValue) {
+                        const pks = join.prop.targetMeta.primaryKeys;
+                        this.andWhere({
+                            $or: [
+                                { [`${join.ownerAlias}.${join.prop.discriminatorColumn}`]: null },
+                                { [`${join.ownerAlias}.${join.prop.discriminatorColumn}`]: { $ne: discriminatorValue } },
+                                { [`${join.alias}.${Utils.getPrimaryKeyHash(pks)}`]: { $ne: null } },
+                            ],
+                        });
+                    }
+                }
             }
         }
     }