From e0c4747a939d271ceeff0dabd87425a2fe0b7446 Mon Sep 17 00:00:00 2001
From: mzn <mzn@opencascade.com>
Date: Wed, 16 Jan 2008 07:16:09 +0000
Subject: [PATCH] Fix for bug NPAL18491(EDF 630 SMESH : group of face appears
 with one hole in SMESH but not in VISU).

---
 src/DriverMED/DriverMED_R_SMESHDS_Mesh.cxx | 2 +-
 src/SMDS/SMDS_MeshGroup.cxx                | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

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);
 }
-- 
2.39.2