Fix for the issue #1196

diff --git a/src/Model/Model_Document.cpp b/src/Model/Model_Document.cpp
index ec071b40300a35ca2d586f902764a204002897ed..645291f72e0ae720f3090cc3b72234a3559af720 100755 (executable)
--- a/src/Model/Model_Document.cpp
+++ b/src/Model/Model_Document.cpp
@@ -979,13 +979,18 @@ void Model_Document::setCurrentFeature(
     }
     // update for everyone the concealment flag immideately: on edit feature in the midle of history
     if (aWasChanged) {
-      const std::list<std::shared_ptr<ModelAPI_Result> >& aResList = anIter->results();
-      std::list<std::shared_ptr<ModelAPI_Result> >::const_iterator aRes = aResList.begin();
-      for(; aRes != aResList.end(); aRes++) {
+      std::list<ResultPtr> aResults;
+      ModelAPI_Tools::allResults(anIter, aResults);
+      std::list<ResultPtr>::const_iterator aRes = aResults.begin();
+      for(; aRes != aResults.end(); aRes++) {
         if ((*aRes).get() && (*aRes)->data()->isValid() && !(*aRes)->isDisabled())
           std::dynamic_pointer_cast<Model_Data>((*aRes)->data())->updateConcealmentFlag();
       }
-
+      // update the concealment status for disply in isConcealed of ResultBody
+      for(aRes = aResults.begin(); aRes != aResults.end(); aRes++) {
+        if ((*aRes).get() && (*aRes)->data()->isValid() && !(*aRes)->isDisabled())
+          (*aRes)->isConcealed();
+      }
     }
   }
   // unblock  the flush signals and up them after this

diff --git a/src/Model/Model_Objects.cpp b/src/Model/Model_Objects.cpp
index f8ea0099ee299260e57f43c3a8520ac582a49d4c..aa8e2751c0980532839c2fbb18ea9c247aa4ee42 100644 (file)
--- a/src/Model/Model_Objects.cpp
+++ b/src/Model/Model_Objects.cpp
@@ -820,6 +820,16 @@ void Model_Objects::synchronizeBackRefs()
       }
     }
   }
+  for(aFeatures.Initialize(myFeatures); aFeatures.More(); aFeatures.Next()) {
+    FeaturePtr aFeature = aFeatures.Value();
+    std::list<ResultPtr> aResults;
+    ModelAPI_Tools::allResults(aFeature, aResults);
+    // update the concealment status for disply in isConcealed of ResultBody
+    std::list<ResultPtr>::iterator aRIter = aResults.begin();
+    for(; aRIter != aResults.cend(); aRIter++) {
+      (*aRIter)->isConcealed();
+    }
+  }
 }
 
 TDF_Label Model_Objects::resultLabel(

diff --git a/src/Model/Model_ResultBody.cpp b/src/Model/Model_ResultBody.cpp
index c4abb777c014ee86b92384a375cf3cdc677208ea..5fb26d84309209720f6c8689237aa89c7e4aa759 100644 (file)
--- a/src/Model/Model_ResultBody.cpp
+++ b/src/Model/Model_ResultBody.cpp
@@ -9,6 +9,7 @@
 #include <ModelAPI_AttributeIntArray.h>
 #include <ModelAPI_Tools.h>
 #include <Config_PropManager.h>
+#include <ModelAPI_Events.h>
 // DEB
 //#include <TCollection_AsciiString.hxx>
 //#include <TDF_Tool.hxx>
@@ -17,6 +18,7 @@
 Model_ResultBody::Model_ResultBody()
 {
   myBuilder = new Model_BodyBuilder(this);
+  myWasConcealed = false;
 }
 
 void Model_ResultBody::initAttributes()
@@ -50,15 +52,32 @@ bool Model_ResultBody::isLatestEqual(const std::shared_ptr<GeomAPI_Shape>& theSh
 
 bool Model_ResultBody::isConcealed()
 {
-  if (ModelAPI_ResultBody::isConcealed())
-    return true;
-  ResultPtr aThis = std::dynamic_pointer_cast<ModelAPI_Result>(data()->owner());
-  if (aThis.get()) {
-    ResultCompSolidPtr aParent = ModelAPI_Tools::compSolidOwner(aThis);
-    if (aParent.get()) {
-      if (aParent->isConcealed())
-        return true;
+  bool aResult = false;
+  if (ModelAPI_ResultBody::isConcealed()) {
+    aResult = true;
+  } else {
+    ResultPtr aThis = std::dynamic_pointer_cast<ModelAPI_Result>(data()->owner());
+    if (aThis.get()) {
+      ResultCompSolidPtr aParent = ModelAPI_Tools::compSolidOwner(aThis);
+      if (aParent.get()) {
+        if (aParent->isConcealed())
+          aResult = true;
+      }
     }
   }
-  return false;
+  if (myWasConcealed != aResult) {
+    myWasConcealed = aResult;
+    if (aResult) { // hidden unit must be redisplayed (hidden)
+      ModelAPI_EventCreator::get()->sendDeleted(document(), this->groupName());
+      // redisplay for the viewer (it must be disappeared also)
+      static Events_ID EVENT_DISP = 
+        Events_Loop::loop()->eventByName(EVENT_OBJECT_TO_REDISPLAY);
+      ModelAPI_EventCreator::get()->sendUpdated(data()->owner(), EVENT_DISP);
+    } else { // was not concealed become concealed => delete event
+      static Events_ID anEvent = Events_Loop::eventByName(EVENT_OBJECT_CREATED);
+      ModelAPI_EventCreator::get()->sendUpdated(data()->owner(), anEvent);
+    }
+  }
+
+  return aResult;
 }

diff --git a/src/Model/Model_ResultBody.h b/src/Model/Model_ResultBody.h
index 282c070470d48bf5c21df2c2eee497dbf55094a9..bbb23da35ed80a810fa26396569f16852fe75baf 100644 (file)
--- a/src/Model/Model_ResultBody.h
+++ b/src/Model/Model_ResultBody.h
@@ -28,6 +28,10 @@ class Model_ResultBody : public ModelAPI_ResultBody
   /// builders that tores the naming history: one per label to allow store several shapes to one 
   /// label; index in vector corresponds to the label tag
   //std::vector<TNaming_Builder*> myBuilders;
+
+  /// Flag that stores the previous state of "concealed": if it is changed,
+  /// The event is used to redisplay the body.
+  bool myWasConcealed;
 public:
   /// Request for initialization of data model of the result: adding all attributes
   virtual void initAttributes();