From ff1e21d254ea2aaa6ecc7bdad7c445be7dda7aa5 Mon Sep 17 00:00:00 2001
From: azv <azv@opencascade.com>
Date: Tue, 8 Nov 2016 12:53:30 +0300
Subject: [PATCH] Split has incorrectly modified coincidences (issue #1784)

Do not add feature (ConstraintCoincidence) into a list to remove, if the coincident point is not found. Because, there may be a coincidence between any attribute of split feature and another feature.
---
 src/SketchPlugin/SketchPlugin_ConstraintSplit.cpp | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/src/SketchPlugin/SketchPlugin_ConstraintSplit.cpp b/src/SketchPlugin/SketchPlugin_ConstraintSplit.cpp
index 156dc33fa..4ac441841 100755
--- a/src/SketchPlugin/SketchPlugin_ConstraintSplit.cpp
+++ b/src/SketchPlugin/SketchPlugin_ConstraintSplit.cpp
@@ -532,13 +532,9 @@ void SketchPlugin_ConstraintSplit::getConstraints(std::set<FeaturePtr>& theFeatu
         if (anAttribute.get())
           aCoincidentPoint = std::dynamic_pointer_cast<GeomDataAPI_Point2D>(anAttribute);
       }
-      if (aCoincidentPoint.get()) {
-        if (isToFeature)
-          theCoincidenceToFeature[aRefFeature] = std::make_pair(anAttributeToBeModified,
-                                                                aCoincidentPoint);
-      }
-      else
-        theFeaturesToDelete.insert(aRefFeature); /// this case should not happen
+      if (aCoincidentPoint.get() && isToFeature)
+        theCoincidenceToFeature[aRefFeature] = std::make_pair(anAttributeToBeModified,
+                                                              aCoincidentPoint);
     }
   }
 }
-- 
2.39.2