Fix

Author: CubikingChill

src/engraving/dom/instrchange.cpp

@@ -34,7 +34,6 @@
 #include "score.h"
 #include "segment.h"
 #include "staff.h"
-
 #include "log.h"
 
 using namespace mu;
@@ -82,8 +81,15 @@ InstrumentChange::~InstrumentChange()
 void InstrumentChange::setInstrument(const Instrument& i)
 {
     *m_instrument = i;
-    //delete _instrument;
-    //_instrument = new Instrument(i);
+    // Number of staves for this instrument
+    size_t n = i.cleffTypeCount();
+    _concertClefs.resize(n);
+    _transposingClefs.resize(n);
+    for (size_t s = 0; s < n; ++s) {
+        ClefTypeList ct = i.clefType(s);
+        _concertClefs[s]     = ct.concertClef;
+        _transposingClefs[s] = ct.transposingClef;
+    }
 }
 
 void InstrumentChange::setupInstrument(const Instrument* instrument)
@@ -101,17 +107,26 @@ void InstrumentChange::setupInstrument(const Instrument* instrument)
 
     // change the clef for each staff
     for (size_t i = 0; i < part->nstaves(); i++) {
-        ClefType oldClefType = concPitch ? part->instrument(tickStart)->clefType(i).concertClef
-                               : part->instrument(tickStart)->clefType(i).transposingClef;
-        ClefType newClefType = concPitch ? instrument->clefType(i).concertClef
-                               : instrument->clefType(i).transposingClef;
-        // Introduce cleff change only if the new clef *symbol* is different from the old one
-        if (ClefInfo::symId(oldClefType) != ClefInfo::symId(newClefType)) {
-            // If instrument change is at the start of a measure, use the measure as the element, as this will place the instrument change before the barline.
-            EngravingItem* element = rtick().isZero() ? toEngravingItem(findMeasure()) : toEngravingItem(this);
-            score()->undoChangeClef(part->staff(i), element, newClefType, true);
-        }
+    ClefType oldClefType = concPitch ? part->instrument(tickStart)->clefType(i).concertClef
+                           : part->instrument(tickStart)->clefType(i).transposingClef;
+    ClefTypeList ct = instrument->clefType(i);
+    ClefType newClefType = concPitch ? ct.concertClef : ct.transposingClef;
+    if (ClefInfo::symId(oldClefType) != ClefInfo::symId(newClefType)) {
+        EngravingItem* element = rtick().isZero() ? toEngravingItem(findMeasure()) : toEngravingItem(this);
+        score()->undoChangeClef(part->staff(i), element, newClefType, true);
     }
+}
+
+for (size_t i = 0; i < part->nstaves(); i++) {
+    Staff* staff = part->staff(i);
+
+    ClefType oldClef = staff->clef(tickStart);
+    ClefType newClef = clefTypeForCurrentLayout(i);
+
+    if (ClefInfo::symId(oldClef) != ClefInfo::symId(newClef)) {
+        score()->undoChangeClef(staff, this, newClef, true);
+    }
+}
 
     // Change key signature if necessary. CAUTION: not necessary in case of octave-transposing!
     if ((v.chromatic - oldV.chromatic) % 12) {
@@ -213,4 +228,13 @@ engraving::PropertyValue InstrumentChange::propertyDefault(Pid propertyId) const
         return TextBase::propertyDefault(propertyId);
     }
 }
+
+ClefType InstrumentChange::clefTypeForCurrentLayout(size_t staffIndex) const
+{
+    bool isConcert = score()->style().styleB(Sid::concertPitch);
+    return isConcert
+        ? _concertClefs[staffIndex]
+        : _transposingClefs[staffIndex];
+}
+
 }

src/engraving/dom/instrchange.h

@@ -67,6 +67,11 @@ class InstrumentChange final : public TextBase
 
     bool placeMultiple() const override { return false; }
 
+    ClefType clefTypeForCurrentLayout(size_t staffIndex) const;
+
+    std::vector<ClefType> _concertClefs;
+    std::vector<ClefType> _transposingClefs;
+
 private:
 
     Instrument* m_instrument = nullptr;     // Staff holds ownership if part of score