From: mzn <mzn@opencascade.com>
Date: Wed, 16 Jan 2008 07:16:09 +0000 (+0000)
Subject: Fix for bug NPAL18491(EDF 630 SMESH : group of face appears with one hole in SMESH... 
X-Git-Tag: V3_2_10~23
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=e0c4747a939d271ceeff0dabd87425a2fe0b7446;p=modules%2Fsmesh.git

Fix for bug NPAL18491(EDF 630 SMESH : group of face appears with one hole in SMESH but not in VISU).
---

diff --git a/src/DriverMED/DriverMED_R_SMESHDS_Mesh.cxx b/src/DriverMED/DriverMED_R_SMESHDS_Mesh.cxx
index 7271a5c86..e500ed3ad 100644
--- a/src/DriverMED/DriverMED_R_SMESHDS_Mesh.cxx
+++ b/src/DriverMED/DriverMED_R_SMESHDS_Mesh.cxx
@@ -795,7 +795,7 @@ void DriverMED_R_SMESHDS_Mesh::GetGroup(SMESHDS_Group* theGroup)
 	theGroup->SMDSGroup().Add(element);
       }
       if ( element )
-        theGroup->SetType( element->GetType() );
+        theGroup->SetType( theGroup->SMDSGroup().GetType() );
     }
   }
 }
diff --git a/src/SMDS/SMDS_MeshGroup.cxx b/src/SMDS/SMDS_MeshGroup.cxx
index adbde5081..e003c9219 100644
--- a/src/SMDS/SMDS_MeshGroup.cxx
+++ b/src/SMDS/SMDS_MeshGroup.cxx
@@ -124,8 +124,10 @@ void SMDS_MeshGroup::Add(const SMDS_MeshElement * theElem)
 {
 	// the type of the group is determined by the first element added
 	if (myElements.empty()) myType = theElem->GetType();
-	else if (theElem->GetType() != myType)
+	else if (theElem->GetType() != myType) {
 	  MESSAGE("SMDS_MeshGroup::Add : Type Mismatch "<<theElem->GetType()<<"!="<<myType);
+	  return;
+	}
 	
 	myElements.insert(theElem);
 }