Salome HOME
23368: [CEA 1865] Possibility to define faces to mesh as a single one: transpatch...
[modules/smesh.git] / src / StdMeshersGUI / StdMeshersGUI_SubShapeSelectorWdg.cxx
index 4f63c559b5027472855678b4c77d0070aca20791..0ba30408664397aa29ab0530d5bebb0d15f58942 100644 (file)
@@ -1,9 +1,9 @@
-// Copyright (C) 2007-2013  CEA/DEN, EDF R&D, OPEN CASCADE
+// Copyright (C) 2007-2016  CEA/DEN, EDF R&D, OPEN CASCADE
 //
 // 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 "StdMeshersGUI_SubShapeSelectorWdg.h"
 
 // SMESH Includes
-#include "SMESH_Type.h"
-#include "SMESHGUI_MeshUtils.h"
+#include "SMESHGUI_Utils.h"
+#include "SMESHGUI_VTKUtils.h"
 #include "SMESH_Actor.h"
-#include "SMESH_PreviewActorsCollection.h"
-#include "SMESH_ActorUtils.h"
-#include "SMESHGUI_GroupUtils.h"
 #include "SMESH_Gen_i.hxx"
-#include "SMESHGUI_GEOMGenUtils.h"
 #include "SMESH_LogicalFilter.hxx"
-
-// SVTK Includes
-#include <SVTK_ViewWindow.h>
-#include <SVTK_ViewModel.h>
-#include <SVTK_ViewWindow.h>
-#include <SVTK_Selector.h>
+#include "SMESH_PreviewActorsCollection.h"
+#include "SMESH_Type.h"
 
 // SALOME GUI includes
-#include <SALOME_ListIO.hxx>
 #include <LightApp_SelectionMgr.h>
-#include <SALOME_ListIteratorOfListIO.hxx>
-
-// SUIT Includes
+#include <SALOME_ListIO.hxx>
+#include <SUIT_OverrideCursor.h>
 #include <SUIT_ResourceMgr.h>
+#include <SVTK_Selector.h>
+#include <SVTK_ViewModel.h>
+#include <SVTK_ViewWindow.h>
 
 // GEOM Includes
 #include <GEOMBase.h>
 
 // OCCT includes
 #include <TColStd_MapOfInteger.hxx>
-#include <TColStd_IndexedMapOfInteger.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopExp.hxx>
 #include <TopExp_Explorer.hxx>
-#include <StdSelect_TypeOfEdge.hxx>
 
 
 #define SPACING 6
 //================================================================================
 
 StdMeshersGUI_SubShapeSelectorWdg
-::StdMeshersGUI_SubShapeSelectorWdg( QWidget * parent, TopAbs_ShapeEnum aSubShType ): 
+::StdMeshersGUI_SubShapeSelectorWdg( QWidget *        parent,
+                                     TopAbs_ShapeEnum subShType,
+                                     const bool       toShowList ):
   QWidget( parent ),
-  myPreviewActor( 0 ),
-  myMaxSize( -1 )
+  myMaxSize( -1 ),
+  myPreviewActor( 0 )
 {
   QPixmap image0( SMESH::GetResourceMgr( mySMESHGUI )->loadPixmap( "SMESH", tr( "ICON_SELECT" ) ) );
 
   QGridLayout* edgesLayout = new QGridLayout( this );
   edgesLayout->setMargin( MARGIN );
   edgesLayout->setSpacing( SPACING );
-  
-  myListWidget   = new QListWidget( this );
-  myAddButton    = new QPushButton( tr( "SMESH_BUT_ADD" ),    this );
-  myRemoveButton = new QPushButton( tr( "SMESH_BUT_REMOVE" ), this );      
+
+  if ( toShowList )
+  {
+    myListWidget   = new QListWidget( this );
+    myAddButton    = new QPushButton( tr( "SMESH_BUT_ADD" ),    this );
+    myRemoveButton = new QPushButton( tr( "SMESH_BUT_REMOVE" ), this );
+    myListWidget->setSelectionMode( QListWidget::ExtendedSelection );
+    myListWidget->setMinimumWidth(300);
+  }
+  else
+  {
+    myListWidget   = 0;
+    myAddButton    = 0;
+    myRemoveButton = 0;
+  }
   myInfoLabel    = new QLabel( this );
   myPrevButton   = new QPushButton( "<<", this );
   myNextButton   = new QPushButton( ">>", this );
-  myListWidget->setSelectionMode( QListWidget::ExtendedSelection );
-
-  edgesLayout->addWidget(myListWidget,   0, 0, 3, 3);
-  edgesLayout->addWidget(myAddButton,    0, 3);
-  edgesLayout->addWidget(myRemoveButton, 1, 3);
-  edgesLayout->addWidget(myInfoLabel,    3, 0, 1, 3);
-  edgesLayout->addWidget(myPrevButton,   4, 0);
-  edgesLayout->addWidget(myNextButton,   4, 2);
 
-  edgesLayout->setRowStretch(2, 5);
-  edgesLayout->setColumnStretch(1, 5);
-
-  myListWidget->setMinimumWidth(300);
-  myInfoLabel->setMinimumWidth(300);
-  myInfoLabel->setSizePolicy(QSizePolicy::MinimumExpanding, QSizePolicy::Fixed);
+  if ( myListWidget )
+  {
+    edgesLayout->addWidget(myListWidget,   0, 0, 3, 3);
+    edgesLayout->addWidget(myAddButton,    0, 3);
+    edgesLayout->addWidget(myRemoveButton, 1, 3);
+    edgesLayout->addWidget(myInfoLabel,    3, 0, 1, 3);
+    edgesLayout->addWidget(myPrevButton,   4, 0);
+    edgesLayout->addWidget(myNextButton,   4, 2);
+
+    edgesLayout->setRowStretch(2, 5);
+    edgesLayout->setColumnStretch(1, 5);
+  }
+  else // show only Prev and Next buttons
+  {
+    edgesLayout->addWidget(myInfoLabel,    0, 0, 1, 2);
+    edgesLayout->addWidget(myPrevButton,   1, 0);
+    edgesLayout->addWidget(myNextButton,   1, 1);
+  }
+  //myInfoLabel->setMinimumWidth(300);
+  //myInfoLabel->setSizePolicy(QSizePolicy::MinimumExpanding, QSizePolicy::Fixed);
   myInfoLabel->setAlignment(Qt::AlignCenter);
 
-  mySubShType = aSubShType;
+  mySubShType = subShType;
 
   init();
 }
@@ -143,6 +153,8 @@ StdMeshersGUI_SubShapeSelectorWdg::~StdMeshersGUI_SubShapeSelectorWdg()
     mySelectionMgr->removeFilter( myFilter );
   delete myFilter; myFilter=0;
 
+  mySelectionMgr->clearSelected();
+
   SUIT_SelectionFilter* filter;
   foreach( filter, myGeomFilters )
     delete filter;
@@ -157,30 +169,31 @@ StdMeshersGUI_SubShapeSelectorWdg::~StdMeshersGUI_SubShapeSelectorWdg()
 void StdMeshersGUI_SubShapeSelectorWdg::init()
 {
   myParamValue = "";
-  myIsNotCorrected = true; // to dont call the GetCorrectedValue method twice
   myListOfIDs.clear();
   mySelectedIDs.clear();
 
-  myAddButton->setEnabled( false );
-  myRemoveButton->setEnabled( false );
-
   mySMESHGUI     = SMESHGUI::GetSMESHGUI();
   mySelectionMgr = SMESH::GetSelectionMgr( mySMESHGUI );
-  mySelector = (SMESH::GetViewWindow( mySMESHGUI ))->GetSelector();
+  mySelector     = (SMESH::GetViewWindow( mySMESHGUI ))->GetSelector();
 
   if ( SVTK_ViewWindow* aViewWindow = SMESH::GetViewWindow( mySMESHGUI ))
     aViewWindow->SetSelectionMode( ActorSelection );
 
   myFilter=0;
-  //setFilter();
 
-  connect( myAddButton,    SIGNAL(clicked()), SLOT(onAdd()));
-  connect( myRemoveButton, SIGNAL(clicked()), SLOT(onRemove()));
+  if ( myListWidget )
+  {
+    myAddButton->setEnabled( false );
+    myRemoveButton->setEnabled( false );
+
+    connect( myListWidget,   SIGNAL(itemSelectionChanged()),    this, SLOT(onListSelectionChanged()));
+    connect( myAddButton,    SIGNAL(clicked()), SLOT(onAdd()));
+    connect( myRemoveButton, SIGNAL(clicked()), SLOT(onRemove()));
+  }
   connect( myPrevButton,   SIGNAL(clicked()), SLOT(onPrevious()));
   connect( myNextButton,   SIGNAL(clicked()), SLOT(onNext()));
-  
-  connect( mySelectionMgr, SIGNAL(currentSelectionChanged()), this, SLOT(SelectionIntoArgument()));
-  connect( myListWidget,   SIGNAL(itemSelectionChanged()),    this, SLOT(onListSelectionChanged()));
+
+  connect( mySelectionMgr, SIGNAL(currentSelectionChanged()), this, SLOT(selectionIntoArgument()));
 
   updateState();
 }
@@ -193,14 +206,17 @@ void StdMeshersGUI_SubShapeSelectorWdg::init()
 
 void StdMeshersGUI_SubShapeSelectorWdg::setFilter()
 {
-  SalomeApp_Study* study = mySMESHGUI->activeStudy();
-  GEOM_TypeFilter* typeFilter = new GEOM_TypeFilter(study, mySubShType, /*isShapeType=*/true );
-  GEOM_CompoundFilter* gpoupFilter = new GEOM_CompoundFilter(study);
-  gpoupFilter->addSubType( mySubShType );
-  myGeomFilters.append( typeFilter );
-  myGeomFilters.append( gpoupFilter );
-  myFilter = new SMESH_LogicalFilter( myGeomFilters, SMESH_LogicalFilter::LO_OR );
-  mySelectionMgr->installFilter( myFilter );
+  // if ( !myFilter )
+  // {
+  //   SalomeApp_Study* study = mySMESHGUI->activeStudy();
+  //   GEOM_TypeFilter* typeFilter = new GEOM_TypeFilter(study, mySubShType, /*isShapeType=*/true );
+  //   GEOM_CompoundFilter* gpoupFilter = new GEOM_CompoundFilter(study);
+  //   gpoupFilter->addSubType( mySubShType );
+  //   myGeomFilters.append( typeFilter );
+  //   myGeomFilters.append( gpoupFilter );
+  //   myFilter = new SMESH_LogicalFilter( myGeomFilters, SMESH_LogicalFilter::LO_OR );
+  // }
+  // mySelectionMgr->installFilter( myFilter );
 }
 
 //================================================================================
@@ -209,7 +225,7 @@ void StdMeshersGUI_SubShapeSelectorWdg::setFilter()
  */
 //================================================================================
 
-void StdMeshersGUI_SubShapeSelectorWdg::showPreview( bool visible)
+void StdMeshersGUI_SubShapeSelectorWdg::ShowPreview( bool visible)
 {
   if ( !myPreviewActor )
     return;
@@ -224,11 +240,44 @@ void StdMeshersGUI_SubShapeSelectorWdg::showPreview( bool visible)
   }
 }
 
+//================================================================================
+/*!
+ * \brief Connect/disconnect to change of selection
+ */
+//================================================================================
+
+void StdMeshersGUI_SubShapeSelectorWdg::ActivateSelection( bool toActivate )
+{
+  if ( !mySelectionMgr ) return;
+
+  if ( toActivate )
+  {
+    connect( mySelectionMgr, SIGNAL(currentSelectionChanged()), this, SLOT(selectionIntoArgument()));
+  }
+  else
+  {
+    disconnect(mySelectionMgr, 0, this, 0 );
+  }
+}
+
+//================================================================================
+/*!
+ * \brief Clears selected IDs. This is a workaround of a bug that
+ *        SUIT_SelectionMgr::clearSelected() does not emit currentSelectionChanged
+ */
+//================================================================================
+
+void StdMeshersGUI_SubShapeSelectorWdg::ClearSelected()
+{
+  mySelectedIDs.clear();
+  selectionIntoArgument();
+}
+
 //=================================================================================
-// function : SelectionIntoArgument()
-// purpose  : Called when selection as changed or other case
+// function : selectionIntoArgument()
+// purpose  : Called when selection has changed or in other cases
 //=================================================================================
-void StdMeshersGUI_SubShapeSelectorWdg::SelectionIntoArgument()
+void StdMeshersGUI_SubShapeSelectorWdg::selectionIntoArgument()
 {
   if ( !myPreviewActor )
     return;
@@ -242,31 +291,36 @@ void StdMeshersGUI_SubShapeSelectorWdg::SelectionIntoArgument()
 
   if (nbSel > 0) {
     SALOME_ListIteratorOfListIO anIt (aList);
-    
-    for ( ; anIt.More(); anIt.Next()) { // Loop on selected objects
+
+    for ( ; anIt.More(); anIt.Next()) // Loop on selected objects
+    {
       Handle(SALOME_InteractiveObject) IO = anIt.Value();
-      
+
       GEOM::GEOM_Object_var aGeomObj = GetGeomObjectByEntry( IO->getEntry() );
       if ( !CORBA::is_nil( aGeomObj ) ) { // Selected Object From Study
-        GEOM::GEOM_Object_var aGeomFatherObj = aGeomObj->GetMainShape();
-        QString aFatherEntry = "";
-        QString aMainFatherEntry = "";
-        TopoDS_Shape shape;
-        if ( !CORBA::is_nil( aGeomFatherObj ) ) {
-          // Get Main Shape
-          GEOM::GEOM_Object_var aGeomMain = GetGeomObjectByEntry( myEntry );
-          if ( !CORBA::is_nil( aGeomMain ) && aGeomMain->GetType() == 37 ) {  // Main Shape is a Group
-            GEOM::GEOM_Object_var aMainFatherObj = aGeomMain->GetMainShape();
-            if ( !CORBA::is_nil( aMainFatherObj ) )
-              aMainFatherEntry = aMainFatherObj->GetStudyEntry();
-          }
-          aFatherEntry = aGeomFatherObj->GetStudyEntry();
-        }
-
-        if ( aFatherEntry != "" && ( aFatherEntry == myEntry || aFatherEntry == aMainFatherEntry ) )
+        // commented for IPAL52836
+        //
+        // GEOM::GEOM_Object_var aGeomFatherObj = aGeomObj->GetMainShape();
+        // QString aFatherEntry = "";
+        // QString aMainFatherEntry = "";
+        // TopoDS_Shape shape;
+        // if ( !CORBA::is_nil( aGeomFatherObj ) ) {
+        //   // Get Main Shape
+        //   GEOM::GEOM_Object_var aGeomMain = GetGeomObjectByEntry( myEntry.c_str() );
+        //   if ( !CORBA::is_nil( aGeomMain ) && aGeomMain->GetType() == 37 ) {  // Main Shape is a Group
+        //     GEOM::GEOM_Object_var aMainFatherObj = aGeomMain->GetMainShape();
+        //     if ( !CORBA::is_nil( aMainFatherObj ) )
+        //       aMainFatherEntry = aMainFatherObj->GetStudyEntry();
+        //   }
+        //   aFatherEntry = aGeomFatherObj->GetStudyEntry();
+        // }
+
+        // if (( ! aFatherEntry.isEmpty() ) &&
+        //     ( aFatherEntry == myEntry.c_str() || aFatherEntry == aMainFatherEntry ) )
         {
+          TopoDS_Shape shape;
           if ( aGeomObj->GetType() == 37 /*GEOM_GROUP*/ ) { // Selected Group that belongs the main object
-            GEOMBase::GetShape(aGeomObj, shape); 
+            GEOMBase::GetShape(aGeomObj, shape);
             if ( !shape.IsNull() ) {
               TopExp_Explorer exp( shape, mySubShType );
               for ( ; exp.More(); exp.Next() ) {
@@ -277,8 +331,10 @@ void StdMeshersGUI_SubShapeSelectorWdg::SelectionIntoArgument()
                 }
               }
             }
-          } else if ( aGeomObj->GetType() == 28 /*GEOM_SUBSHAPE*/  ) {
-            GEOMBase::GetShape(aGeomObj, shape); 
+          } else if ( aGeomObj->GetType() == 28 /*GEOM_SUBSHAPE*/ ||
+                      myEntry == IO->getEntry() )
+          {
+            GEOMBase::GetShape(aGeomObj, shape);
             if ( !shape.IsNull() && shape.ShapeType() == mySubShType ) {
               int index = myPreviewActor->GetIndexByShape( shape );
               if ( index ) {
@@ -300,23 +356,28 @@ void StdMeshersGUI_SubShapeSelectorWdg::SelectionIntoArgument()
     }
   }
   // update add button
-  myAddButton->setEnabled( ( myListWidget->count() < myMaxSize || myMaxSize == -1 ) &&
-                           mySelectedIDs.size() > 0 &&
-                           ( mySelectedIDs.size() <= myMaxSize || myMaxSize == -1 ) );
-
-  //Connect Selected Ids in viewer and dialog's Ids list
-  bool signalsBlocked = myListWidget->blockSignals( true );
-  myListWidget->clearSelection();
-  if ( mySelectedIDs.size() > 0 ) {
-    for (int i = 0; i < mySelectedIDs.size(); i++) {
-      QString anID = QString(" %1").arg( mySelectedIDs.at(i) );
-      QList<QListWidgetItem*> anItems = myListWidget->findItems ( anID, Qt::MatchExactly );
-      QListWidgetItem* item;
-      foreach(item, anItems)
-        item->setSelected(true);
+  if ( myListWidget )
+  {
+    myAddButton->setEnabled(( myListWidget->count() < myMaxSize || myMaxSize == -1 ) &&
+                            ( mySelectedIDs.size() > 0                             ) &&
+                            ( mySelectedIDs.size() <= myMaxSize || myMaxSize == -1 ) );
+
+    //Connect Selected Ids in viewer and dialog's Ids list
+    bool signalsBlocked = myListWidget->blockSignals( true );
+    myListWidget->clearSelection();
+    if ( mySelectedIDs.size() > 0 ) {
+      for (int i = 0; i < mySelectedIDs.size(); i++) {
+        QString anID = QString(" %1").arg( mySelectedIDs.at(i) );
+        QList<QListWidgetItem*> anItems = myListWidget->findItems ( anID, Qt::MatchExactly );
+        QListWidgetItem* item;
+        foreach(item, anItems)
+          item->setSelected(true);
+      }
     }
+    myListWidget->blockSignals( signalsBlocked );
   }
-  myListWidget->blockSignals( signalsBlocked );
+
+  emit shapeSelected();
 }
 
 //=================================================================================
@@ -325,7 +386,7 @@ void StdMeshersGUI_SubShapeSelectorWdg::SelectionIntoArgument()
 //=================================================================================
 void StdMeshersGUI_SubShapeSelectorWdg::onAdd()
 {
-  if ( mySelectedIDs.size() < 1 )
+  if ( mySelectedIDs.size() < 1 || !myListWidget )
     return;
 
   myListWidget->blockSignals( true );
@@ -341,7 +402,9 @@ void StdMeshersGUI_SubShapeSelectorWdg::onAdd()
   }
   onListSelectionChanged();
   myListWidget->blockSignals( false );
-  myAddButton->setEnabled( myMaxSize == -1 || myListOfIDs.size() < myMaxSize );
+
+  mySelectedIDs.clear();
+  myAddButton->setEnabled( false );
 }
          
 //=================================================================================
@@ -350,7 +413,7 @@ void StdMeshersGUI_SubShapeSelectorWdg::onAdd()
 //=================================================================================
 void StdMeshersGUI_SubShapeSelectorWdg::onRemove()
 {
-  if ( myListWidget->count() < 1 )
+  if ( myListWidget->count() < 1 || !myListWidget )
     return;
 
   myListWidget->blockSignals( true );
@@ -368,14 +431,15 @@ void StdMeshersGUI_SubShapeSelectorWdg::onRemove()
   onListSelectionChanged();
   myListWidget->blockSignals( false );
   
-  myAddButton->setEnabled( true );
+  myAddButton->setEnabled( !mySelectedIDs.isEmpty() );
 }
 
 void StdMeshersGUI_SubShapeSelectorWdg::onPrevious()
 {
   if ( myPreviewActor ) {
     myPreviewActor->previous();
-    myListWidget->clearSelection();
+    if ( myListWidget )
+      myListWidget->clearSelection();
     updateButtons();
     if ( SVTK_ViewWindow* aViewWindow = SMESH::GetViewWindow( mySMESHGUI ))
       aViewWindow->Repaint();
@@ -386,7 +450,8 @@ void StdMeshersGUI_SubShapeSelectorWdg::onNext()
 {
   if ( myPreviewActor ) {
     myPreviewActor->next();
-    myListWidget->clearSelection();
+    if ( myListWidget )
+      myListWidget->clearSelection();
     updateButtons();
     if ( SVTK_ViewWindow* aViewWindow = SMESH::GetViewWindow( mySMESHGUI ))
       aViewWindow->Repaint();
@@ -402,29 +467,46 @@ void StdMeshersGUI_SubShapeSelectorWdg::onListSelectionChanged()
   if ( !myPreviewActor )
     return;
 
-  mySelectionMgr->clearSelected();
-  TColStd_MapOfInteger aIndexes;
+  myPreviewActor->HighlightAll( false );
   QList<QListWidgetItem*> selItems = myListWidget->selectedItems();
   QListWidgetItem* anItem;
   foreach(anItem, selItems)
     myPreviewActor->HighlightID( anItem->text().toInt() );
 
+  if ( SVTK_ViewWindow* aViewWindow = SMESH::GetViewWindow( mySMESHGUI ))
+    aViewWindow->Repaint();
+
   // update remove button
   myRemoveButton->setEnabled( selItems.size() > 0 );
+
+  emit selectionChanged();
 }
 
 //=================================================================================
 // function : setGeomShape
 // purpose  : Called to set geometry whose sub-shapes are selected
 //================================================================================
-void StdMeshersGUI_SubShapeSelectorWdg::SetGeomShapeEntry( const QString& theEntry )
+void StdMeshersGUI_SubShapeSelectorWdg::SetGeomShapeEntry( const QString& theEntry,
+                                                           const QString& theMainShapeEntry )
 {
-  if ( theEntry != "") {
+  if ( !theEntry.isEmpty() || theMainShapeEntry.isEmpty() )
+  {
     myParamValue = theEntry;
-    myEntry = theEntry;
-    myGeomShape = GetTopoDSByEntry( theEntry );
+    myEntry      = theEntry.toStdString();
+    myMainEntry  = theMainShapeEntry.toStdString();
+
+    if ( myMainEntry.empty() ) myMainEntry = myEntry;
+    if ( myEntry.empty() )     myEntry     = myMainEntry;
+    if ( myMainEntry.length() > myEntry.length() &&
+         theMainShapeEntry.startsWith( theEntry ))
+      std::swap( myMainEntry, myEntry );
+
+    myGeomShape = GetTopoDSByEntry( myEntry.c_str() );
+    if ( myEntry == myMainEntry )
+      myMainShape = myGeomShape;
+    else
+      myMainShape = GetTopoDSByEntry( myMainEntry.c_str() );
     updateState();
-    myIsNotCorrected = true;
   }
 }
 
@@ -434,20 +516,23 @@ void StdMeshersGUI_SubShapeSelectorWdg::SetGeomShapeEntry( const QString& theEnt
 //=================================================================================
 void StdMeshersGUI_SubShapeSelectorWdg::updateState()
 {
-  bool state = false;
-  if ( !myGeomShape.IsNull() )
-    state = true;
-  myInfoLabel->setVisible( false );
+  bool state = ( !myGeomShape.IsNull() );
+
+  myInfoLabel ->setVisible( false );
   myPrevButton->setVisible( false );
   myNextButton->setVisible( false );
-  
-  myListWidget->setEnabled( state );
-  myAddButton->setEnabled( mySelectedIDs.size() > 0 );
-  
-  if (state) {
-    myPreviewActor = new SMESH_PreviewActorsCollection();
+
+  if ( myListWidget )
+  {
+    myListWidget->setEnabled( state );
+    myAddButton->setEnabled( mySelectedIDs.size() > 0 );
+  }
+  if ( state ) {
+    SUIT_OverrideCursor wc;
+    if ( !myPreviewActor )
+      myPreviewActor = new SMESH_PreviewActorsCollection();
     myPreviewActor->SetSelector( mySelector );
-    myPreviewActor->Init( myGeomShape, mySubShType, myEntry );
+    myPreviewActor->Init( myGeomShape, myMainShape, mySubShType, myEntry.c_str() );
     myPreviewActor->SetShown( false );
     myIsShown = false;
     if ( SVTK_ViewWindow* aViewWindow = SMESH::GetViewWindow( mySMESHGUI )) {
@@ -500,24 +585,19 @@ SMESH::long_array_var StdMeshersGUI_SubShapeSelectorWdg::GetListOfIDs()
 {
   SMESH::long_array_var anArray = new SMESH::long_array;
 
-  if ( myMainEntry != "" && myIsNotCorrected )
-    myListOfIDs = GetCorrectedListOfIDs( true );
-
   int size = myListOfIDs.size();
   anArray->length( size );
-  if ( size ) {
-    for (int i = 0; i < size; i++) {
-        anArray[i] = myListOfIDs.at(i);
-    }
-  }
+  for (int i = 0; i < size; i++)
+    anArray[i] = myListOfIDs.at(i);
+
   return anArray;
 }
 
 //=================================================================================
 // function : SetListOfIds
-// purpose  : Called to set the list of SubShapes IDs
+// purpose  : Called to set the list of SubShapes IDs. Returns false if any ID is invalid
 //=================================================================================
-void StdMeshersGUI_SubShapeSelectorWdg::SetListOfIDs( SMESH::long_array_var theIds)
+bool StdMeshersGUI_SubShapeSelectorWdg::SetListOfIDs( SMESH::long_array_var theIds)
 {
   mySelectedIDs.clear();
   myListOfIDs.clear();
@@ -525,19 +605,28 @@ void StdMeshersGUI_SubShapeSelectorWdg::SetListOfIDs( SMESH::long_array_var theI
   for ( int i = 0; i < size; i++ )
     mySelectedIDs.append( theIds[ i ] );
 
-  mySelectedIDs = GetCorrectedListOfIDs( false );
-  onAdd();
-}
+  if ( myListWidget )
+  {
+    myListWidget->blockSignals( true );
+    myListWidget->clear();
+    myListWidget->blockSignals( false );
+  }
 
-//=================================================================================
-// function : SetMainShapeEntry
-// purpose  : Called to set the Entry of main shape of the mesh
-//=================================================================================
-void StdMeshersGUI_SubShapeSelectorWdg::SetMainShapeEntry( const QString& theEntry )
-{
-  myMainEntry = theEntry;
-  myMainShape = GetTopoDSByEntry( theEntry );
-  myIsNotCorrected = true;
+  bool isOk = true;
+  if ( myPreviewActor )
+  {
+    for ( int i = 0; i < size && isOk; i++ )
+      isOk = myPreviewActor->IsValidIndex( theIds[ i ] );
+  }
+  else if ( !myMainShape.IsNull() )
+  {
+    TopTools_IndexedMapOfShape aMainMap;
+    TopExp::MapShapes(myMainShape, aMainMap);
+    for ( int i = 0; i < size && isOk; i++ )
+      isOk = ( theIds[ i ] > 0 && theIds[ i ] <= aMainMap.Extent() );
+  }
+  onAdd();
+  return isOk;
 }
 
 //=================================================================================
@@ -546,47 +635,8 @@ void StdMeshersGUI_SubShapeSelectorWdg::SetMainShapeEntry( const QString& theEnt
 //=================================================================================
 const char* StdMeshersGUI_SubShapeSelectorWdg::GetMainShapeEntry()
 {
-  if ( myMainEntry == "")
-    return myEntry.toLatin1().data();
-
-  return myMainEntry.toLatin1().data();
-}
-
-//=================================================================================
-// function : GetCorrectedListOfIds
-// purpose  : Called to convert the list of IDs from sub-shape IDs to main shape IDs
-//=================================================================================
-QList<int> StdMeshersGUI_SubShapeSelectorWdg::GetCorrectedListOfIDs( bool fromSubshapeToMainshape )
-{
-  if ( ( myMainShape.IsNull() || myGeomShape.IsNull() ) &&  fromSubshapeToMainshape )
-    return myListOfIDs;
-  else   if ( ( myMainShape.IsNull() || myGeomShape.IsNull() ) &&  !fromSubshapeToMainshape )
-    return mySelectedIDs;
-
-  QList<int> aList;
-  TopTools_IndexedMapOfShape   aGeomMap;
-  TopTools_IndexedMapOfShape   aMainMap;
-  TopExp::MapShapes(myGeomShape, aGeomMap);
-  TopExp::MapShapes(myMainShape, aMainMap);
-
-  if ( fromSubshapeToMainshape ) { // convert indexes from sub-shape to mainshape
-    int size = myListOfIDs.size();
-    for (int i = 0; i < size; i++) {
-      TopoDS_Shape aSubShape = aGeomMap.FindKey( myListOfIDs.at(i) );
-      int index = aMainMap.FindIndex( aSubShape );
-      aList.append( index );
-    }
-    myIsNotCorrected = false;
-  } else { // convert indexes from main shape to sub-shape
-    int size = mySelectedIDs.size();
-    for (int i = 0; i < size; i++) {
-      TopoDS_Shape aSubShape = aMainMap.FindKey( mySelectedIDs.at(i) );
-      int index = aGeomMap.FindIndex( aSubShape );
-      aList.append( index );
-    }
-  }
-
-  return aList;
+  if ( myMainEntry.empty() ) myMainEntry = "";
+  return myMainEntry.c_str();
 }
 
 void StdMeshersGUI_SubShapeSelectorWdg::updateButtons()