Simplify SetApplicationStateCmd, remove SurfaceManagerGui parts

src/Application/Application.cpp

@@ -98,29 +98,25 @@ void Application::onKeyPressed(ofKeyEventArgs & args){
 	 case '1':
 		 _cmdManager.exec(
 			 new ofx::piMapper::SetApplicationStateCmd(
-				 this, PresentationState::instance(),
+				 this, PresentationState::instance()));
-				 &_gui, GuiMode::NONE));
 		 break;
 
 	 case '2':
 		 _cmdManager.exec(
 			 new ofx::piMapper::SetApplicationStateCmd(
-				 this, TextureMappingState::instance(),
+				 this, TextureMappingState::instance()));
-				 &_gui, GuiMode::TEXTURE_MAPPING));
 		 break;
 
 	 case '3':
 		 _cmdManager.exec(
 			 new ofx::piMapper::SetApplicationStateCmd(
-				 this, ProjectionMappingState::instance(),
+				 this, ProjectionMappingState::instance()));
-				 &_gui, GuiMode::PROJECTION_MAPPING));
 		 break;
 
 	 case '4':
 		 _cmdManager.exec(
 			 new ofx::piMapper::SetApplicationStateCmd(
-				 this, SourceSelectionState::instance(),
+				 this, SourceSelectionState::instance()));
-				 &_gui, GuiMode::SOURCE_SELECTION));
 		 break;
 
 	 case 'f':

src/Application/States/PresentationState.cpp

@@ -22,8 +22,7 @@ void PresentationState::draw(Application * app){
 void PresentationState::onMousePressed(Application * app, ofMouseEventArgs & args){
     app->getCmdManager()->exec(
         new ofx::piMapper::SetApplicationStateCmd(
-            app, ProjectionMappingState::instance(),
+            app, ProjectionMappingState::instance()));
-            app->getGui(), GuiMode::PROJECTION_MAPPING));
 }
 
 } // namespace piMapper

src/Commands/SetApplicationStateCmd.cpp

@@ -4,18 +4,11 @@ namespace ofx {
 namespace piMapper {
 
 SetApplicationStateCmd::SetApplicationStateCmd(Application * app,
-											   ApplicationBaseState * st,
+											   ApplicationBaseState * st){
-											   SurfaceManagerGui * gui,
-											   int mode){
 
 	_application = app;
 	_prevApplicationState = 0;
 	_applicationState = st;
-
-	// TODO: To be removed
-	_gui = gui;
-	_prevGuiMode = -1;
-	_mode = mode;
 }
 
 void SetApplicationStateCmd::exec(){
@@ -27,9 +20,11 @@ void SetApplicationStateCmd::exec(){
 	Gui::instance()->getTextureEditorWidget().setSurface(
 		_application->getSurfaceManager()->getSelectedSurface());
 	
-	// TODO: To be removed.
+	if(_applicationState != PresentationState::instance()){
-	_prevGuiMode = _gui->getMode();
+		ofShowCursor();
-	_gui->setMode(_mode);
+	}else{
+		ofHideCursor();
+	}
 }
 
 void SetApplicationStateCmd::undo(){
@@ -37,8 +32,14 @@ void SetApplicationStateCmd::undo(){
 	_application->setState(_prevApplicationState);
 	_application->getState()->setTranslation(_translation);
 	
-	// TODO: To be removed.
+	Gui::instance()->getTextureEditorWidget().setSurface(
-	_gui->setMode(_prevGuiMode);
+		_application->getSurfaceManager()->getSelectedSurface());
+	
+	if(_prevApplicationState != PresentationState::instance()){
+		ofShowCursor();
+	}else{
+		ofHideCursor();
+	}
 }
 
 } // namespace piMapper

src/Commands/SetApplicationStateCmd.h

@@ -2,8 +2,6 @@
 
 #include "BaseCmd.h"
 #include "Application.h"
-#include "SurfaceManagerGui.h"
-#include "Gui.h"
 
 namespace ofx {
 namespace piMapper {
@@ -15,9 +13,7 @@ class SetApplicationStateCmd : public BaseUndoCmd {
 
 	public:
 		SetApplicationStateCmd(Application * app,
-							   ApplicationBaseState * st,
+							   ApplicationBaseState * st);
-							   SurfaceManagerGui * gui,
-							   int mode);
 
 		void exec();
 		void undo();
@@ -29,11 +25,6 @@ class SetApplicationStateCmd : public BaseUndoCmd {
 	
 		ofPoint _translation;
 
-		// TODO: Remove these after porting to app state system is done
-		SurfaceManagerGui * _gui;
-		int _prevGuiMode;
-		int _mode;
-
 };
 
 } // namespace piMapper