From 56e455a1835b68a43ffc6fca7209ad094635e4f2 Mon Sep 17 00:00:00 2001
From: Benoit LELANDAIS <benoit.lelandais@cea.fr>
Date: Fri, 10 Oct 2025 15:44:17 +0200
Subject: [PATCH] Fix undo clear group bug

---
 src/Core/Internal/InfoCommand.cpp       |  7 +++++--
 src/Core/Mesh/CommandClearGroupName.cpp | 27 +++++++++++++------------
 2 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/src/Core/Internal/InfoCommand.cpp b/src/Core/Internal/InfoCommand.cpp
index e30cf6a2..fd8701aa 100644
--- a/src/Core/Internal/InfoCommand.cpp
+++ b/src/Core/Internal/InfoCommand.cpp
@@ -217,12 +217,15 @@ void InfoCommand::addGroupInfoEntity(Group::GroupEntity* entity, type t)
 	TkUtil::AutoMutex	autoMutex (&m_mutex);
 #ifdef _DEBUG2
     std::cout<<"addGroupInfoEntity("<<entity->getName()<<" (id "<<entity->getUniqueId()<<"),"
-            <<type2String(m_group_entities_info[entity])<<" => "<<type2String(t)
-            <<")"<<std::endl;
+            <<type2String(m_group_entities_info[entity])<<" -> "<<type2String(t);
 #endif
     type old_t = m_group_entities_info[entity];
     if (old_t < t)
         m_group_entities_info[entity] = t;
+
+#ifdef _DEBUG2
+    std::cout<<" ==> "<<type2String(m_group_entities_info[entity])<<")"<<std::endl;
+#endif
 }
 /*----------------------------------------------------------------------------*/
 void InfoCommand::getGeomInfoEntity(uint ind, Geom::GeomEntity* &entity, type &t)
diff --git a/src/Core/Mesh/CommandClearGroupName.cpp b/src/Core/Mesh/CommandClearGroupName.cpp
index 18916a20..a9731802 100644
--- a/src/Core/Mesh/CommandClearGroupName.cpp
+++ b/src/Core/Mesh/CommandClearGroupName.cpp
@@ -183,6 +183,15 @@ void CommandClearGroupName::internalExecute()
 /*----------------------------------------------------------------------------*/
 void CommandClearGroupName::internalUndo()
 {
+
+    getInfoCommand().permCreatedDeleted();
+
+    // suppression de ce qui a été ajouté
+    deleteCreatedMeshEntities();
+
+    // permute toutes les propriétés internes avec leur sauvegarde
+    permInternalsStats();
+
 	Group::GroupManager& gm = getContext().getGroupManager();
 	std::string default_group_name = gm.getDefaultName(m_dim);
 
@@ -268,7 +277,10 @@ void CommandClearGroupName::internalUndo()
 	}
 	break;
 	}
-
+}
+/*----------------------------------------------------------------------------*/
+void CommandClearGroupName::internalRedo()
+{
     getInfoCommand().permCreatedDeleted();
 
     // suppression de ce qui a été ajouté
@@ -276,10 +288,7 @@ void CommandClearGroupName::internalUndo()
 
     // permute toutes les propriétés internes avec leur sauvegarde
     permInternalsStats();
-}
-/*----------------------------------------------------------------------------*/
-void CommandClearGroupName::internalRedo()
-{
+
 	switch(m_dim){
 	case(0):{
 		for (std::vector<Geom::GeomEntity*>::iterator iter = m_geom_entities.begin();
@@ -360,14 +369,6 @@ void CommandClearGroupName::internalRedo()
 	}
 	break;
 	}
-
-    getInfoCommand().permCreatedDeleted();
-
-    // suppression de ce qui a été ajouté
-    deleteCreatedMeshEntities();
-
-    // permute toutes les propriétés internes avec leur sauvegarde
-    permInternalsStats();
 }
 /*----------------------------------------------------------------------------*/
 void CommandClearGroupName::