From 170d28ffbd39b1f0406e437eaeac39a07389f525 Mon Sep 17 00:00:00 2001
From: Kevin Zander <veratil@gmail.com>
Date: Mon, 14 Oct 2019 21:22:41 -0500
Subject: [PATCH] Remove global automation from MIDI import (#5223)

---
 plugins/MidiImport/MidiImport.cpp | 35 +++++++++++++++++--------------
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/plugins/MidiImport/MidiImport.cpp b/plugins/MidiImport/MidiImport.cpp
index 1f2ab1f5777..b8653fe1886 100644
--- a/plugins/MidiImport/MidiImport.cpp
+++ b/plugins/MidiImport/MidiImport.cpp
@@ -309,10 +309,20 @@ bool MidiImport::readSMF( TrackContainer* tc )
 	smfMidiChannel chs[256];
 
 	MeterModel & timeSigMM = Engine::getSong()->getTimeSigModel();
-	AutomationPattern * timeSigNumeratorPat = 
-		AutomationPattern::globalAutomationPattern( &timeSigMM.numeratorModel() );
-	AutomationPattern * timeSigDenominatorPat = 
-		AutomationPattern::globalAutomationPattern( &timeSigMM.denominatorModel() );
+	AutomationTrack * nt = dynamic_cast<AutomationTrack*>(
+		Track::create(Track::AutomationTrack, Engine::getSong()));
+	nt->setName(tr("MIDI Time Signature Numerator"));
+	AutomationTrack * dt = dynamic_cast<AutomationTrack*>(
+		Track::create(Track::AutomationTrack, Engine::getSong()));
+	dt->setName(tr("MIDI Time Signature Denominator"));
+	AutomationPattern * timeSigNumeratorPat =
+		new AutomationPattern(nt);
+	timeSigNumeratorPat->setDisplayName(tr("Numerator"));
+	timeSigNumeratorPat->addObject(&timeSigMM.numeratorModel());
+	AutomationPattern * timeSigDenominatorPat =
+		new AutomationPattern(dt);
+	timeSigDenominatorPat->setDisplayName(tr("Denominator"));
+	timeSigDenominatorPat->addObject(&timeSigMM.denominatorModel());
 	
 	// TODO: adjust these to Time.Sig changes
 	double beatsPerTact = 4; 
@@ -323,19 +333,12 @@ bool MidiImport::readSMF( TrackContainer* tc )
 	for( int s = 0; s < timeSigs->length(); ++s )
 	{
 		Alg_time_sig timeSig = (*timeSigs)[s];
-		// Initial timeSig, set song-default value
-		if(/* timeSig.beat == 0*/ true )
-		{
-			// TODO set song-global default value
-			printf("Another timesig at %f\n", timeSig.beat);
-			timeSigNumeratorPat->putValue( timeSig.beat*ticksPerBeat, timeSig.num );
-			timeSigDenominatorPat->putValue( timeSig.beat*ticksPerBeat, timeSig.den );
-		}
-		else
-		{
-		}
-
+		timeSigNumeratorPat->putValue(timeSig.beat * ticksPerBeat, timeSig.num);
+		timeSigDenominatorPat->putValue(timeSig.beat * ticksPerBeat, timeSig.den);
 	}
+	// manually call otherwise the pattern shows being 1 bar
+	timeSigNumeratorPat->updateLength();
+	timeSigDenominatorPat->updateLength();
 
 	pd.setValue( 2 );