Salome HOME
Double editor in table
[modules/hydro.git] / src / HYDROCurveCreator / CurveCreator_Widget.cxx
index ec33d24990a920e03bb8affe7e50858369d74676..56ae6c123358bb31b87f131dac788d00fccb8e93 100644 (file)
@@ -25,6 +25,7 @@
 //#include "CurveCreator_NewPointDlg.h"
 #include "CurveCreator_NewSectionDlg.h"
 #include "CurveCreator_Utils.h"
+#include "CurveCreator_TableView.h"
 
 #include <SUIT_Session.h>
 #include <SUIT_Desktop.h>
 #include <QTableWidget>
 
 const double LOCAL_SELECTION_TOLERANCE = 0.0001;
-const int POINT_INDEX_COLUMN_WIDTH = 50;
+const int SECTION_NAME_COLUMN_WIDTH = 75;
+const int POINT_INDEX_COLUMN_WIDTH = 40;
 
 const int SCENE_PIXEL_TOLERANCE = 10;
 
 CurveCreator_Widget::CurveCreator_Widget(QWidget* parent,
                                          CurveCreator_ICurve *theCurve,
                                          Qt::WindowFlags fl)
-: QWidget(parent), myNewSectionEditor(NULL), myCurve(theCurve), mySection(0), myDragStarted( false ),
+: QWidget(parent), myNewSectionEditor(NULL), myCurve(theCurve), mySection(0),
+  myDragStarted( false ), myDragInteractionStyle( SUIT_ViewModel::STANDARD ),
   myOCCViewer( 0 )
 {
   myNewSectionEditor = new CurveCreator_NewSectionDlg( this );
@@ -94,15 +97,17 @@ CurveCreator_Widget::CurveCreator_Widget(QWidget* parent,
   connect( mySectionView, SIGNAL(customContextMenuRequested(QPoint)), this, SLOT(onContextMenu(QPoint)) );
 
   myLocalPointView = new QTableWidget();
+  myLocalPointView->setItemDelegate( new CurveCreator_TableItemDelegate( myLocalPointView ) );
   myLocalPointView->setVisible( false );
-  myLocalPointView->setColumnCount( 3 );
-  myLocalPointView->setColumnWidth( 0, POINT_INDEX_COLUMN_WIDTH );
+  myLocalPointView->setColumnCount( 4 );
+  myLocalPointView->setColumnWidth( 0, SECTION_NAME_COLUMN_WIDTH );
+  myLocalPointView->setColumnWidth( 1, POINT_INDEX_COLUMN_WIDTH );
   QStringList aLabels;
-  //aLabels << tr( "IDENTIFIER_LABEL" ) << tr( "X_POSITION_LBL" ) << tr( "Y_POSITION_LBL" );
-  aLabels << tr( "id" ) << tr( "X" ) << tr( "Y" );
+  //aLabels << tr( "SECTION_LABEL" ) << tr( "IDENTIFIER_LABEL" ) << tr( "X_POSITION_LBL" ) << tr( "Y_POSITION_LBL" );
+  aLabels << tr( "Section" ) << "Index" << tr( "X" ) << tr( "Y" );
   myLocalPointView->setHorizontalHeaderLabels( aLabels );
   connect( myLocalPointView, SIGNAL( cellChanged( int, int ) ),
-           this, SLOT( onLocalPointChanged( int, int ) ) );
+           this, SLOT( onCellChanged( int, int ) ) );
 
   QToolBar* aTB = new QToolBar(tr("TOOL_BAR_TLT"), aSectionGroup);
 //    QToolButton* anUndoBtn = new QToolButton(aTB);
@@ -236,9 +241,6 @@ void CurveCreator_Widget::setOCCViewer( OCCViewer_Viewer* theViewer )
 
   myOCCViewer = theViewer;
   if ( myOCCViewer ) {
-    //OCCViewer_ViewWindow* aWnd = dynamic_cast<OCCViewer_ViewWindow*>(myOCCViewer->getViewManager()->getActiveView());
-    //if ( aWnd )
-    //  aWnd->installEventFilter( this );
     OCCViewer_ViewManager* aViewManager = dynamic_cast<OCCViewer_ViewManager*>
                                                     ( myOCCViewer->getViewManager() );
     connect( aViewManager, SIGNAL( mousePress( SUIT_ViewWindow*, QMouseEvent* ) ),
@@ -258,27 +260,6 @@ OCCViewer_Viewer* CurveCreator_Widget::getOCCViewer()
   return myOCCViewer;
 }
 
-/*!
-  \brief Customize event handling
-  \param watched event receiver object
-  \param e event
-  \return \c true if the event processing should be stopped
-*/
-/*bool CurveCreator_Widget::eventFilter( QObject* theWatched, QEvent* theEvent )
-{
-  OCCViewer_Viewer* aViewer = getOCCViewer();
-  if ( !aViewer )
-    return;
-  Handle(AIS_InteractiveContext) aContext = aViewer->getAISContext();
-  bool isLocalContext = aContext->HasOpenedContext();
-  if ( !isLocalContext )
-    return QWidget::eventFilter( theWatched, theEvent );
-
-  bool isProcessed = true;
-
-  return isProcessed;
-}*/
-
 //=======================================================================
 // function: getUniqSectionName
 // purpose: return unique section name
@@ -312,7 +293,9 @@ void CurveCreator_Widget::onSelectionChanged()
 {
   QList<ActionId> anEnabledAct;
   if( myCurve ){
-    anEnabledAct << NEW_SECTION_ID;
+    anEnabledAct << NEW_SECTION_ID << MODIFICATION_MODE_ID;
+    if ( removeEnabled() )
+      anEnabledAct << REMOVE_ID;
     QList<int> aSelSections = mySectionView->getSelectedSections();
     QList< QPair< int, int > > aSelPoints = mySectionView->getSelectedPoints();
     CurveCreator_TreeView::SelectionType aSelType = mySectionView->getSelectionType();
@@ -325,8 +308,7 @@ void CurveCreator_Widget::onSelectionChanged()
         anEnabledAct << UP_ID;
       }*/
       if( aSelSections.size() == 1 ){
-        anEnabledAct << ADDITION_MODE_ID << MODIFICATION_MODE_ID << DETECTION_MODE_ID;
-        anEnabledAct << REMOVE_ID;
+        anEnabledAct << ADDITION_MODE_ID << DETECTION_MODE_ID;
       }
       switch ( getActionMode() ) {
         case AdditionMode: {
@@ -591,9 +573,14 @@ void CurveCreator_Widget::onModifySection()
   bool isClosed = myNewSectionEditor->isClosed();
   CurveCreator::SectionType aSectType = myNewSectionEditor->getSectionType();
 //  myCurve->startOperation();
-  myCurve->setClosed( isClosed, mySection );
-  myCurve->setSectionName( mySection , aName.toStdString() );
-  myCurve->setSectionType( mySection, aSectType );
+  if( myCurve->getSectionName(mySection) != aName.toStdString() )
+    myCurve->setSectionName( mySection , aName.toStdString() );
+
+  if( myCurve->getSectionType(mySection) != aSectType )
+    myCurve->setSectionType( mySection, aSectType );
+
+  if( myCurve->isClosed(mySection) != isClosed )
+    myCurve->setClosed( mySection, isClosed );
 //  myCurve->finishOperation();
   mySectionView->sectionChanged(mySection);
   updateUndoRedo();
@@ -726,7 +713,7 @@ void CurveCreator_Widget::onCloseSections()
 //  myCurve->startOperation();
   QList<int> aSelSections = mySectionView->getSelectedSections();
   for( int i = 0 ; i < aSelSections.size() ; i++ ){
-    myCurve->setClosed(true, aSelSections[i]);
+    myCurve->setClosed(aSelSections[i], true);
     mySectionView->sectionChanged(aSelSections[i]);
   }
 //  myCurve->finishOperation();
@@ -740,7 +727,7 @@ void CurveCreator_Widget::onUncloseSections()
 //  myCurve->startOperation();
   QList<int> aSelSections = mySectionView->getSelectedSections();
   for( int i = 0 ; i < aSelSections.size() ; i++ ){
-    myCurve->setClosed(false, aSelSections[i]);
+    myCurve->setClosed(aSelSections[i], false);
     mySectionView->sectionChanged(aSelSections[i]);
   }
 //  myCurve->finishOperation();
@@ -751,18 +738,23 @@ void CurveCreator_Widget::onUndo()
 {
     if( !myCurve )
       return;
+
+    CurveCreator_Widget::SectionToPointList aPoints;
+    startCurveModification( aPoints, false );
     myCurve->undo();
+    finishCurveModification();
     mySectionView->reset();
-    updateUndoRedo();
 }
 
 void CurveCreator_Widget::onRedo()
 {
     if( !myCurve )
       return;
+    CurveCreator_Widget::SectionToPointList aPoints;
+    startCurveModification( aPoints, false );
     myCurve->redo();
+    finishCurveModification();
     mySectionView->reset();
-    updateUndoRedo();
 }
 
 void CurveCreator_Widget::updateUndoRedo()
@@ -838,6 +830,30 @@ QList< QPair< int, int > > CurveCreator_Widget::getSelectedPoints()
   return mySectionView->getSelectedPoints();
 }
 
+/**
+ * According to the widget state, performs the remove action
+ */
+void CurveCreator_Widget::removeSelected()
+{
+  onRemove();
+}
+
+/**
+ * Checks whether there are some selection to be removed
+ */
+bool CurveCreator_Widget::removeEnabled()
+{
+  bool isEnabled = getActionMode() == ModificationMode;
+  if ( !isEnabled ) {
+    QList<int> aSelSections = mySectionView->getSelectedSections();
+    CurveCreator_TreeView::SelectionType aSelType = mySectionView->getSelectionType();
+    isEnabled = aSelType == CurveCreator_TreeView::ST_SECTIONS &&
+                aSelSections.size() == 1;
+  }
+  return isEnabled;
+}
+
+
 //=================================================================================
 // function : GeometryGUI::onGetCoordsByClick()
 // purpose  : Manage mouse press events in Additon mode
@@ -920,8 +936,11 @@ void CurveCreator_Widget::onMouseRelease( SUIT_ViewWindow*, QMouseEvent* theEven
   if ( getActionMode() != ModificationMode )
     return;
 
-  if ( myDragStarted )
+  if ( myDragStarted ) {
+    if ( myDragged ) // if the drag of some points has happened, restore the drag selection
+      setSelectedPonts( myDragPoints );
     setDragStarted( false );
+  }
   else // check whether the segment is clicked an a new point should be added to the segment
     insertPointToSelectedSegment( theEvent->pos().x(), theEvent->pos().y() );
 
@@ -946,37 +965,24 @@ void CurveCreator_Widget::onMouseMove( SUIT_ViewWindow*, QMouseEvent* theEvent )
   myDragStartPosition = aPos;
 }
 
-void CurveCreator_Widget::onLocalPointChanged( int theRow, int theColumn )
+void CurveCreator_Widget::onCellChanged( int theRow, int theColumn )
 {
-  QList<int> aSelSections = mySectionView->getSelectedSections();
-  if ( aSelSections.size() < 0 )
-    return;
+  int aCurrSect = getSectionId( theRow );
+  int aPntIndex = getPointId( theRow );
 
-  int aSection = aSelSections[0];
+  if ( aPntIndex < 0 )
+    return;
 
-  QList<int> aSelPoints;
+  SectionToPointList aSelPoints;
   startCurveModification( aSelPoints );
 
-  int aPntIndex = -1;
-  int aCurrSect=-1;
+  double aX  = myLocalPointView->item( theRow, 2 )->data( Qt::UserRole ).toDouble();
+  double anY = myLocalPointView->item( theRow, 3 )->data( Qt::UserRole ).toDouble();
   std::deque<float> aChangedPos;
-  float aPrevX, aPrevY, aX, anY;
-  //for( int i = 0 ; i < aSelSections.size() ; i++ ){
-    aCurrSect = aSection;//aSelSections[i];
-
-    aPrevX = myLocalPointView->item( theRow, 1 )->data( Qt::UserRole ).toDouble();
-    aPrevY = myLocalPointView->item( theRow, 2 )->data( Qt::UserRole ).toDouble();
-
-    aPntIndex = findLocalPointIndex( aCurrSect, aPrevX, aPrevY );
-    if ( aPntIndex >= 0 ) {
-      aX  = myLocalPointView->item( theRow, 1 )->text().toDouble();
-      anY = myLocalPointView->item( theRow, 2 )->text().toDouble();
-      aChangedPos.clear();
-      aChangedPos.push_back( aX );
-      aChangedPos.push_back( anY );
-      myCurve->setPoint( aCurrSect, aPntIndex, aChangedPos );
-    }
-  //}
+  aChangedPos.push_back( aX );
+  aChangedPos.push_back( anY );
+  myCurve->setPoint( aCurrSect, aPntIndex, aChangedPos );
+
   finishCurveModification( aSelPoints );
 }
 
@@ -1015,23 +1021,16 @@ void CurveCreator_Widget::removeSection()
  */
 void CurveCreator_Widget::removePoint()
 {
-  int aSectionId = 0;
-  QList<int> aPoints;
-  getSelectedPonts( aSectionId, aPoints );
+  SectionToPointList aPoints;
+  getSelectedPonts( aPoints );
   if ( aPoints.size() == 0 )
     return;
 
-  QList<int> aSelPoints;
+  SectionToPointList aSelPoints;
   startCurveModification( aSelPoints, false );
 
-  // the points should be removed in a decreased order
-  qSort( aPoints );
-  for( int i = aPoints.size()-1; i >= 0; i-- ){
-    int aPntIndx = aPoints[i];
-    myCurve->removePoint( aSectionId, aPntIndx );
-    mySectionView->pointsRemoved( aSectionId, aPntIndx );
-  }
-  finishCurveModification( QList<int>() );
+  myCurve->removeSeveralPoints( aPoints );
+  finishCurveModification( SectionToPointList() );
 }
 
 void CurveCreator_Widget::insertPointToSelectedSegment( const int theX,
@@ -1041,8 +1040,8 @@ void CurveCreator_Widget::insertPointToSelectedSegment( const int theX,
   if ( !aViewer )
     return;
 
-  int aPoint1 = -1, aPoint2 = -1;
   gp_Pnt aPoint;
+  gp_Pnt aPoint1, aPoint2;
   bool isFoundPoint = false;
   Handle(AIS_InteractiveContext) aContext = aViewer->getAISContext();
   for ( aContext->InitSelected(); aContext->MoreSelected() && !isFoundPoint;
@@ -1059,34 +1058,50 @@ void CurveCreator_Widget::insertPointToSelectedSegment( const int theX,
       isFoundPoint = pointOnObject( anAIS, theX, theY, aPoint, aPoint1, aPoint2 );
     }
   }
-  bool isDone = false;
-  if ( !isFoundPoint || aPoint1 < 0 || aPoint2 < 0 )
+  if ( !isFoundPoint )
     return;
 
   // insert the point to the model curve
-  QList<int> aSelPoints;
+  SectionToPointList aSelPoints;
   startCurveModification( aSelPoints );
 
-  int aSection = 0;
   CurveCreator::Coordinates aCoords;
   aCoords.push_back( aPoint.X() );
   aCoords.push_back( aPoint.Y() );
 
+  SectionToPointList aPoints1, aPoints2;
+  findSectionsToPoints( aPoint1.X(), aPoint1.Y(), aPoints1 );
+  findSectionsToPoints( aPoint2.X(), aPoint2.Y(), aPoints2 );
+  SectionToPointList::const_iterator anIt = aPoints1.begin(), aLast = aPoints1.end();
+  int aSectionId = -1;
+  // there can be a case when a new point is added into two sections
+  int aPoint1Id = -1, aPoint2Id = -1;
+  for ( ; anIt != aLast && aSectionId < 0; anIt++ ) {
+    int aSectionCur = anIt->first;
+    SectionToPointList::const_iterator anIt2 = aPoints2.begin(), aLast2 = aPoints2.end();
+    for ( ; anIt2 != aLast2 && aSectionId < 0; anIt2++ ) {
+      if ( anIt2->first == aSectionCur ) {
+        aSectionId = aSectionCur;
+        aPoint1Id = anIt->second;
+        aPoint2Id = anIt2->second;
+      }
+    }
+  }
+
   int anInsertPos = -1;
-  int aLastPoint = myCurve->getNbPoints( 0/*SectionId*/ )-1; 
-  if ( ( aPoint1 == aLastPoint && aPoint2 == 0 ) ||
-       ( aPoint2 == aLastPoint && aPoint1 == 0 ) )
+  int aLastPoint = myCurve->getNbPoints( aSectionId )-1; 
+  if ( ( aPoint1Id == aLastPoint && aPoint2Id == 0 ) ||
+       ( aPoint2Id == aLastPoint && aPoint1Id == 0 ) )
     anInsertPos = -1; // if the section happens between first and last points
   else
-    anInsertPos = aPoint1 < aPoint2 ? aPoint1 + 1 : aPoint2 + 1;
+    anInsertPos = aPoint1Id < aPoint2Id ? aPoint1Id + 1 : aPoint2Id + 1;
 
-  myCurve->addPoints( aCoords, aSection, anInsertPos );
-  mySectionView->pointsAdded( aSection, myCurve->getNbPoints( aSection ) );
+  myCurve->addPoints( aCoords, aSectionId, anInsertPos );
+  mySectionView->pointsAdded( aSectionId, myCurve->getNbPoints( aSectionId ) );
 
   finishCurveModification( aSelPoints );
 
-  setSelectedPonts( 0 );
-  updateUndoRedo();
+  setSelectedPonts();
 }
 
 void CurveCreator_Widget::moveSelectedPoints( const int theXPosition,
@@ -1096,7 +1111,7 @@ void CurveCreator_Widget::moveSelectedPoints( const int theXPosition,
   if ( !aViewer )
     return;
 
-  QList<int> aPoints;
+  SectionToPointList aPoints;
   startCurveModification( aPoints, false );
 
   OCCViewer_ViewWindow* aWindow =
@@ -1109,18 +1124,25 @@ void CurveCreator_Widget::moveSelectedPoints( const int theXPosition,
   double aXDelta = aStartPnt.X() - anEndPnt.X();
   double anYDelta = aStartPnt.Y() - anEndPnt.Y();
 
-  int aSectionId = 0;
-  int aPointId;
+  CurveCreator_ICurve::SectionToPointCoordsList aCoordList;
   std::deque<float> aChangedPos;
-  for ( int i = 0, aNb = myDragPoints.size(); i < aNb; i++ ) {
-    aPointId = myDragPoints[i];
+  SectionToPointList::const_iterator anIt = myDragPoints.begin(), aLast = myDragPoints.end();
+  for ( ; anIt != aLast; anIt++ ) {
+    int aSectionId = anIt->first;
+    int aPointId = anIt->second;
     aChangedPos = myCurve->getPoint( aSectionId, aPointId );
     if ( aChangedPos.size() < 2 )
       continue;
     aChangedPos[0] = aChangedPos[0] - aXDelta;
     aChangedPos[1] = aChangedPos[1] - anYDelta;
-    myCurve->setPoint( aSectionId, aPointId, aChangedPos );
+    
+    aCoordList.push_back(
+      std::make_pair(std::make_pair( aSectionId, aPointId ), 
+                     aChangedPos ));
   }
+  myCurve->setSeveralPoints( aCoordList );
+
+  myDragged = true;
   finishCurveModification( myDragPoints );
 }
 
@@ -1222,33 +1244,57 @@ void CurveCreator_Widget::setLocalPointContext( const bool theOpen, const bool i
 
 void CurveCreator_Widget::addLocalPointToTable( const double theX, const double theY )
 {
+  SectionToPointList aPoints;
+  findSectionsToPoints( theX, theY, aPoints );
+
+  SectionToPointList aSkipList;
+  // table could not contain two equal value rows
   int aRowId = myLocalPointView->rowCount();
   double aCurrentX, aCurrentY;
+  int aSectionId, aPointId;
+  SectionToPoint aPoint;
   for ( int i = 0; i < aRowId; i++ ) {
-    aCurrentX = myLocalPointView->item( i, 1 )->data( Qt::UserRole ).toDouble();
-    aCurrentY = myLocalPointView->item( i, 2 )->data( Qt::UserRole ).toDouble();
+    aCurrentX = myLocalPointView->item( i, 2 )->data( Qt::UserRole ).toDouble();
+    aCurrentY = myLocalPointView->item( i, 3 )->data( Qt::UserRole ).toDouble();
     if ( fabs( aCurrentX - theX ) < LOCAL_SELECTION_TOLERANCE &&
-         fabs( aCurrentY - theY ) < LOCAL_SELECTION_TOLERANCE )
-      return;
+         fabs( aCurrentY - theY ) < LOCAL_SELECTION_TOLERANCE ) {
+           aPoint = std::make_pair<int, int>( getSectionId( i ), getPointId( i ) );
+      if ( !contains( aSkipList, aPoint ) )
+        aSkipList.push_back( aPoint );
+    }
   }
+  if ( aSkipList.size() == aPoints.size() )
+    return;
+
   QTableWidgetItem* anItem;
+  SectionToPointList::const_iterator anIt = aPoints.begin(), aLast = aPoints.end();
+  for ( ; anIt != aLast; anIt++ ) {
+    aPoint = *anIt;
+    if ( contains( aSkipList, aPoint ) )
+      continue;
 
-  myLocalPointView->setRowCount( aRowId+1 );
+    myLocalPointView->setRowCount( aRowId+1 );
+    aSectionId = aPoint.first;
+    aPointId = aPoint.second;
 
-  int aPointSection = 0;
-  int aPointIndex = findLocalPointIndex( aPointSection, theX, theY );
+    anItem = new QTableWidgetItem( myCurve->getSectionName( aSectionId ).c_str() );
+    anItem->setFlags( anItem->flags() & ~Qt::ItemIsEnabled );
+    anItem->setData( Qt::UserRole, aSectionId );
+    myLocalPointView->setItem( aRowId, 0, anItem );
 
-  anItem = new QTableWidgetItem( QString::number( aPointIndex + 1 ) );
-  anItem->setFlags( anItem->flags() & ~Qt::ItemIsEnabled );
-  myLocalPointView->setItem( aRowId, 0, anItem );
+    anItem = new QTableWidgetItem( QString::number( aPointId + 1 ) );
+    anItem->setFlags( anItem->flags() & ~Qt::ItemIsEnabled );
+    anItem->setData( Qt::UserRole, aPointId );
+    myLocalPointView->setItem( aRowId, 1, anItem );
 
-  anItem = new QTableWidgetItem( QString::number( theX ) );
-  anItem->setData( Qt::UserRole, theX );
-  myLocalPointView->setItem( aRowId, 1, anItem );
+    anItem = new QTableWidgetItem( QString::number( theX ) );
+    anItem->setData( Qt::UserRole, theX );
+    myLocalPointView->setItem( aRowId, 2, anItem );
 
-  anItem = new QTableWidgetItem( QString::number( theY ) );
-  anItem->setData( Qt::UserRole, theY );
-  myLocalPointView->setItem( aRowId, 2, anItem );
+    anItem = new QTableWidgetItem( QString::number( theY ) );
+    anItem->setData( Qt::UserRole, theY );
+    myLocalPointView->setItem( aRowId, 3, anItem );
+  }
 }
 
 /**
@@ -1258,34 +1304,34 @@ void CurveCreator_Widget::addLocalPointToTable( const double theX, const double
  */
 void CurveCreator_Widget::setDragStarted( const bool theState, const QPoint& thePoint )
 {
-  if ( !theState ) {
-    myDragStarted = false;
-    myDragPoints.clear();
-  }
-  else {
-    getSelectedPonts( 0, myDragPoints );
+  OCCViewer_Viewer* aViewer = getOCCViewer();
+  if ( theState ) {
+    getSelectedPonts( myDragPoints );
     myDragStarted = myDragPoints.size();
     myDragStartPosition = thePoint;
+    if ( aViewer && myDragStarted ) {
+      // change a viewer interaction style in order to avoid a select rectangle build
+      myDragInteractionStyle = aViewer->interactionStyle();
+      aViewer->setInteractionStyle(SUIT_ViewModel::KEY_FREE);
+    }
+  }
+  else {
+    if ( aViewer && myDragStarted )
+      aViewer->setInteractionStyle( myDragInteractionStyle );
+    myDragStarted = false;
+    myDragPoints.clear();
   }
+  myDragged = false;
 }
 
-void CurveCreator_Widget::getSelectedPonts( int theSectionId, QList<int>& thePoints )
+void CurveCreator_Widget::getSelectedPonts( CurveCreator_Widget::SectionToPointList& thePoints )
 {
   thePoints.clear();
-
-  float aPrevX, aPrevY;
-  int aPntIndex;
-  for ( int i = 0, aNb = myLocalPointView->rowCount(); i < aNb; i++ ) {
-    aPrevX = myLocalPointView->item( i, 1 )->data( Qt::UserRole ).toDouble();
-    aPrevY = myLocalPointView->item( i, 2 )->data( Qt::UserRole ).toDouble();
-
-    aPntIndex = findLocalPointIndex( theSectionId, aPrevX, aPrevY );
-    if ( aPntIndex >= 0 )
-      thePoints.append( aPntIndex );
-  }
+  for ( int i = 0, aNb = myLocalPointView->rowCount(); i < aNb; i++ )
+    thePoints.push_back( std::make_pair( getSectionId( i ), getPointId( i ) ) );
 }
 
-void CurveCreator_Widget::setSelectedPonts( const int theSectionId, const QList<int>& thePoints )
+void CurveCreator_Widget::setSelectedPonts( const CurveCreator_Widget::SectionToPointList& thePoints )
 {
   OCCViewer_Viewer* aViewer = getOCCViewer();
   if ( !aViewer )
@@ -1299,26 +1345,42 @@ void CurveCreator_Widget::setSelectedPonts( const int theSectionId, const QList<
 
   AIS_ListOfInteractive aDisplayedList;
   ic->DisplayedObjects( aDisplayedList );
-  for ( AIS_ListIteratorOfListOfInteractive it( aDisplayedList ); it.More(); it.Next() )
-  {
-    Handle(AIS_InteractiveObject) anAIS = it.Value();
-    if ( anAIS.IsNull() )
-      continue;
-    Handle(AIS_Point) aPoint = Handle(AIS_Point)::DownCast( anAIS );
-    if ( aPoint.IsNull() )
-      continue;
 
-    TopoDS_Vertex aVertex = TopoDS::Vertex( aPoint->Vertex() );
+  SectionToPointList::const_iterator anIt = thePoints.begin(), aLast = thePoints.end();
+  SectionToPoint aSToPoint;
+  for( ; anIt != aLast; anIt++ ) {
+    aSToPoint = *anIt;
 
-    if ( aVertex.IsNull() )
-      continue;
+    for ( AIS_ListIteratorOfListOfInteractive it( aDisplayedList ); it.More(); it.Next() )
+    {
+      Handle(AIS_InteractiveObject) anAIS = it.Value();
+      if ( anAIS.IsNull() )
+        continue;
+      Handle(AIS_Point) anAISPoint = Handle(AIS_Point)::DownCast( anAIS );
+      if ( anAISPoint.IsNull() )
+        continue;
 
-    gp_Pnt aPnt = BRep_Tool::Pnt( aVertex );
-    int aPointIndex = findLocalPointIndex( theSectionId, aPnt.X(), aPnt.Y() );
-    if ( thePoints.contains( aPointIndex ) )
-      aListToSelect.Append( anAIS );
+      TopoDS_Vertex aVertex = TopoDS::Vertex( anAISPoint->Vertex() );
+
+      if ( aVertex.IsNull() )
+        continue;
+
+      gp_Pnt aPnt = BRep_Tool::Pnt( aVertex );
+
+      SectionToPointList aPoints;
+      findSectionsToPoints( aPnt.X(), aPnt.Y(), aPoints );
+
+      SectionToPointList::const_iterator anIt = aPoints.begin(), aLast = aPoints.end();
+      SectionToPoint aPoint;
+      for ( ; anIt != aLast; anIt++ ) {
+        aPoint = *anIt;
+        if ( aPoint.first == aSToPoint.first && aPoint.second == aSToPoint.second )
+          aListToSelect.Append( anAIS );
+      }
+    }
   }
 
+  ic->ClearSelected( Standard_False );
   aViewer->setObjectsSelected( aListToSelect );
   updateLocalPointView();
 }
@@ -1332,12 +1394,13 @@ void CurveCreator_Widget::setSelectedPonts( const int theSectionId, const QList<
  * \param thePoints an output list of curve selected points
  * \param theFillPoints a flag whether the selection list should be filled
  */
-void CurveCreator_Widget::startCurveModification( QList<int>& thePoints, const bool theFillPoints )
+void CurveCreator_Widget::startCurveModification(
+                           CurveCreator_Widget::SectionToPointList& thePoints,
+                           const bool theFillPoints )
 {
   if ( theFillPoints ) {
     thePoints.clear();
-    int aSectionId = 0;
-    getSelectedPonts( aSectionId, thePoints );
+    getSelectedPonts( thePoints );
   }
   setLocalPointContext( false );
 }
@@ -1347,11 +1410,13 @@ void CurveCreator_Widget::startCurveModification( QList<int>& thePoints, const b
  * Open local context and select given points inside it.
  * \param thePoints a list of curve selected points
  */
-void CurveCreator_Widget::finishCurveModification( const QList<int>& thePoints )
+void CurveCreator_Widget::finishCurveModification(
+                           const CurveCreator_Widget::SectionToPointList& thePoints )
 {
-  setLocalPointContext( true );
-  int aSectionId = 0;
-  setSelectedPonts( aSectionId, thePoints );
+  if ( getActionMode() == ModificationMode )
+    setLocalPointContext( true );
+  setSelectedPonts( thePoints );
+  updateUndoRedo();
 }
 
 /**
@@ -1376,6 +1441,43 @@ int CurveCreator_Widget::findLocalPointIndex( int theSectionId, float theX, floa
   return aPntIndex;
 }
 
+void CurveCreator_Widget::findSectionsToPoints( const double theX, const double theY,
+                                 CurveCreator_Widget::SectionToPointList& thePoints )
+{
+  thePoints.clear();
+
+  int aPointId = -1;
+  for ( int i = 0, aNb = myCurve->getNbSections(); i < aNb; i++ ) {
+    aPointId = findLocalPointIndex( i, theX, theY );
+    if ( aPointId < 0 )
+      continue;
+    SectionToPoint aPoint = std::make_pair( i, aPointId );
+    if ( !contains( thePoints, aPoint ) )
+      thePoints.push_back( aPoint );
+  }
+}
+
+void CurveCreator_Widget::convert( const SectionToPointList& thePoints,
+                                   QMap<int, QList<int> >& theConvPoints )
+{
+  theConvPoints.clear();
+
+  SectionToPointList::const_iterator anIt = thePoints.begin(), aLast = thePoints.end();
+  QList<int> aPoints;
+  int aSectionId, aPointId;
+  for ( ; anIt != aLast; anIt++ ) {
+    aSectionId = anIt->first;
+    aPointId = anIt->second;
+    aPoints.clear();
+    if ( theConvPoints.contains( aSectionId ) )
+      aPoints = theConvPoints[aSectionId];
+    if ( aPoints.contains( aPointId ) )
+      continue;
+    aPoints.append( aPointId );
+    theConvPoints[aSectionId] = aPoints;
+  }
+}
+
 /**
  * Checks whether the point belongs to the OCC object
  * \param theObject a line or shape with a bspline inside
@@ -1388,7 +1490,7 @@ int CurveCreator_Widget::findLocalPointIndex( int theSectionId, float theX, floa
 bool CurveCreator_Widget::pointOnObject( Handle(AIS_InteractiveObject) theObject,
                                          const int theX, const int theY,
                                          gp_Pnt& thePoint,
-                                         int& thePoint1, int& thePoint2 )
+                                         gp_Pnt& thePoint1, gp_Pnt& thePoint2 )
 {
   bool isFound = false;
 
@@ -1405,12 +1507,18 @@ bool CurveCreator_Widget::pointOnObject( Handle(AIS_InteractiveObject) theObject
     isFound = hasProjectPointOnCurve( theX, theY, aGLine, aParameter );
     if ( isFound ) {
       aPoint = aGLine->Value( aParameter );
-      
-    Handle(Geom_Point) aPStart;
-    Handle(Geom_Point) aPEnd;
-    aLine->Points( aPStart, aPEnd );
-    aPnt1 = aPStart->Pnt();
-    aPnt2 = aPEnd->Pnt();
+
+      Handle(Geom_Point) aPStart;
+      Handle(Geom_Point) aPEnd;
+      aLine->Points( aPStart, aPEnd );
+      aPnt1 = aPStart->Pnt();
+      aPnt2 = aPEnd->Pnt();
+
+      // in case of Geom line a projection is performed to the infinite line,
+      // so it is necessary to bound it by the line size
+      Bnd_Box aLineBox;
+      aLineBox.Set( aPnt1, gp_Vec( aPnt1, aPnt2 ) );
+      isFound = !aLineBox.IsOut( aPoint );
     }
   }
   else {
@@ -1447,9 +1555,8 @@ bool CurveCreator_Widget::pointOnObject( Handle(AIS_InteractiveObject) theObject
   }
   if ( isFound ) {
     thePoint = aPoint;
-    thePoint1 = findLocalPointIndex( 0, aPnt1.X(), aPnt1.Y() );
-    thePoint2 = findLocalPointIndex( 0, aPnt2.X(), aPnt2.Y() );
-    isFound = thePoint1 >= 0 && thePoint2 >= 0;
+    thePoint1 = aPnt1;
+    thePoint2 = aPnt2;
   }
   return isFound;
 }
@@ -1492,3 +1599,38 @@ bool CurveCreator_Widget::hasProjectPointOnCurve( const int theX, const int theY
   }
   return isFound;
 }
+
+/**
+ * Returns a section index from the table
+ * \param theRowId a table row
+ */
+int CurveCreator_Widget::getSectionId( const int theRowId ) const
+{
+  return myLocalPointView->item( theRowId, 0 )->data( Qt::UserRole ).toInt();
+}
+
+/**
+ * Returns a point index from the table
+ * \param theRowId a table row
+ */
+int CurveCreator_Widget::getPointId( const int theRowId ) const
+{
+  return myLocalPointView->item( theRowId, 1 )->data( Qt::UserRole ).toInt();
+}
+
+/**
+ * Returns whethe the container has the value
+ * \param theList a container of values
+ * \param theValue a value
+ */
+bool CurveCreator_Widget::contains( const CurveCreator_Widget::SectionToPointList& theList,
+                                    const CurveCreator_Widget::SectionToPoint& theValue ) const
+{
+  bool isFound = false;
+
+  SectionToPointList::const_iterator anIt = theList.begin(), aLast = theList.end();
+  for ( ; anIt != aLast && !isFound; anIt++ )
+    isFound = anIt->first == theValue.first && anIt->second == theValue.second;
+
+  return isFound;
+}