From f712e1cfc3b1d8320514de0c05aef8a6cd036550 Mon Sep 17 00:00:00 2001
From: wmayer <wmayer@users.sourceforge.net>
Date: Sun, 9 Oct 2016 19:45:30 +0200
Subject: [PATCH] move disabling of some commands to task panel

---
 src/Gui/SoFCCSysDragger.cpp |  8 --------
 src/Gui/TaskCSysDragger.cpp | 12 +++++++++---
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/src/Gui/SoFCCSysDragger.cpp b/src/Gui/SoFCCSysDragger.cpp
index ca70e8baa..52e69a5b5 100644
--- a/src/Gui/SoFCCSysDragger.cpp
+++ b/src/Gui/SoFCCSysDragger.cpp
@@ -49,8 +49,6 @@
 
 #include <Base/Quantity.h>
 #include <Gui/MainWindow.h>
-#include <Gui/Application.h>
-#include <Gui/Command.h>
 #include "SoFCDB.h"
 #include "SoFCCSysDragger.h"
 
@@ -809,16 +807,10 @@ SoFCCSysDragger::SoFCCSysDragger()
     this->addFinishCallback(&SoFCCSysDragger::finishDragCB, this);
 
     this->setUpConnections(TRUE, TRUE);
-    
-    //we can't have user switching camera types while dragger is shown.
-    Gui::Application::Instance->commandManager().getCommandByName("Std_OrthographicCamera")->setEnabled(false);
-    Gui::Application::Instance->commandManager().getCommandByName("Std_PerspectiveCamera")->setEnabled(false);
 }
 
 SoFCCSysDragger::~SoFCCSysDragger()
 {
-  Gui::Application::Instance->commandManager().getCommandByName("Std_OrthographicCamera")->setEnabled(true);
-  Gui::Application::Instance->commandManager().getCommandByName("Std_PerspectiveCamera")->setEnabled(true);
 }
 
 
diff --git a/src/Gui/TaskCSysDragger.cpp b/src/Gui/TaskCSysDragger.cpp
index bbc968f30..f70d3df50 100644
--- a/src/Gui/TaskCSysDragger.cpp
+++ b/src/Gui/TaskCSysDragger.cpp
@@ -33,6 +33,7 @@
 #include <Gui/TaskView/TaskView.h>
 #include "QuantitySpinBox.h"
 #include <Gui/Application.h>
+#include <Gui/Command.h>
 #include <Gui/Document.h>
 #include <Gui/BitmapFactory.h>
 #include <Gui/ViewProviderGeometryObject.h>
@@ -70,6 +71,8 @@ TaskCSysDragger::TaskCSysDragger(Gui::ViewProviderGeometryObject* vpObjectIn, Gu
 TaskCSysDragger::~TaskCSysDragger()
 {
   dragger->unref();
+  Gui::Application::Instance->commandManager().getCommandByName("Std_OrthographicCamera")->setEnabled(true);
+  Gui::Application::Instance->commandManager().getCommandByName("Std_PerspectiveCamera")->setEnabled(true);
 }
 
 void TaskCSysDragger::setupGui()
@@ -104,7 +107,7 @@ void TaskCSysDragger::setupGui()
   
   incrementsBox->groupLayout()->addLayout(gridLayout);
   Content.push_back(incrementsBox);
-  
+
   connect(tSpinBox, SIGNAL(valueChanged(double)), this, SLOT(onTIncrementSlot(double)));
   connect(rSpinBox, SIGNAL(valueChanged(double)), this, SLOT(onRIncrementSlot(double)));
 }
@@ -121,11 +124,14 @@ void TaskCSysDragger::onRIncrementSlot(double freshValue)
 
 void TaskCSysDragger::open()
 {
+  //we can't have user switching camera types while dragger is shown.
+  Gui::Application::Instance->commandManager().getCommandByName("Std_OrthographicCamera")->setEnabled(false);
+  Gui::Application::Instance->commandManager().getCommandByName("Std_PerspectiveCamera")->setEnabled(false);
 //   dragger->translationIncrement.setValue(lastTranslationIncrement);
 //   dragger->rotationIncrement.setValue(lastRotationIncrement);
   tSpinBox->setValue(lastTranslationIncrement);
   rSpinBox->setValue(radiansToDegrees(lastRotationIncrement));
-  
+
   Gui::TaskView::TaskDialog::open();
 }
 
@@ -133,7 +139,7 @@ bool TaskCSysDragger::accept()
 {
   lastTranslationIncrement = dragger->translationIncrement.getValue();
   lastRotationIncrement = dragger->rotationIncrement.getValue();
-  
+
   App::DocumentObject* dObject = vpObject.getObject();
   if (dObject) {
     Gui::Document* document = Gui::Application::Instance->getDocument(dObject->getDocument());