Fix incorrect linking of extrusion if the sketch has non-default name (issue #1839)

diff --git a/src/ModelHighAPI/ModelHighAPI_Dumper.cpp b/src/ModelHighAPI/ModelHighAPI_Dumper.cpp
index b43241e367a254033c4d852f3380cfe77b58e68c..bb5c4672678a1a47c62dbade24f8bcde33a9c764 100644 (file)
--- 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;
 }