Clean up, rm `duplicateSurface` method from `CommandManager`

src/Commands/DuplicateSurfaceCmd.cpp

@@ -9,7 +9,9 @@ DuplicateSurfaceCmd::DuplicateSurfaceCmd(BaseSurface * surface, SurfaceManager *
 }
 
 void DuplicateSurfaceCmd::exec(){
-	_duplicate = _surfaceManager->duplicateSurface(_surface);
+	ofLogNotice("DuplicateSurfaceCmd", "exec");
+	_duplicate = _surface->clone();
+	_surfaceManager->addSurface(_duplicate);
 	_duplicate->moveBy(ofVec2f(10.0f, 10.0f));
 	_surfaceManager->selectSurface(_duplicate);
 }

src/Surfaces/SurfaceManager.cpp

@@ -166,12 +166,6 @@ BaseSurface * SurfaceManager::getSelectedSurface(){
 	return selectedSurface;
 }
 
-BaseSurface * SurfaceManager::duplicateSurface(BaseSurface * surface){
-	BaseSurface * duplicate = surface->clone();
-	addSurface(duplicate);
-	return duplicate;
-}
-
 // TODO: select vertex should be implemented ad BaseSurface level
 void SurfaceManager::selectNextVertex(){
 	if(selectedSurface == 0){

src/Surfaces/SurfaceManager.h

@@ -43,7 +43,6 @@ class SurfaceManager {
 		BaseSurface * selectNextSurface();
 		BaseSurface * selectPrevSurface();
 		BaseSurface * getSelectedSurface();
-		BaseSurface * duplicateSurface(BaseSurface * surface);
 	
 		// These should trigger an event for the GUI layer to catch
 		void selectNextVertex();