Fix AudioNode and RecordNode state updating

Author: kmichaelfox

Source/Processors/AudioNode/AudioNode.cpp

@@ -113,7 +113,11 @@ void AudioNode::addInputChannel(GenericProcessor* sourceNode, int chan)
 
     setPlayConfigDetails(channelIndex+1,0,44100.0,128);
 
-    dataChannelArray.add(new DataChannel(*sourceNode->getDataChannel(chan)));
+    auto dataChannel = sourceNode->getDataChannel(chan);
+    auto dataChannelCopy = new DataChannel(*dataChannel);
+    dataChannelCopy->setMonitored(dataChannel->isMonitored());
+    
+    dataChannelArray.add(dataChannelCopy);
 
 }
 
@@ -534,4 +538,4 @@ void Expander::process(float* sampleData, int numSamples)
 
         sampleData[i] = sampleData[i] * gain;
     }
-}
+}

Source/Processors/Channel/InfoObjects.cpp

@@ -214,8 +214,8 @@ DataChannel::DataChannel(const DataChannel& ch)
 		m_type(ch.m_type),
 		m_bitVolts(ch.m_bitVolts),
 		m_isEnabled(true),
-		m_isMonitored(ch.m_isMonitored),
-		m_isRecording(ch.m_isRecording)
+		m_isMonitored(false),
+		m_isRecording(false)
 {
 }
 

Source/Processors/Editors/ChannelSelector.cpp

@@ -680,9 +680,16 @@ void ChannelSelector::buttonClicked(Button* button)
 
             if (acquisitionIsActive) // use setParameter to change parameter safely
             {
-                AccessClass::getProcessorGraph()->
+                if ( AccessClass::getProcessorGraph()->
                 getRecordNode()->
-                setChannelStatus(ch, status);
+                setChannelStatus(ch, status) )
+                {
+                    const_cast<DataChannel*>(ch)->setRecordState(status);
+                }
+                
+                // make sure that the button matches the system's actual state, in case
+                // user's interaction was disallowed
+                b->setToggleState(const_cast<DataChannel*>(ch)->getRecordState(), dontSendNotification);
             }
             else     // change parameter directly
             {

Source/Processors/RecordNode/RecordNode.cpp

@@ -69,15 +69,16 @@ RecordNode::~RecordNode()
 
 void RecordNode::setChannel(const DataChannel* ch)
 {
-    int channelNum = dataChannelArray.indexOf(ch);
+//    int channelNum = dataChannelArray.indexOf(ch);
+    int channelNum = getDataChannelIndex(ch->getSourceIndex(), ch->getSourceNodeID(), ch->getSubProcessorIdx());
 
     std::cout << "Record node setting channel to " << channelNum << std::endl;
 
     setCurrentChannel(channelNum);
 
 }
 
-void RecordNode::setChannelStatus(const DataChannel* ch, bool status)
+bool RecordNode::setChannelStatus(const DataChannel* ch, bool status)
 {
 
     //std::cout << "Setting channel status!" << std::endl;
@@ -87,7 +88,8 @@ void RecordNode::setChannelStatus(const DataChannel* ch, bool status)
         setParameter(2, 1.0f);
     else
         setParameter(2, 0.0f);
-
+    
+    return status == dataChannelArray[currentChannel]->getRecordState();
 }
 
 
@@ -555,4 +557,4 @@ void RecordNode::updateRecordChannelIndexes()
 void RecordNode::addSpecialProcessorChannels(Array<EventChannel*>& channels)
 {
 	eventChannelArray.addArray(channels);
-}
+}

Source/Processors/RecordNode/RecordNode.h

@@ -106,8 +106,11 @@ class RecordNode : public GenericProcessor,
     /** Turns recording on and off for a particular channel.
 
         Channel numbers are absolute (based on RecordNode channel mapping).
+        
+        Returns a bool indicating whether the status update was successful (true)
+        or blocked (false) because recording is currently in progress.
     */
-    void setChannelStatus(const DataChannel* ch, bool status);
+    bool setChannelStatus(const DataChannel* ch, bool status);
 
     /** Used to clear all connections prior to the start of acquisition.
     */