From: nds <natalia.donis@opencascade.com>
Date: Thu, 23 Apr 2015 14:51:31 +0000 (+0300)
Subject: Non-pushed modification for the next bug:
X-Git-Tag: V_1.2.0~110^2~3^2~3
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=957a92adb93d332b3b33b5d3796e45c01c0fe8c5;p=modules%2Fshaper.git

Non-pushed modification for the next bug:

Start a sketch creation, do not select a plane, instead of it, click in any place in the viewer.
Result is - all nested sketch entities actions are enabled.
---

diff --git a/src/XGUI/XGUI_ActionsMgr.cpp b/src/XGUI/XGUI_ActionsMgr.cpp
index bdc91203b..beb901ee0 100644
--- a/src/XGUI/XGUI_ActionsMgr.cpp
+++ b/src/XGUI/XGUI_ActionsMgr.cpp
@@ -141,11 +141,15 @@ void XGUI_ActionsMgr::updateOnViewSelection()
   //QString aFeatureId = QString::fromStdString(anActiveFeature->getKind());
   XGUI_Selection* aSelection = myWorkshop->selector()->selection();
   if (aSelection->getSelected().size() == 0) {
-    foreach(QString aFeatureId, aIdList) {
+    // it seems that this code is not nesessary anymore. It leads to incorrect case:
+    // sketch operation start, click in any place in the viewer. The result is all nested
+    // entities are enabled(but the sketch plane is not selected yet). Any sketch operation
+    // can be started but will be incorrect on preview build before it uses the sketch unset plane.
+    /*foreach(QString aFeatureId, aIdList) {
       foreach(QString aId, nestedCommands(aFeatureId)) {
         setActionEnabled(aId, true);
       }
-    }
+    }*/
   } else { 
     SessionPtr aMgr = ModelAPI_Session::get();
     ModelAPI_ValidatorsFactory* aFactory = aMgr->validators();