Issue #1343. Improvement of Extrusion and Revolution operations: Bug correction:...

diff --git a/src/XGUI/XGUI_Workshop.cpp b/src/XGUI/XGUI_Workshop.cpp
index 455bebcd6c88b95167f89ea24da47f3a40a3a62a..8290f2c72c6ce9eabdbb0b46c11881d7b6fe3590 100755 (executable)
--- a/src/XGUI/XGUI_Workshop.cpp
+++ b/src/XGUI/XGUI_Workshop.cpp
@@ -1202,6 +1202,7 @@ void XGUI_Workshop::deleteObjects()
     ModuleBase_OperationAction* anOpAction = new ModuleBase_OperationAction(aDescription, module());
     operationMgr()->startOperation(anOpAction);
 
+    // WORKAROUND, should be done before each object remove, if it presents in XGUI_DataModel tree
     // It is necessary to clear selection in order to avoid selection changed event during
     // deletion and negative consequences connected with processing of already deleted items
     mySelector->clearSelection();
@@ -1296,6 +1297,11 @@ void XGUI_Workshop::cleanHistory()
     ModuleBase_OperationAction* anOpAction = new ModuleBase_OperationAction(aDescription, module());
     operationMgr()->startOperation(anOpAction);
 
+    // WORKAROUND, should be done before each object remove, if it presents in XGUI_DataModel tree
+    // It is necessary to clear selection in order to avoid selection changed event during
+    // deletion and negative consequences connected with processing of already deleted items
+    mySelector->clearSelection();
+
     std::set<FeaturePtr> anIgnoredFeatures;
     if (removeFeatures(anUnusedObjects, anIgnoredFeatures, anActionId)) {
       operationMgr()->commitOperation();