From 19835844bfb44278afa9683c06c1f9401a1513af Mon Sep 17 00:00:00 2001
From: joeydelarago <joeydelarago@gmail.com>
Date: Wed, 31 Aug 2022 10:50:20 +0200
Subject: [PATCH] Change listType -> componentType. Makes it a bit more clear
 that this variable decides which component should be used for the model.

CURA-9514
---
 cura/Machines/Models/MachineListModel.py              | 10 +++++-----
 resources/qml/PrinterSelector/MachineListButton.qml   |  2 +-
 resources/qml/PrinterSelector/MachineSelectorList.qml |  2 +-
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/cura/Machines/Models/MachineListModel.py b/cura/Machines/Models/MachineListModel.py
index 10bc2d13c9..a429418e09 100644
--- a/cura/Machines/Models/MachineListModel.py
+++ b/cura/Machines/Models/MachineListModel.py
@@ -25,7 +25,7 @@ class MachineListModel(ListModel):
     IsOnlineRole = Qt.ItemDataRole.UserRole + 5
     MachineCountRole = Qt.ItemDataRole.UserRole + 6
     IsAbstractMachineRole = Qt.ItemDataRole.UserRole + 7
-    ListTypeRole = Qt.ItemDataRole.UserRole + 8
+    ComponentTypeRole = Qt.ItemDataRole.UserRole + 8
 
     def __init__(self, parent=None) -> None:
         super().__init__(parent)
@@ -41,7 +41,7 @@ class MachineListModel(ListModel):
         self.addRoleName(self.IsOnlineRole, "isOnline")
         self.addRoleName(self.MachineCountRole, "machineCount")
         self.addRoleName(self.IsAbstractMachineRole, "isAbstractMachine")
-        self.addRoleName(self.ListTypeRole, "listType")
+        self.addRoleName(self.ComponentTypeRole, "componentType")
 
         self._change_timer = QTimer()
         self._change_timer.setInterval(200)
@@ -103,13 +103,13 @@ class MachineListModel(ListModel):
 
         if len(abstract_machine_stacks) > 0:
             if self._show_cloud_printers:
-                self.appendItem({"listType": "HIDE_BUTTON",
+                self.appendItem({"componentType": "HIDE_BUTTON",
                                  "isOnline": True,
                                  "isAbstractMachine": False,
                                  "machineCount": 0
                                  })
             else:
-                self.appendItem({"listType": "SHOW_BUTTON",
+                self.appendItem({"componentType": "SHOW_BUTTON",
                                  "isOnline": True,
                                  "isAbstractMachine": False,
                                  "machineCount": 0
@@ -130,7 +130,7 @@ class MachineListModel(ListModel):
             has_connection |= connection_type in container_stack.configuredConnectionTypes
 
         self.appendItem({
-                        "listType": "MACHINE",
+                        "componentType": "MACHINE",
                         "name": container_stack.getName(),
                          "id": container_stack.getId(),
                          "metadata": container_stack.getMetaData().copy(),
diff --git a/resources/qml/PrinterSelector/MachineListButton.qml b/resources/qml/PrinterSelector/MachineListButton.qml
index f492e87574..b158cb20c0 100644
--- a/resources/qml/PrinterSelector/MachineListButton.qml
+++ b/resources/qml/PrinterSelector/MachineListButton.qml
@@ -11,7 +11,7 @@ Loader {
     id: loader
     width: parent.width
     sourceComponent: {
-        switch (model.listType) {
+        switch (model.componentType) {
             case "HIDE_BUTTON":
                 hideButtonComponent
                 break;
diff --git a/resources/qml/PrinterSelector/MachineSelectorList.qml b/resources/qml/PrinterSelector/MachineSelectorList.qml
index d4a4c4a5ce..a328ae69d9 100644
--- a/resources/qml/PrinterSelector/MachineSelectorList.qml
+++ b/resources/qml/PrinterSelector/MachineSelectorList.qml
@@ -35,7 +35,7 @@ ListView
 
         onClicked: function()
         {
-            switch (model.listType) {
+            switch (model.componentType) {
                 case "HIDE_BUTTON":
                     listView.model.setShowCloudPrinters(false);
                     break;