From 90bf7c921291b7ce15f9902978e667bacb797021 Mon Sep 17 00:00:00 2001
From: Klaus Rettinghaus <klaus.rettinghaus@gmail.com>
Date: Wed, 11 Sep 2024 15:07:25 +0200
Subject: [PATCH 1/5] fix typo

---
 src/importexport/mei/internal/meiexporter.cpp | 8 ++++----
 src/importexport/mei/internal/meiexporter.h   | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/importexport/mei/internal/meiexporter.cpp b/src/importexport/mei/internal/meiexporter.cpp
index c6c8f6af4a8e5..e00459fee5a1e 100644
--- a/src/importexport/mei/internal/meiexporter.cpp
+++ b/src/importexport/mei/internal/meiexporter.cpp
@@ -1183,7 +1183,7 @@ bool MeiExporter::writeChord(const Chord* chord, const Staff* staff)
             meiChord.SetDots(chord->dots());
         }
         this->writeBeamTypeAtt(chord, meiChord);
-        this->writeStaffIdenAtt(chord, staff, meiChord);
+        this->writeStaffIdentAtt(chord, staff, meiChord);
         this->writeStemAtt(chord, meiChord);
         this->writeArtics(chord);
         this->writeVerses(chord);
@@ -1273,7 +1273,7 @@ bool MeiExporter::writeNote(const Note* note, const Chord* chord, const Staff* s
             meiNote.SetDots(chord->dots());
         }
         this->writeBeamTypeAtt(chord, meiNote);
-        this->writeStaffIdenAtt(chord, staff, meiNote);
+        this->writeStaffIdentAtt(chord, staff, meiNote);
         this->writeStemAtt(chord, meiNote);
         this->writeArtics(chord);
         this->writeVerses(chord);
@@ -1343,7 +1343,7 @@ bool MeiExporter::writeRest(const Rest* rest, const Staff* staff)
         }
         Convert::colorToMEI(rest, meiRest);
         this->writeBeamTypeAtt(rest, meiRest);
-        this->writeStaffIdenAtt(rest, staff, meiRest);
+        this->writeStaffIdentAtt(rest, staff, meiRest);
         // this->writeVerses(rest);
         const char prefix = (rest->visible()) ? 'r' : 's';
         std::string xmlId = this->getXmlIdFor(rest, prefix);
@@ -2053,7 +2053,7 @@ bool MeiExporter::writeBeamTypeAtt(const ChordRest* chordRest, libmei::AttTyped&
  * Write the cross-staff attribute (@staff) for a ChordRest (i.e., chord, note, rest or space).
  */
 
-bool MeiExporter::writeStaffIdenAtt(const ChordRest* chordRest, const Staff* staff, libmei::AttStaffIdent& staffIdentAtt)
+bool MeiExporter::writeStaffIdentAtt(const ChordRest* chordRest, const Staff* staff, libmei::AttStaffIdent& staffIdentAtt)
 {
     if (chordRest->staffMove() != 0) {
         staff_idx_t staffN = staff->idx() + chordRest->staffMove() + 1;
diff --git a/src/importexport/mei/internal/meiexporter.h b/src/importexport/mei/internal/meiexporter.h
index 3f1d61326aa8b..706fa8f9e1f6f 100644
--- a/src/importexport/mei/internal/meiexporter.h
+++ b/src/importexport/mei/internal/meiexporter.h
@@ -147,7 +147,7 @@ class MeiExporter
      * Methods for writing specific MEI attribute classes within elements
      */
     bool writeBeamTypeAtt(const engraving::ChordRest* chordRest, libmei::AttTyped& typeAtt);
-    bool writeStaffIdenAtt(const engraving::ChordRest* chordRest, const engraving::Staff* staff, libmei::AttStaffIdent& staffIdentAtt);
+    bool writeStaffIdentAtt(const engraving::ChordRest* chordRest, const engraving::Staff* staff, libmei::AttStaffIdent& staffIdentAtt);
     bool writeStemAtt(const engraving::Chord* chord, libmei::AttStems& stemsAtt);
 
     /**

From a944952042b2be7be8ad3122587b8b4fa799cfff Mon Sep 17 00:00:00 2001
From: Klaus Rettinghaus <klaus.rettinghaus@gmail.com>
Date: Thu, 12 Sep 2024 13:51:50 +0200
Subject: [PATCH 2/5] add support for staff and layer attributes

---
 .../mei/internal/meiconverter.cpp             | 70 +++++++++++++++++++
 src/importexport/mei/internal/meiconverter.h  |  5 ++
 2 files changed, 75 insertions(+)

diff --git a/src/importexport/mei/internal/meiconverter.cpp b/src/importexport/mei/internal/meiconverter.cpp
index 79f74fe39145d..1fbe80c57c04e 100644
--- a/src/importexport/mei/internal/meiconverter.cpp
+++ b/src/importexport/mei/internal/meiconverter.cpp
@@ -1258,6 +1258,9 @@ void Convert::dynamFromMEI(engraving::Dynamic* dynamic, const StringList& meiLin
         dynamic->setDynamicType(meiLines.at(0));
     }
 
+    // @layer
+    Convert::layerIdentFromMEI(dynamic, meiDynam);
+
     // text content
     StringList lines;
     // For each line in the dynamic text
@@ -1307,6 +1310,12 @@ libmei::Dynam Convert::dynamToMEI(const engraving::Dynamic* dynamic, StringList&
         meiDynam.SetPlace(Convert::directionToMEI(dynamic->direction()));
     }
 
+    // @layer
+    Convert::layerIdentToMEI(dynamic, meiDynam);
+
+    // @staff
+    Convert::staffIdentToMEI(dynamic, meiDynam);
+
     // @label
     if (dynamic->dynamicType() != engraving::DynamicType::OTHER) {
         meiDynam.SetLabel(engraving::TConv::toXml(dynamic->dynamicType()).ascii());
@@ -1605,6 +1614,9 @@ void Convert::hairpinFromMEI(engraving::Hairpin* hairpin, const libmei::Hairpin&
 
     // @color
     Convert::colorlineFromMEI(hairpin, meiHairpin);
+
+    // @layer
+    Convert::layerIdentFromMEI(hairpin, meiHairpin);
 }
 
 libmei::Hairpin Convert::hairpinToMEI(const engraving::Hairpin* hairpin)
@@ -1631,6 +1643,12 @@ libmei::Hairpin Convert::hairpinToMEI(const engraving::Hairpin* hairpin)
     // @color
     Convert::colorlineToMEI(hairpin, meiHairpin);
 
+    // @layer
+    Convert::layerIdentToMEI(hairpin, meiHairpin);
+
+    // @staff
+    Convert::staffIdentToMEI(hairpin, meiHairpin);
+
     return meiHairpin;
 }
 
@@ -2296,6 +2314,9 @@ libmei::Octave Convert::octaveToMEI(const engraving::Ottava* ottava)
     // @color
     Convert::colorlineToMEI(ottava, meiOctave);
 
+    // @staff
+    Convert::staffIdentToMEI(ottava, meiOctave);
+
     return meiOctave;
 }
 
@@ -2946,6 +2967,9 @@ libmei::Tempo Convert::tempoToMEI(const engraving::TempoText* tempoText, StringL
     // text content - only split lines
     meiLines = String(tempoText->plainText()).split(u"\n");
 
+    // @staff
+    Convert::staffIdentToMEI(tempoText, meiTempo);
+
     return meiTempo;
 }
 
@@ -3333,6 +3357,52 @@ std::list<std::string> Convert::getTypeValuesWithPrefix(const std::string& typeS
     return values;
 }
 
+void Convert::layerIdentFromMEI(engraving::EngravingItem* item, const libmei::Element& meiElement)
+{
+    const libmei::AttLayerIdent* layerAtt = dynamic_cast<const libmei::AttLayerIdent*>(&meiElement);
+
+    IF_ASSERT_FAILED(layerAtt) {
+        return;
+    }
+
+    // @color
+    if (layerAtt->HasLayer()) {
+        // without further check we assume the layer to match
+        if (item->hasVoiceAssignmentProperties()) {
+            item->setProperty(engraving::Pid::VOICE_ASSIGNMENT, engraving::VoiceAssignment::CURRENT_VOICE_ONLY);
+        }
+    }
+}
+
+void Convert::layerIdentToMEI(const engraving::EngravingItem* item, libmei::Element& meiElement)
+{
+    libmei::AttLayerIdent* layerAtt = dynamic_cast<libmei::AttLayerIdent*>(&meiElement);
+
+    IF_ASSERT_FAILED(layerAtt) {
+        return;
+    }
+
+    if (item->hasVoiceAssignmentProperties()
+        && (item->getProperty(engraving::Pid::VOICE_ASSIGNMENT).value<engraving::VoiceAssignment>()
+            == engraving::VoiceAssignment::CURRENT_VOICE_ONLY)) {
+        layerAtt->SetLayer(item->voice() + 1);
+    }
+}
+
+void Convert::staffIdentToMEI(const engraving::EngravingItem* item, libmei::Element& meiElement)
+{
+    libmei::AttStaffIdent* staffAtt = dynamic_cast<libmei::AttStaffIdent*>(&meiElement);
+
+    IF_ASSERT_FAILED(staffAtt) {
+        return;
+    }
+
+    libmei::xsdPositiveInteger_List staffList;
+    staffList.push_back(item->staff()->idx() + 1);
+    // TODO: add staff number if centered between staves
+    staffAtt->SetStaff(staffList);
+}
+
 double Convert::tstampFromFraction(const engraving::Fraction& fraction, const engraving::Fraction& timesig)
 {
     return (double)fraction.numerator() / fraction.denominator() * timesig.denominator() + 1.0;
diff --git a/src/importexport/mei/internal/meiconverter.h b/src/importexport/mei/internal/meiconverter.h
index 8d16360c9ce63..b90f2f15b461d 100644
--- a/src/importexport/mei/internal/meiconverter.h
+++ b/src/importexport/mei/internal/meiconverter.h
@@ -223,6 +223,9 @@ class Convert
     static void harmFromMEI(engraving::Harmony* harmony, const muse::StringList& meiLines, const libmei::Harm& meiHarm, bool& warning);
     static libmei::Harm harmToMEI(const engraving::Harmony* harmony, muse::StringList& meiLines);
 
+    static void layerIdentFromMEI(engraving::EngravingItem* item, const libmei::Element& meiElement);
+    static void layerIdentToMEI(const engraving::EngravingItem* item, libmei::Element& meiElement);
+
     static void lvFromMEI(engraving::Articulation* lv, const libmei::Lv& meiLv, bool& warning);
     static libmei::Lv lvToMEI(const engraving::Articulation* lv);
 
@@ -281,6 +284,8 @@ class Convert
     static StaffStruct staffFromMEI(const libmei::StaffDef& meiStaffDef, bool& warning);
     static libmei::StaffDef staffToMEI(const engraving::Staff* staff);
 
+    static void staffIdentToMEI(const engraving::EngravingItem* item, libmei::Element& meiElement);
+
     static std::pair<engraving::DirectionV, bool> stemFromMEI(const libmei::AttStems& meiStemsAtt, bool& warning);
     static std::pair<libmei::data_STEMDIRECTION, double> stemToMEI(const engraving::DirectionV direction, bool noStem);
 

From 1adaaa489d5c3bc982402846a3564457a1c3ad3c Mon Sep 17 00:00:00 2001
From: Klaus Rettinghaus <klaus.rettinghaus@gmail.com>
Date: Sat, 14 Sep 2024 21:42:11 +0200
Subject: [PATCH 3/5] update tests

---
 src/importexport/mei/tests/data/color-01.mei  |  4 ++--
 .../mei/tests/data/dynamic-01.mei             | 16 +++++++--------
 src/importexport/mei/tests/data/ending-01.mei |  2 +-
 .../mei/tests/data/hairpin-01.mei             | 14 ++++++-------
 src/importexport/mei/tests/data/octave-01.mei | 12 +++++------
 src/importexport/mei/tests/data/tempo-01.mei  | 20 +++++++++----------
 6 files changed, 34 insertions(+), 34 deletions(-)

diff --git a/src/importexport/mei/tests/data/color-01.mei b/src/importexport/mei/tests/data/color-01.mei
index a2a9d1926aceb..599d550b905f5 100644
--- a/src/importexport/mei/tests/data/color-01.mei
+++ b/src/importexport/mei/tests/data/color-01.mei
@@ -88,8 +88,8 @@
                            <note xml:id="n12667r3" dur="4" pname="c" oct="5" />
                         </layer>
                      </staff>
-                     <octave xml:id="oggpsks" dis="8" dis.place="above" startid="#nx5z8ek" color="#0433FF" endid="#n1p30nx0" />
-                     <hairpin xml:id="h1ofeby0" form="cres" startid="#nx5z8ek" color="#0433FF" endid="#n12667r3" />
+                     <octave xml:id="oggpsks" staff="1" dis="8" dis.place="above" startid="#nx5z8ek" color="#0433FF" endid="#n1p30nx0" />
+                     <hairpin xml:id="h1ofeby0" form="cres" staff="1" startid="#nx5z8ek" color="#0433FF" endid="#n12667r3" />
                      <tie xml:id="t1cu6c7m" startid="#n12667r3" color="#0433FF" endid="#n1oshtpn" />
                   </measure>
                   <measure xml:id="m1dsz34o" n="4">
diff --git a/src/importexport/mei/tests/data/dynamic-01.mei b/src/importexport/mei/tests/data/dynamic-01.mei
index b2b4a0bdf2f69..3259b2c7975c4 100644
--- a/src/importexport/mei/tests/data/dynamic-01.mei
+++ b/src/importexport/mei/tests/data/dynamic-01.mei
@@ -54,11 +54,11 @@
                            <note xml:id="ns8t4tw" dur="1" pname="c" oct="4" />
                         </layer>
                      </staff>
-                     <dynam xml:id="dvuqhls" label="p" startid="#nrgzdcj" place="above">p</dynam>
-                     <dynam xml:id="d1hg5le3" label="mp" startid="#n6si3n7">mp</dynam>
-                     <dynam xml:id="d1n0kf96" label="sffz" startid="#n7c44ti">sffz</dynam>
-                     <dynam xml:id="d1btxom9" label="sfz" startid="#nvtsktx">sfz</dynam>
-                     <dynam xml:id="d1rbma6l" label="pp" startid="#ns8t4tw">pp</dynam>
+                     <dynam xml:id="dvuqhls" label="p" staff="1" startid="#nrgzdcj" place="above">p</dynam>
+                     <dynam xml:id="d1hg5le3" label="mp" staff="1" startid="#n6si3n7">mp</dynam>
+                     <dynam xml:id="d1n0kf96" label="sffz" staff="1" startid="#n7c44ti">sffz</dynam>
+                     <dynam xml:id="d1btxom9" label="sfz" staff="1" startid="#nvtsktx">sfz</dynam>
+                     <dynam xml:id="d1rbma6l" label="pp" staff="1" startid="#ns8t4tw">pp</dynam>
                   </measure>
                   <measure xml:id="m428gem" right="end" n="2">
                      <staff xml:id="m2s1" n="1">
@@ -76,9 +76,9 @@
                            </chord>
                         </layer>
                      </staff>
-                     <dynam xml:id="d1pv19cd" label="mp" startid="#noxjpqs">sempre mp e <lb />dolce</dynam>
-                     <dynam xml:id="da76h31" label="p" startid="#c1bio1o2">p e sempre<lb />dolce</dynam>
-                     <dynam xml:id="dqa2pt" label="ff" startid="#c19y1p0c">molto ff</dynam>
+                     <dynam xml:id="d1pv19cd" label="mp" staff="1" startid="#noxjpqs">sempre mp e <lb />dolce</dynam>
+                     <dynam xml:id="da76h31" label="p" staff="1" startid="#c1bio1o2">p e sempre<lb />dolce</dynam>
+                     <dynam xml:id="dqa2pt" label="ff" staff="1" startid="#c19y1p0c">molto ff</dynam>
                   </measure>
                </section>
             </score>
diff --git a/src/importexport/mei/tests/data/ending-01.mei b/src/importexport/mei/tests/data/ending-01.mei
index fb1743472374f..ff66724657503 100644
--- a/src/importexport/mei/tests/data/ending-01.mei
+++ b/src/importexport/mei/tests/data/ending-01.mei
@@ -57,7 +57,7 @@
                            <note xml:id="nj5abn7" dur="1" pname="b" oct="2" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t1sz2k2m" type="mscore-infer-from-text" tstamp="1.000000" midi.bpm="320.000000">
+                     <tempo xml:id="t1sz2k2m" type="mscore-infer-from-text" staff="1" tstamp="1.000000" midi.bpm="320.000000">
                         <rend glyph.auth="smufl"></rend> = 160</tempo>
                   </measure>
                   <ending xml:id="e56vjvc" label="1." type="mscore-ending-1">
diff --git a/src/importexport/mei/tests/data/hairpin-01.mei b/src/importexport/mei/tests/data/hairpin-01.mei
index bf9b65390edeb..17ac53df82c55 100644
--- a/src/importexport/mei/tests/data/hairpin-01.mei
+++ b/src/importexport/mei/tests/data/hairpin-01.mei
@@ -58,7 +58,7 @@
                            <mRest xml:id="mfs9r97" />
                         </layer>
                      </staff>
-                     <hairpin xml:id="h1dor5os" form="cres" startid="#n1n2q0nd" endid="#n14xf1ls" />
+                     <hairpin xml:id="h1dor5os" form="cres" staff="1" startid="#n1n2q0nd" endid="#n14xf1ls" />
                   </measure>
                   <measure xml:id="mcatpri" n="2">
                      <staff xml:id="m2s1" n="1">
@@ -74,8 +74,8 @@
                            <mRest xml:id="m1sz0e05" />
                         </layer>
                      </staff>
-                     <dynam xml:id="d1eogas6" label="f" startid="#nspxw9k">f</dynam>
-                     <hairpin xml:id="hniw3gu" form="dim" startid="#nspxw9k" endid="#n1w9pjbk" />
+                     <dynam xml:id="d1eogas6" label="f" staff="1" startid="#nspxw9k">f</dynam>
+                     <hairpin xml:id="hniw3gu" form="dim" staff="1" startid="#nspxw9k" endid="#n1w9pjbk" />
                   </measure>
                   <measure xml:id="mzs8qux" n="3">
                      <staff xml:id="m3s1" n="1">
@@ -121,7 +121,7 @@
                            <note xml:id="naukxk7" dur="4" pname="g" oct="2" />
                         </layer>
                      </staff>
-                     <dynam xml:id="d16couhq" label="f" startid="#n1h73eje">f</dynam>
+                     <dynam xml:id="d16couhq" label="f" staff="2" startid="#n1h73eje">f</dynam>
                      <dir xml:id="dxfp0cj" type="mscore-hairpin mscore-decresc" startid="#n1h73eje" extender="true" endid="#n4ihxav">dim.</dir>
                   </measure>
                   <measure xml:id="m1suce6e" n="6">
@@ -166,7 +166,7 @@
                            <mRest xml:id="mks294k" />
                         </layer>
                      </staff>
-                     <hairpin xml:id="hjs1sym" form="cres" startid="#n9ff3t1" lform="dashed" place="above" endid="#n2tj1jf" />
+                     <hairpin xml:id="hjs1sym" form="cres" staff="1" startid="#n9ff3t1" lform="dashed" place="above" endid="#n2tj1jf" />
                   </measure>
                   <measure xml:id="moaa1gv" n="9">
                      <staff xml:id="m9s1" n="1">
@@ -182,7 +182,7 @@
                            <mRest xml:id="m1okzahl" />
                         </layer>
                      </staff>
-                     <hairpin xml:id="hhnn0kt" form="dim" startid="#nq8sgb7" lform="dotted" place="above" endid="#n19df3r8" />
+                     <hairpin xml:id="hhnn0kt" form="dim" staff="1" startid="#nq8sgb7" lform="dotted" place="above" endid="#n19df3r8" />
                   </measure>
                   <measure xml:id="m1vhuhz5" n="10">
                      <staff xml:id="m10s1" n="1">
@@ -212,7 +212,7 @@
                            <note xml:id="ntcvrux" dur="4" pname="d" oct="3" />
                         </layer>
                      </staff>
-                     <hairpin xml:id="heiz76k" form="cres" startid="#n1ahnf5i" place="above" endid="#nsqyndt" />
+                     <hairpin xml:id="heiz76k" form="cres" staff="2" startid="#n1ahnf5i" place="above" endid="#nsqyndt" />
                   </measure>
                   <measure xml:id="mjmh4f6" n="12">
                      <staff xml:id="m12s1" n="1">
diff --git a/src/importexport/mei/tests/data/octave-01.mei b/src/importexport/mei/tests/data/octave-01.mei
index 20d75ac6f10ba..29987fab49ff8 100644
--- a/src/importexport/mei/tests/data/octave-01.mei
+++ b/src/importexport/mei/tests/data/octave-01.mei
@@ -58,7 +58,7 @@
                            <mRest xml:id="s2l1_t0_1" />
                         </layer>
                      </staff>
-                     <octave xml:id="m1ot1" dis="8" dis.place="above" startid="#s1l1_t1_2" endid="#s1l1_t5_4" />
+                     <octave xml:id="m1ot1" staff="1" dis="8" dis.place="above" startid="#s1l1_t1_2" endid="#s1l1_t5_4" />
                   </measure>
                   <measure xml:id="m2" n="2">
                      <staff xml:id="m2s1" n="1">
@@ -74,7 +74,7 @@
                            <mRest xml:id="s2l1_t1_1" />
                         </layer>
                      </staff>
-                     <octave xml:id="m2ot1" dis="15" dis.place="above" startid="#s1l1_t3_2" lform="solid" endid="#s1l1_t5_2" />
+                     <octave xml:id="m2ot1" staff="1" dis="15" dis.place="above" startid="#s1l1_t3_2" lform="solid" endid="#s1l1_t5_2" />
                   </measure>
                   <measure xml:id="m3" n="3">
                      <staff xml:id="m3s1" n="1">
@@ -104,7 +104,7 @@
                            <note xml:id="s2l1_t15_4" dur="4" pname="d" oct="3" oct.ges="2" />
                         </layer>
                      </staff>
-                     <octave xml:id="m4ot1" dis="8" dis.place="below" startid="#s2l1_t7_2" lform="dotted" endid="#s2l1_t17_4" />
+                     <octave xml:id="m4ot1" staff="2" dis="8" dis.place="below" startid="#s2l1_t7_2" lform="dotted" endid="#s2l1_t17_4" />
                   </measure>
                   <measure xml:id="m5" n="5">
                      <staff xml:id="m5s1" n="1">
@@ -120,7 +120,7 @@
                            <note xml:id="s2l1_t19_4" dur="4" pname="g" oct="3" oct.ges="1" />
                         </layer>
                      </staff>
-                     <octave xml:id="m5ot1" dis="15" dis.place="below" startid="#s2l1_t9_2" lendsym="none" endid="#s2l1_t11_2" />
+                     <octave xml:id="m5ot1" staff="2" dis="15" dis.place="below" startid="#s2l1_t9_2" lendsym="none" endid="#s2l1_t11_2" />
                   </measure>
                   <measure xml:id="m6" n="6">
                      <staff xml:id="m6s1" n="1">
@@ -149,8 +149,8 @@
                            <rest xml:id="s2l1_t13_2" dur="2" />
                         </layer>
                      </staff>
-                     <octave xml:id="m7ot1" dis="22" dis.place="above" startid="#s1l1_t6_1" extender="false" endid="#s1l1_t13_2" />
-                     <octave xml:id="m7ot2" dis="22" dis.place="below" startid="#s2l1_t6_1" endid="#s2l1_t13_2" />
+                     <octave xml:id="m7ot1" staff="1" dis="22" dis.place="above" startid="#s1l1_t6_1" extender="false" endid="#s1l1_t13_2" />
+                     <octave xml:id="m7ot2" staff="2" dis="22" dis.place="below" startid="#s2l1_t6_1" endid="#s2l1_t13_2" />
                   </measure>
                </section>
             </score>
diff --git a/src/importexport/mei/tests/data/tempo-01.mei b/src/importexport/mei/tests/data/tempo-01.mei
index 9f05a34bac8f7..577e87d173ecd 100644
--- a/src/importexport/mei/tests/data/tempo-01.mei
+++ b/src/importexport/mei/tests/data/tempo-01.mei
@@ -61,7 +61,7 @@
                            <note xml:id="nrmm3ez" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t19ctd6p" type="mscore-infer-from-text" tstamp="1.000000" midi.bpm="80.000000">
+                     <tempo xml:id="t19ctd6p" type="mscore-infer-from-text" staff="1" tstamp="1.000000" midi.bpm="80.000000">
                         <rend glyph.auth="smufl"></rend> = 80</tempo>
                   </measure>
                   <measure xml:id="m1yzdfof" n="2">
@@ -79,7 +79,7 @@
                            <note xml:id="n1oc3itk" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t1qcdn3j" type="mscore-infer-from-text" tstamp="1.000000" midi.bpm="120.000000">
+                     <tempo xml:id="t1qcdn3j" type="mscore-infer-from-text" staff="1" tstamp="1.000000" midi.bpm="120.000000">
                         <rend glyph.auth="smufl"></rend> = 120</tempo>
                   </measure>
                   <measure xml:id="m1h89rz0" n="3">
@@ -97,7 +97,7 @@
                            <note xml:id="n1rf6jms" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="td9e89e" type="mscore-infer-from-text" tstamp="1.000000" midi.bpm="160.000000">
+                     <tempo xml:id="td9e89e" type="mscore-infer-from-text" staff="1" tstamp="1.000000" midi.bpm="160.000000">
                         <rend glyph.auth="smufl"></rend> = 80</tempo>
                   </measure>
                   <measure xml:id="m13fajcg" n="4">
@@ -115,7 +115,7 @@
                            <note xml:id="nt1fnmr" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t1ydvqo1" tstamp="1.000000" midi.bpm="172.000000">Vivace</tempo>
+                     <tempo xml:id="t1ydvqo1" staff="1" tstamp="1.000000" midi.bpm="172.000000">Vivace</tempo>
                   </measure>
                   <measure xml:id="mti300x" n="5">
                      <staff xml:id="m5s1" n="1">
@@ -132,7 +132,7 @@
                            <note xml:id="nk9ewxf" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t13pakdc" tstamp="1.000000" midi.bpm="187.000000">Presto</tempo>
+                     <tempo xml:id="t13pakdc" staff="1" tstamp="1.000000" midi.bpm="187.000000">Presto</tempo>
                   </measure>
                   <measure xml:id="m86cvw0" n="6">
                      <staff xml:id="m6s1" n="1">
@@ -149,7 +149,7 @@
                            <note xml:id="nxo2jts" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t1pqrmiv" type="mscore-infer-from-text" tstamp="1.000000" midi.bpm="120.000000">
+                     <tempo xml:id="t1pqrmiv" type="mscore-infer-from-text" staff="1" tstamp="1.000000" midi.bpm="120.000000">
                         <rend glyph.auth="smufl"></rend> = <rend glyph.auth="smufl"></rend>
                      </tempo>
                   </measure>
@@ -168,7 +168,7 @@
                            <note xml:id="nzl59q9" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t1egyk2b" type="mscore-infer-from-text" tstamp="1.000000" midi.bpm="120.000000">
+                     <tempo xml:id="t1egyk2b" type="mscore-infer-from-text" staff="1" tstamp="1.000000" midi.bpm="120.000000">
                         <rend glyph.auth="smufl"></rend> = <rend glyph.auth="smufl"></rend>
                      </tempo>
                   </measure>
@@ -187,7 +187,7 @@
                            <note xml:id="nowkwf8" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t16u6fwq" tstamp="1.000000" midi.bpm="92.000000">Andante (<rend glyph.auth="smufl"></rend> = 120)</tempo>
+                     <tempo xml:id="t16u6fwq" staff="1" tstamp="1.000000" midi.bpm="92.000000">Andante (<rend glyph.auth="smufl"></rend> = 120)</tempo>
                   </measure>
                   <measure xml:id="mdoorsi" n="9">
                      <staff xml:id="m9s1" n="1">
@@ -268,7 +268,7 @@
                            <note xml:id="nopyavo" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="t4t06bt" tstamp="1.000000" midi.bpm="144.000000">Allegro<lb />molto</tempo>
+                     <tempo xml:id="t4t06bt" staff="1" tstamp="1.000000" midi.bpm="144.000000">Allegro<lb />molto</tempo>
                   </measure>
                   <measure xml:id="m1fq65a7" n="14">
                      <staff xml:id="m14s1" n="1">
@@ -285,7 +285,7 @@
                            <note xml:id="nzgcaps" dur="2" pname="g" oct="3" />
                         </layer>
                      </staff>
-                     <tempo xml:id="tm7hfns" startid="#nmjnl8c" midi.bpm="172.000000">Vivace</tempo>
+                     <tempo xml:id="tm7hfns" staff="1" startid="#nmjnl8c" midi.bpm="172.000000">Vivace</tempo>
                   </measure>
                   <measure xml:id="m6uij9w" n="15">
                      <staff xml:id="m15s1" n="1">

From 3b89951bb42cba141d00286cf0680edd77d8dc72 Mon Sep 17 00:00:00 2001
From: Klaus Rettinghaus <klaus.rettinghaus@gmail.com>
Date: Sun, 15 Sep 2024 15:27:20 +0200
Subject: [PATCH 4/5] add voice assignment to test

---
 src/importexport/mei/tests/data/dynamic-01.mei  | 2 +-
 src/importexport/mei/tests/data/dynamic-01.mscx | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/importexport/mei/tests/data/dynamic-01.mei b/src/importexport/mei/tests/data/dynamic-01.mei
index 3259b2c7975c4..de9ca3cb67dda 100644
--- a/src/importexport/mei/tests/data/dynamic-01.mei
+++ b/src/importexport/mei/tests/data/dynamic-01.mei
@@ -57,7 +57,7 @@
                      <dynam xml:id="dvuqhls" label="p" staff="1" startid="#nrgzdcj" place="above">p</dynam>
                      <dynam xml:id="d1hg5le3" label="mp" staff="1" startid="#n6si3n7">mp</dynam>
                      <dynam xml:id="d1n0kf96" label="sffz" staff="1" startid="#n7c44ti">sffz</dynam>
-                     <dynam xml:id="d1btxom9" label="sfz" staff="1" startid="#nvtsktx">sfz</dynam>
+                     <dynam xml:id="d1btxom9" label="sfz" layer="1" staff="1" startid="#nvtsktx">sfz</dynam>
                      <dynam xml:id="d1rbma6l" label="pp" staff="1" startid="#ns8t4tw">pp</dynam>
                   </measure>
                   <measure xml:id="m428gem" right="end" n="2">
diff --git a/src/importexport/mei/tests/data/dynamic-01.mscx b/src/importexport/mei/tests/data/dynamic-01.mscx
index 385a7de6bcfba..0d173df2da8d4 100644
--- a/src/importexport/mei/tests/data/dynamic-01.mscx
+++ b/src/importexport/mei/tests/data/dynamic-01.mscx
@@ -104,6 +104,7 @@
           <Dynamic>
             <subtype>sfz</subtype>
             <velocity>112</velocity>
+            <voiceAssignment>currentVoiceOnly</voiceAssignment>
             </Dynamic>
           <Chord>
             <durationType>eighth</durationType>

From fcbca570806799887d5eda52eb8fbcb8ab47a033 Mon Sep 17 00:00:00 2001
From: Klaus Rettinghaus <klaus.rettinghaus@gmail.com>
Date: Mon, 16 Sep 2024 10:44:35 +0200
Subject: [PATCH 5/5] early return in voice assignment

---
 src/importexport/mei/internal/meiconverter.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/importexport/mei/internal/meiconverter.cpp b/src/importexport/mei/internal/meiconverter.cpp
index 1fbe80c57c04e..2fe6d90881bac 100644
--- a/src/importexport/mei/internal/meiconverter.cpp
+++ b/src/importexport/mei/internal/meiconverter.cpp
@@ -3359,18 +3359,19 @@ std::list<std::string> Convert::getTypeValuesWithPrefix(const std::string& typeS
 
 void Convert::layerIdentFromMEI(engraving::EngravingItem* item, const libmei::Element& meiElement)
 {
+    if (!item->hasVoiceAssignmentProperties()) {
+        return;
+    }
+
     const libmei::AttLayerIdent* layerAtt = dynamic_cast<const libmei::AttLayerIdent*>(&meiElement);
 
     IF_ASSERT_FAILED(layerAtt) {
         return;
     }
 
-    // @color
     if (layerAtt->HasLayer()) {
         // without further check we assume the layer to match
-        if (item->hasVoiceAssignmentProperties()) {
-            item->setProperty(engraving::Pid::VOICE_ASSIGNMENT, engraving::VoiceAssignment::CURRENT_VOICE_ONLY);
-        }
+        item->setProperty(engraving::Pid::VOICE_ASSIGNMENT, engraving::VoiceAssignment::CURRENT_VOICE_ONLY);
     }
 }