From a2cef23dbd2e85e64a733f870efdb089edf5af4e Mon Sep 17 00:00:00 2001
From: Chris Hennes <chennes@pioneerlibrarysystem.org>
Date: Sun, 16 Feb 2025 11:25:45 -0600
Subject: [PATCH] PD: Combine onlyHaveRefined and onlyHasToRefine

Co-authored-by: Florian Foinant-Willig <FlachyJoe@users.noreply.github.com>
---
 src/Mod/PartDesign/App/FeatureRefine.cpp | 16 +++-------------
 1 file changed, 3 insertions(+), 13 deletions(-)

diff --git a/src/Mod/PartDesign/App/FeatureRefine.cpp b/src/Mod/PartDesign/App/FeatureRefine.cpp
index 6106129914..b56e9c5ba5 100644
--- a/src/Mod/PartDesign/App/FeatureRefine.cpp
+++ b/src/Mod/PartDesign/App/FeatureRefine.cpp
@@ -58,7 +58,7 @@ FeatureRefine::FeatureRefine()
     this->Refine.setValue(hGrp->GetBool("RefineModel", true));
 }
 
-bool FeatureRefine::onlyHasToRefine() const
+bool FeatureRefine::onlyHaveRefined()
 {
     if (!Refine.isTouched()) {
         return false;
@@ -75,20 +75,10 @@ bool FeatureRefine::onlyHasToRefine() const
             return false;
         }
     }
+    TopoShape result = refineShapeIfActive(rawShape);
+    Shape.setValue(result);
     return true;
 }
-
-
-bool FeatureRefine::onlyHaveRefined()
-{
-    if (onlyHasToRefine()) {
-        TopoShape result = refineShapeIfActive(rawShape);
-        Shape.setValue(result);
-        return true;
-    }
-    return false;
-}
-
 TopoShape FeatureRefine::refineShapeIfActive(const TopoShape& oldShape,
                                              const RefineErrorPolicy onError) const
 {