From eab95def22ec1527eea81185f89263fa68ed037d Mon Sep 17 00:00:00 2001
From: szy <szy@opencascade.com>
Date: Thu, 25 Dec 2014 18:41:12 +0300
Subject: [PATCH] Small fix to register selected name.

---
 src/Model/Model_AttributeSelection.cpp     | 21 ++++++++++++++++-----
 src/ModelAPI/ModelAPI_AttributeSelection.h |  5 ++++-
 2 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/src/Model/Model_AttributeSelection.cpp b/src/Model/Model_AttributeSelection.cpp
index ea00eaaec..f418c7b77 100644
--- a/src/Model/Model_AttributeSelection.cpp
+++ b/src/Model/Model_AttributeSelection.cpp
@@ -442,7 +442,7 @@ TDF_Label Model_AttributeSelection::selectionLabel()
   return myRef.myRef->Label().FindChild(1);
 }
 
-#define DEB_NAME 1
+#define FIX_BUG1 1
 std::string GetShapeName(std::shared_ptr<Model_Document> theDoc, const TopoDS_Shape& theShape, 
 					     const TDF_Label& theLabel)
 {
@@ -573,21 +573,30 @@ std::string Model_AttributeSelection::buildSubShapeName(std::shared_ptr<GeomAPI_
 		  break;
 
 	  case TopAbs_VERTEX:
-		  // name structure (Monifold Topology): F1 | F2 | F3; intersection of 3 faces defines a vertex.
+		  // name structure (Monifold Topology): 
+		  // 1) F1 | F2 | F3 - intersection of 3 faces defines a vertex - trivial case.
+		  // 2) F1 | F2      - intersection of 2 faces definses a vertex - applicable for case
+		  //                   when 1 faces is cylindrical, conical, spherical or revolution and etc.
+		  // 3) E1 | E2 | E3 - intersection of 3 edges defines a vertex - when we have case of a shell
+		  //                   or compound of 2 open faces.
+		  // 4) E1 | E2      - intesection of 2 edges defines a vertex - when we have a case of 
+		  //                   two independent edges (wire or compound)
+		  // implemented 2 first cases
 		  {
 			TopTools_IndexedDataMapOfShapeListOfShape aMap;
 		    TopExp::MapShapesAndAncestors(aContext, TopAbs_VERTEX, TopAbs_FACE, aMap);
 			const TopTools_ListOfShape& aList2  = aMap.FindFromKey(aSubShape);
 			TopTools_ListOfShape aList;
 			TopTools_MapOfShape aFMap;
-#ifdef DEB_NAME
-			int n = aList2.Extent(); //bug!
+#ifdef FIX_BUG1
+			//int n = aList2.Extent(); //bug!
+			// fix is below
 			TopTools_ListIteratorOfListOfShape itl2(aList2);
 		    for (int i = 1;itl2.More();itl2.Next(),i++) {
 			  if(aFMap.Add(itl2.Value()))
 				aList.Append(itl2.Value());
 			}
-			n = aList.Extent();
+			//n = aList.Extent();
 #endif
 			TopTools_ListIteratorOfListOfShape itl(aList);
 		    for (int i = 1;itl.More();itl.Next(),i++) {
@@ -601,6 +610,8 @@ std::string Model_AttributeSelection::buildSubShapeName(std::shared_ptr<GeomAPI_
 		  }
 		  break;
 	}
+    // register name			
+    aDoc->addNamingName(selectionLabel(), aName);
   }
   return aName;
 }
diff --git a/src/ModelAPI/ModelAPI_AttributeSelection.h b/src/ModelAPI/ModelAPI_AttributeSelection.h
index 4e8f2a51e..b34a24d1c 100644
--- a/src/ModelAPI/ModelAPI_AttributeSelection.h
+++ b/src/ModelAPI/ModelAPI_AttributeSelection.h
@@ -44,9 +44,12 @@ class ModelAPI_AttributeSelection : public ModelAPI_Attribute
     return type();
   }
 
-  /// builds name of the SubShape 
+  /// Builds name of the SubShape 
   virtual std::string buildSubShapeName(std::shared_ptr<GeomAPI_Shape> theSubShape, 
 	                                    const ResultPtr& theContext) = 0;
+
+  /// Selects sub-shape by Name
+  //virtual selectSubShape(const std::string& theSubShapeName, const std::string& theContextShapeName)
   /// To virtually destroy the fields of successors
   virtual ~ModelAPI_AttributeSelection()
   {
-- 
2.39.2