From 565ea47bca70316604a59dcc22b03f37602011ba Mon Sep 17 00:00:00 2001
From: nds <natalia.donis@opencascade.com>
Date: Tue, 20 May 2014 10:16:02 +0400
Subject: [PATCH] refs #30 - Sketch base GUI: create, draw lines

Check the following scenario: create sketch, line. Undo/Redo. As a result the line is displayed. Undo/Redo. The line is displayed again.
---
 src/SketchPlugin/SketchPlugin_Feature.cpp | 25 ++++++++++++++---------
 src/XGUI/XGUI_Displayer.cpp               |  2 +-
 2 files changed, 16 insertions(+), 11 deletions(-)

diff --git a/src/SketchPlugin/SketchPlugin_Feature.cpp b/src/SketchPlugin/SketchPlugin_Feature.cpp
index e75bc01e6..dbaa6f824 100644
--- a/src/SketchPlugin/SketchPlugin_Feature.cpp
+++ b/src/SketchPlugin/SketchPlugin_Feature.cpp
@@ -2,6 +2,7 @@
 #include "SketchPlugin_Sketch.h"
 #include <ModelAPI_Document.h>
 #include <ModelAPI_Data.h>
+#include <ModelAPI_Object.h>
 #include <ModelAPI_AttributeRefList.h>
 
 SketchPlugin_Feature::SketchPlugin_Feature()
@@ -16,16 +17,20 @@ void SketchPlugin_Feature::setData(boost::shared_ptr<ModelAPI_Data> theData)
   // find sketch that references to this feature
   int aSketches = document()->size("Construction");
   for(int a = 0; a < aSketches && !mySketch; a++) {
-    boost::shared_ptr<SketchPlugin_Sketch> aSketch = 
-      boost::dynamic_pointer_cast<SketchPlugin_Sketch>(document()->feature("Construction", a));
-    if (aSketch) {
-      std::list<boost::shared_ptr<ModelAPI_Feature> > aList = 
-        aSketch->data()->reflist(SKETCH_ATTR_FEATURES)->list();
-      std::list<boost::shared_ptr<ModelAPI_Feature> >::iterator aSub = aList.begin();
-      for(; aSub != aList.end(); aSub++) {
-        if ((*aSub)->data()->isEqual(theData)) {
-          mySketch = aSketch.get();
-          break;
+    boost::shared_ptr<ModelAPI_Object> anObj = 
+      boost::dynamic_pointer_cast<ModelAPI_Object>(document()->feature("Construction", a));
+    if (anObj) {
+      boost::shared_ptr<SketchPlugin_Sketch> aSketch = 
+        boost::dynamic_pointer_cast<SketchPlugin_Sketch>(anObj->featureRef());
+      if (aSketch) {
+        std::list<boost::shared_ptr<ModelAPI_Feature> > aList = 
+          aSketch->data()->reflist(SKETCH_ATTR_FEATURES)->list();
+        std::list<boost::shared_ptr<ModelAPI_Feature> >::iterator aSub = aList.begin();
+        for(; aSub != aList.end(); aSub++) {
+          if ((*aSub)->data()->isEqual(theData)) {
+            mySketch = aSketch.get();
+            break;
+          }
         }
       }
     }
diff --git a/src/XGUI/XGUI_Displayer.cpp b/src/XGUI/XGUI_Displayer.cpp
index 73515eae7..7f6bb00d7 100644
--- a/src/XGUI/XGUI_Displayer.cpp
+++ b/src/XGUI/XGUI_Displayer.cpp
@@ -247,7 +247,7 @@ void XGUI_Displayer::EraseDeletedFeatures(const bool isUpdateViewer)
   for (; aFIt != aFLast; aFIt++)
   {
     boost::shared_ptr<ModelAPI_Feature> aFeature = (*aFIt).first;
-    if (!aFeature || !aFeature->data()->isValid()) {
+    if (!aFeature || !aFeature->data() || !aFeature->data()->isValid()) {
       Handle(AIS_InteractiveObject) anAIS = (*aFIt).second;
       if (!anAIS.IsNull()) {
         aContext->Erase(anAIS, false);
-- 
2.39.2