Merge pull request #26773 from rettinghaus/xml/fix-arpeggio-color

[MusicXML] read non-arpeggiate color and avoid double checks

Author: cbjeukendrup

src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp

@@ -8503,10 +8503,7 @@ void MusicXmlParserNotations::arpeggio()
     if (m_arpeggioNo == 0) {
         m_arpeggioNo = 1;
     }
-    Color color = Color::fromString(m_e.attribute("color"));
-    if (color.isValid()) {
-        m_arpeggioColor = color;
-    }
+    m_arpeggioColor = Color::fromString(m_e.attribute("color"));
     m_e.skipCurrentElement();  // skip but don't log
 }
 
@@ -8993,6 +8990,7 @@ void MusicXmlParserNotations::parse()
         } else if (m_e.name() == "glissando") {
             glissandoSlide();
         } else if (m_e.name() == "non-arpeggiate") {
+            m_arpeggioColor = Color::fromString(m_e.attribute("color"));
             m_arpeggioType = u"non-arpeggiate";
             m_e.skipCurrentElement();  // skip but don't log
         } else if (m_e.name() == "ornaments") {