Add menu entry for sanity check

musescore · Feb 28, 2025 · 0f41f08 · 0f41f08
1 parent ca820ef
commit 0f41f08
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 1 deletion.
diff --git a/src/appshell/view/appmenumodel.cpp b/src/appshell/view/appmenumodel.cpp
@@ -465,7 +465,8 @@ MenuItem* AppMenuModel::makeDiagnosticsMenu()
             makeMenuItem("show-skylines"),
             makeMenuItem("show-system-bounding-rects"),
             makeMenuItem("show-element-masks"),
-            makeMenuItem("show-corrupted-measures")
+            makeMenuItem("show-corrupted-measures"),
+            makeMenuItem("check-for-score-corruptions")
         };
 
         MenuItemList extensionsItems {

diff --git a/src/notation/internal/notationactioncontroller.cpp b/src/notation/internal/notationactioncontroller.cpp
@@ -73,6 +73,28 @@ const std::unordered_map<ActionCode, bool EngravingDebuggingOptions::*> Notation
     { "show-corrupted-measures", &EngravingDebuggingOptions::showCorruptedMeasures }
 };
 
+void NotationActionController::checkForScoreCorruptions()
+{
+    String name = currentMasterNotation()->masterScore()->name();
+    Ret ret = currentMasterNotation()->masterScore()->sanityCheck();
+    if (ret) {
+        std::string title = muse::mtrc("project", "Score “%1” seems to be sane and healthy").arg(name).toStdString();
+        std::string body = muse::trc("project",
+                                     "This score does not seem to contain errors that could cause MuseScore Studio to malfunction.");
+
+        interactive()->info(title, body, {
+            interactive()->buttonData(IInteractive::Button::Ok) }).button();
+    } else {
+        std::string title = muse::mtrc("project", "Score “%1” is corrupted").arg(name).toStdString();
+        std::string body = muse::trc("project", "This score contains errors that could cause MuseScore Studio to malfunction.\n"
+                                                "Fix those at the earliest to prevent crashes and further corruptions.");
+
+        interactive()->warning(title, body, ret.text(), {
+            interactive()->buttonData(IInteractive::Button::Ok) }).button();
+    }
+    return;
+}
+
 void NotationActionController::init()
 {
     TRACEFUNC;
@@ -547,6 +569,7 @@ void NotationActionController::init()
             engravingConfiguration()->setDebuggingOptions(options);
         });
     }
+    dispatcher()->reg(this, "check-for-score-corruptions", [this] { checkForScoreCorruptions(); });
 }
 
 bool NotationActionController::canReceiveAction(const ActionCode& code) const

diff --git a/src/notation/internal/notationactioncontroller.h b/src/notation/internal/notationactioncontroller.h
@@ -53,6 +53,8 @@ class NotationActionController : public muse::actions::Actionable, public muse::
 public:
     void init();
 
+    void checkForScoreCorruptions();
+
     bool canReceiveAction(const muse::actions::ActionCode& code) const override;
 
     muse::async::Notification currentNotationChanged() const;

diff --git a/src/notation/internal/notationuiactions.cpp b/src/notation/internal/notationuiactions.cpp
@@ -2611,6 +2611,11 @@ const UiActionList NotationUiActions::m_engravingDebuggingActions = {
              TranslatableString("action", "Show/hide corrupted measures"),
              Checkable::Yes
              ),
+    UiAction("check-for-score-corruptions",
+             mu::context::UiCtxProjectOpened,
+             mu::context::CTX_NOTATION_OPENED,
+             TranslatableString("action", "Check for score corruptions")
+             ),
     UiAction("edit-strings",
              mu::context::UiCtxProjectOpened,
              mu::context::CTX_NOTATION_OPENED