Fix bug when undoing a MOVE operation

Author: jsiegle

Source/Processors/ProcessorGraph/ProcessorGraph.cpp

@@ -160,9 +160,17 @@ void ProcessorGraph::moveProcessor(GenericProcessor* processor,
     checkForNewRootNodes(processor, false, true);
 
     if (moveDownstream) // processor is further down the signal chain, its original dest may have changed
+    {
+        LOGD("MOVE: Updating settings for ", originalDest->getNodeId());
         updateSettings(originalDest);
+    }
+        
     else // processor is upstream of its original dest, so we can just update that
+    {
+        LOGD("MOVE: Updating settings for ", processor->getNodeId());
         updateSettings(processor);
+    }
+        
 }
 
 GenericProcessor* ProcessorGraph::createProcessor(Plugin::Description& description,

Source/UI/EditorViewportActions.cpp

@@ -291,7 +291,7 @@ MoveProcessor::~MoveProcessor()
 
 bool MoveProcessor::perform()
 {
-    LOGDD("Peforming MOVE for processor ", nodeId);
+    LOGD("Peforming MOVE for processor ", nodeId);
 
     GenericProcessor* processor = AccessClass::getProcessorGraph()->getProcessorWithNodeId(nodeId);
 
@@ -310,7 +310,7 @@ bool MoveProcessor::perform()
 bool MoveProcessor::undo()
 {
 
-    LOGDD("Undoing MOVE for processor ", nodeId);
+    LOGD("Undoing MOVE for processor ", nodeId);
 
     GenericProcessor* processor = AccessClass::getProcessorGraph()->getProcessorWithNodeId(nodeId);
 
@@ -321,7 +321,7 @@ bool MoveProcessor::undo()
     AccessClass::getProcessorGraph()->moveProcessor(processor,
                                                     sourceProcessor,
                                                     destProcessor,
-                                                    moveDownstream);
+                                                    !moveDownstream);
 
     if (processor->isSource() && originalDestNodeDestNodeId > -1)
     {
@@ -330,7 +330,7 @@ bool MoveProcessor::undo()
         AccessClass::getProcessorGraph()->moveProcessor(originalDest,
                                                         destProcessor,
                                                         originalDest->getDestNode(),
-                                                        moveDownstream);
+                                                        !moveDownstream);
     }
 
     return true;