Salome HOME
0020895: EDF 1428 SMESH: Bug in creation of groups using Set Filter
authorvsr <vsr@opencascade.com>
Thu, 3 Jun 2010 05:05:38 +0000 (05:05 +0000)
committervsr <vsr@opencascade.com>
Thu, 3 Jun 2010 05:05:38 +0000 (05:05 +0000)
src/SMESHGUI/SMESHGUI_FilterDlg.cxx

index 711d9a220059d2e1d75a74a713843a18b7969a41..9253da04bfa2d01dbd964beab7ac10c2b1b4ab2b 100755 (executable)
@@ -2987,14 +2987,12 @@ void SMESHGUI_FilterDlg::onSelectionDone()
     }
   }
 
-  int aCriterionType = myTable->GetCriterionType(aRow);
-  if (aList.Extent() != 1 ||
-      !myTable->CurrentCell(aRow, aCol) ||
-      aCriterionType != SMESH::FT_BelongToGeom &&
-      aCriterionType != SMESH::FT_BelongToPlane &&
-      aCriterionType != SMESH::FT_BelongToCylinder &&
-      aCriterionType != SMESH::FT_BelongToGenSurface &&
-      aCriterionType != SMESH::FT_LyingOnGeom)
+  QList<int> types; 
+  types << SMESH::FT_BelongToGeom     << SMESH::FT_BelongToPlane 
+       << SMESH::FT_BelongToCylinder << SMESH::FT_BelongToGenSurface
+       << SMESH::FT_LyingOnGeom;
+  if (aList.Extent() != 1 || !myTable->CurrentCell(aRow, aCol) || 
+      !types.contains(myTable->GetCriterionType(aRow)))
     return;
 
   Handle(SALOME_InteractiveObject) anIO = aList.First();