@mikro-orm/core 7.0.8-dev.3 → 7.0.8-dev.5

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/core",
-  "version": "7.0.8-dev.3",
+  "version": "7.0.8-dev.5",
   "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/unit-of-work/UnitOfWork.js

@@ -967,7 +967,13 @@ export class UnitOfWork {
         }
         await this.#changeSetPersister.executeInserts(changeSets, { ctx });
         for (const changeSet of changeSets) {
-            this.register(changeSet.entity, changeSet.payload, { refresh: true });
+            // For TPT entities, use the full entity snapshot instead of the partial changeset payload,
+            // since each table's changeset only contains its own properties. Without this, the snapshot
+            // would only have the last table's properties, causing spurious UPDATEs on next flush (GH #7454).
+            const data = changeSet.meta.inheritanceType === 'tpt'
+                ? this.#comparator.prepareEntity(changeSet.entity)
+                : changeSet.payload;
+            this.register(changeSet.entity, data, { refresh: true });
             await this.runHooks(EventType.afterCreate, changeSet);
         }
     }

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.3';
+    static #ORM_VERSION = '7.0.8-dev.5';
     /**
      * Checks if the argument is instance of `Object`. Returns false for arrays.
      */