Merge pull request #9471 from Ultimaker/Fix_QML_warnings_in_Qt5.15

Fix QML warnings produced due to signal calls in Connections

plugins/MachineSettingsAction/MachineSettingsAction.qml

@@ -27,7 +27,7 @@ Cura.MachineAction
     Connections
     {
         target: extrudersModel
-        onItemsChanged: tabNameModel.update()
+        function onItemsChanged() { tabNameModel.update() }
     }
 
     ListModel

plugins/MachineSettingsAction/MachineSettingsPrinterTab.qml

@@ -326,7 +326,7 @@ Item
                 Connections
                 {
                     target: Cura.MachineManager
-                    onGlobalContainerChanged: extruderCountModel.update()
+                    function onGlobalContainerChanged() { extruderCountModel.update() }
                 }
             }
 

plugins/PerObjectSettingsTool/PerObjectItem.qml

@@ -35,7 +35,7 @@ UM.TooltipArea
     Connections
     {
         target: addedSettingsModel
-        onVisibleCountChanged:
+        function onVisibleCountChanged()
         {
             check.checked = addedSettingsModel.getVisible(model.key)
         }

plugins/PerObjectSettingsTool/PerObjectSettingsPanel.qml

@@ -334,13 +334,13 @@ Item
                         Connections
                         {
                             target: inheritStackProvider
-                            onPropertiesChanged: provider.forcePropertiesChanged()
+                            function onPropertiesChanged() { provider.forcePropertiesChanged() }
                         }
 
                         Connections
                         {
                             target: UM.ActiveTool
-                            onPropertiesChanged:
+                            function onPropertiesChanged()
                             {
                                 // the values cannot be bound with UM.ActiveTool.properties.getValue() calls,
                                 // so here we connect to the signal and update the those values.

plugins/PostProcessingPlugin/PostProcessingPlugin.qml

@@ -414,7 +414,7 @@ UM.Dialog
                         {
                             target: item
 
-                            onShowTooltip:
+                            function onShowTooltip(text)
                             {
                                 tooltip.text = text
                                 var position = settingLoader.mapToItem(settingsPanel, settingsPanel.x, 0)

plugins/SimulationView/SimulationViewMainComponent.qml

@@ -55,8 +55,8 @@ Item
         Connections
         {
             target: UM.SimulationView
-            onMaxPathsChanged: pathSlider.setHandleValue(UM.SimulationView.currentPath)
-            onCurrentPathChanged:
+            function onMaxPathsChanged() { pathSlider.setHandleValue(UM.SimulationView.currentPath) }
+            function onCurrentPathChanged()
             {
                 // Only pause the simulation when the layer was changed manually, not when the simulation is running
                 if (pathSlider.manuallyChanged)
@@ -89,7 +89,7 @@ Item
         Connections
         {
             target: UM.Preferences
-            onPreferenceChanged:
+            function onPreferenceChanged(preference)
             {
                 if (preference !== "view/only_show_top_layers" && preference !== "view/top_layer_count" && ! preference.match("layerview/"))
                 {
@@ -221,9 +221,9 @@ Item
         Connections
         {
             target: UM.SimulationView
-            onMaxLayersChanged: layerSlider.setUpperValue(UM.SimulationView.currentLayer)
-            onMinimumLayerChanged: layerSlider.setLowerValue(UM.SimulationView.minimumLayer)
-            onCurrentLayerChanged:
+            function onMaxLayersChanged() { layerSlider.setUpperValue(UM.SimulationView.currentLayer) }
+            function onMinimumLayerChanged() { layerSlider.setLowerValue(UM.SimulationView.minimumLayer) }
+            function onCurrentLayerChanged()
             {
                 // Only pause the simulation when the layer was changed manually, not when the simulation is running
                 if (layerSlider.manuallyChanged)

plugins/SimulationView/SimulationViewMenuComponent.qml

@@ -22,7 +22,7 @@ Cura.ExpandableComponent
     Connections
     {
         target: UM.Preferences
-        onPreferenceChanged:
+        function onPreferenceChanged(preference)
         {
             if (preference !== "view/only_show_top_layers" && preference !== "view/top_layer_count" && ! preference.match("layerview/"))
             {

plugins/Toolbox/resources/qml/Toolbox.qml

@@ -100,8 +100,8 @@ Window
         Connections
         {
             target: toolbox
-            onShowLicenseDialog: { licenseDialog.show() }
-            onCloseLicenseDialog: { licenseDialog.close() }
+            function onShowLicenseDialog() { licenseDialog.show() }
+            function onCloseLicenseDialog() { licenseDialog.close() }
         }
         
         ToolboxLicenseDialog

plugins/Toolbox/resources/qml/components/ToolboxDetailTileActions.qml

@@ -112,8 +112,8 @@ Column
     Connections
     {
         target: toolbox
-        onInstallChanged: installed = toolbox.isInstalled(model.id)
-        onFilterChanged:
+        function onInstallChanged() { installed = toolbox.isInstalled(model.id) }
+        function onFilterChanged()
         {
             installed = toolbox.isInstalled(model.id)
         }

plugins/Toolbox/resources/qml/components/ToolboxInstalledTile.qml

@@ -117,7 +117,7 @@ Item
         Connections
         {
             target: toolbox
-            onEnabledChanged: isEnabled = toolbox.isEnabled(model.id)
+            function onToolboxEnabledChanged() { isEnabled = toolbox.isEnabled(model.id) }
         }
     }
 }