fix binary sensors

add new events, fix binsensor
fix mqtt
2025-12-12 01:24:29 +05:00 · 2021-01-28 10:29:16 +03:00 · 2021-01-27 21:40:55 +03:00 · 2021-01-26 08:34:27 +03:00 · 2021-01-25 21:35:06 +03:00 · 2021-01-25 20:17:15 +03:00
15 changed files with 379 additions and 150 deletions
--- a/.experiment.py
+++ b/.experiment.py
@@ -0,0 +1,32 @@
 import asyncio
 async def handle_echo(reader: asyncio.StreamReader, writer: asyncio.StreamWriter):
    data = await reader.read(100)
    message = data.decode()
    addr = writer.get_extra_info('peername')
    print(f"Received {message!r} from {addr!r}")
    print(f"Send: {message!r}")
    ans = '''HTTP/1.1 200 OK\nContent-Length: 6\n\nhello\n'''.encode()
    writer.write(ans)
    await writer.drain()
    print("Close the connection")
    writer.transport.close()
    writer.close()
    await writer.wait_closed()
 async def main():
    server = await asyncio.start_server(
        handle_echo, '127.0.0.1', 8888)
    addr = server.sockets[0].getsockname()
    print(f'Serving on {addr}')
    async with server:
        await server.serve_forever()
 asyncio.run(main())
--- a/.github/ISSUE_TEMPLATE/bug-report.md
+++ b/.github/ISSUE_TEMPLATE/bug-report.md
@@ -0,0 +1,26 @@
 ---
 name: Bug report
 about: Create a report to help us improve
 title: ''
 labels: ''
 assignees: ''
 ---
 **Описание**
 A clear and concise description of what the bug is.
 **Версии систем**
 Enviroment: raspberry/linux/windows/macos/docker
 HA version: 
 mega_hacs version:
 megad firmware version:
 **Ожидаемое поведение**
 A clear and concise description of what you expected to happen.
 **Screenshots**
 If applicable, add screenshots to help explain your problem.
 **LOG**
 Прочитайте в документации как включить подробный лог интеграции и приложите его здесь
--- a/custom_components/mega/.experiment.py
+++ b/custom_components/mega/.experiment.py
@@ -1,36 +0,0 @@
 import asyncio
 from bs4 import BeautifulSoup
 import aiohttp
 host = '192.168.88.14/sec'
 # page = '''
 # <html><head><style>input,select{margin:1px}</style></head><body><a href="/sec/?cf=3">Back</a><br>P7/ON<br><a href="/sec/?pt=7&amp;cmd=7:1">ON</a> <a href="/sec/?pt=7&amp;cmd=7:0">OFF</a><br><form action="/sec/"><input type="hidden" name="pn" value="7">Type <select name="pty"><option value="255">NC</option><option value="0">In</option><option value="1" selected="">Out</option><option value="3">DSen</option><option value="4">I2C</option></select><br>Default: <select name="d"><option value="0" selected="">0</option><option value="1">1</option></select><br>Mode <select name="m"><option value="0" selected="">SW</option><option value="3">SW LINK</option><option value="2">DS2413</option></select><br>Group <input name="grp" size="2" value=""><br><input type="submit" value="Save"></form></body></html>
 # <head><style>input,select{margin:1px}</style></head>
 # <body><a href="/sec/?cf=3">Back</a><br>P7/ON<br><a href="/sec/?pt=7&amp;cmd=7:1">ON</a> <a href="/sec/?pt=7&amp;cmd=7:0">OFF</a><br><form action="/sec/"><input type="hidden" name="pn" value="7">Type <select name="pty"><option value="255">NC</option><option value="0">In</option><option value="1" selected="">Out</option><option value="3">DSen</option><option value="4">I2C</option></select><br>Default: <select name="d"><option value="0" selected="">0</option><option value="1">1</option></select><br>Mode <select name="m"><option value="0" selected="">SW</option><option value="3">SW LINK</option><option value="2">DS2413</option></select><br>Group <input name="grp" size="2" value=""><br><input type="submit" value="Save"></form></body>
 # <a href="/sec/?cf=3">Back</a>
 # <br>
 # P7/ON
 # <br>
 # <a href="/sec/?pt=7&amp;cmd=7:1">ON</a>
 # <a href="/sec/?pt=7&amp;cmd=7:0">OFF</a>
 # <br>
 # <form action="/sec/"><input type="hidden" name="pn" value="7">Type <select name="pty"><option value="255">NC</option><option value="0">In</option><option value="1" selected="">Out</option><option value="3">DSen</option><option value="4">I2C</option></select><br>Default: <select name="d"><option value="0" selected="">0</option><option value="1">1</option></select><br>Mode <select name="m"><option value="0" selected="">SW</option><option value="3">SW LINK</option><option value="2">DS2413</option></select><br>Group <input name="grp" size="2" value=""><br><input type="submit" value="Save"></form>
 # <body><a href="/sec/?cf=3">Back</a><br>P7/ON<br><a href="/sec/?pt=7&amp;cmd=7:1">ON</a> <a href="/sec/?pt=7&amp;cmd=7:0">OFF</a><br><form action="/sec/"><input type="hidden" name="pn" value="7">Type <select name="pty"><option value="255">NC</option><option value="0">In</option><option value="1" selected="">Out</option><option value="3">DSen</option><option value="4">I2C</option></select><br>Default: <select name="d"><option value="0" selected="">0</option><option value="1">1</option></select><br>Mode <select name="m"><option value="0" selected="">SW</option><option value="3">SW LINK</option><option value="2">DS2413</option></select><br>Group <input name="grp" size="2" value=""><br><input type="submit" value="Save"></form></body>
 # <html><head><style>input,select{margin:1px}</style></head><body><a href="/sec/?cf=3">Back</a><br>P7/ON<br><a href="/sec/?pt=7&amp;cmd=7:1">ON</a> <a href="/sec/?pt=7&amp;cmd=7:0">OFF</a><br><form action="/sec/"><input type="hidden" name="pn" value="7">Type <select name="pty"><option value="255">NC</option><option value="0">In</option><option value="1" selected="">Out</option><option value="3">DSen</option><option value="4">I2C</option></select><br>Default: <select name="d"><option value="0" selected="">0</option><option value="1">1</option></select><br>Mode <select name="m"><option value="0" selected="">SW</option><option value="3">SW LINK</option><option value="2">DS2413</option></select><br>Group <input name="grp" size="2" value=""><br><input type="submit" value="Save"></form></body></html>
 # '''
 # tree = BeautifulSoup(page, features="lxml")
 # pty = tree.find('select', attrs={'name': 'pty'}).find(selected=True)['value']
 # m = tree.find('select', attrs={'name': 'm'})
 # if m:
 #     m = m.find(selected=True)['value']
 #
 # print(pty, m)
 if __name__ == '__main__':
    asyncio.get_event_loop().run_until_complete(
        scan_port(0)
    )
--- a/custom_components/mega/init.py
+++ b/custom_components/mega/init.py
@@ -16,7 +16,7 @@ from homeassistant.helpers import config_validation as cv
 from homeassistant.components import mqtt
 from homeassistant.config_entries import ConfigEntry
 from .const import DOMAIN, CONF_INVERT, CONF_RELOAD, PLATFORMS, CONF_PORTS, CONF_CUSTOM, CONF_SKIP, CONF_PORT_TO_SCAN, \
-    CONF_MQTT_INPUTS, CONF_HTTP, CONF_RESPONSE_TEMPLATE, CONF_ACTION, CONF_GET_VALUE
+    CONF_MQTT_INPUTS, CONF_HTTP, CONF_RESPONSE_TEMPLATE, CONF_ACTION, CONF_GET_VALUE, CONF_ALLOW_HOSTS
 from .hub import MegaD
 from .config_flow import ConfigFlow
 from .http import MegaView
@@ -26,6 +26,7 @@ _LOGGER = logging.getLogger(__name__)
 CONFIG_SCHEMA = vol.Schema(
    {
        DOMAIN: {
            vol.Optional(CONF_ALLOW_HOSTS): [str],
            vol.Required(str, description='id меги из веб-интерфейса'): {
                vol.Optional(int, description='номер порта'): {
                    vol.Optional(CONF_SKIP, description='исключить порт из сканирования', default=False): bool,
@@ -62,6 +63,7 @@ async def async_setup(hass: HomeAssistant, config: dict):
    """YAML-конфигурация содержит только кастомизации портов"""
    hass.data[DOMAIN] = {CONF_CUSTOM: config.get(DOMAIN, {})}
    hass.data[DOMAIN][CONF_HTTP] = view = MegaView(cfg=config.get(DOMAIN, {}))
    view.allowed_hosts |= set(config.get(DOMAIN, {}).get(CONF_ALLOW_HOSTS, []))
    hass.http.register_view(view)
    hass.services.async_register(
        DOMAIN, 'save', partial(_save_service, hass), schema=vol.Schema({
--- a/custom_components/mega/binary_sensor.py
+++ b/custom_components/mega/binary_sensor.py
@@ -78,7 +78,10 @@ class MegaBinarySensor(BinarySensorEntity, MegaPushEntity):
        if val is None and self._state is not None:
            return self._state == 'ON'
        elif val is not None:
-            return val == 'ON' or val == 1
+            if val in ['ON', 'OFF']:
                return val == 'ON'
            else:
                return val != 1
    def _update(self, payload: dict):
        self.mega.values[self.port] = payload
--- a/custom_components/mega/config_flow.py
+++ b/custom_components/mega/config_flow.py
@@ -32,8 +32,8 @@ STEP_USER_DATA_SCHEMA = vol.Schema(
 async def get_hub(hass: HomeAssistant, data):
    _mqtt = hass.data.get(mqtt.DOMAIN)
-    if not isinstance(_mqtt, mqtt.MQTT):
+    # if not isinstance(_mqtt, mqtt.MQTT):
-        raise exceptions.MqttNotConfigured("mqtt must be configured first")
+    #     raise exceptions.MqttNotConfigured("mqtt must be configured first")
    hub = MegaD(hass, **data, lg=_LOGGER, mqtt=_mqtt, loop=asyncio.get_event_loop())
    hub.mqtt_id = await hub.get_mqtt_id()
    if not await hub.authenticate():
@@ -135,7 +135,7 @@ class OptionsFlowHandler(config_entries.OptionsFlow):
                vol.Optional(CONF_MQTT_INPUTS, default=e.get(CONF_MQTT_INPUTS, True)): bool,
                vol.Optional(CONF_NPORTS, default=e.get(CONF_NPORTS, 37)): int,
                vol.Optional(CONF_RELOAD, default=False): bool,
-                # vol.Optional(CONF_UPDATE_ALL, default=e.get(CONF_UPDATE_ALL, True)): bool,
+                vol.Optional(CONF_UPDATE_ALL, default=e.get(CONF_UPDATE_ALL, True)): bool,
                # vol.Optional(CONF_INVERT, default=''): str,
            }),
        )
--- a/custom_components/mega/const.py
+++ b/custom_components/mega/const.py
@@ -23,6 +23,7 @@ CONF_RESPONSE_TEMPLATE = 'response_template'
 CONF_ACTION = 'action'
 CONF_UPDATE_ALL = 'update_all'
 CONF_GET_VALUE = 'get_value'
 CONF_ALLOW_HOSTS = 'allow_hosts'
 PLATFORMS = [
    "light",
    "switch",
@@ -30,4 +31,13 @@ PLATFORMS = [
    "sensor",
 ]
 EVENT_BINARY_SENSOR = f'{DOMAIN}.sensor'
-PATT_SPLIT = re.compile('[;/]')
+EVENT_BINARY = f'{DOMAIN}.binary'
 PATT_SPLIT = re.compile('[;/]')
 LONG = 'long'
 RELEASE = 'release'
 LONG_RELEASE = 'long_release'
 PRESS = 'press'
 SINGLE_CLICK = 'single'
 DOUBLE_CLICK = 'double'
--- a/custom_components/mega/entities.py
+++ b/custom_components/mega/entities.py
@@ -1,12 +1,25 @@
 import logging
 import asyncio
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.const import CONF_NAME
 from homeassistant.core import State
 from homeassistant.helpers.update_coordinator import CoordinatorEntity
 from homeassistant.helpers.restore_state import RestoreEntity
 from .hub import MegaD
-from .const import DOMAIN, CONF_CUSTOM, CONF_INVERT
+from .const import DOMAIN, CONF_CUSTOM, CONF_INVERT, EVENT_BINARY_SENSOR, LONG, \
    LONG_RELEASE, RELEASE, PRESS, SINGLE_CLICK, DOUBLE_CLICK, EVENT_BINARY
 _events_on = False
 _LOGGER = logging.getLogger(__name__)
 async def _set_events_on():
    global _events_on
    await asyncio.sleep(10)
    _LOGGER.debug('events on')
    _events_on = True
 _task_set_ev_on = None
 class BaseMegaEntity(CoordinatorEntity, RestoreEntity):
@@ -59,7 +72,7 @@ class BaseMegaEntity(CoordinatorEntity, RestoreEntity):
            "config_entries": [
                self.config_entry,
            ],
-            "name": f'port {self.port}',
+            "name": f'{self._mega_id} port {self.port}',
            "manufacturer": 'ab-log.ru',
            # "model": self.light.productname,
            # "sw_version": self.light.swversion,
@@ -88,8 +101,11 @@ class BaseMegaEntity(CoordinatorEntity, RestoreEntity):
        return self._unique_id
    async def async_added_to_hass(self) -> None:
        global _task_set_ev_on
        await super().async_added_to_hass()
        self._state = await self.async_get_last_state()
        if self.mega.mqtt_inputs and _task_set_ev_on is None:
            _task_set_ev_on = asyncio.create_task(_set_events_on())
    async def get_state(self):
        if self.mega.mqtt is None:
@@ -111,7 +127,55 @@ class MegaPushEntity(BaseMegaEntity):
        self._update(value)
        self.async_write_ha_state()
        self.lg.debug(f'state after update %s', self.state)
-        self.is_first_update = False
+        if self.mega.mqtt_inputs and not _events_on:
            _LOGGER.debug('skip event because events are off')
            return
        if not self.entity_id.startswith('binary_sensor'):
            _LOGGER.debug('skip event because not a bnary sens')
            return
        ll: bool = self.mega.last_long.get(self.port, False)
        if safe_int(value.get('click', 0)) == 1:
            self.hass.bus.async_fire(
                event_type=EVENT_BINARY,
                event_data={
                    'entity_id': self.entity_id,
                    'type': SINGLE_CLICK
                }
            )
        elif safe_int(value.get('click', 0)) == 2:
            self.hass.bus.async_fire(
                event_type=EVENT_BINARY,
                event_data={
                    'entity_id': self.entity_id,
                    'type': DOUBLE_CLICK
                }
            )
        elif safe_int(value.get('m', 0)) == 2:
            self.mega.last_long[self.port] = True
            self.hass.bus.async_fire(
                event_type=EVENT_BINARY,
                event_data={
                    'entity_id': self.entity_id,
                    'type': LONG
                }
            )
        elif safe_int(value.get('m', 0)) == 1:
            self.hass.bus.async_fire(
                event_type=EVENT_BINARY,
                event_data={
                    'entity_id': self.entity_id,
                    'type': LONG_RELEASE if ll else RELEASE,
                }
            )
        elif safe_int(value.get('m', None)) == 0:
            self.hass.bus.async_fire(
                event_type=EVENT_BINARY,
                event_data={
                    'entity_id': self.entity_id,
                    'type': PRESS,
                }
            )
            self.mega.last_long[self.port] = False
        return
    def _update(self, payload: dict):
@@ -155,10 +219,12 @@ class MegaOutPort(MegaPushEntity):
    @property
    def is_on(self) -> bool:
-        val = self.mega.values.get(self.port, {}).get("value")
+        val = self.mega.values.get(self.port, {})
        if val is None and self._state is not None:
            return self._state == 'ON'
        elif val is not None:
            val = val.get("value")
            if not self.invert:
                return val == 'ON' or str(val) == '1' or (safe_int(val) is not None and safe_int(val) > 0)
            else:
@@ -186,10 +252,11 @@ class MegaOutPort(MegaPushEntity):
        self.mega.values[self.port] = {'value': cmd}
        await self.get_state()
 def safe_int(v):
    if v in ['ON', 'OFF']:
        return None
    try:
        return int(v)
-    except ValueError:
+    except (ValueError, TypeError):
        return None
--- a/custom_components/mega/http.py
+++ b/custom_components/mega/http.py
@@ -8,11 +8,11 @@ from aiohttp.web_request import Request
 from aiohttp.web_response import Response
 from homeassistant.helpers.template import Template
-from .const import EVENT_BINARY_SENSOR, CONF_HTTP, DOMAIN, CONF_CUSTOM, CONF_RESPONSE_TEMPLATE
+from .const import EVENT_BINARY_SENSOR, DOMAIN, CONF_RESPONSE_TEMPLATE
 from homeassistant.components.http import HomeAssistantView
-from homeassistant.core import callback, HomeAssistant
+from homeassistant.core import HomeAssistant
-from . import hub
+from .tools import make_ints
-
+from . import hub as h
 _LOGGER = logging.getLogger(__name__).getChild('http')
@@ -26,8 +26,7 @@ class MegaView(HomeAssistantView):
    def __init__(self, cfg: dict):
        self._try = 0
        self.allowed_hosts = {'::1'}
-        self.callbacks: typing.DefaultDict[int, typing.List[typing.Callable[[dict], typing.Coroutine]]] \
+        self.callbacks = defaultdict(lambda: defaultdict(list))
            = defaultdict(list)
        self.templates: typing.Dict[str, typing.Dict[str, Template]] = {
            mid: {
                pt: cfg[mid][pt][CONF_RESPONSE_TEMPLATE]
@@ -38,6 +37,7 @@ class MegaView(HomeAssistantView):
        _LOGGER.debug('templates: %s', self.templates)
    async def get(self, request: Request) -> Response:
        auth = False
        for x in self.allowed_hosts:
            if request.remote.startswith(x):
@@ -48,7 +48,7 @@ class MegaView(HomeAssistantView):
            return Response(status=401)
        hass: HomeAssistant = request.app['hass']
-        hub: 'hub.MegaD' = hass.data.get(DOMAIN).get(request.remote)  # TODO: проверить какой remote
+        hub: 'h.MegaD' = hass.data.get(DOMAIN).get(request.remote)  # TODO: проверить какой remote
        if hub is None and request.remote == '::1':
            hub = hass.data.get(DOMAIN).get('__def')
        if hub is None:
@@ -62,9 +62,11 @@ class MegaView(HomeAssistantView):
        make_ints(data)
        port = data.get('pt')
        data = data.copy()
        data['mega_id'] = hub.id
        ret = 'd'
        if port is not None:
-            for cb in self.callbacks[port]:
+            hub.values[port] = data
            for cb in self.callbacks[hub.id][port]:
                cb(data)
            template: Template = self.templates.get(hub.id, {}).get(port)
            if hub.update_all:
@@ -73,8 +75,7 @@ class MegaView(HomeAssistantView):
                template.hass = hass
                ret = template.async_render(data)
        _LOGGER.debug('response %s', ret)
-        ret = Response(body=ret or 'd', content_type='text/plain', headers={})
+        ret = Response(body=ret or 'd', content_type='text/plain', headers={'Server': 's', 'Date': 'n'})
        ret.headers.clear()
        return ret
    async def later_update(self, hub):
@@ -82,14 +83,3 @@ class MegaView(HomeAssistantView):
        await asyncio.sleep(1)
        await hub.updater.async_refresh()
 def make_ints(d: dict):
    for x in d:
        try:
            d[x] = float(d[x])
        except ValueError:
            pass
    if 'm' not in d:
        d['m'] = 0
    if 'click' not in d:
        d['click'] = 0
--- a/custom_components/mega/hub.py
+++ b/custom_components/mega/hub.py
@@ -14,9 +14,9 @@ from homeassistant.const import DEVICE_CLASS_TEMPERATURE, DEVICE_CLASS_HUMIDITY
 from homeassistant.core import HomeAssistant
 from homeassistant.helpers.entity import Entity
 from homeassistant.helpers.update_coordinator import DataUpdateCoordinator
-from .const import TEMP, HUM, PATT_SPLIT, DOMAIN, CONF_HTTP
+from .const import TEMP, HUM, PATT_SPLIT, DOMAIN, CONF_HTTP, EVENT_BINARY_SENSOR
-from .exceptions import CannotConnect, MqttNotConfigured
+from .exceptions import CannotConnect
-from .http import MegaView
+from .tools import make_ints
 TEMP_PATT = re.compile(r'temp:([01234567890\.]+)')
 HUM_PATT = re.compile(r'hum:([01234567890\.]+)')
@@ -60,8 +60,9 @@ class MegaD:
    ):
        """Initialize."""
        if mqtt_inputs is None or mqtt_inputs == 'None' or mqtt_inputs is False:
-            self.http = hass.data[DOMAIN][CONF_HTTP]
+            self.http = hass.data.get(DOMAIN, {}).get(CONF_HTTP)
-            self.http.allowed_hosts |= {host}
+            if not self.http is None:
                self.http.allowed_hosts |= {host}
        else:
            self.http = None
        self.update_all = update_all if update_all is not None else True
@@ -74,6 +75,7 @@ class MegaD:
        self.mqtt = mqtt
        self.id = id
        self.lck = asyncio.Lock()
        self.last_long = {}
        self._http_lck = asyncio.Lock()
        self._notif_lck = asyncio.Lock()
        self.cnd = asyncio.Condition()
@@ -123,10 +125,16 @@ class MegaD:
        async with self.lck:
            self.entities.append(ent)
-    async def get_sensors(self):
+    async def get_sensors(self, only_list=False):
        self.lg.debug(self.sensors)
        ports = []
        for x in self.sensors:
-            await self.get_port(x)
+            if only_list and x.http_cmd != 'list':
                continue
            if x.port in ports:
                continue
            await self.get_port(x.port, force_http=True, http_cmd=x.http_cmd)
            ports.append(x.port)
    @property
    def is_online(self):
@@ -157,6 +165,7 @@ class MegaD:
        self.lg.debug('poll')
        if self.mqtt is None:
            await self.get_all_ports()
            await self.get_sensors(only_list=True)
            return
        if len(self.sensors) > 0:
            await self.get_sensors()
@@ -188,7 +197,9 @@ class MegaD:
                    self.lg.warning('%s returned %s (%s)', url, req.status, await req.text())
                    return None
                else:
-                    return await req.text()
+                    ret = await req.text()
                    self.lg.debug('response %s', ret)
                    return ret
    async def save(self):
        await self.send_command(cmd='s')
@@ -209,15 +220,20 @@ class MegaD:
            ret = {'value': ret}
        return ret
-    async def get_port(self, port):
+    async def get_port(self, port, force_http=False, http_cmd='get'):
        """
        Запрос состояния порта. Состояние всегда возвращается в виде объекта, всегда сохраняется в центральное
        хранилище values
        """
        self.lg.debug(f'get port %s', port)
-        if self.mqtt is None:
+        if self.mqtt is None or force_http:
-            ret = await self.request(pt=port, cmd='get')
+            ret = await self.request(pt=port, cmd=http_cmd)
            ret = self.parse_response(ret)
            self.lg.debug('parsed: %s', ret)
            if http_cmd == 'list' and isinstance(ret, dict) and 'value' in ret:
                await asyncio.sleep(1)
                ret = await self.request(pt=port, http_cmd=http_cmd)
                ret = self.parse_response(ret)
            self.values[port] = ret
            return ret
@@ -236,7 +252,7 @@ class MegaD:
                except asyncio.TimeoutError:
                    self.lg.error(f'timeout when getting port {port}')
-    async def get_all_ports(self):
+    async def get_all_ports(self, only_out=False):
        if not self.mqtt_inputs:
            ret = await self.request(cmd='all')
            for port, x in enumerate(ret.split(';')):
@@ -275,9 +291,18 @@ class MegaD:
        value = None
        try:
            value = json.loads(msg.payload)
            if isinstance(value, dict):
                make_ints(value)
            self.values[port] = value
            for cb in self._callbacks[port]:
                cb(value)
            if isinstance(value, dict):
                value = value.copy()
                value['mega_id'] = self.id
                self.hass.bus.async_fire(
                    EVENT_BINARY_SENSOR,
                    value,
                )
        except Exception as exc:
            self.lg.warning(f'could not parse json ({msg.payload}): {exc}')
            return
@@ -292,7 +317,7 @@ class MegaD:
        if self.mqtt_inputs:
            self._callbacks[port].append(callback)
        else:
-            self.http.callbacks[port].append(callback)
+            self.http.callbacks[self.id][port].append(callback)
    async def authenticate(self) -> bool:
        """Test if we can authenticate with the host."""
@@ -346,7 +371,7 @@ class MegaD:
                return pty, m
    async def scan_ports(self, nports=37):
-        for x in range(nports+1):
+        for x in range(0, nports+1):
            ret = await self.scan_port(x)
            if ret:
                yield [x, *ret]
@@ -357,11 +382,15 @@ class MegaD:
        async for port, pty, m in self.scan_ports(nports):
            if pty == "0":
                ret['binary_sensor'][port].append({})
-            elif pty == "1" and m in ['0', '1']:
+            elif pty == "1" and (m in ['0', '1', '3'] or m is None):
                ret['light'][port].append({'dimmer': m == '1'})
            elif pty == '3':
                try:
-                    values = await self.get_port(port)
+                    http_cmd = 'get'
                    values = await self.get_port(port, force_http=True)
                    if values is None or (isinstance(values, dict) and str(values.get('value')) in ('', 'None')):
                        values = await self.get_port(port, force_http=True, http_cmd='list')
                        http_cmd = 'list'
                except asyncio.TimeoutError:
                    self.lg.warning(f'timout on port {port}')
                    continue
@@ -382,6 +411,7 @@ class MegaD:
                        unit_of_measurement=UNITS.get(key, UNITS[TEMP]),
                        device_class=CLASSES.get(key, CLASSES[TEMP]),
                        id_suffix=key,
                        http_cmd=http_cmd,
                    ))
        return ret
--- a/custom_components/mega/sensor.py
+++ b/custom_components/mega/sensor.py
@@ -101,6 +101,7 @@ class Mega1WSensor(MegaPushEntity):
            unit_of_measurement,
            device_class,
            key=None,
            http_cmd='get',
            *args,
            **kwargs
    ):
@@ -115,8 +116,8 @@ class Mega1WSensor(MegaPushEntity):
        self.key = key
        self._device_class = device_class
        self._unit_of_measurement = unit_of_measurement
-        if self.port not in self.mega.sensors:
+        self.mega.sensors.append(self)
-            self.mega.sensors.append(self.port)
+        self.http_cmd = http_cmd
    @property
    def unit_of_measurement(self):
@@ -143,16 +144,32 @@ class Mega1WSensor(MegaPushEntity):
    @property
    def state(self):
        ret = None
        if self.key:
-            ret = self.mega.values.get(self.port, {}).get('value', {}).get(self.key)
+            try:
                ret = self.mega.values.get(self.port, {})
                if isinstance(ret, dict):
                    ret = ret.get('value', {})
                    if isinstance(ret, dict):
                        ret = ret.get(self.key)
            except:
                self.lg.error(self.mega.values.get(self.port, {}).get('value', {}))
                return
        else:
            ret = self.mega.values.get(self.port, {}).get('value')
        if ret is None and self._state is not None:
            ret = self._state.state
        try:
            ret = float(ret)
            ret = str(ret)
        except:
            ret = None
        return ret
    @property
    def name(self):
        n = super().name
-        c = self.customize.get(CONF_NAME, {}).get(self.key)
+        c = self.customize.get(CONF_NAME, {})
        if isinstance(c, dict):
            c = c.get(self.key)
        return c or n
--- a/custom_components/mega/strings.json
+++ b/custom_components/mega/strings.json
@@ -37,7 +37,8 @@
          "reload": "[%key:common::config_flow::data::reload%]",
          "invert": "[%key:common::config_flow::data::invert%]",
          "mqtt_inputs": "[%key:common::config_flow::data::mqtt_inputs%]",
-          "nports": "[%key:common::config_flow::data::nports%]"
+          "nports": "[%key:common::config_flow::data::nports%]",
          "update_all": "[%key:common::config_flow::data::update_all%]"
        }
      }
    }
--- a/custom_components/mega/tools.py
+++ b/custom_components/mega/tools.py
@@ -0,0 +1,12 @@
 _params = ['m', 'click', 'cnt', 'pt']
 def make_ints(d: dict):
    for x in _params:
        try:
            d[x] = int(d.get(x, 0))
        except (ValueError, TypeError):
            pass
    if 'm' not in d:
        d['m'] = 0
    if 'click' not in d:
        d['click'] = 0
--- a/custom_components/mega/translations/en.json
+++ b/custom_components/mega/translations/en.json
@@ -21,7 +21,8 @@
                    "scan_interval": "Scan interval (sec), 0 - don't update",
                    "port_to_scan": "Port to poll aliveness (needed only if no sensors used)",
                    "nports": "Number of ports",
-                    "update_all": "Update all outs when input"
+                    "update_all": "Update all outs when input",
                    "mqtt_inputs": "Use MQTT"
                }
            }
        }
--- a/readme.md
+++ b/readme.md
@@ -14,66 +14,6 @@
  большого кол-ва команд (например в сценах). Каждая следующая команда отправляется только после получения ответа о
  выполнении предыдущей.
 ## Зависимости
 **Важно!!** Для максимальной совместимости необходимо настроить интеграцию [mqtt](https://www.home-assistant.io/integrations/mqtt/) 
 в HomeAssistant, а так же обновить ваш контроллер до последней версии, тк были важные обновления в части mqtt
 ## HTTP in
 Начиная с версии `0.3.1` интеграция стала поддерживать обратную связь без mqtt, используя http-сервер. Для этого в настройках
 интеграции необходимо снять галку с `использовать mqtt`
 В самой меге необходимо прописать настройки:
 ```yaml
 srv: "192.168.1.4:8123" # ip:port вашего HA
 script: "mega" # это api интеграции, к которому будет обращаться контроллер
 ```
 Входы будут доступны как binary_sensor, а так же в виде событий `mega.sensor`.
 События можно обрабатывать так:
 ```yaml
 - alias: some double click
  trigger:
    - platform: event
      event_type: mega.sensor
      event_data:
        pt: 1
  action:
    - service: light.toggle
      entity_id: light.some_light
 ```
 Для binary_sensor имеет смысл использовать режим P&R, для остальных режимов - лучше пользоваться событиями.
 ## Ответ на входящие события от контроллера
 Контроллер ожидает ответ от сервера, который может быть сценарием (по умолчанию интеграция отвечает `d`, что означает 
 запустить то что прописано в поле act в настройках порта).
 Поддерживаеются шаблоны HA. Это может быть использовано, например, для запоминания яркости (тк сам контроллер этого не 
 умеет). В шаблоне можно использовать параметры, которые передает контроллер (m, click, pt, value)
 Примеры:
 ```yaml
 mega:
  mega1: # id меги, который вы сами придумываете в конфиге в UI
    4: # номер порта, с которого ожидаются события
      response_template: 5:2 # простейший пример без шаблона. Каждый раз когда будет приходить сообщение на этот порт, 
                             # будем менять состояние на противоположное
    5:
      # пример с использованием шаблона, порт 1 будет выключен если он сейчас включен и включен с последней сохраненной 
      # яркостью если он сейчас выключен     
      response_template: >-
        {% if is_state('light.some_port_1', 'on') %}
        1:0
        {% else %}
        1:{{state_attr('light.some_port_1', 'brightness')}}
        {% endif %}
    6:
      # в шаблон так же передаются все параметры, которые передает контроллер (pt, cnt, m, click)
      # эти параметры можно использовать в условиях или непосредственно в шаблоне в виде {{pt}}
      response_template: >-
        {% if m==2 %}1:0{% else %}d{% endif %}
 ```
 ## Установка
 Рекомендованный способ с поддержкой обновлений - [HACS](https://hacs.xyz/docs/installation/installation):
@@ -118,6 +58,108 @@ mega:
      unit_of_measurement: "°C" # если датчик один, то просто строчкой
 ```
 ## Зависимости
 Для совместимости c mqtt необходимо настроить интеграцию [mqtt](https://www.home-assistant.io/integrations/mqtt/) 
 в HomeAssistant, а так же обновить ваш контроллер до последней версии, тк были важные обновления в части mqtt
 ## HTTP in
 Начиная с версии `0.3.1` интеграция стала поддерживать обратную связь без mqtt, используя http-сервер. Для этого в настройках
 интеграции необходимо снять галку с `использовать mqtt`
 В самой меге необходимо прописать настройки:
 ```yaml
 srv: "192.168.1.4:8123" # ip:port вашего HA
 script: "mega" # это api интеграции, к которому будет обращаться контроллер
 ```
 Входы будут доступны как binary_sensor, а так же в виде событий `mega.sensor`.
 События можно обрабатывать так:
 ```yaml
 - alias: some double click
  trigger:
    - platform: event
      event_type: mega.sensor
      event_data:
        pt: 1
        click: 2
  action:
    - service: light.toggle
      entity_id: light.some_light
 ```
 Для binary_sensor имеет смысл использовать режим P&R, для остальных режимов - лучше пользоваться событиями.
 Примеры использования binary_sensor:
 ```yaml
 - alias: обработка долгих/коротких нажатий
  trigger:
    - platform: state
      entity_id: binary_sensor.some_sensor
      to: on
      for: 1 # задержка на секунду
  action:
    - choose:
          # если кнопка все еще нажата - значит это долгое нажатие
        - conditions: "{{ is_state('binary_sensor.some_sensor', 'on')}}"
          sequence:
          - service: light.turn_on
            entity_id: light.some_light
          # если кнопка уже не нажата - значит это короткое нажатие
        - conditions: "{{ is_state('binary_sensor.some_sensor', 'off')}}"
          sequence:
          - service: light.turn_off
            entity_id: light.some_light
 ```
 ## Ответ на входящие события от контроллера
 Контроллер ожидает ответ от сервера, который может быть сценарием (по умолчанию интеграция отвечает `d`, что означает 
 запустить то что прописано в поле act в настройках порта).
 Поддерживаеются шаблоны HA. Это может быть использовано, например, для запоминания яркости (тк сам контроллер этого не 
 умеет). В шаблоне можно использовать параметры, которые передает контроллер (m, click, pt, mdid, mega_id)
 Примеры:
 ```yaml
 mega:
  mega1: # id меги, который вы сами придумываете в конфиге в UI
    4: # номер порта, с которого ожидаются события
      response_template: 5:2 # простейший пример без шаблона. Каждый раз когда будет приходить сообщение на этот порт, 
                             # будем менять состояние на противоположное
    5:
      # пример с использованием шаблона, порт 1 будет выключен если он сейчас включен и включен с последней сохраненной 
      # яркостью если он сейчас выключен     
      response_template: >-
        {% if is_state('light.some_port_1', 'on') %}
        1:0
        {% else %}
        1:{{state_attr('light.some_port_1', 'brightness')}}
        {% endif %}
    6:
      # в шаблон так же передаются все параметры, которые передает контроллер (pt, cnt, m, click)
      # эти параметры можно использовать в условиях или непосредственно в шаблоне в виде {{pt}}
      response_template: >-
        {% if m==2 %}1:0{% else %}d{% endif %}
 ```
 ## Отладка ответов
 Для отладки ответов сервера можно самим имитировать запросы контроллера, если у вас есть доступ к консоли
 HA:
 ```shell
 curl -v -X GET 'http://localhost:8123/mega?pt=5&m=1'
 ```
 Если доступа нет, нужно в файл конфигурации добавить ip компьюетра, с которого вы хотите делать запросы, например:
 ```yaml
 mega:
  allow_hosts:
    - 192.168.1.1
 ```
 И тогда можно с локальной машины делать запросы на ваш сервер HA:
 ```shell
 curl -v -X GET 'http://192.168.88.1.4:8123/mega?pt=5&m=1'
 ```
 В ответ будет приходить либо `d`, либо скрипт, который вы настроили
 ## События
 `binary_sensor` срабатывает когда цифровой выход принимает значение 'ON'. `binary_sensor` имеет смысл использовать
 только с режимом входа P&R
@@ -125,6 +167,7 @@ mega:
 При каждом срабатывании `binary_sensor` так же сообщает о событии типа `mega.sensor`.
 События можно использовать в автоматизациях, например так:
 ```yaml
 # Пример события с полями как есть прямо из меги
 - alias: some double click
  trigger:
    - platform: event
@@ -136,6 +179,37 @@ mega:
    - service: light.toggle
      entity_id: light.some_light
 ```
 События могут содержать следующие поля: 
 - mega_id: id как в конфиге HA
 - pt: номер порта
 - cnt: счетчик срабатываний
 - mdid: if как в конфиге контроллера
 - click: клик (подробнее в документации меги)
 - value: текущее значение (только для mqtt)
 - port: номер порта
 Начиная с версии 0.3.7 появилось так же событие типа mega.binary:
 ```yaml
 # Пример события с полями как есть прямо из меги
 - alias: some long click
  trigger:
    - platform: event
      event_type: mega.binary
      event_data:
        entity_id: binary_sensor.some_id
        type: long
  action:
    - service: light.toggle
      entity_id: light.some_light
 ```
 Возможные варианты поля `type`:
 - `long`: долгое нажатие
 - `release`: размыкание (с гарантией что не было долгого нажатия)
 - `long_release`: размыкание после долгого нажатия
 - `press`: замыкание
 - `single`: одинарный клик (в режиме кликов)
 - `double`: двойной клик
 Чтобы понять, какие события происходят, лучше всего воспользоваться панелью разработчика и подписаться
 на вкладке события на событие `mega.sensor`, понажимать кнопки.
Author	SHA1	Message	Date
Andrey	c36c1369aa	fix binary sensors	2021-01-28 10:29:16 +03:00
Andrey	6d3391bc45	add new events, fix binsensor	2021-01-27 21:40:55 +03:00
Andrey	a7d7738a5c	fix mqtt	2021-01-26 08:34:27 +03:00
Andrey	c0b1247b9e	smaller headers	2021-01-25 21:35:06 +03:00
Andrey	1548e8c364	fix multiple megas	2021-01-25 20:17:15 +03:00
Andrey	39c4ab0e3b	fix device name	2021-01-25 18:45:23 +03:00
Andrey	a002e48e04	fix old mega out type 0	2021-01-25 18:13:25 +03:00
Andrey	dc6bdfc8f4	fix yaml exclusion	2021-01-25 17:56:47 +03:00
Andrey	e51b50797c	fix yaml exclusion	2021-01-25 17:46:26 +03:00
Andrey	c4205c7ddc	fix sw-link	2021-01-25 17:23:33 +03:00
Andrey	6164966d0b	fix scanning	2021-01-25 17:21:55 +03:00
Andrey	ff6225a959	fix scanning	2021-01-25 17:18:42 +03:00
Andrey	57f355d479	fix scanning	2021-01-25 16:37:48 +03:00
Andrey	7d6273539e	fix scanning	2021-01-25 15:57:10 +03:00
Andrey	5da2973351	fix scanning	2021-01-25 15:52:24 +03:00
Andrey	5eadd295f1	Merge branch 'master' of https://github.com/andvikt/mega_hacs	2021-01-25 15:48:29 +03:00
Andrey	5bf432a27f	change readme	2021-01-25 15:47:55 +03:00
Andrey	d934e87ae5	force use http while scanning	2021-01-25 15:46:40 +03:00
andvikt	0b54db9c44	Update issue templates	2021-01-25 13:05:31 +03:00
Andrey	2d15b60929	add allowed hosts config	2021-01-25 12:54:15 +03:00