Remove dead playlistDirty methods from Score/MasterScore

src/braille/internal/braille.cpp

@@ -2402,8 +2402,10 @@ QString Braille::brailleMeasure(Measure* measure, int staffCount)
             resetOctave(staffCount);
 
             // Undo filling the missing beats with rests, so we don't have an altered score.
+/* see FIXME above
             m_score->undoRedo(true, nullptr);
             m_score->undoRedo(true, nullptr);
+*/
             m_score->deselectAll();
         }
     }

src/braille/tests/braille_tests.cpp

@@ -40,16 +40,6 @@ class Braille_Tests : public ::testing::Test
     void brailleSaveTest(const char* file);
 };
 
-//---------------------------------------------------------
-//   fixupScore -- do required fixups after reading/importing score
-//---------------------------------------------------------
-
-static void fixupScore(MasterScore* score)
-{
-    score->connectTies();
-    score->masterScore()->rebuildMidiMapping();
-}
-
 static bool saveBraille(MasterScore* score, const String& saveName)
 {
     QFile file(saveName);
@@ -73,7 +63,6 @@ void Braille_Tests::brailleSaveTest(const char* file)
     String fileName = String::fromUtf8(file);
     MasterScore* score = ScoreRW::readScore(BRAILLE_DIR + fileName + u".mscx", false);
     EXPECT_TRUE(score);
-    fixupScore(score);
     score->doLayout();
     EXPECT_TRUE(saveCompareBrailleScore(score, fileName + ".brf", BRAILLE_DIR + fileName + "_ref.brf"));
     delete score;

src/engraving/CMakeLists.txt

@@ -167,10 +167,12 @@ target_sources(engraving PRIVATE
     editing/splitjoinmeasure.h
     editing/textedit.cpp
     editing/textedit.h
+    editing/transaction/undoablecommand.cpp
+    editing/transaction/undoablecommand.h
+    editing/transaction/undostack.cpp
+    editing/transaction/undostack.h
     editing/transpose.cpp
     editing/transpose.h
-    editing/undo.cpp
-    editing/undo.h
 
     rw/ireader.h
     rw/iwriter.h

src/engraving/api/v1/elements.cpp

@@ -335,7 +335,6 @@ void Chord::setPlayEventType(mu::engraving::PlayEventType v)
 {
     // Only create undo operation if the value has changed.
     if (v != chord()->playEventType()) {
-        chord()->score()->setPlaylistDirty();
         chord()->score()->undo(new ChangeChordPlayEventType(chord(), v));
     }
 }

src/engraving/api/v1/elements.h

@@ -54,8 +54,6 @@
 #include "engraving/dom/tie.h"
 #include "engraving/dom/accidental.h"
 
-#include "engraving/editing/undo.h"
-
 #include "playevent.h"
 
 // api

src/engraving/api/v1/selection.cpp

@@ -22,7 +22,7 @@
 
 #include "selection.h"
 
-#include "engraving/editing/undo.h"
+#include "engraving/editing/transaction/undostack.h"
 
 // api
 #include "score.h"
@@ -83,8 +83,8 @@ bool Selection::select(EngravingItem* elWrapper, bool add)
     mu::engraving::EngravingItem* e = elWrapper->element();
 
     // Check whether it's safe to select this element:
-    // use types list from UndoMacro for now
-    if (!mu::engraving::UndoMacro::canRecordSelectedElement(e)) {
+    // use types list from UndoableTransaction for now
+    if (!mu::engraving::UndoableTransaction::canRecordSelectedElement(e)) {
         LOGW("Cannot select element of type %s", e->typeName());
         return false;
     }
@@ -136,7 +136,7 @@ bool Selection::selectRange(int startTick, int endTick, int startStaff, int endS
         return false;
     }
 
-    if (segEnd && m_selection->score()->undoStack()->hasActiveCommand()) {
+    if (segEnd && m_selection->score()->undoStack()->hasActiveTransaction()) {
         m_selection->setRangeTicks(segStart->tick(), segEnd->tick(), startStaff, endStaff);
     } else {
         m_selection->setRange(segStart, segEnd, startStaff, endStaff);

src/engraving/compat/midi/compatmidirender.cpp

@@ -35,7 +35,6 @@
 #include "dom/trill.h"
 #include "dom/utils.h"
 #include "dom/volta.h"
-#include "editing/undo.h"
 #include "types/constants.h"
 
 namespace mu::engraving {

src/engraving/dom/arpeggio.cpp

@@ -24,7 +24,6 @@
 
 #include "../editing/elementeditdata.h"
 #include "../editing/mscoreview.h"
-#include "../editing/undo.h"
 #include "../types/typesconv.h"
 
 #include "accidental.h"

src/engraving/dom/barline.cpp

@@ -24,7 +24,6 @@
 
 #include "translation.h"
 
-#include "../editing/undo.h"
 #include "../types/symnames.h"
 
 #include "articulation.h"

src/engraving/dom/box.cpp

@@ -25,7 +25,6 @@
 #include <cmath>
 
 #include "../editing/elementeditdata.h"
-#include "../editing/undo.h"
 #include "../editing/editfretboarddiagram.h"
 
 #include "actionicon.h"

src/engraving/dom/capo.cpp

@@ -22,6 +22,8 @@
 
 #include "capo.h"
 
+#include "score.h"
+
 #include "translation.h"
 
 using namespace mu::engraving;
@@ -109,6 +111,11 @@ bool Capo::setProperty(Pid id, const PropertyValue& val)
     }
 
     triggerLayout();
+
+    if (Score* s = score()) {
+        s->updateCapo();
+    }
+
     return true;
 }
 
@@ -171,3 +178,21 @@ muse::String Capo::generateText(size_t stringCount) const
 
     return text;
 }
+
+void Capo::added()
+{
+    StaffTextBase::added();
+
+    if (Score* s = score()) {
+        s->updateCapo();
+    }
+}
+
+void Capo::removed()
+{
+    StaffTextBase::removed();
+
+    if (Score* s = score()) {
+        s->updateCapo();
+    }
+}

src/engraving/dom/capo.h

@@ -20,8 +20,7 @@
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
 
-#ifndef MU_ENGRAVING_CAPO_H
-#define MU_ENGRAVING_CAPO_H
+#pragma once
 
 #include "stafftextbase.h"
 
@@ -49,11 +48,12 @@ class Capo final : public StaffTextBase
     bool shouldAutomaticallyGenerateText() const;
     String generateText(size_t stringCount) const;
 
+    void added() override;
+    void removed() override;
+
 private:
     CapoParams m_params;
     bool m_shouldAutomaticallyGenerateText = true;
     String m_customText;
 };
-} // namespace mu::engraving
-
-#endif // MU_ENGRAVING_CAPO_H
+}

src/engraving/dom/chord.cpp

@@ -658,8 +658,7 @@ void Chord::add(EngravingItem* e)
             measure()->setHasVoices(staffIdx(), true);
         }
     }
-        score()->setPlaylistDirty();
-        break;
+    break;
     case ElementType::ARPEGGIO:
         m_arpeggio = toArpeggio(e);
         break;
@@ -761,7 +760,6 @@ void Chord::remove(EngravingItem* e)
         if (voice() && measure() && note->visible()) {
             measure()->checkMultiVoices(staffIdx());
         }
-        score()->setPlaylistDirty();
     }
     break;
 

src/engraving/dom/clef.cpp

@@ -29,7 +29,6 @@
 
 #include "translation.h"
 
-#include "../editing/undo.h"
 #include "../editing/editproperty.h"
 #include "../types/typesconv.h"
 

src/engraving/dom/durationelement.cpp

@@ -22,13 +22,13 @@
 
 #include "durationelement.h"
 
-#include "../editing/undo.h"
-
 #include "property.h"
 #include "score.h"
 #include "staff.h"
 #include "tuplet.h"
 
+#include "editing/transaction/undostack.h"
+
 using namespace mu;
 using namespace mu::engraving;
 
@@ -123,7 +123,7 @@ Fraction DurationElement::actualTicks() const
 void DurationElement::readAddTuplet(Tuplet* t)
 {
     setTuplet(t);
-    if (!score()->undoStack()->hasActiveCommand()) { // HACK, also added in Undo::AddElement()
+    if (!score()->undoStack()->hasActiveTransaction()) { // HACK, also added in Undo::AddElement()
         t->add(this);
     }
 }

src/engraving/dom/dynamichairpingroup.cpp

@@ -22,8 +22,6 @@
 
 #include "dynamichairpingroup.h"
 
-#include "../editing/undo.h"
-
 #include "dynamic.h"
 #include "expression.h"
 #include "hairpin.h"

src/engraving/dom/engravingitem.cpp

@@ -362,7 +362,7 @@ void EngravingItem::deleteLater()
     if (selected()) {
         score()->deselect(this);
     }
-    masterScore()->deleteLater(this);
+    masterScore()->cmdState().deleteLater(this);
 }
 
 //---------------------------------------------------------

src/engraving/dom/engravingobject.cpp

@@ -24,9 +24,9 @@
 
 #include "global/containers.h"
 
-#include "../editing/undo.h"
 #include "../editing/addremoveelement.h"
 #include "../editing/editproperty.h"
+#include "../editing/transaction/undostack.h"
 #include "style/textstyle.h"
 #include "types/typesconv.h"
 

src/engraving/dom/excerpt.cpp

@@ -481,7 +481,7 @@ void Excerpt::createExcerpt(Excerpt* excerpt)
     }
 
     // second layout of score
-    score->setPlaylistDirty();
+    score->invalidateRepeatList();
     masterScore->rebuildMidiMapping();
     masterScore->updateChannel();
     score->remapBracketsAndBarlines();

src/engraving/dom/figuredbass.cpp

@@ -24,7 +24,6 @@
 
 #include "io/file.h"
 
-#include "../editing/undo.h"
 #include "rw/xmlreader.h"
 #include "style/textstyle.h"
 

src/engraving/dom/harmony.cpp

@@ -26,11 +26,7 @@
 #include "translation.h"
 
 #include "draw/fontmetrics.h"
-#include "draw/types/brush.h"
-#include "draw/types/pen.h"
 
-#include "../editing/textedit.h"
-#include "../editing/undo.h"
 #include "../editing/transpose.h"
 
 #include "chordlist.h"

src/engraving/dom/jump.cpp

@@ -157,7 +157,7 @@ bool Jump::setProperty(Pid propertyId, const PropertyValue& v)
         break;
     }
     triggerLayout();
-    score()->setPlaylistDirty();
+    score()->invalidateRepeatList();
     return true;
 }
 

src/engraving/dom/lyrics.cpp

@@ -25,7 +25,6 @@
 #include "types/translatablestring.h"
 
 #include "../editing/textedit.h"
-#include "../editing/undo.h"
 
 #include "measure.h"
 #include "navigate.h"

src/engraving/dom/masterscore.cpp

@@ -25,6 +25,7 @@
 
 #include "compat/writescorehook.h"
 #include "editing/editmeasures.h"
+#include "editing/transaction/undostack.h"
 #include "rw/mscloader.h"
 #include "rw/xmlreader.h"
 #include "rw/rwregister.h"
@@ -151,12 +152,11 @@ IAutomation* MasterScore::automation() const
 }
 
 //---------------------------------------------------------
-//   setPlaylistDirty
+//   invalidateRepeatLists
 //---------------------------------------------------------
 
-void MasterScore::setPlaylistDirty()
+void MasterScore::invalidateRepeatList()
 {
-    m_playlistDirty = true;
     m_expandedRepeatList->setScoreChanged();
     m_nonExpandedRepeatList->setScoreChanged();
 }
@@ -171,7 +171,7 @@ void MasterScore::setExpandRepeats(bool expand)
         return;
     }
     m_expandRepeats = expand;
-    setPlaylistDirty();
+    invalidateRepeatList();
 }
 
 //---------------------------------------------------------