@mikro-orm/sql 7.2.5-dev.7 → 7.2.5-dev.9

package/AbstractSqlDriver.d.ts

@@ -127,6 +127,12 @@ export declare abstract class AbstractSqlDriver<Connection extends AbstractSqlCo
      * @internal
      */
     protected addTPTPolymorphicJoinsForRelation<T extends object>(qb: AnyQueryBuilder<T>, meta: EntityMetadata<T>, baseAlias: string, fields: InternalField<T>[]): void;
+    /**
+     * Registers the ancestor table aliases of each polymorphically joined TPT descendant,
+     * so formulas on descendants resolve inherited columns to the table that owns them.
+     * @internal
+     */
+    registerTPTAncestorAliases(qb: AnyQueryBuilder<any>, meta: EntityMetadata, baseAlias: string, childAliases: Dictionary<string>): void;
     /**
      * Find the alias for a TPT child table in the query builder.
      * @internal

package/AbstractSqlDriver.js

@@ -1989,6 +1989,10 @@ export class AbstractSqlDriver extends DatabaseDriver {
             qb.createAlias(childMeta.class, childAlias);
             childAliases[childMeta.className] = childAlias;
             qb.addPropertyJoin(childMeta.tptInverseProp, baseAlias, childAlias, JoinType.leftJoin, `[tpt]${meta.className}`);
+        }
+        this.registerTPTAncestorAliases(qb, meta, baseAlias, childAliases);
+        for (const childMeta of descendants) {
+            const childAlias = childAliases[childMeta.className];
             // Add fields from this child (only ownProps, skip PKs)
             const schema = childMeta.schema === '*' ? '*' : this.getSchemaName(childMeta);
             childMeta
@@ -2000,6 +2004,21 @@ export class AbstractSqlDriver extends DatabaseDriver {
             fields.push(this.buildTPTDiscriminatorExpression(meta, descendants, childAliases, baseAlias));
         }
     }
+    /**
+     * Registers the ancestor table aliases of each polymorphically joined TPT descendant,
+     * so formulas on descendants resolve inherited columns to the table that owns them.
+     * @internal
+     */
+    registerTPTAncestorAliases(qb, meta, baseAlias, childAliases) {
+        const tptAlias = qb.state.tptAlias;
+        for (const childMeta of meta.allTPTDescendants) {
+            for (let parent = childMeta.tptParent; parent; parent = parent.tptParent) {
+                tptAlias[`${childAliases[childMeta.className]}:${parent.className}`] =
+                    childAliases[parent.className] ??
+                        (parent === meta ? baseAlias : tptAlias[`${baseAlias}:${parent.className}`]);
+            }
+        }
+    }
     /**
      * Find the alias for a TPT child table in the query builder.
      * @internal
@@ -2090,7 +2109,7 @@ export class AbstractSqlDriver extends DatabaseDriver {
         if (prop.formula) {
             const quotedAlias = this.platform.quoteIdentifier(tableAlias).toString();
             const table = this.createFormulaTable(quotedAlias, meta, schema);
-            const columns = meta.createColumnMappingObject(tableAlias);
+            const columns = meta.createColumnMappingObject(p => qb.helper.getTPTAliasForProperty(p.name, tableAlias), tableAlias);
             return [raw(`${this.evaluateFormula(prop.formula, columns, table)} as ${aliased}`)];
         }
         if (prop.customTypes?.some(type => !!type?.convertToJSValueSQL)) {

package/package.json

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

package/query/QueryBuilder.js

@@ -1943,7 +1943,8 @@ export class QueryBuilder {
             if (parentAlias) {
                 const schema = parentMeta.schema === '*' ? '*' : this.driver.getSchemaName(parentMeta);
                 parentMeta
-                    .ownProps.filter(prop => this.platform.shouldHaveColumn(prop, []))
+                    // inherited formulas are selected on the main alias in `finalize()`
+                    .ownProps.filter(prop => !prop.formula && this.platform.shouldHaveColumn(prop, []))
                     .forEach(prop => this.#state.fields.push(...this.driver.mapPropToFieldNames(this, prop, parentAlias, parentMeta, schema)));
             }
             parentMeta = parentMeta.tptParent;
@@ -1962,12 +1963,16 @@ export class QueryBuilder {
         if (!this.#state.fields?.includes('*') && !this.#state.fields?.includes(`${this.mainAlias.aliasName}.*`)) {
             return;
         }
-        // LEFT JOIN each descendant table and add their fields
+        // LEFT JOIN each descendant table
         for (const childMeta of descendants) {
             const childAlias = this.getNextAlias(childMeta.className);
             this.createAlias(childMeta.class, childAlias);
             this.#state.tptAlias[childMeta.className] = childAlias;
             this.addPropertyJoin(childMeta.tptInverseProp, this.mainAlias.aliasName, childAlias, JoinType.leftJoin, `[tpt]${meta.className}`);
+        }
+        this.driver.registerTPTAncestorAliases(this, meta, this.mainAlias.aliasName, this.#state.tptAlias);
+        for (const childMeta of descendants) {
+            const childAlias = this.#state.tptAlias[childMeta.className];
             // Add child fields
             const schema = childMeta.schema === '*' ? '*' : this.driver.getSchemaName(childMeta);
             childMeta
@@ -1976,7 +1981,7 @@ export class QueryBuilder {
         }
         // Add computed discriminator (CASE WHEN to determine concrete type)
         // descendants is pre-sorted by depth (deepest first) during discovery
-        if (meta.tptDiscriminatorColumn) {
+        if (meta.root.tptDiscriminatorColumn) {
             this.#state.fields.push(this.driver.buildTPTDiscriminatorExpression(meta, descendants, this.#state.tptAlias, this.mainAlias.aliasName));
         }
     }