Browse Source

Fix checking for cases where machine creation failed

ChrisTerBeke 5 years ago
parent
commit
5f31551184

+ 3 - 0
plugins/UM3NetworkPrinting/src/Cloud/CloudOutputDeviceManager.py

@@ -144,6 +144,9 @@ class CloudOutputDeviceManager:
         # Create a new machine and activate it.
         # We do not use use MachineManager.addMachine here because we need to set the cluster ID before activating it.
         new_machine = CuraStackBuilder.createMachine(device.name, device.printerType)
+        if not new_machine:
+            Logger.log("e", "Failed creating a new machine")
+            return
         new_machine.setMetaDataEntry(self.META_CLUSTER_ID, device.key)
         CuraApplication.getInstance().getMachineManager().setActiveMachine(new_machine.getId())
         self._connectToOutputDevice(device, new_machine)

+ 3 - 0
plugins/UM3NetworkPrinting/src/Network/LocalClusterOutputDeviceManager.py

@@ -208,6 +208,9 @@ class LocalClusterOutputDeviceManager:
         # We do not use use MachineManager.addMachine here because we need to set the network key before activating it.
         # If we do not do this the auto-pairing with the cloud-equivalent device will not work.
         new_machine = CuraStackBuilder.createMachine(device.name, device.printerType)
+        if not new_machine:
+            Logger.log("e", "Failed creating a new machine")
+            return
         new_machine.setMetaDataEntry(self.META_NETWORK_KEY, device.key)
         CuraApplication.getInstance().getMachineManager().setActiveMachine(new_machine.getId())
         self._connectToOutputDevice(device, new_machine)