From 94405e33c70dfbc176a52ea25c8f1bedb0204011 Mon Sep 17 00:00:00 2001 From: vosk Date: Fri, 1 Jan 2021 16:42:35 +0200 Subject: [PATCH] [PartDesign]: Fix refine boolean not being obeyed at multiple places Related to issue #3927 --- src/Mod/PartDesign/App/FeatureChamfer.cpp | 2 +- src/Mod/PartDesign/App/FeatureFillet.cpp | 1 + src/Mod/PartDesign/App/FeaturePipe.cpp | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/Mod/PartDesign/App/FeatureChamfer.cpp b/src/Mod/PartDesign/App/FeatureChamfer.cpp index bb70f423e6..7980872c15 100644 --- a/src/Mod/PartDesign/App/FeatureChamfer.cpp +++ b/src/Mod/PartDesign/App/FeatureChamfer.cpp @@ -185,7 +185,7 @@ App::DocumentObjectExecReturn *Chamfer::execute(void) if (solidCount > 1) { return new App::DocumentObjectExecReturn("Chamfer: Result has multiple solids. This is not supported at this time."); } - + shape = refineShapeIfActive(shape); this->Shape.setValue(getSolid(shape)); return App::DocumentObject::StdReturn; } diff --git a/src/Mod/PartDesign/App/FeatureFillet.cpp b/src/Mod/PartDesign/App/FeatureFillet.cpp index 37a9286dfb..646d3fde59 100644 --- a/src/Mod/PartDesign/App/FeatureFillet.cpp +++ b/src/Mod/PartDesign/App/FeatureFillet.cpp @@ -123,6 +123,7 @@ App::DocumentObjectExecReturn *Fillet::execute(void) return new App::DocumentObjectExecReturn("Fillet: Result has multiple solids. This is not supported at this time."); } + shape = refineShapeIfActive(shape); this->Shape.setValue(getSolid(shape)); return App::DocumentObject::StdReturn; } diff --git a/src/Mod/PartDesign/App/FeaturePipe.cpp b/src/Mod/PartDesign/App/FeaturePipe.cpp index 3ee4f68a5d..6007e1d6d4 100644 --- a/src/Mod/PartDesign/App/FeaturePipe.cpp +++ b/src/Mod/PartDesign/App/FeaturePipe.cpp @@ -303,6 +303,7 @@ App::DocumentObjectExecReturn *Pipe::execute(void) AddSubShape.setValue(result); if(base.IsNull()) { + result = refineShapeIfActive(result); Shape.setValue(getSolid(result)); return App::DocumentObject::StdReturn; }