Browse Source

CURA-5264 Don't need the if statement since the hardware-typeid is the
right value to look at and it is in the firmware api.

Diego Prado Gesto 6 years ago
parent
commit
8c4dc1cc2b
1 changed files with 2 additions and 6 deletions
  1. 2 6
      plugins/UM3NetworkPrinting/UM3OutputDevicePlugin.py

+ 2 - 6
plugins/UM3NetworkPrinting/UM3OutputDevicePlugin.py

@@ -189,14 +189,10 @@ class UM3OutputDevicePlugin(OutputDevicePlugin):
                 b"name": system_info["name"].encode("utf-8"),
                 b"address": address.encode("utf-8"),
                 b"firmware_version": system_info["firmware"].encode("utf-8"),
-                b"manual": b"true"
+                b"manual": b"true",
+                b"machine": str(system_info['hardware']["typeid"]).encode("utf-8")
             }
 
-            if "hardware" in system_info and 'typeid' in system_info["hardware"]:
-                properties[b"machine"] = str(system_info['hardware']["typeid"]).encode("utf-8")
-            else:
-                properties[b"machine"] = system_info["variant"].encode("utf-8")
-
             if has_cluster_capable_firmware:
                 # Cluster needs an additional request, before it's completed.
                 properties[b"incomplete"] = b"true"