Skip to content

Commit

Permalink
Fix #307530: MusicXML export for hidden staves
Browse files Browse the repository at this point in the history 
Backport of musescore#11953
  • Loading branch information
@HemantAntony @Jojo-Schmitz
HemantAntony authored and Jojo-Schmitz committed Nov 7, 2023
1 parent 108b559 commit d1ab553
Show file tree
Hide file tree
Showing 4 changed files with 295 additions and 4 deletions.
10 changes: 6 additions & 4 deletions importexport/musicxml/exportxml.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6509,11 +6509,13 @@ static void writeStaffDetails(XmlWriter& xml, const Part* part)
// currently exported as a two staff part ...
for (int i = 0; i < staves; i++) {
Staff* st = part->staff(i);
if (st->lines(Fraction(0,1)) != 5 || st->isTabStaff(Fraction(0,1))) {
if (st->lines(Fraction(0,1)) != 5 || st->isTabStaff(Fraction(0,1)) || !st->show()) {
QString details = "staff-details";
if (staves > 1)
xml.stag(QString("staff-details number=\"%1\"").arg(i+1));
else
xml.stag("staff-details");
details += QString(" number=\"%1\"").arg(i+1);
if (!st->show())
details += " print-object=\"no\"";
xml.stag(details);
xml.tag("staff-lines", st->lines(Fraction(0,1)));
if (st->isTabStaff(Fraction(0,1)) && instrument->stringData()) {
QList<instrString> l = instrument->stringData()->stringList();
Expand Down
16 changes: 16 additions & 0 deletions importexport/musicxml/importmxmlpass2.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2258,6 +2258,16 @@ void MusicXMLParserPass2::part()
setStaffTypePercussion(msPart, drumset);
}

bool showPart = false;
for (Staff* staff : *msPart->staves()) {
if (!staff->invisible(Fraction(0,1))) {
showPart = true;
break;
}
}

msPart->setShow(showPart);

addError(checkAtEndElement(_e, "part"));
}

Expand Down Expand Up @@ -2824,6 +2834,12 @@ void MusicXMLParserPass2::staffDetails(const QString& partId)
int staffIdx = _score->staffIdx(part) + n;

StringData* t = new StringData;
QString visible = _e.attributes().value("print-object").toString();
if (visible == "no" )
_score->staff(staffIdx)->setInvisible(Fraction(0,1), true);
else if (!visible.isEmpty() && visible != "yes")
_logger->logError(QString("print-object should be \"yes\" or \"no\""));

int staffLines = 0;
while (_e.readNextStartElement()) {
if (_e.name() == "staff-lines") {
Expand Down
272 changes: 272 additions & 0 deletions mtest/musicxml/io/testHiddenStaves.xml
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,272 @@
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE score-partwise PUBLIC "-//Recordare//DTD MusicXML 4.0 Partwise//EN" "http://www.musicxml.org/dtds/partwise.dtd">
<score-partwise version="4.0">
<work>
<work-title>Untitled Score</work-title>
</work>
<identification>
<creator type="composer">Composer / arranger</creator>
<encoding>
<software>MuseScore 0.7.0</software>
<encoding-date>2007-09-10</encoding-date>
<supports element="accidental" type="yes"/>
<supports element="beam" type="yes"/>
<supports element="print" attribute="new-page" type="no"/>
<supports element="print" attribute="new-system" type="no"/>
<supports element="stem" type="yes"/>
</encoding>
</identification>
<part-list>
<part-group type="start" number="1">
<group-symbol>bracket</group-symbol>
</part-group>
<score-part id="P1">
<part-name>Flute</part-name>
<part-abbreviation>Fl.</part-abbreviation>
<score-instrument id="P1-I1">
<instrument-name>Flute</instrument-name>
</score-instrument>
<midi-device id="P1-I1" port="1"></midi-device>
<midi-instrument id="P1-I1">
<midi-channel>1</midi-channel>
<midi-program>74</midi-program>
<volume>78.7402</volume>
<pan>0</pan>
</midi-instrument>
</score-part>
<score-part id="P2">
<part-name>Bassoon</part-name>
<part-abbreviation>Bsn.</part-abbreviation>
<score-instrument id="P2-I1">
<instrument-name>Bassoon</instrument-name>
</score-instrument>
<midi-device id="P2-I1" port="1"></midi-device>
<midi-instrument id="P2-I1">
<midi-channel>2</midi-channel>
<midi-program>71</midi-program>
<volume>78.7402</volume>
<pan>0</pan>
</midi-instrument>
</score-part>
<part-group type="stop" number="1"/>
<score-part id="P3">
<part-name>Harpsichord</part-name>
<part-abbreviation>Hch.</part-abbreviation>
<score-instrument id="P3-I1">
<instrument-name>Harpsichord</instrument-name>
</score-instrument>
<midi-device id="P3-I1" port="1"></midi-device>
<midi-instrument id="P3-I1">
<midi-channel>3</midi-channel>
<midi-program>7</midi-program>
<volume>78.7402</volume>
<pan>0</pan>
</midi-instrument>
</score-part>
<score-part id="P4">
<part-name>Piano</part-name>
<part-abbreviation>Pno.</part-abbreviation>
<score-instrument id="P4-I1">
<instrument-name>Piano</instrument-name>
</score-instrument>
<midi-device id="P4-I1" port="1"></midi-device>
<midi-instrument id="P4-I1">
<midi-channel>4</midi-channel>
<midi-program>1</midi-program>
<volume>78.7402</volume>
<pan>0</pan>
</midi-instrument>
</score-part>
</part-list>
<part id="P1">
<measure number="1">
<attributes>
<divisions>1</divisions>
<key>
<fifths>0</fifths>
</key>
<time>
<beats>4</beats>
<beat-type>4</beat-type>
</time>
<clef>
<sign>G</sign>
<line>2</line>
</clef>
</attributes>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
</note>
</measure>
<measure number="2">
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
</note>
<barline location="right">
<bar-style>light-heavy</bar-style>
</barline>
</measure>
</part>
<part id="P2">
<measure number="1">
<attributes>
<divisions>1</divisions>
<key>
<fifths>0</fifths>
</key>
<time>
<beats>4</beats>
<beat-type>4</beat-type>
</time>
<clef>
<sign>F</sign>
<line>4</line>
</clef>
</attributes>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
</note>
</measure>
<measure number="2">
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
</note>
<barline location="right">
<bar-style>light-heavy</bar-style>
</barline>
</measure>
</part>
<part id="P3">
<measure number="1">
<attributes>
<divisions>1</divisions>
<key>
<fifths>0</fifths>
</key>
<time>
<beats>4</beats>
<beat-type>4</beat-type>
</time>
<staves>2</staves>
<clef number="1">
<sign>G</sign>
<line>2</line>
</clef>
<clef number="2">
<sign>F</sign>
<line>4</line>
</clef>
<staff-details number="1" print-object="no">
<staff-lines>5</staff-lines>
</staff-details>
<staff-details number="2" print-object="no">
<staff-lines>5</staff-lines>
</staff-details>
</attributes>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
<staff>1</staff>
</note>
<backup>
<duration>4</duration>
</backup>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>5</voice>
<staff>2</staff>
</note>
</measure>
<measure number="2">
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
<staff>1</staff>
</note>
<backup>
<duration>4</duration>
</backup>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>5</voice>
<staff>2</staff>
</note>
<barline location="right">
<bar-style>light-heavy</bar-style>
</barline>
</measure>
</part>
<part id="P4">
<measure number="1">
<attributes>
<divisions>1</divisions>
<key>
<fifths>0</fifths>
</key>
<time>
<beats>4</beats>
<beat-type>4</beat-type>
</time>
<staves>2</staves>
<clef number="1">
<sign>G</sign>
<line>2</line>
</clef>
<clef number="2">
<sign>F</sign>
<line>4</line>
</clef>
<staff-details number="2" print-object="no">
<staff-lines>5</staff-lines>
</staff-details>
</attributes>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
<staff>1</staff>
</note>
<backup>
<duration>4</duration>
</backup>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>5</voice>
<staff>2</staff>
</note>
</measure>
<measure number="2">
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>1</voice>
<staff>1</staff>
</note>
<backup>
<duration>4</duration>
</backup>
<note>
<rest measure="yes"/>
<duration>4</duration>
<voice>5</voice>
<staff>2</staff>
</note>
<barline location="right">
<bar-style>light-heavy</bar-style>
</barline>
</measure>
</part>
</score-partwise>
1 change: 1 addition & 0 deletions mtest/musicxml/io/tst_mxml_io.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,7 @@ private slots:
void hello() { mxmlIoTest("testHello"); }
void helloReadCompr() { mxmlReadTestCompr("testHello"); }
void helloReadWriteCompr() { mxmlReadWriteTestCompr("testHello"); }
void hiddenStaves() { mxmlIoTest("testHiddenStaves"); }
void implicitMeasure1() { mxmlIoTest("testImplicitMeasure1"); }
void incompleteTuplet() { mxmlIoTestRef("testIncompleteTuplet"); }
void incorrectStaffNumber1() { mxmlIoTestRef("testIncorrectStaffNumber1"); }
Expand Down

0 comments on commit d1ab553

Please sign in to comment.