From b9e87dcef9246d590eb8706be38ad1fe970b4c6b Mon Sep 17 00:00:00 2001
From: azv <azv@opencascade.com>
Date: Mon, 31 Oct 2016 14:14:06 +0300
Subject: [PATCH] Fix incorrect linking of extrusion if the sketch has
 non-default name (issue #1839)

---
 src/ModelHighAPI/ModelHighAPI_Dumper.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/ModelHighAPI/ModelHighAPI_Dumper.cpp b/src/ModelHighAPI/ModelHighAPI_Dumper.cpp
index b43241e36..bb5c46726 100644
--- a/src/ModelHighAPI/ModelHighAPI_Dumper.cpp
+++ b/src/ModelHighAPI/ModelHighAPI_Dumper.cpp
@@ -303,16 +303,17 @@ bool ModelHighAPI_Dumper::processSubs(const std::shared_ptr<ModelAPI_CompositeFe
       myEntitiesStack.top().myEntity == EntityPtr(theComposite);
   bool isForceModelDo = isSubDumped && isDumpSetName &&
       (myEntitiesStack.top().myUserName || !myEntitiesStack.top().myResults.empty());
+
+  // dump "setName" for composite feature before "model.do()" command to apply the name too
+  if (isDumpSetName)
+    dumpEntitySetName();
+
   // It is necessary for the sketch to create its result when complete (command "model.do()").
   // This option is set by flat theDumpModelDo.
   // However, nested sketches are rebuilt by parent feature, so, they do not need
   // explicit call of "model.do()". This will be controlled by the depth of the stack.
   if (isForceModelDo || (theDumpModelDo && gCompositeStackDepth <= 1))
     *this << "model.do()" << std::endl;
-
-  // dump "setName" for composite feature
-  if (isDumpSetName)
-    dumpEntitySetName();
   return isOk;
 }
 
-- 
2.39.2