Fix issue#215

Author: lelandaisb

src/Core/Mesh/CommandAddRemoveGroupName.cpp

@@ -574,10 +574,18 @@ void CommandAddRemoveGroupName::internalExecute()
 /*----------------------------------------------------------------------------*/
 void CommandAddRemoveGroupName::internalUndo()
 {
+	// on commence par remettre les anciens groupes
+    getInfoCommand().permCreatedDeleted();
+
+    // suppression de ce qui a été ajouté
+    deleteCreatedMeshEntities();
+
+    // permute toutes les propriétés internes avec leur sauvegarde
+    permInternalsStats();
 
 	switch(m_dim){
 	case(0):{
-		Group::Group0D* grp = getContext().getGroupManager().getNewGroup0D(m_groupName, &getInfoCommand());
+		Group::Group0D* grp = getContext().getGroupManager().getGroup0D(m_groupName);
 
 		for (std::vector<Geom::GeomEntity*>::iterator iter = m_geom_entities.begin();
 				iter != m_geom_entities.end(); ++iter){
@@ -615,7 +623,7 @@ void CommandAddRemoveGroupName::internalUndo()
 	}
 	break;
 	case(1):{
-		Group::Group1D* grp = getContext().getGroupManager().getNewGroup1D(m_groupName, &getInfoCommand());
+		Group::Group1D* grp = getContext().getGroupManager().getGroup1D(m_groupName);
 
 		for (std::vector<Geom::GeomEntity*>::iterator iter = m_geom_entities.begin();
 				iter != m_geom_entities.end(); ++iter){
@@ -653,7 +661,7 @@ void CommandAddRemoveGroupName::internalUndo()
 	}
 	break;
 	case(2):{
-		Group::Group2D* grp = getContext().getGroupManager().getNewGroup2D(m_groupName, &getInfoCommand());
+		Group::Group2D* grp = getContext().getGroupManager().getGroup2D(m_groupName);
 
 		for (std::vector<Geom::GeomEntity*>::iterator iter = m_geom_entities.begin();
 				iter != m_geom_entities.end(); ++iter){
@@ -691,7 +699,7 @@ void CommandAddRemoveGroupName::internalUndo()
 	}
 	break;
 	case(3):{
-		Group::Group3D* grp = getContext().getGroupManager().getNewGroup3D(m_groupName, &getInfoCommand());
+		Group::Group3D* grp = getContext().getGroupManager().getGroup3D(m_groupName);
 
 		for (std::vector<Geom::GeomEntity*>::iterator iter = m_geom_entities.begin();
 				iter != m_geom_entities.end(); ++iter){
@@ -734,14 +742,6 @@ void CommandAddRemoveGroupName::internalUndo()
 	}
 	break;
 	}
-
-    getInfoCommand().permCreatedDeleted();
-
-    // suppression de ce qui a été ajouté
-    deleteCreatedMeshEntities();
-
-    // permute toutes les propriétés internes avec leur sauvegarde
-    permInternalsStats();
 }
 /*----------------------------------------------------------------------------*/
 void CommandAddRemoveGroupName::internalRedo()

test_link/test_groups.py

@@ -94,15 +94,40 @@ def test_section_by_plane():
     ctx.clearSession() # Clean the session after the previous test
     # Création d'une boite avec une topologie
     ctx.getTopoManager().newBoxWithTopo (Mgx3D.Point(0, 0, 0), Mgx3D.Point(1, 1, 1), 10, 10, 10, "box")
+    vol_info = ctx.getGeomManager().getInfos("Vol0000",3)
+    assert "box" in vol_info.groupsName()
     # Section par un plan entre géométries avec topologies
     ctx.getGeomManager ( ).sectionByPlane (["Vol0000"], Mgx3D.Plane(Mgx3D.Point(0, 0, .5), Mgx3D.Vector(0, 0, 1)), "aaa")
     # Les entités de dimension inférieure à 2 sont affectées au groupe aaa de manière explicite
+    vol_info = ctx.getGeomManager().getInfos("Vol0001",3)
+    assert "box" in vol_info.groupsName()
+    surf_info = ctx.getGeomManager().getInfos("Surf0010",2)
+    assert "aaa" in surf_info.groupsName()
+    crb_info = ctx.getGeomManager().getInfos("Crb0014",1)
+    assert "aaa" in crb_info.groupsName()
+    pt_info = ctx.getGeomManager().getInfos("Pt0008",0)
+    assert "aaa" in pt_info.groupsName()
+    # Destruction de  Pt0008
+    ctx.getGeomManager().destroy(["Pt0008"], False)
+    assert "DETRUIT" in ctx.getGroupManager().getInfos("aaa", 2)
+    assert "DETRUIT" in ctx.getGroupManager().getInfos("box", 3)
+    # Annulation de : Destruction de  Pt0008
+    ctx.undo()
+    # Les entités de dimension inférieure à 2 sont affectées au groupe aaa de manière explicite
+    vol_info = ctx.getGeomManager().getInfos("Vol0001",3)
+    assert "box" in vol_info.groupsName()
     surf_info = ctx.getGeomManager().getInfos("Surf0010",2)
     assert "aaa" in surf_info.groupsName()
     crb_info = ctx.getGeomManager().getInfos("Crb0014",1)
     assert "aaa" in crb_info.groupsName()
     pt_info = ctx.getGeomManager().getInfos("Pt0008",0)
     assert "aaa" in pt_info.groupsName()
+    # Annulation de : Section par un plan entre géométries avec topologies
+    ctx.undo()
+    vol_info = ctx.getGeomManager().getInfos("Vol0000",3)
+    assert "box" in vol_info.groupsName()
+    assert "DETRUIT" in ctx.getGroupManager().getInfos("aaa", 1)
+    assert "DETRUIT" in ctx.getGroupManager().getInfos("aaa", 2)
 
 def test_new_vertices_curves_and_planar_surface():
     ctx = Mgx3D.getStdContext()
@@ -135,3 +160,16 @@ def test_new_planar_surface():
     assert "aaa" not in crb_info.groupsName()
     pt_info = ctx.getGeomManager().getInfos("Pt0002",0)
     assert "aaa" not in pt_info.groupsName()
+
+# Issue #215
+def test_undo_add_to_group():
+    ctx = Mgx3D.getStdContext()
+    ctx.clearSession() # Clean the session after the previous test
+    # Création d'une boite avec une topologie
+    ctx.getTopoManager().newBoxWithTopo (Mgx3D.Point(0, 0, 0), Mgx3D.Point(1, 1, 1), 10, 10, 10)
+    # Modifie le groupe BOX
+    ctx.getGeomManager().addToGroup (["Vol0000"], 3, "BOX")
+    # Annulation de : Modifie le groupe BOX
+    ctx.undo()
+    # Hors_Groupe_3D n'est plus détruit après fix de l'issue#215
+    assert "DETRUIT" not in ctx.getGroupManager().getInfos("Hors_Groupe_3D", 3)

test_link/test_unresolved_problems.py

@@ -190,15 +190,3 @@ def test_undo_clear_group():
     # ctx.undo()
     # Vol0000 ne devrait pas être dans Hors_Groupe_3D mais seulement dans BOX
     # assert "Vol0000" not in ctx.getGroupManager().getGeomEntities(["Hors_Groupe_3D"])
-
-# Issue #215
-def test_undo_add_to_group():
-    ctx = Mgx3D.getStdContext()
-    ctx.clearSession() # Clean the session after the previous test
-    # Création d'une boite avec une topologie
-    ctx.getTopoManager().newBoxWithTopo (Mgx3D.Point(0, 0, 0), Mgx3D.Point(1, 1, 1), 10, 10, 10)
-    # Modifie le groupe BOX
-    ctx.getGeomManager().addToGroup (["Vol0000"], 3, "BOX")
-    # Annulation de : Modifie le groupe BOX
-    # ctx.undo()
-    #Vol0000 est bien dans Hors_Groupe_3D qui est destroyed (en rouge dans l'ihm)