v2.13.2 bugfix

Author: HAEdwin

custom_components/apsystems_ecu_reader/binary_sensor.py

@@ -1,7 +1,6 @@
 """binary_sensor.py"""
 
 import logging
-from homeassistant.util import dt as dt_util
 
 from homeassistant.components.binary_sensor import BinarySensorEntity
 from homeassistant.helpers.entity import EntityCategory
@@ -64,8 +63,7 @@ def extra_state_attributes(self):
         """Return the state attributes of the sensor."""
         return {
             "timezone": self._ecu.ecu.timezone,
-            "Last data update": self._ecu.ecu.last_update,
-            "Last UI update": dt_util.now().isoformat(),
+            "last_data_update": self._ecu.ecu.last_update,
         }
 
     @property

custom_components/apsystems_ecu_reader/ecu_api.py

@@ -38,6 +38,8 @@ def __init__(self, ipaddr, raw_ecu=None, raw_inverter=None, timeout=5):
         self.timeout = timeout
         # how big of a buffer to read at a time from the socket
         self.recv_size = DEFAULT_RECV_SIZE
+        # buffer used to store the latest socket read
+        self.read_buffer = b""
 
         self.ecu_cmd = "APS1100160001END\n"
         self.inverter_query_prefix = "APS1100280002"
@@ -321,7 +323,7 @@ def process_inverter_data(self, show_graphs, data=None):
 
                 while cnt1 < inverter_qty:
                     inv = {}
-                    if aps_str(data, 15, 2) == "01": # 01 = Not replaced inverter
+                    if aps_str(data, 15, 2) == "01":  # 01 = Not replaced inverter
                         inverter_uid = aps_uid(data, cnt2)
                         inv["uid"] = inverter_uid
                         inv["online"] = bool(aps_int_from_bytes(data, cnt2 + 6, 1))

custom_components/apsystems_ecu_reader/manifest.json

@@ -10,5 +10,5 @@
   "issue_tracker": "https://github.com/haedwin/homeassistant-apsystems_ecu_reader/issues",
   "loggers": ["custom_components.apsystems_ecu_reader"],
   "requirements": [],
-  "version": "2.13.1"
+  "version": "2.13.2"
 }

custom_components/apsystems_ecu_reader/sensor.py

@@ -1,7 +1,6 @@
 """sensor.py"""
 
 import logging
-from datetime import datetime
 
 from homeassistant.components.sensor import SensorEntity
 from homeassistant.components.binary_sensor import BinarySensorEntity
@@ -27,7 +26,6 @@
 
 
 from homeassistant.core import callback
-from homeassistant.util import dt as dt_util
 
 from .const import (
     DOMAIN,
@@ -403,9 +401,8 @@ def is_on(self):
     def extra_state_attributes(self):
         """Return the extra state attributes."""
         return {
-            "Timezone": self._ecu.ecu.timezone,
-            "Last data update": self._ecu.ecu.last_update,
-            "Last UI update": dt_util.now().isoformat(),
+            "timezone": self._ecu.ecu.timezone,
+            "last_data_update": self._ecu.ecu.last_update,
         }
 
     @callback
@@ -501,12 +498,10 @@ def native_unit_of_measurement(self):
 
     @property
     def extra_state_attributes(self):
-        attrs = {
-            "Timezone": self._ecu.ecu.timezone,
-            "Last data update": self._ecu.ecu.last_update,
-            "Last UI update": dt_util.now().isoformat(),
+        return {
+            "timezone": self._ecu.ecu.timezone,
+            "last_data_update": self._ecu.ecu.last_update,
         }
-        return attrs
 
     @property
     def state_class(self):
@@ -605,9 +600,8 @@ def native_unit_of_measurement(self):
     @property
     def extra_state_attributes(self):
         return {
-            "Timezone": self._ecu.ecu.timezone,
-            "Last data update": self._ecu.ecu.last_update,
-            "Last UI update": dt_util.now().isoformat(),
+            "timezone": self._ecu.ecu.timezone,
+            "last_data_update": self._ecu.ecu.last_update,
         }
 
     @property
@@ -687,10 +681,9 @@ def icon(self):
     def extra_state_attributes(self):
         """Return the state attributes of the entity."""
         return {
-            "IP-Address": self._ecu.ipaddr,
-            "Timezone": self._ecu.ecu.timezone,
-            "Last data update": self._ecu.ecu.last_update,
-            "Last UI update": dt_util.now().isoformat(),
+            "ip_address": self._ecu.ipaddr,
+            "timezone": self._ecu.ecu.timezone,
+            "last_data_update": self._ecu.ecu.last_update,
         }
 
     @property