Check if attribute setPointTemperature before use

Make use of hasattr to see if the attribute setPointTemperature really exists before using its value. This should fix auhlig#4
hanzZzZzZ · Jan 6, 2021 · a31ef18 · a31ef18
1 parent a1003fd
commit a31ef18
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/exporter.py b/exporter.py
@@ -167,14 +167,14 @@ def __collect_thermostat_metrics(self, room, device):
         if device.actualTemperature:
             self.metric_temperature_actual.labels(room=room, device_label=device.label).set(device.actualTemperature)
 
-        if device.setPointTemperature:
+        if hasattr(device, 'setPointTemperature'):
             self.metric_temperature_setpoint.labels(room=room, device_label=device.label).set(device.setPointTemperature)
 
         if device.humidity:
             self.metric_humidity_actual.labels(room=room, device_label=device.label).set(device.humidity)
         logging.info(
-            "room: {}, label: {}, temperature_actual: {}, temperature_setpoint: {}, humidity_actual: {}"
-            .format(room, device.label, device.actualTemperature, device.setPointTemperature, device.humidity)
+            "found device: room: {}, label: {}, temperature_actual: {}, temperature_setpoint: {}, humidity_actual: {}"
+            .format(room, device.label, device.actualTemperature, device.setPointTemperature if hasattr(device, 'setPointTemperature') else "n/a", device.humidity)
         )
 
     def __collect_heating_metrics(self, room, device):