@mikro-orm/core 7.0.8-dev.4 → 7.0.8-dev.6

package/entity/EntityHelper.js

@@ -228,7 +228,8 @@ export class EntityHelper {
                     // Inverse already points to owner — propagation is not needed,
                     // but we still need to clean up old's inverse side.
                     helper(old).__pk ??= helper(old).getPrimaryKey();
-                    if (old[prop2.name] != null) {
+                    // Don't nullify the FK if it's part of the PK — the entity will be deleted via orphan removal
+                    if (old[prop2.name] != null && !(prop.orphanRemoval && prop2.primary)) {
                         delete helper(old).__data[prop2.name];
                         old[prop2.name] = null;
                     }
@@ -267,7 +268,8 @@ export class EntityHelper {
         else {
             entity[prop2.name] = Reference.wrapReference(owner, prop);
         }
-        if (old?.[prop2.name] != null) {
+        // Don't nullify the FK if it's part of the PK — the entity will be deleted via orphan removal
+        if (old?.[prop2.name] != null && !(prop.orphanRemoval && prop2.primary)) {
             delete helper(old).__data[prop2.name];
             old[prop2.name] = null;
         }

package/entity/EntityLoader.js

@@ -53,6 +53,32 @@ export class EntityLoader {
         for (const pop of populate) {
             await this.populateField(entityName, entities, pop, options);
         }
+        // Child-specific relations exist only on child metadata, so the parent-scoped populate loop above skips them (GH #7453).
+        if (Array.isArray(populate) &&
+            populate.some(p => p.all) &&
+            meta.inheritanceType === 'tpt' &&
+            meta.tptChildren?.length) {
+            const byType = new Map();
+            for (const entity of entities) {
+                const entityMeta = helper(entity).__meta;
+                if (entityMeta !== meta) {
+                    const group = byType.get(entityMeta);
+                    if (group) {
+                        group.push(entity);
+                    }
+                    else {
+                        byType.set(entityMeta, [entity]);
+                    }
+                }
+            }
+            for (const [childMeta, group] of byType) {
+                await this.populate(childMeta.class, group, true, {
+                    ...options,
+                    lookup: false,
+                    validate: false,
+                });
+            }
+        }
         for (const entity of entities) {
             visited.delete(entity);
         }
@@ -711,6 +737,11 @@ export class EntityLoader {
                 all: true,
             });
         });
+        // For TPT parents with child types, keep an all:true sentinel so the populate
+        // loop doesn't exit early and the TPT child relation population can run after it.
+        if (ret.length === 0 && meta.inheritanceType === 'tpt' && meta.tptChildren?.length) {
+            ret.push({ field: meta.primaryKeys[0], strategy: LoadStrategy.SELECT_IN, all: true });
+        }
         return ret;
     }
     getRelationName(meta, prop) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/core",
-  "version": "7.0.8-dev.4",
+  "version": "7.0.8-dev.6",
   "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",

package/utils/Utils.js

@@ -132,7 +132,7 @@ export function parseJsonSafe(value) {
 /** Collection of general-purpose utility methods used throughout the ORM. */
 export class Utils {
     static PK_SEPARATOR = '~~~';
-    static #ORM_VERSION = '7.0.8-dev.4';
+    static #ORM_VERSION = '7.0.8-dev.6';
     /**
      * Checks if the argument is instance of `Object`. Returns false for arrays.
      */