- Better error handling on ha_remove ..

git-svn-id: https://redmine.imt-atlantique.fr/svn/xaal/code/Python/branches/0.7@3042 b32b6428-25c9-4566-ad07-03861ab6144f

apps/homeassistant/xaal/bridge.py

@@ -300,10 +300,13 @@ class Bridge(object):
     def remove_entities(self, addr: bindings.UUID) -> None:
         """remove entities for a given xAAL address"""
         _LOGGER.debug(f"Removing entities: {addr}")
-        self._entities.pop(addr)
-        # if device not already auto-washed remove it
-        if self._mon.devices.get_with_addr(addr):
+        try:
+            self._entities.pop(addr)
             self._mon.devices.remove(addr)
+        except KeyError:
+            # device already auto-washed or
+            # and old entity
+            _LOGGER.warn(f"Unknow entity w/ addr {addr}")
 
     def get_entities(self, addr: bindings.UUID) -> list[XAALEntity] | None:
         """ return entities for a given xAAL address"""

apps/homeassistant/xaal/light.py

@@ -60,7 +60,6 @@ class Lamp(XAALEntity, LightEntity):
         return self.get_attribute('light')
 
     def turn_on(self, **kwargs) -> None:
-        _LOGGER.warn(kwargs)
         color = kwargs.get(ATTR_HS_COLOR, None)
         brightness = kwargs.get(ATTR_BRIGHTNESS, None)
         color_temp = kwargs.get(ATTR_COLOR_TEMP_KELVIN, None)

apps/homeassistant/xaal/sensor.py

@@ -18,11 +18,11 @@ _LOGGER = logging.getLogger(__name__)
 async def async_setup_entry(hass: HomeAssistant,
                             config_entry: ConfigEntry,
                             async_add_entities: AddEntitiesCallback) -> None:
-    binding = {'thermometer.'     : [Thermometer ],
+    binding = {'thermometer.'     : [Thermometer],
                'hygrometer.'      : [Hygrometer],
                'barometer.'       : [Barometer],
                'battery.'         : [Battery],
-               'powermeter.full'  : [PowerMeter,CurrentMeter, VoltMeter],
+               'powermeter.full'  : [PowerMeter, CurrentMeter, VoltMeter],
                'powermeter.'      : [PowerMeter],
                'wifimeter.'       : [WifiMeter],
                'luxmeter.'        : [LuxMeter],
@@ -63,6 +63,7 @@ class Barometer(XAALSensorEntity):
 
 
 class Battery(XAALSensorEntity):
+    _attr_state_class = None
     _attr_device_class = SensorDeviceClass.BATTERY
     _attr_unit_of_measurement = const.PERCENTAGE
     _xaal_attribute = 'level'