From 6339093425f6055e2491220a54ae23936642768d Mon Sep 17 00:00:00 2001
From: dbv <dbv@opencascade.com>
Date: Tue, 14 Jun 2016 17:22:29 +0300
Subject: [PATCH] Fixed features creation from CPP High API

---
 src/FeaturesAPI/FeaturesAPI_Boolean.cpp   |  8 +++++---
 src/FeaturesAPI/FeaturesAPI_Extrusion.cpp | 22 +++++++++++-----------
 src/PythonAPI/examples/MakeBrick1.py      |  2 ++
 3 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/src/FeaturesAPI/FeaturesAPI_Boolean.cpp b/src/FeaturesAPI/FeaturesAPI_Boolean.cpp
index 6f68890cf..128b9abfc 100644
--- a/src/FeaturesAPI/FeaturesAPI_Boolean.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Boolean.cpp
@@ -25,9 +25,11 @@ FeaturesAPI_Boolean::FeaturesAPI_Boolean(const std::shared_ptr<ModelAPI_Feature>
 : ModelHighAPI_Interface(theFeature)
 {
   if(initialize()) {
-    setBoolType(theBoolType);
-    setMainObjects(theMainObjects);
-    setToolObjects(theToolObjects);
+    fillAttribute(theBoolType, myboolType);
+    fillAttribute(theMainObjects, mymainObjects);
+    fillAttribute(theToolObjects, mytoolObjects);
+
+    execute();
   }
 }
 
diff --git a/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp b/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp
index da6f0cf3e..8bb243a02 100644
--- a/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp
@@ -23,8 +23,8 @@ FeaturesAPI_Extrusion::FeaturesAPI_Extrusion(const std::shared_ptr<ModelAPI_Feat
 : ModelHighAPI_Interface(theFeature)
 {
   if(initialize()) {
-    setBase(theBaseObjects);
-    setSize(theSize);
+    fillAttribute(theBaseObjects, mybaseObjects);
+    setSizes(theSize, ModelHighAPI_Double());
   }
 }
 
@@ -36,9 +36,9 @@ FeaturesAPI_Extrusion::FeaturesAPI_Extrusion(const std::shared_ptr<ModelAPI_Feat
 : ModelHighAPI_Interface(theFeature)
 {
   if(initialize()) {
-    setBase(theBaseObjects);
-    setDirection(theDirection);
-    setSize(theSize);
+    fillAttribute(theBaseObjects, mybaseObjects);
+    fillAttribute(theDirection, mydirection);
+    setSizes(theSize, ModelHighAPI_Double());
   }
 }
 
@@ -50,7 +50,7 @@ FeaturesAPI_Extrusion::FeaturesAPI_Extrusion(const std::shared_ptr<ModelAPI_Feat
 : ModelHighAPI_Interface(theFeature)
 {
   if(initialize()) {
-    setBase(theBaseObjects);
+    fillAttribute(theBaseObjects, mybaseObjects);
     setSizes(theToSize, theFromSize);
   }
 }
@@ -64,8 +64,8 @@ FeaturesAPI_Extrusion::FeaturesAPI_Extrusion(const std::shared_ptr<ModelAPI_Feat
 : ModelHighAPI_Interface(theFeature)
 {
   if(initialize()) {
-    setBase(theBaseObjects);
-    setDirection(theDirection);
+    fillAttribute(theBaseObjects, mybaseObjects);
+    fillAttribute(theDirection, mydirection);
     setSizes(theToSize, theFromSize);
   }
 }
@@ -80,7 +80,7 @@ FeaturesAPI_Extrusion::FeaturesAPI_Extrusion(const std::shared_ptr<ModelAPI_Feat
 : ModelHighAPI_Interface(theFeature)
 {
   if(initialize()) {
-    setBase(theBaseObjects);
+    fillAttribute(theBaseObjects, mybaseObjects);
     setPlanesAndOffsets(theToObject, theToOffset, theFromObject, theFromOffset);
   }
 }
@@ -96,8 +96,8 @@ FeaturesAPI_Extrusion::FeaturesAPI_Extrusion(const std::shared_ptr<ModelAPI_Feat
 : ModelHighAPI_Interface(theFeature)
 {
   if(initialize()) {
-    setBase(theBaseObjects);
-    setDirection(theDirection);
+    fillAttribute(theBaseObjects, mybaseObjects);
+    fillAttribute(theDirection, mydirection);
     setPlanesAndOffsets(theToObject, theToOffset, theFromObject, theFromOffset);
   }
 }
diff --git a/src/PythonAPI/examples/MakeBrick1.py b/src/PythonAPI/examples/MakeBrick1.py
index 4efed0b15..07a2629ec 100644
--- a/src/PythonAPI/examples/MakeBrick1.py
+++ b/src/PythonAPI/examples/MakeBrick1.py
@@ -56,6 +56,8 @@ c1 = mystand.addCircle(0, 25, 5)
 mystand.setDistance(c1.center(), mystand.addLine(thisxmin), 10)
 mystand.setDistance(c1.center(), mystand.addLine(thiszmax), 10)
 
+mystand.execute()
+
 myboss = model.addExtrusion(mypart, mystand.selectFace(), -5)
 
 # Subtracting the cylinder to the box
-- 
2.39.2