From 6d9886775c3cfc3099ffdbd68c9f2fd81768c00a Mon Sep 17 00:00:00 2001
From: wmayer <wmayer@users.sourceforge.net>
Date: Mon, 25 Sep 2017 22:36:02 +0200
Subject: [PATCH] fixes #0002722: Attached cube doesn't follow support

---
 src/Mod/Part/App/FeaturePartBox.cpp    | 3 +--
 src/Mod/Part/App/FeaturePartCircle.cpp | 3 +--
 src/Mod/Part/App/PrimitiveFeature.cpp  | 8 +-------
 3 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/src/Mod/Part/App/FeaturePartBox.cpp b/src/Mod/Part/App/FeaturePartBox.cpp
index 1005b02ab1..2d72824193 100644
--- a/src/Mod/Part/App/FeaturePartBox.cpp
+++ b/src/Mod/Part/App/FeaturePartBox.cpp
@@ -75,12 +75,11 @@ App::DocumentObjectExecReturn *Box::execute(void)
         BRepPrimAPI_MakeBox mkBox(L, W, H);
         TopoDS_Shape ResultShape = mkBox.Shape();
         this->Shape.setValue(ResultShape);
+        return Primitive::execute();
     }
     catch (Standard_Failure& e) {
         return new App::DocumentObjectExecReturn(e.GetMessageString());
     }
-
-    return App::DocumentObject::StdReturn;
 }
 
 /**
diff --git a/src/Mod/Part/App/FeaturePartCircle.cpp b/src/Mod/Part/App/FeaturePartCircle.cpp
index 6b522dd763..9bc1ef7da0 100644
--- a/src/Mod/Part/App/FeaturePartCircle.cpp
+++ b/src/Mod/Part/App/FeaturePartCircle.cpp
@@ -70,8 +70,7 @@ App::DocumentObjectExecReturn *Circle::execute(void)
                                                Base::toRadians<double>(this->Angle1.getValue()));
     const TopoDS_Edge& edge = clMakeEdge.Edge();
     this->Shape.setValue(edge);
-
-    return App::DocumentObject::StdReturn;
+    return Primitive::execute();
 }
 
 void Circle::onChanged(const App::Property* prop)
diff --git a/src/Mod/Part/App/PrimitiveFeature.cpp b/src/Mod/Part/App/PrimitiveFeature.cpp
index 466bce3834..ac83b19896 100644
--- a/src/Mod/Part/App/PrimitiveFeature.cpp
+++ b/src/Mod/Part/App/PrimitiveFeature.cpp
@@ -954,16 +954,11 @@ App::DocumentObjectExecReturn *Spiral::execute(void)
         BRepProj_Projection proj(wire, mkFace.Face(), gp::DZ());
         this->Shape.setValue(proj.Shape());
 
-        Primitive::execute();
+        return Primitive::execute();
     }
     catch (Standard_Failure& e) {
-
         return new App::DocumentObjectExecReturn(e.GetMessageString());
     }
-
-
-
-    return Primitive::execute();
 }
 
 PROPERTY_SOURCE(Part::Wedge, Part::Primitive)
@@ -1044,7 +1039,6 @@ App::DocumentObjectExecReturn *Wedge::execute(void)
         this->Shape.setValue(mkSolid.Solid());
     }
     catch (Standard_Failure& e) {
-
         return new App::DocumentObjectExecReturn(e.GetMessageString());
     }