From eb3ce44b58a38c59a923d9327e442f564e4ada65 Mon Sep 17 00:00:00 2001
From: wmayer <wmayer@freecad.org>
Date: Mon, 18 Nov 2024 19:01:00 +0100
Subject: [PATCH] Core: Do not detach object from document before the call of
 breakDependency

This fixes #17900
---
 src/App/Document.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/App/Document.cpp b/src/App/Document.cpp
index aef283502b..131a18ba85 100644
--- a/src/App/Document.cpp
+++ b/src/App/Document.cpp
@@ -3989,15 +3989,15 @@ void Document::_removeObject(DocumentObject* pcObject)
     if (!d->rollback && d->activeUndoTransaction) {
         // Undo stuff
         signalTransactionRemove(*pcObject, d->activeUndoTransaction);
+        breakDependency(pcObject, true);
         d->activeUndoTransaction->addObjectNew(pcObject);
     }
     else {
         // for a rollback delete the object
         signalTransactionRemove(*pcObject, 0);
+        breakDependency(pcObject, true);
     }
 
-    breakDependency(pcObject, true);
-
     // remove from map
     pcObject->setStatus(ObjectStatus::Remove, false);  // Unset the bit to be on the safe side
     d->objectIdMap.erase(pcObject->_Id);