fixes #2722: Attached cube doesn't follow support

FreeCAD · Sep 25, 2017 · 3ec3079 · 3ec3079
1 parent 47cd41f
commit 3ec3079
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 11 deletions.
diff --git 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
@@ -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
@@ -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());
     }