Salome HOME
IPAL52704: TC7.6.0: "Modification of orientation" don't work on volumes
[modules/smesh.git] / src / SMESHGUI / SMESHGUI_FilterDlg.cxx
index 07f8a30a5cb2a966b399583e6d9c5976f109111b..4e39b5da5d9a2acd1f9e31e84768304598a6248e 100755 (executable)
@@ -1,4 +1,4 @@
-// Copyright (C) 2007-2013  CEA/DEN, EDF R&D, OPEN CASCADE
+// Copyright (C) 2007-2015  CEA/DEN, EDF R&D, OPEN CASCADE
 //
 // Copyright (C) 2003-2007  OPEN CASCADE, EADS/CCR, LIP6, CEA/DEN,
 // CEDRAT, EDF R&D, LEG, PRINCIPIA R&D, BUREAU VERITAS
@@ -6,7 +6,7 @@
 // This library is free software; you can redistribute it and/or
 // modify it under the terms of the GNU Lesser General Public
 // License as published by the Free Software Foundation; either
-// version 2.1 of the License.
+// version 2.1 of the License, or (at your option) any later version.
 //
 // This library is distributed in the hope that it will be useful,
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
 #include <SUIT_ResourceMgr.h>
 #include <SUIT_Session.h>
 #include <SUIT_MessageBox.h>
+#include <SUIT_OverrideCursor.h>
 #include <QtxColorButton.h>
 
 #include <LightApp_Application.h>
 #include <LightApp_SelectionMgr.h>
-#include <SalomeApp_Tools.h>
-#include <SalomeApp_Study.h>
-#include <SalomeApp_IntSpinBox.h>
+#include <SalomeApp_Application.h>
 #include <SalomeApp_DoubleSpinBox.h>
+#include <SalomeApp_IntSpinBox.h>
+#include <SalomeApp_Study.h>
+#include <SalomeApp_Tools.h>
 
 #include <SALOME_ListIO.hxx>
-#include <SALOME_ListIteratorOfListIO.hxx>
-#include <SALOME_DataMapIteratorOfDataMapOfIOMapOfInteger.hxx>
 
 #include <SVTK_ViewWindow.h>
 
@@ -1135,6 +1135,7 @@ bool SMESHGUI_FilterTable::IsValid (const bool theMess, const int theEntityType)
         errMsg = tr( "GROUPCOLOR_ERROR" );
     }
     else if (aCriterion == SMESH::FT_RangeOfIds ||
+             aCriterion == SMESH::FT_BelongToMeshGroup ||
              aCriterion == SMESH::FT_BelongToGeom ||
              aCriterion == SMESH::FT_BelongToPlane ||
              aCriterion == SMESH::FT_BelongToCylinder ||
@@ -1148,7 +1149,7 @@ bool SMESHGUI_FilterTable::IsValid (const bool theMess, const int theEntityType)
       if (aTable->text(i, 2).isEmpty())
         errMsg = tr( "ERROR" );
     }
-    else
+    else // check correctness of a numeric value
     {
       bool aRes = false;
       bool isSignalsBlocked = aTable->signalsBlocked();
@@ -1158,10 +1159,6 @@ bool SMESHGUI_FilterTable::IsValid (const bool theMess, const int theEntityType)
 
       if (!aRes && aTable->isEditable(i, 2))
         errMsg = tr( "ERROR" );
-      else if (aType == SMESH::EDGE &&
-               GetCriterionType(i, aType) == SMESH::FT_MultiConnection &&
-               aThreshold == 1)
-        errMsg = tr( "MULTIEDGES_ERROR" );
     }
 
     if (!errMsg.isEmpty()) {
@@ -1288,6 +1285,7 @@ void SMESHGUI_FilterTable::GetCriterion (const int                 theRow,
     }
   }
   else if ( aCriterionType != SMESH::FT_RangeOfIds &&
+            aCriterionType != SMESH::FT_BelongToMeshGroup &&
             aCriterionType != SMESH::FT_BelongToGeom &&
             aCriterionType != SMESH::FT_BelongToPlane &&
             aCriterionType != SMESH::FT_BelongToCylinder &&
@@ -1387,6 +1385,7 @@ void SMESHGUI_FilterTable::SetCriterion (const int                       theRow,
     }
   }
   else if (theCriterion.Type != SMESH::FT_RangeOfIds &&
+           theCriterion.Type != SMESH::FT_BelongToMeshGroup &&
            theCriterion.Type != SMESH::FT_BelongToGeom &&
            theCriterion.Type != SMESH::FT_BelongToPlane &&
            theCriterion.Type != SMESH::FT_BelongToCylinder &&
@@ -1403,6 +1402,7 @@ void SMESHGUI_FilterTable::SetCriterion (const int                       theRow,
            theCriterion.Type != SMESH::FT_OverConstrainedVolume &&
            theCriterion.Type != SMESH::FT_LinearOrQuadratic)
   {
+    // Numberic criterion
     aTable->item( theRow, 2 )->setText(QString("%1").arg(theCriterion.Threshold, 0, 'g', 15));
   }
   else
@@ -1559,18 +1559,29 @@ void SMESHGUI_FilterTable::updateAdditionalWidget()
 
   ComboItem* anItem = ((ComboItem*)aTable->item(aRow, 0));
   int aCriterion = GetCriterionType(aRow);
-  bool toEnable = ((((ComboItem*)aTable->item(aRow, 1))->value() == SMESH::FT_EqualTo &&
-                   aCriterion != SMESH::FT_RangeOfIds &&
-                   aCriterion != SMESH::FT_FreeEdges &&
-                   aCriterion != SMESH::FT_FreeFaces &&
-                   aCriterion != SMESH::FT_BadOrientedVolume &&
-                   aCriterion != SMESH::FT_BareBorderFace &&
-                   aCriterion != SMESH::FT_BareBorderVolume &&
-                   aCriterion != SMESH::FT_OverConstrainedFace &&
-                   aCriterion != SMESH::FT_OverConstrainedVolume)
-                   ||
-                   aCriterion == SMESH::FT_CoplanarFaces);
-  
+  bool isDbl = ( aCriterion == SMESH::FT_AspectRatio        ||
+                 aCriterion == SMESH::FT_AspectRatio3D      ||
+                 aCriterion == SMESH::FT_Warping            ||
+                 aCriterion == SMESH::FT_MinimumAngle       ||
+                 aCriterion == SMESH::FT_Taper              ||
+                 aCriterion == SMESH::FT_Skew               ||
+                 aCriterion == SMESH::FT_Area               ||
+                 aCriterion == SMESH::FT_Volume3D           ||
+                 aCriterion == SMESH::FT_MaxElementLength2D ||
+                 aCriterion == SMESH::FT_MaxElementLength3D ||
+                 aCriterion == SMESH::FT_Length             ||
+                 aCriterion == SMESH::FT_Length2D           ||
+                 aCriterion == SMESH::FT_BallDiameter );
+
+  bool toEnable = (( isDbl && ((ComboItem*)aTable->item(aRow, 1))->value() == SMESH::FT_EqualTo) ||
+                   aCriterion == SMESH::FT_BelongToPlane                                         ||
+                   aCriterion == SMESH::FT_BelongToCylinder                                      ||
+                   aCriterion == SMESH::FT_BelongToGenSurface                                    ||
+                   aCriterion == SMESH::FT_BelongToGeom                                          ||
+                   aCriterion == SMESH::FT_LyingOnGeom                                           ||
+                   aCriterion == SMESH::FT_CoplanarFaces                                         ||
+                   aCriterion == SMESH::FT_EqualNodes);
+
   if (!myAddWidgets.contains(anItem))
   {
     myAddWidgets[ anItem ] = new AdditionalWidget(myWgStack);
@@ -1608,6 +1619,7 @@ const char* SMESHGUI_FilterTable::getPrecision( const int aType )
   case SMESH::FT_Length2D:
   case SMESH::FT_MaxElementLength2D:
   case SMESH::FT_MaxElementLength3D:
+  case SMESH::FT_BallDiameter:
     retval = "length_precision"; break;
   case SMESH::FT_Volume3D:
     retval = "vol_precision"; break;
@@ -1765,184 +1777,226 @@ void SMESHGUI_FilterTable::onCriterionChanged (const int row, const int col, con
   Table* aTable = myTables[ aType ];
   ComboItem* aCompareItem = (ComboItem*)aTable->item(row, 1);
 
-  int aCriterionType = GetCriterionType(row);
+  // find out type of a existing threshould table item
   QtxColorButton* clrBtn = qobject_cast<QtxColorButton*>(aTable->cellWidget(row, 2));
-  int aComboType = ComboItem::Type();
-  int aIntSpinType = IntSpinItem::Type();
-  int aDoubleSpinType = DoubleSpinItem::Type();
-  QTableWidgetItem* aTableItem = aTable->item(row, 2);
-  bool isComboItem = false;
-  bool isIntSpinItem = false;
-  bool isDoubleSpinItem = false;
-  if (aTableItem) {
-    int aTableType = aTable->item(row, 2)->type();
-    isComboItem = ( aTableType == aComboType );
-    isIntSpinItem = ( aTableType == aIntSpinType );
-    isDoubleSpinItem = ( aTableType == aDoubleSpinType );
+  bool isComboItem       = false;
+  bool isIntSpinItem     = false;
+  bool isDoubleSpinItem  = false;
+  if ( QTableWidgetItem* aTableItem = aTable->item(row, 2) )
+  {
+    int aTableType   = aTableItem->type();
+    isComboItem      = ( aTableType == ComboItem::Type() );
+    isIntSpinItem    = ( aTableType == IntSpinItem::Type() );
+    isDoubleSpinItem = ( aTableType == DoubleSpinItem::Type() );
+  }
+
+  // find out a type of item required by a new criterion and other table features
+  int aCriterionType       = GetCriterionType(row);
+  bool anIsDoubleCriterion = false;
+  bool anIsIntCriterion    = false;
+  bool anIsComboCriterion  = false;
+  // other features:
+  QList<int> comboIDs; // values to show in a combo item
+  int nbCompareSigns = 0; // possible values are 0,1,3
+  bool isThresholdEditable = false; // actual for "simple" item types
+  switch ( aCriterionType )
+  {
+  case SMESH::FT_AspectRatio:
+  case SMESH::FT_AspectRatio3D:
+  case SMESH::FT_Warping:
+  case SMESH::FT_MinimumAngle:
+  case SMESH::FT_Taper:
+  case SMESH::FT_Skew:
+  case SMESH::FT_Area:
+  case SMESH::FT_Volume3D:
+  case SMESH::FT_MaxElementLength2D:
+  case SMESH::FT_MaxElementLength3D:
+    anIsDoubleCriterion = true; break;
+
+  case SMESH::FT_FreeBorders:
+  case SMESH::FT_FreeEdges:
+  case SMESH::FT_FreeNodes:
+  case SMESH::FT_FreeFaces:
+  case SMESH::FT_EqualNodes:
+  case SMESH::FT_EqualEdges:
+  case SMESH::FT_EqualFaces:
+  case SMESH::FT_EqualVolumes: break;
+
+  case SMESH::FT_MultiConnection:
+  case SMESH::FT_MultiConnection2D: anIsIntCriterion = true; nbCompareSigns = 3; break;
+
+  case SMESH::FT_Length:
+  case SMESH::FT_Length2D: anIsDoubleCriterion = true; break;
+
+  case SMESH::FT_BelongToMeshGroup: break;
+
+  case SMESH::FT_BelongToGeom:
+  case SMESH::FT_BelongToPlane:
+  case SMESH::FT_BelongToCylinder:
+  case SMESH::FT_BelongToGenSurface:
+  case SMESH::FT_LyingOnGeom: nbCompareSigns = 0; isThresholdEditable = true; break;
+
+  case SMESH::FT_RangeOfIds: nbCompareSigns = 0; isThresholdEditable = true; break;
+
+  case SMESH::FT_BadOrientedVolume:
+  case SMESH::FT_BareBorderVolume:
+  case SMESH::FT_BareBorderFace:
+  case SMESH::FT_OverConstrainedVolume:
+  case SMESH::FT_OverConstrainedFace:
+  case SMESH::FT_LinearOrQuadratic: break;
+
+  case SMESH::FT_GroupColor: nbCompareSigns = 1; isThresholdEditable = true; break;
+
+  case SMESH::FT_ElemGeomType:
+    comboIDs = geomTypes( aType ); anIsComboCriterion = true; nbCompareSigns = 1; break;
+
+  case SMESH::FT_EntityType:
+    comboIDs = entityTypes( aType ); anIsComboCriterion = true; nbCompareSigns = 1; break;
+
+  case SMESH::FT_CoplanarFaces: isThresholdEditable = true; break;
+
+  case SMESH::FT_BallDiameter: anIsDoubleCriterion = true; break;
+
+  case SMESH::FT_ConnectedElements: isThresholdEditable = true; break;
+
+  case SMESH::FT_LessThan:
+  case SMESH::FT_MoreThan:
+  case SMESH::FT_EqualTo:
+  case SMESH::FT_LogicalNOT:
+  case SMESH::FT_LogicalAND:
+  case SMESH::FT_LogicalOR:
+  case SMESH::FT_Undefined:
+  default: return;
   }
-  
-  bool anIsDoubleCriterion =
-    aCriterionType == SMESH::FT_AspectRatio ||
-    aCriterionType == SMESH::FT_AspectRatio3D ||
-    aCriterionType == SMESH::FT_Taper ||
-    aCriterionType == SMESH::FT_Warping ||
-    aCriterionType == SMESH::FT_MinimumAngle ||
-    aCriterionType == SMESH::FT_Skew ||
-    aCriterionType == SMESH::FT_Area ||
-    aCriterionType == SMESH::FT_Length ||
-    aCriterionType == SMESH::FT_Length2D ||
-    aCriterionType == SMESH::FT_MaxElementLength2D ||
-    aCriterionType == SMESH::FT_MaxElementLength3D ||
-    aCriterionType == SMESH::FT_Volume3D ||
-    aCriterionType == SMESH::FT_EntityType;
+
+  // get a precision of a double criterion
   int aPrecision = 0;
   if ( anIsDoubleCriterion ) {
-    const char* aPrecisionType = getPrecision( aCriterionType );
+    const char*     aPrecisionType = getPrecision( aCriterionType );
     SUIT_ResourceMgr* aResourceMgr = SMESH::GetResourceMgr( mySMESHGUI );
     if( aPrecisionType && aResourceMgr )
       aPrecision = aResourceMgr->integerValue( "SMESH", aPrecisionType, aPrecision );
   }
 
-  // if the precision is to be changed we should remove the existing
-  // spin item and create another one with new precision
-  bool anIsPrecisionChanged = false;
-  if ( anIsDoubleCriterion && isDoubleSpinItem ) {
-    if ( DoubleSpinItem* aDoubleSpinItem = dynamic_cast<DoubleSpinItem*>( aTable->item( row, 2 ) ) ) {
-      anIsPrecisionChanged = aDoubleSpinItem->precision() != aPrecision;
+  // check if the current item type satisfies the criterion
+  bool itemTypeKO =
+    ( aCriterionType == SMESH::FT_GroupColor && !clrBtn) ||
+    ( anIsComboCriterion                     && !isComboItem ) ||
+    ( anIsIntCriterion                       && !isIntSpinItem ) ||
+    ( anIsDoubleCriterion                    && !isDoubleSpinItem );
+
+  if ( !itemTypeKO )
+  {
+    if ( anIsDoubleCriterion )
+    {
+      // if the precision is to be changed we should remove the existing
+      // spin item and create another one with new precision
+      if ( DoubleSpinItem* aDoubleSpinItem = dynamic_cast<DoubleSpinItem*>( aTable->item( row, 2 )))
+        itemTypeKO = ( aDoubleSpinItem->precision() != aPrecision );
+      else
+        itemTypeKO = true;
+    }
+    else if ( anIsComboCriterion )
+    {
+      if ( ComboItem* aComboItem = dynamic_cast<ComboItem*>( aTable->item( row, 2 )))
+        itemTypeKO = ( aComboItem->count() != comboIDs.count() );
+      else
+        itemTypeKO = true;
+    }
+    else if ( !anIsIntCriterion && aCriterionType != SMESH::FT_GroupColor )
+    {
+      itemTypeKO = ( clrBtn || isComboItem || isIntSpinItem || isDoubleSpinItem );
     }
   }
 
-  if ( (aCriterionType != SMESH::FT_GroupColor && clrBtn) ||
-       (aCriterionType != SMESH::FT_ElemGeomType && isComboItem) ||
-       (aCriterionType != SMESH::FT_EntityType && isComboItem) ||
-       (aCriterionType != SMESH::FT_MultiConnection && isIntSpinItem) ||
-       (!anIsDoubleCriterion && isDoubleSpinItem) ||
-       anIsPrecisionChanged )
+  // update the table row
+
+  bool isSignalsBlocked = aTable->signalsBlocked();
+  aTable->blockSignals( true );
+
+  // update threshold table item
+  if ( itemTypeKO )
   {
-    bool isSignalsBlocked = aTable->signalsBlocked();
-    aTable->blockSignals( true );
     aTable->removeCellWidget( row, 2 );
-    aTable->setItem( row, 2, new QTableWidgetItem() );
-    aTable->blockSignals( isSignalsBlocked );
-  }
-  if ( (aCriterionType == SMESH::FT_GroupColor && !clrBtn) ||
-       (aCriterionType == SMESH::FT_ElemGeomType && !isComboItem) ||
-       (aCriterionType == SMESH::FT_EntityType && !isComboItem) ||
-       (aCriterionType == SMESH::FT_MultiConnection && !isIntSpinItem) ||
-       (anIsDoubleCriterion && !isDoubleSpinItem) ||
-       anIsPrecisionChanged )
-  {
-    bool isSignalsBlocked = aTable->signalsBlocked();
-    aTable->blockSignals( true );
-    if ( aCriterionType == SMESH::FT_GroupColor )
+
+    if ( aCriterionType == SMESH::FT_GroupColor ) // ---------------------- QtxColorButton
+    {
       aTable->setCellWidget( row, 2, new QtxColorButton( aTable ) );
-    else if ( aCriterionType == SMESH::FT_ElemGeomType ) {
-      QList<int> typeIds = geomTypes( aType );
-      QMap<int, QString> typeNames;
-      QList<int>::const_iterator anIter = typeIds.begin();
-      for ( int i = 0; anIter != typeIds.end(); ++anIter, ++i)
-      {
-        QString typeKey = QString( "GEOM_TYPE_%1" ).arg( *anIter );
-        typeNames[ *anIter ] = tr( typeKey.toLatin1().data() );
-      }
-      ComboItem* typeBox = new ComboItem( typeNames );
-      aTable->setItem( row, 2, typeBox );
-    }
-    else if ( aCriterionType == SMESH::FT_EntityType ) {
-      QList<int> typeIds = entityTypes( aType );
-      QMap<int, QString> typeNames;
-      QList<int>::const_iterator anIter = typeIds.begin();
-      for ( int i = 0; anIter != typeIds.end(); ++anIter, ++i)
+    }
+    else if ( anIsComboCriterion ) // -------------------------------------------ComboItem
+    {
+      QString msgPrefix
+        ( aCriterionType == SMESH::FT_ElemGeomType  ? "GEOM_TYPE_%1" :  "ENTITY_TYPE_%1" );
+      QMap<int, QString> names;
+      QList<int>::const_iterator id = comboIDs.begin();
+      for ( ; id != comboIDs.end(); ++id )
       {
-        QString typeKey = QString( "ENTITY_TYPE_%1" ).arg( *anIter );
-        typeNames[ *anIter ] = tr( typeKey.toLatin1().data() );
+        QString name = msgPrefix.arg( *id );
+        names[ *id ] = tr( name.toLatin1().data() );
       }
-      ComboItem* typeBox = new ComboItem( typeNames );
-      aTable->setItem( row, 2, typeBox );
+      ComboItem* comboBox = new ComboItem( names );
+      aTable->setItem( row, 2, comboBox );
     }
-    else if ( aCriterionType == SMESH::FT_MultiConnection ) {
+    else if ( anIsIntCriterion ) // ------------------------------------------ IntSpinItem
+    {
       IntSpinItem* intSpin = new IntSpinItem( 0 );
       aTable->setItem( row, 2, intSpin );
     }
-    else if ( anIsDoubleCriterion ) {
+    else if ( anIsDoubleCriterion ) // -------------------------------------DoubleSpinItem
+    {
       DoubleSpinItem* dblSpin = new DoubleSpinItem( 0 );
       dblSpin->setPrecision( aPrecision );
       aTable->setItem( row, 2, dblSpin );
     }
-    aTable->blockSignals( isSignalsBlocked );
-  }
-
-  // set Compare and enable/desable Threshold
-  if ((aType == SMESH::NODE && (aCriterionType == SMESH::FT_FreeNodes               ||
-                                aCriterionType == SMESH::FT_EqualNodes ))           ||
-      (aType == SMESH::EDGE && (aCriterionType == SMESH::FT_FreeBorders             ||
-                                aCriterionType == SMESH::FT_EqualEdges ))           ||
-      (aType == SMESH::FACE && (aCriterionType == SMESH::FT_BareBorderFace          ||
-                                aCriterionType == SMESH::FT_OverConstrainedFace     ||
-                                aCriterionType == SMESH::FT_FreeEdges               ||
-                                aCriterionType == SMESH::FT_FreeFaces               ||
-                                aCriterionType == SMESH::FT_EqualFaces))            ||
-      (aType == SMESH::VOLUME && (aCriterionType == SMESH::FT_BadOrientedVolume     ||
-                                  aCriterionType == SMESH::FT_OverConstrainedVolume ||
-                                  aCriterionType == SMESH::FT_BareBorderVolume      ||
-                                  aCriterionType == SMESH::FT_EqualVolumes ))       ||
-      aCriterionType == SMESH::FT_LinearOrQuadratic ||
-      aCriterionType == SMESH::FT_CoplanarFaces ||
-      aCriterionType == SMESH::FT_ConnectedElements
-      )
-  {
-    // - No compare
-    bool isSignalsBlocked = aTable->signalsBlocked();
-    aTable->blockSignals( true );
-
-    if (aCompareItem->count() > 0)
-      aCompareItem->clear();
-    aTable->setEditable(false, row, 1);
-    // - Threshold is NOT editable for most of criteria
-    aTable->item(row, 2)->setText( QString("") );
-    aTable->setEditable(( aCriterionType == SMESH::FT_ConnectedElements ||
-                          aCriterionType == SMESH::FT_CoplanarFaces       ), row, 2);
-    aTable->blockSignals( isSignalsBlocked );
-  }
-  else if (aCriterionType == SMESH::FT_RangeOfIds ||
-           aCriterionType == SMESH::FT_GroupColor ||
-           aCriterionType == SMESH::FT_ElemGeomType ||
-           aCriterionType == SMESH::FT_EntityType ||
-           aCriterionType == SMESH::FT_BelongToGeom ||
-           aCriterionType == SMESH::FT_BelongToPlane ||
-           aCriterionType == SMESH::FT_BelongToCylinder ||
-           aCriterionType == SMESH::FT_BelongToGenSurface ||
-           aCriterionType == SMESH::FT_LyingOnGeom)
-  {
-    // - EQUAL_TO compare ONLY
-    QMap<int, QString> aMap;
-    aMap[ SMESH::FT_EqualTo ] = tr("EQUAL_TO");
-    aCompareItem->setItems(aMap);
-    // if (!aTable->isEditable(row, 2))
-    aTable->setEditable(false, row, 1);
-    // - Threshold is editable
-    if (!aTable->isEditable(row, 2))
-      aTable->setEditable(true, row, 2);
+    else // --------------------------------------------------------------QTableWidgetItem
+    {
+      aTable->setItem( row, 2, new QTableWidgetItem() );
+    }
   }
-  else
+
+  // set Compare
+  if ( anIsDoubleCriterion )
+    nbCompareSigns = 3;
+  if ( aCompareItem->count() != nbCompareSigns )
   {
-    // All compare signs
-    if (aCompareItem && aCompareItem->count() != 3)
-    {
+    switch ( nbCompareSigns ) {
+    case 0: {
+      aCompareItem->clear();
+      break;
+    }
+    case 1: {
+      QMap<int, QString> aMap;
+      aMap[ SMESH::FT_EqualTo ] = tr("EQUAL_TO");
+      aCompareItem->setItems(aMap);
+      break;
+    }
+    case 3: {
+      int oldValue = aCompareItem->value();
       aCompareItem->setItems(getCompare());
+      if ( oldValue >= 0 )
+        aCompareItem->setValue( oldValue );
+      break;
     }
-    // Threshold is editable
-    if (aTable->item( row, 2 )) {
-      QString aText = aTable->text(row, 2);
-      bool isOk = false;
-      aText.toDouble(&isOk);
-      aTable->item( row, 2 )->setText(isOk ? aText : QString(""));
-      if (!aTable->isEditable(row, 1))
-        aTable->setEditable(true, row, 1);
-      if (!aTable->isEditable(row, 2))
-        aTable->setEditable(true, row, 2);
     }
   }
+  aTable->setEditable( nbCompareSigns == 3, row, 1);
+
+  // enable/desable Threshold
+  if ( aCriterionType == SMESH::FT_GroupColor ||
+       anIsComboCriterion ||
+       anIsIntCriterion ||
+       anIsDoubleCriterion )
+  {
+    isThresholdEditable = true;
+  }
+  if ( !isThresholdEditable )
+  {
+    aTable->setItem( row, 2, new QTableWidgetItem() );
+  }
+  aTable->setEditable( isThresholdEditable, row, 2);
+
+
+  aTable->blockSignals( isSignalsBlocked );
 
   updateAdditionalWidget();
 
@@ -2125,6 +2179,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
     if (aCriteria.isEmpty())
     {
       aCriteria[ SMESH::FT_RangeOfIds         ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup  ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom       ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_BelongToPlane      ] = tr("BELONG_TO_PLANE");
       aCriteria[ SMESH::FT_BelongToCylinder   ] = tr("BELONG_TO_CYLINDER");
@@ -2146,6 +2201,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
       aCriteria[ SMESH::FT_MultiConnection    ] = tr("MULTI_BORDERS");
       aCriteria[ SMESH::FT_Length             ] = tr("LENGTH");
       aCriteria[ SMESH::FT_RangeOfIds         ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup  ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom       ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_BelongToPlane      ] = tr("BELONG_TO_PLANE");
       aCriteria[ SMESH::FT_BelongToCylinder   ] = tr("BELONG_TO_CYLINDER");
@@ -2174,6 +2230,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
       aCriteria[ SMESH::FT_MaxElementLength2D ] = tr("MAX_ELEMENT_LENGTH_2D");
       aCriteria[ SMESH::FT_FreeEdges          ] = tr("FREE_EDGES");
       aCriteria[ SMESH::FT_RangeOfIds         ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup  ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom       ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_BelongToPlane      ] = tr("BELONG_TO_PLANE");
       aCriteria[ SMESH::FT_BelongToCylinder   ] = tr("BELONG_TO_CYLINDER");
@@ -2201,6 +2258,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
     {
       aCriteria[ SMESH::FT_AspectRatio3D        ] = tr("ASPECT_RATIO_3D");
       aCriteria[ SMESH::FT_RangeOfIds           ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup    ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom         ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_LyingOnGeom          ] = tr("LYING_ON_GEOM");
       aCriteria[ SMESH::FT_BadOrientedVolume    ] = tr("BAD_ORIENTED_VOLUME");
@@ -2223,6 +2281,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
     if (aCriteria.isEmpty())
     {
       aCriteria[ SMESH::FT_RangeOfIds         ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup  ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom       ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_BelongToPlane      ] = tr("BELONG_TO_PLANE");
       aCriteria[ SMESH::FT_BelongToCylinder   ] = tr("BELONG_TO_CYLINDER");
@@ -2239,6 +2298,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
     {
       aCriteria[ SMESH::FT_BallDiameter       ] = tr("BALL_DIAMETER");
       aCriteria[ SMESH::FT_RangeOfIds         ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup  ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom       ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_BelongToPlane      ] = tr("BELONG_TO_PLANE");
       aCriteria[ SMESH::FT_BelongToCylinder   ] = tr("BELONG_TO_CYLINDER");
@@ -2254,6 +2314,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
     if (aCriteria.isEmpty())
     {
       aCriteria[ SMESH::FT_RangeOfIds         ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup  ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom       ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_BelongToPlane      ] = tr("BELONG_TO_PLANE");
       aCriteria[ SMESH::FT_BelongToCylinder   ] = tr("BELONG_TO_CYLINDER");
@@ -2269,6 +2330,7 @@ const QMap<int, QString>& SMESHGUI_FilterTable::getCriteria (const int theType)
     if (aCriteria.isEmpty())
     {
       aCriteria[ SMESH::FT_RangeOfIds         ] = tr("RANGE_OF_IDS");
+      aCriteria[ SMESH::FT_BelongToMeshGroup  ] = tr("BELONG_TO_MESH_GROUP");
       aCriteria[ SMESH::FT_BelongToGeom       ] = tr("BELONG_TO_GEOM");
       aCriteria[ SMESH::FT_LyingOnGeom        ] = tr("LYING_ON_GEOM");
       aCriteria[ SMESH::FT_LinearOrQuadratic  ] = tr("LINEAR");
@@ -2618,7 +2680,9 @@ SMESHGUI_FilterDlg::SMESHGUI_FilterDlg( SMESHGUI*         theModule,
 : QDialog( SMESH::GetDesktop( theModule ) ),
   mySMESHGUI( theModule ),
   mySelectionMgr( SMESH::GetSelectionMgr( theModule ) ),
-  myInitSourceWgOnApply( true )
+  myInitSourceWgOnApply( true ),
+  myInsertEnabled( true ),
+  myDiffSourcesEnabled( true )
 {
   if ( SVTK_ViewWindow* aViewWindow = SMESH::GetViewWindow( mySMESHGUI ))
     mySelector = aViewWindow->GetSelector();
@@ -2635,7 +2699,9 @@ SMESHGUI_FilterDlg::SMESHGUI_FilterDlg( SMESHGUI*   theModule,
 : QDialog( SMESH::GetDesktop( theModule ) ),
   mySMESHGUI( theModule ),
   mySelectionMgr( SMESH::GetSelectionMgr( theModule ) ),
-  myInitSourceWgOnApply( true )
+  myInitSourceWgOnApply( true ),
+  myInsertEnabled( true ),
+  myDiffSourcesEnabled( true )
 {
   if ( SVTK_ViewWindow* aViewWindow = SMESH::GetViewWindow( mySMESHGUI ))
     mySelector = aViewWindow->GetSelector();
@@ -2694,6 +2760,7 @@ QWidget* SMESHGUI_FilterDlg::createMainFrame (QWidget* theParent)
   int rows = aLay->rowCount();
   int cols = aLay->columnCount();
 
+  // This line looks strange when all additional parameters and mySetInViewer are hidden
   QFrame* aLine = new QFrame(aGrp);
   aLine->setFrameStyle(QFrame::HLine | QFrame::Sunken);
   aLay->addWidget(aLine, rows++, 0, 1, cols);
@@ -2827,6 +2894,7 @@ void SMESHGUI_FilterDlg::Init (const QList<int>& theTypes, const bool setInViewe
   mySourceWg  = 0;
   myTypes     = theTypes;
   myMesh      = SMESH::SMESH_Mesh::_nil();
+  myGroup     = SMESH::SMESH_GroupOnFilter::_nil();
   myIObjects.Clear();
   myIsSelectionChanged = false;
   myToRestoreSelMode = false;
@@ -3105,8 +3173,7 @@ bool SMESHGUI_FilterDlg::isValid() const
       if (aType == SMESH::FT_BelongToCylinder ||
           aType == SMESH::FT_BelongToPlane    ||
           aType == SMESH::FT_BelongToGenSurface ) {
-        CORBA::Object_var anObject = SMESH::SObjectToObject(aList[ 0 ]);
-        //GEOM::GEOM_Object_var aGeomObj = GEOM::GEOM_Object::_narrow(aList[ 0 ]->GetObject());
+        CORBA::Object_var     anObject = SMESH::SObjectToObject(aList[ 0 ]);
         GEOM::GEOM_Object_var aGeomObj = GEOM::GEOM_Object::_narrow(anObject);
         if (!aGeomObj->_is_nil()) {
           TopoDS_Shape aFace;
@@ -3179,6 +3246,21 @@ void SMESHGUI_FilterDlg::SetSourceWg (QWidget* theWg,
   myInitSourceWgOnApply = initOnApply;
 }
 
+//=======================================================================
+//function : EnableFiltering
+//purpose  : Enables "Insert filter in the viewer"
+//           and different "Source"s (Mesh, Initial Selection, Current Group)
+//=======================================================================
+
+void SMESHGUI_FilterDlg::SetEnabled( bool setInViewer, bool diffSources )
+{
+  myInsertEnabled = setInViewer;
+  myDiffSourcesEnabled = diffSources;
+
+  mySetInViewer->setVisible( myInsertEnabled );
+  mySourceGrp->button(0)->parentWidget()->setVisible( myDiffSourcesEnabled );
+}
+
 //=======================================================================
 // name    : SMESHGUI_FilterDlg::SetMesh
 // Purpose : Set mesh
@@ -3195,6 +3277,15 @@ void SMESHGUI_FilterDlg::SetMesh (SMESH::SMESH_Mesh_var theMesh)
   myButtons[BTN_Apply]->setEnabled(isEnable);
 }
 
+//=======================================================================
+// name    : SMESHGUI_FilterDlg::SetGroup
+// Purpose : Set a group being edited
+//=======================================================================
+void SMESHGUI_FilterDlg::SetGroup(SMESH::SMESH_GroupOnFilter_var group)
+{
+  myGroup = group;
+}
+
 //=======================================================================
 // name    : SMESHGUI_FilterDlg::SetSelection
 // Purpose : Get filtered ids
@@ -3232,6 +3323,7 @@ bool SMESHGUI_FilterDlg::onApply()
   if (!isValid())
     return false;
 
+  SUIT_OverrideCursor wc;
   try {
     int aCurrType = myTable->GetType();
 
@@ -3241,13 +3333,18 @@ bool SMESHGUI_FilterDlg::onApply()
     insertFilterInViewer();
 
     if (!myFilter[ aCurrType ]->GetPredicate()->_is_nil()) {
-      QList<int> aResultIds;
-      filterSource(aCurrType, aResultIds);
-      // select in viewer
-      selectInViewer(aCurrType, aResultIds);
+      // 
+      bool toFilter = (( SMESH::FindActorByObject( myMesh )) ||
+                       ( myInitSourceWgOnApply && mySourceWg ) ||
+                       ( mySourceGrp->checkedId() == Dialog && mySourceWg ));
+      if ( toFilter ) {
+        QList<int> aResultIds;
+        filterSource(aCurrType, aResultIds);
+        // select in viewer
+        selectInViewer(aCurrType, aResultIds);
+      }
     }
 
-
     myInsertState[ aCurrType ] = mySetInViewer->isChecked();
     myApplyToState[ aCurrType ] = mySourceGrp->checkedId();
   }
@@ -3292,6 +3389,8 @@ bool SMESHGUI_FilterDlg::createFilter (const int theType)
     aCriteria[ i ] = aCriterion;
   }
 
+  if ( !myFilter[ theType ]->_is_nil() )
+    myFilter[ theType ]->UnRegister();
   myFilter[ theType ] = aFilterMgr->CreateFilter();
   myFilter[ theType ]->SetCriteria(aCriteria.inout());
 
@@ -3325,9 +3424,29 @@ SMESH::Filter_var SMESHGUI_FilterDlg::GetFilter() const
 
 void SMESHGUI_FilterDlg::SetFilter(SMESH::Filter_var filter, int type)
 {
+  if ( !filter->_is_nil() )
+    filter->Register();
+  if ( !myFilter[ type ]->_is_nil() )
+    myFilter[ type ]->UnRegister();
+
   myFilter[ type ] = filter;
 }
 
+//================================================================================
+/*!
+ * \brief call UnRegister() for myFilter's
+ */
+//================================================================================
+
+void SMESHGUI_FilterDlg::UnRegisterFilters()
+{
+  QMap< int, SMESH::Filter_var >::iterator i_f = myFilter.begin();
+  for ( ; i_f != myFilter.end(); ++i_f )
+    if ( !i_f.value()->_is_nil() )
+      i_f.value()->UnRegister();
+  myFilter.clear();
+}
+
 //=======================================================================
 // name    : SMESHGUI_FilterDlg::insertFilterInViewer
 // Purpose : Insert filter in viewer
@@ -3338,8 +3457,10 @@ void SMESHGUI_FilterDlg::insertFilterInViewer()
     SMESH::ElementType anEntType = (SMESH::ElementType)myTable->GetType();
 
     if (myFilter[ myTable->GetType() ]->_is_nil() ||
-         myFilter[ myTable->GetType() ]->GetPredicate()->_is_nil() ||
-         !mySetInViewer->isChecked()) {
+        myFilter[ myTable->GetType() ]->GetPredicate()->_is_nil() ||
+        !mySetInViewer->isChecked() ||
+        !myInsertEnabled )
+    {
       SMESH::RemoveFilter(getFilterId(anEntType), aSelector);
     }
     else {
@@ -3364,7 +3485,7 @@ void SMESHGUI_FilterDlg::filterSource (const int theType,
 
   int aSourceId = mySourceGrp->checkedId();
 
-  if (aSourceId == Mesh)
+  if (aSourceId == Mesh || !myDiffSourcesEnabled )
   {
     if (myMesh->_is_nil())
       return;
@@ -3572,10 +3693,10 @@ void SMESHGUI_FilterDlg::onSelectionDone()
 
   const int type = myTable->GetCriterionType(aRow);
   QList<int> types; 
-  types << SMESH::FT_BelongToGeom     << SMESH::FT_BelongToPlane 
-        << SMESH::FT_BelongToCylinder << SMESH::FT_BelongToGenSurface
-        << SMESH::FT_LyingOnGeom      << SMESH::FT_CoplanarFaces
-        << SMESH::FT_ConnectedElements;
+  types << SMESH::FT_BelongToGeom      << SMESH::FT_BelongToPlane 
+        << SMESH::FT_BelongToCylinder  << SMESH::FT_BelongToGenSurface
+        << SMESH::FT_LyingOnGeom       << SMESH::FT_CoplanarFaces
+        << SMESH::FT_ConnectedElements << SMESH::FT_BelongToMeshGroup;
   if ( !types.contains( type ))
     return;
 
@@ -3607,13 +3728,31 @@ void SMESHGUI_FilterDlg::onSelectionDone()
       }
       break;
     }
+  case SMESH::FT_BelongToMeshGroup: // get a group Name and Entry
+    {
+      SMESH::SMESH_GroupBase_var grp = SMESH::IObjectToInterface<SMESH::SMESH_GroupBase>(anIO);
+      if ( !grp->_is_nil() )
+      {
+        if ( !myMesh->_is_nil() )
+        {
+          SMESH::SMESH_Mesh_var mesh = grp->GetMesh();
+          if ( ! myMesh->_is_equivalent( mesh ))
+            return;
+        }
+        if ( !myGroup->_is_nil() && myGroup->IsInDependency( grp ))
+          return; // avoid cyclic dependencies between Groups on Filter
+
+        myTable->SetThreshold(aRow, SMESH::toQStr( grp->GetName() ));
+        myTable->SetID       (aRow, anIO->getEntry() );
+      }
+    }
   default: // get a GEOM object
     {
       GEOM::GEOM_Object_var anObj = SMESH::IObjectToInterface<GEOM::GEOM_Object>(anIO);
       if (!anObj->_is_nil())
       {
         myTable->SetThreshold(aRow, GEOMBase::GetName(anObj));
-        myTable->SetID(aRow, anIO->getEntry());
+        myTable->SetID       (aRow, anIO->getEntry());
       }
     }
   }
@@ -3698,6 +3837,23 @@ void SMESHGUI_FilterDlg::updateSelection()
     myIsSelectionChanged = true;
 
   }
+  else if ( aCriterionType == SMESH::FT_BelongToMeshGroup )
+  {
+    SMESH_TypeFilter* typeFilter = 0;
+    switch ( myTable->GetType() )
+    {
+    case SMESH::NODE   : typeFilter = new SMESH_TypeFilter( SMESH::GROUP_NODE   ); break;
+    case SMESH::ELEM0D : typeFilter = new SMESH_TypeFilter( SMESH::GROUP_0D     ); break;
+    case SMESH::BALL   : typeFilter = new SMESH_TypeFilter( SMESH::GROUP_BALL   ); break;
+    case SMESH::EDGE   : typeFilter = new SMESH_TypeFilter( SMESH::GROUP_EDGE   ); break;
+    case SMESH::FACE   : typeFilter = new SMESH_TypeFilter( SMESH::GROUP_FACE   ); break;
+    case SMESH::VOLUME : typeFilter = new SMESH_TypeFilter( SMESH::GROUP_VOLUME ); break;
+    case SMESH::ALL    : typeFilter = new SMESH_TypeFilter( SMESH::GROUP        ); break;
+    default            : typeFilter = 0;
+    }
+    if ( typeFilter )
+      mySelectionMgr->installFilter( typeFilter );
+  }
   else if ( aCriterionType == SMESH::FT_ConnectedElements )
   {
     QList<SUIT_SelectionFilter*> fList;