spare_parts/qml_connections_check.sh

+#!/bin/bash
+
+# Find all .qml files
+find . -type f -name "*.qml" | while read -r file; do
+    inside_connections=0
+    while IFS= read -r line || [[ -n "$line" ]]; do
+        ((line_number++))
+
+        # Detect the start of a Connections block
+        if [[ $line =~ "Connections"[[:space:]]*\{ ]]; then
+            inside_connections=1
+        fi
+
+        # Detect the end of a Connections block
+        if [[ $inside_connections -eq 1 && $line =~ \} ]]; then
+            inside_connections=0
+        fi
+
+        # Check for incorrect signal handlers inside Connections {}
+        if [[ $inside_connections -eq 1 && $line =~ ^[[:space:]]*on[A-Z] ]]; then
+            echo "Incorrect signal handler in $file at line $line_number: $line"
+        fi
+    done < "$file"
+    line_number=0
+done

src/qml/ScoreMenuContent.qml

@@ -43,7 +43,7 @@ Tmenu {
 
     Connections {
         target: score.notesMenuAct
-        onTriggered: {
+        function onTriggered() : void {
             notesLoader.active = true;
             notesLoader.item.open();
             close();
@@ -123,8 +123,8 @@ Tmenu {
 
     Connections {
         target: score.transposeAct
-        onTriggered: {
-            var tPop = transComp.createObject(nootkaWindow);
+        function onTriggered() : void {
+            let tPop = transComp.createObject(nootkaWindow);
             tPop.initKey = score.keySignature;
             close();
         }

src/qml/instruments/Ukulele.qml

@@ -142,7 +142,7 @@ TguitarBg {
     Connections {
         target: instrFlick
         enabled: !NOO.isAndroid()
-        onMovementEnded: instrument.pressedAt(0, 0)
+        function onMovementEnded() : void { instrument.pressedAt(0, 0) }
     }
 
     MouseArea {

src/qml/level/MelodyPage.qml

@@ -256,24 +256,20 @@ Tflickable {
                         bold: true
                         pixelSize: NOO.factor() * 0.9
                     }
-
                 }
-
             }
-
         }
-
     }
 
     Connections {
         target: creator
-        onUpdateNotesList: updateNotesList()
-        onUpdateMelodyList: checkMelodyView()
+        function onUpdateNotesList() : void { updateNotesList() }
+        function onUpdateMelodyList() : void { checkMelodyView() }
     }
 
     Connections {
         target: melCombo
-        onCurrentIndexChanged: checkMelodyView()
+        function onCurrentIndexChanged() : void { checkMelodyView() }
     }
 
 }

src/qml/score/MelGenDialog.qml

@@ -149,7 +149,7 @@ TmelGenItem {
 
     Connections {
         target: genButt
-        onClicked: {
+        function onClicked() : void {
             generate();
             dialLoader.close();
         }

src/qml/score/Scordature.qml

@@ -65,7 +65,7 @@ Grid {
 
     Connections {
         target: GLOB
-        onNoteNameStyleChanged: {
+        function onNoteNameStyleChanged() : void {
             for (var s = 0; s < scordModel.count; ++s) scordModel.get(s).name = GLOB.tuning.stringName(scordModel.get(s).strNr)
         }
     }

src/qml/settings/GlobalPage.qml

@@ -361,10 +361,9 @@ Tflickable {
 
     Connections {
         target: dialogObj
-        onUpdateSummary: {
+        function onUpdateSummary() : void {
             if (!updater)
                 updater = Qt.createComponent("qrc:/updater/UpdaterPopup.qml").createObject(globalPage);
-
             updater.open();
         }
     }

src/qml/sound/CountdownItem.qml

@@ -23,10 +23,9 @@ TipRect {
 
     Connections {
         target: tempoBar
-        onCntChanged: {
+        function onCntChanged() : void {
             if (SOUND.metroRunning)
                 counter++;
-
         }
     }
 

src/qml/sound/NotesDiffBar.qml

@@ -91,7 +91,7 @@ TnotesBarItem {
 
     Connections {
         target: GLOB
-        onGuitarParamsChanged: setAmbitus(GLOB.loNote(), GLOB.hiNote())
+        function onGuitarParamsChanged() : void { setAmbitus(GLOB.loNote(), GLOB.hiNote()) }
     }
 
 }

src/qml/updater/TupdateSummary.qml

@@ -22,7 +22,7 @@ TupdateItem {
 
     Connections {
         target: dialogObj
-        onUpdateSummary: {
+        function onUpdateSummary(version: string, changes: string, rules: var) : void {
             console.log(version);
             updateIt.version = version;
             updateIt.changes = changes;

src/qml/wizard/WizardClef.qml

@@ -84,7 +84,7 @@ Tflickable {
 
     Connections {
         target: nootkaWindow
-        onInstrumentChanged: changeInstrument()
+        function onInstrumentChanged() : void { changeInstrument() }
     }
 
 }

src/qml/wizard/WizardInstrument.qml

@@ -63,10 +63,9 @@ Tflickable {
 
     Connections {
         target: nootkaWindow.swipe
-        onCurrentIndexChanged: {
+        function onCurrentIndexChanged() : void {
             if (nootkaWindow.swipe.currentIndex != 1 && instrPage.instrument != instrSel.instrument)
                 instrPage.instrument = instrSel.instrument;
-
         }
     }
 

src/qml/wizard/WizardSax.qml

@@ -83,7 +83,7 @@ Column {
 
     Connections {
         target: nootkaWindow
-        onInstrumentChanged: transpose.shift = NOO.instr(nootkaWindow.instrument).transposition
+        function onInstrumentChanged() : void { transpose.shift = NOO.instr(nootkaWindow.instrument).transposition }
     }
 
 }