Use correct none checking (hopefully)

This commit is contained in:
ChrisTerBeke 2019-08-02 15:59:55 +02:00
parent a2345d3004
commit 66dbbf7fb6

View File

@ -196,12 +196,11 @@ class UltimakerNetworkedPrinterOutputDevice(NetworkedPrinterOutputDevice):
# Check which printers need to be created or updated. # Check which printers need to be created or updated.
for index, printer_data in enumerate(remote_printers): for index, printer_data in enumerate(remote_printers):
printer = next(iter(printer for printer in self._printers if printer.key == printer_data.uuid), None) printer = next(iter(printer for printer in self._printers if printer.key == printer_data.uuid), None)
if printer: if printer is None:
printer_data.updateOutputModel(printer)
new_printers.append(printer)
else:
printer = printer_data.createOutputModel(ClusterOutputController(self)) printer = printer_data.createOutputModel(ClusterOutputController(self))
new_printers.append(printer) else:
printer_data.updateOutputModel(printer)
new_printers.append(printer)
# Check which printers need to be removed (de-referenced). # Check which printers need to be removed (de-referenced).
remote_printers_keys = [printer_data.uuid for printer_data in remote_printers] remote_printers_keys = [printer_data.uuid for printer_data in remote_printers]