From 9ddf765aa139febfce68653fb50282c68759591b Mon Sep 17 00:00:00 2001
From: mpv <mpv@opencascade.com>
Date: Wed, 8 Jul 2015 17:27:33 +0300
Subject: [PATCH] Remove of composite boolean feature implementation: with
 sub-sketch

---
 .../FeaturesPlugin_CompositeBoolean.cpp              | 12 ++++++++++++
 src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.h |  3 +++
 src/Model/Model_ResultPart.cpp                       |  2 +-
 src/ModelAPI/ModelAPI_Tools.cpp                      |  2 +-
 4 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.cpp b/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.cpp
index f8fff4cdd..c11ed54d5 100644
--- a/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.cpp
+++ b/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.cpp
@@ -79,6 +79,18 @@ void FeaturesPlugin_CompositeBoolean::removeFeature(std::shared_ptr<ModelAPI_Fea
 {
 }
 
+//=================================================================================================
+void FeaturesPlugin_CompositeBoolean::erase()
+{
+  FeaturePtr aSketch =
+    std::dynamic_pointer_cast<ModelAPI_Feature>(data()->reference(SKETCH_OBJECT_ID())->value());
+  if (aSketch.get() && aSketch->data()->isValid()) {
+    document()->removeFeature(aSketch);
+  }
+  ModelAPI_CompositeFeature::erase();
+}
+
+
 //=================================================================================================
 void FeaturesPlugin_CompositeBoolean::execute()
 {
diff --git a/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.h b/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.h
index c5ef7f3e6..6838e931b 100644
--- a/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.h
+++ b/src/FeaturesPlugin/FeaturesPlugin_CompositeBoolean.h
@@ -58,6 +58,9 @@ class FeaturesPlugin_CompositeBoolean : public ModelAPI_CompositeFeature
   /// structures of the owner (the remove from the document will be done outside just after)
   FEATURESPLUGIN_EXPORT virtual void removeFeature(std::shared_ptr<ModelAPI_Feature> theFeature);
 
+  /// removes also all sub-sketch
+  FEATURESPLUGIN_EXPORT virtual void erase();
+
 protected:
   FeaturesPlugin_CompositeBoolean(){};
 
diff --git a/src/Model/Model_ResultPart.cpp b/src/Model/Model_ResultPart.cpp
index 5023340b1..5523e7fbd 100644
--- a/src/Model/Model_ResultPart.cpp
+++ b/src/Model/Model_ResultPart.cpp
@@ -112,7 +112,7 @@ std::shared_ptr<GeomAPI_Shape> Model_ResultPart::shape()
 {
   if (myShape.IsNull()) {
     DocumentPtr aDoc = Model_ResultPart::partDoc();
-    if (aDoc.get()) {
+    if (aDoc.get() && aDoc->isOpened()) {
       const std::string& aBodyGroup = ModelAPI_ResultBody::group();
       TopoDS_Compound aResultComp;
       BRep_Builder aBuilder;
diff --git a/src/ModelAPI/ModelAPI_Tools.cpp b/src/ModelAPI/ModelAPI_Tools.cpp
index 2390d72d5..3ffd15ec4 100644
--- a/src/ModelAPI/ModelAPI_Tools.cpp
+++ b/src/ModelAPI/ModelAPI_Tools.cpp
@@ -174,7 +174,7 @@ CompositeFeaturePtr compositeOwner(const FeaturePtr& theFeature)
     for(; aRefIter != aRefs.end(); aRefIter++) {
       CompositeFeaturePtr aComp = std::dynamic_pointer_cast<ModelAPI_CompositeFeature>
         ((*aRefIter)->owner());
-      if (aComp.get() && aComp->isSub(theFeature))
+      if (aComp.get() && aComp->data()->isValid() && aComp->isSub(theFeature))
         return aComp;
     }
   }
-- 
2.39.2