Issue #146 Do not start line creation if the existing point was selected

diff --git a/src/PartSet/PartSet_OperationFeatureCreate.cpp b/src/PartSet/PartSet_OperationFeatureCreate.cpp
index 759356cd24b40c0530f00ff45977d5af9600c7f7..38768a0ca2913d934cda57575905918aa46a227a 100644 (file)
--- a/src/PartSet/PartSet_OperationFeatureCreate.cpp
+++ b/src/PartSet/PartSet_OperationFeatureCreate.cpp
@@ -106,24 +106,24 @@ void PartSet_OperationFeatureCreate::mouseReleased(
 {
   gp_Pnt aPoint = PartSet_Tools::convertClickToPoint(theEvent->pos(), theView);
   double aX = aPoint.X(), anY = aPoint.Y();
+  bool isClosedContour = false;
 
   if (theSelected.empty()) {
     PartSet_Tools::convertTo2D(aPoint, sketch(), theView, aX, anY);
   } else {
     ModuleBase_ViewerPrs aPrs = theSelected.front();
     const TopoDS_Shape& aShape = aPrs.shape();
-    if (!aShape.IsNull())  // the point is selected
+    if (!aShape.IsNull())
     {
-      if (aShape.ShapeType() == TopAbs_VERTEX) {
+      if (aShape.ShapeType() == TopAbs_VERTEX) { // a point is selected
         const TopoDS_Vertex& aVertex = TopoDS::Vertex(aShape);
         if (!aVertex.IsNull()) {
           aPoint = BRep_Tool::Pnt(aVertex);
           PartSet_Tools::convertTo2D(aPoint, sketch(), theView, aX, anY);
-
           PartSet_Tools::setConstraints(sketch(), feature(), myActiveWidget->attributeID(), aX, anY);
+          isClosedContour = true;
         }
-      } else if (aShape.ShapeType() == TopAbs_EDGE)  // the line is selected
-          {
+      } else if (aShape.ShapeType() == TopAbs_EDGE) { // a line is selected
         PartSet_Tools::convertTo2D(aPoint, sketch(), theView, aX, anY);
         // move to selected line
         if (feature()->getKind() == SketchPlugin_Line::ID()) {
@@ -146,7 +146,7 @@ void PartSet_OperationFeatureCreate::mouseReleased(
     emit activateNextWidget(myActiveWidget);
   }
 
-  if (commit()) {
+  if (commit() && !isClosedContour) {
     // if the point creation is finished, the next mouse release should commit the modification
     // the next release can happens by double click in the viewer
     restartOperation(feature()->getKind(), feature());