From 2f2ba281560cb4afc2dd352269a241e2ae81e92a Mon Sep 17 00:00:00 2001
From: wmayer <wmayer@users.sourceforge.net>
Date: Sat, 4 Nov 2017 13:46:42 +0100
Subject: [PATCH] remove pointless Section option from boolean operation in PD

---
 src/Mod/PartDesign/App/FeatureBoolean.cpp        | 13 +------------
 src/Mod/PartDesign/Gui/TaskBooleanParameters.cpp |  1 -
 src/Mod/PartDesign/Gui/TaskBooleanParameters.ui  |  5 -----
 3 files changed, 1 insertion(+), 18 deletions(-)

diff --git a/src/Mod/PartDesign/App/FeatureBoolean.cpp b/src/Mod/PartDesign/App/FeatureBoolean.cpp
index c518e9c90a..af3eb0dda6 100644
--- a/src/Mod/PartDesign/App/FeatureBoolean.cpp
+++ b/src/Mod/PartDesign/App/FeatureBoolean.cpp
@@ -26,13 +26,11 @@
 # include <BRepAlgoAPI_Fuse.hxx>
 # include <BRepAlgoAPI_Cut.hxx>
 # include <BRepAlgoAPI_Common.hxx>
-# include <BRepAlgoAPI_Section.hxx>
 # include <gp_Trsf.hxx>
 # include <gp_Pnt.hxx>
 # include <gp_Dir.hxx>
 # include <gp_Vec.hxx>
 # include <gp_Ax1.hxx>
-#include <BRepBuilderAPI_GTransform.hxx>
 #endif
 
 #include "Body.h"
@@ -48,7 +46,7 @@ namespace PartDesign {
 
 PROPERTY_SOURCE_WITH_EXTENSIONS(PartDesign::Boolean, PartDesign::Feature)
 
-const char* Boolean::TypeEnums[]= {"Fuse","Cut","Common","Section",NULL};
+const char* Boolean::TypeEnums[]= {"Fuse","Cut","Common",NULL};
 
 Boolean::Boolean()
 {
@@ -139,15 +137,6 @@ App::DocumentObjectExecReturn *Boolean::execute(void)
             if (!mkCommon.IsDone())
                 return new App::DocumentObjectExecReturn("Common operation failed");
             boolOp = mkCommon.Shape();
-        } else if (type == "Section") {
-            BRepAlgoAPI_Section mkSection(result, shape);
-            if (!mkSection.IsDone())
-                return new App::DocumentObjectExecReturn("Section failed");
-            // we have to get the solids
-            boolOp = this->getSolid(mkSection.Shape());
-            // lets check if the result is a solid
-            if (boolOp.IsNull())
-                return new App::DocumentObjectExecReturn("Resulting shape is not a solid");
         }
 
         result = boolOp; // Use result of this operation for fuse/cut of next body
diff --git a/src/Mod/PartDesign/Gui/TaskBooleanParameters.cpp b/src/Mod/PartDesign/Gui/TaskBooleanParameters.cpp
index 9bba9cc992..8642825598 100644
--- a/src/Mod/PartDesign/Gui/TaskBooleanParameters.cpp
+++ b/src/Mod/PartDesign/Gui/TaskBooleanParameters.cpp
@@ -211,7 +211,6 @@ void TaskBooleanParameters::onTypeChanged(int index)
         case 0: pcBoolean->Type.setValue("Fuse"); break;
         case 1: pcBoolean->Type.setValue("Cut"); break;
         case 2: pcBoolean->Type.setValue("Common"); break;
-        case 3: pcBoolean->Type.setValue("Section"); break;
         default: pcBoolean->Type.setValue("Fuse");
     }
 
diff --git a/src/Mod/PartDesign/Gui/TaskBooleanParameters.ui b/src/Mod/PartDesign/Gui/TaskBooleanParameters.ui
index 8384eb9fda..954635e07c 100644
--- a/src/Mod/PartDesign/Gui/TaskBooleanParameters.ui
+++ b/src/Mod/PartDesign/Gui/TaskBooleanParameters.ui
@@ -58,11 +58,6 @@
        <string>Common</string>
       </property>
      </item>
-     <item>
-      <property name="text">
-       <string>Section</string>
-      </property>
-     </item>
     </widget>
    </item>
   </layout>