Updated PR to add support to regression scenarios

This PR still has an scenario to deal with (testTuplets5.xml). A thread has been opened within this PR to determine how to deal with it

Author: pacebes

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

@@ -1017,9 +1017,9 @@ static void resizeTitleBox(VBox* vbox)
         }
     }
 
-    const auto height = Spatium::fromMM(calculatedVBoxHeight, vbox->spatium());
-    if (height > vbox->propertyDefault(Pid::BOX_HEIGHT).value<Spatium>()) {
-        vbox->undoChangeProperty(Pid::BOX_HEIGHT, height);
+    double heightInSp = calculatedVBoxHeight / vbox->spatium();
+    if (heightInSp > vbox->propertyDefault(Pid::BOX_HEIGHT).toDouble()) {
+        vbox->undoChangeProperty(Pid::BOX_HEIGHT, heightInSp);
     }
 }
 
@@ -2373,9 +2373,6 @@ void MusicXmlParserPass1::partGroup(const int scoreParts,
             if (m_e.readText() == "no") {
                 barlineSpan = false;
             }
-        } else if (m_e.name() == "group-time") {
-            m_score->style().set(Sid::timeSigPlacement, TimeSigPlacement::ACROSS_STAVES);
-            m_e.skipCurrentElement();          // skip but don't log
         } else {
             skipLogCurrElem();
         }
@@ -2426,7 +2423,6 @@ void MusicXmlParserPass1::scorePart(const String& curPartGroupName)
             String name = m_e.readText();
             m_parts[id].setName(name);
         } else if (m_e.name() == "part-name-display") {
-            m_parts[id].setPrintName(m_e.asciiAttribute("print-object") != "no");
             String name;
             while (m_e.readNextStartElement()) {
                 if (m_e.name() == "display-text") {
@@ -2449,7 +2445,6 @@ void MusicXmlParserPass1::scorePart(const String& curPartGroupName)
             String name = m_e.readText();
             m_parts[id].setAbbr(name);
         } else if (m_e.name() == "part-abbreviation-display") {
-            m_parts[id].setPrintAbbr(m_e.asciiAttribute("print-object") != "no");
             String name;
             while (m_e.readNextStartElement()) {
                 if (m_e.name() == "display-text") {
@@ -2892,7 +2887,6 @@ void MusicXmlParserPass1::measure(const String& partId,
 
     // delete tuplets
     nesTuplets.clear();
-
     addError(checkAtEndElement(m_e, u"measure"));
 }
 
@@ -3330,6 +3324,10 @@ void MusicXmlParserPass1::notations(MusicXmlStartStop& tupletStartStop, unsigned
     tupletsTimeMod.clear();
 
     while (m_e.readNextStartElement()) {
+        if (m_e.isStartElement()) {
+        }
+        if (m_e.isEndElement()) {
+        }
         if (m_e.name() == "tuplet") {
             String tupletType = m_e.attribute("type");
 
@@ -3545,31 +3543,30 @@ void MusicXmlParserPass1::note(const String& partId,
 
         if (tupletStartStop == MusicXmlStartStop::START) {
             unsigned int tupletDepth = nestedTupletState.currentTupletDepth();
-            Fraction realTimeMod;
+            Fraction currentTupletTimeMod;
 
             for (unsigned int i = 1; i <= numberOfStartsOrStops; ++i) {
                 ++tupletDepth;
                 nesTuplets[voice][tupletDepth] = tupletsTimeMod[i];
-                realTimeMod = nesTuplets[voice][tupletDepth];
-
-                nestedTupletState.determineTupletAction(mnd.duration(), realTimeMod, tupletStartStop,
+                currentTupletTimeMod = nesTuplets[voice][tupletDepth];
+                nestedTupletState.determineTupletAction(mnd.duration(), mnd.timeMod(), currentTupletTimeMod, tupletStartStop,
                                                         mnd.normalType(), missingPrev, missingCurr);
             }
         } else if (tupletStartStop == MusicXmlStartStop::STOP) {
             unsigned int tupletDepth;
-            Fraction realTimeMod;
+            Fraction currentTupletTimeMod;
 
             for (unsigned int i = numberOfStartsOrStops; i > 0; --i) {
                 tupletDepth = nestedTupletState.currentTupletDepth();
-                realTimeMod = nesTuplets[voice][tupletDepth];
-                nestedTupletState.determineTupletAction(mnd.duration(), realTimeMod, tupletStartStop,
+                currentTupletTimeMod = nesTuplets[voice][tupletDepth];
+                nestedTupletState.determineTupletAction(mnd.duration(), mnd.timeMod(), currentTupletTimeMod, tupletStartStop,
                                                         mnd.normalType(), missingPrev, missingCurr);
             }
         } else {
             unsigned int tupletDepth = nestedTupletState.currentTupletDepth();
-            Fraction realTimeMod = nesTuplets[voice][tupletDepth];
-            nestedTupletState.determineTupletAction(mnd.duration(), realTimeMod, tupletStartStop, mnd.normalType(), missingPrev,
-                                                    missingCurr);
+            Fraction currentTupletTimeMod = nesTuplets[voice][tupletDepth];
+            nestedTupletState.determineTupletAction(mnd.duration(), mnd.timeMod(), currentTupletTimeMod, tupletStartStop,
+                                                    mnd.normalType(), missingPrev, missingCurr);
         }
     }