Fix hristo-atanasov#78 Add new option dnt_send_list

This is a setting that does not transmit the data if the On transmission
is a toggle operation of a function, and if tasmota_irhvac sends On each
time it does not result in normal operation. If you want a toggle
action, it can be sent by the service.

dnt_send_list: #optional - default []
    - Beep
    - Clean
    - Econo
    - Filter
    - Light
    - Quiet
    - Sleep
    - SwingH
    - SwingV
    - Turbo

custom_components/tasmota_irhvac/climate.py

@@ -138,6 +138,7 @@
     CONF_KEEP_MODE,
     CONF_SWINGV,
     CONF_SWINGH,
+    CONF_DNT_SEND_LIST,
     DATA_KEY,
     DOMAIN,
     DEFAULT_NAME,
@@ -170,6 +171,7 @@
     SERVICE_SLEEP_MODE,
     SERVICE_SET_SWINGV,
     SERVICE_SET_SWINGH,
+    DNT_SEND_ALL_LIST,
 )
 
 DEFAULT_MODES_LIST = [
@@ -265,6 +267,10 @@
         vol.Optional(CONF_KEEP_MODE, default=DEFAULT_CONF_KEEP_MODE): cv.boolean,
         vol.Optional(CONF_SWINGV): cv.string,
         vol.Optional(CONF_SWINGH): cv.string,
+        vol.Optional(CONF_DNT_SEND_LIST, default=[]): vol.All(
+            cv.ensure_list,
+            [vol.In(DNT_SEND_ALL_LIST)],
+        ),
     }
 )
 
@@ -465,7 +471,7 @@ def __init__(
         self._model = config[CONF_MODEL]
         self._celsius = config[CONF_CELSIUS]
         self._light = config[CONF_LIGHT].lower()
-        self._filters = config[CONF_FILTER].lower()
+        self._filter = config[CONF_FILTER].lower()
         self._clean = config[CONF_CLEAN].lower()
         self._beep = config[CONF_BEEP].lower()
         self._sleep = config[CONF_SLEEP].lower()
@@ -476,6 +482,8 @@ def __init__(
         self._swingh = config.get(CONF_SWINGH).lower() if config.get(CONF_SWINGH) is not None else None
         self._fix_swingv = None
         self._fix_swingh = None
+        self._dnt_send_list = config[CONF_DNT_SEND_LIST]
+        self._force_send_list = []
 
         availability_topic = config.get(CONF_AVAILABILITY_TOPIC)
         if (availability_topic) is None:
@@ -513,10 +521,10 @@ async def async_added_to_hass(self):
             if old_state.attributes.get(ATTR_LAST_ON_MODE) is not None:
                 self._last_on_mode = old_state.attributes.get(ATTR_LAST_ON_MODE)
 
-            for attr in ATTRIBUTES_IRHVAC:
+            for attr, prop in ATTRIBUTES_IRHVAC.items():
                 val = old_state.attributes.get(attr)
                 if val is not None:
-                    setattr(self, "_" + attr, val)
+                    setattr(self, "_" + prop, val)
             if old_state.state:
                 self._hvac_mode = old_state.state
                 self._enabled = self._hvac_mode != HVAC_MODE_OFF
@@ -603,7 +611,7 @@ async def state_message_received(msg):
                 if "Light" in payload:
                     self._light = payload["Light"].lower()
                 if "Filter" in payload:
-                    self._filters = payload["Filter"].lower()
+                    self._filter = payload["Filter"].lower()
                 if "Clean" in payload:
                     self._clean = payload["Clean"].lower()
                 if "Beep" in payload:
@@ -720,8 +728,8 @@ async def async_will_remove_from_hass(self):
     @property
     def extra_state_attributes(self):
         """Return the state attributes of the device."""
-        return {attribute: getattr(self, '_' + attribute)
-             for attribute in ATTRIBUTES_IRHVAC}
+        return {attr: getattr(self, '_' + prop)
+             for attr, prop in ATTRIBUTES_IRHVAC.items()}
 
     @property
     def should_poll(self):
@@ -922,60 +930,77 @@ async def async_set_swing_mode(self, swing_mode):
         # note: set _swingv and _swingh in send_ir() later
         if not self._hvac_mode == HVAC_MODE_OFF:
             self.power_mode = STATE_ON
+        if self._swing_mode == SWING_BOTH or self._swing_mode == SWING_OFF:
+            if SWING_VERTICAL in self._swing_list:
+                self._force_send_list.append('SwingV')
+            if SWING_HORIZONTAL in self._swing_list:
+                self._force_send_list.append('SwingH')
+        elif self._swing_mode == SWING_VERTICAL:
+            self._force_send_list.append('SwingV')
+        elif self._swing_mode == SWING_HORIZONTAL:
+            self._force_send_list.append('SwingH')
         await self.async_send_cmd()
 
     async def async_set_econo(self, econo):
         """Set new target econo mode."""
         if econo not in ON_OFF_LIST:
             return
         self._econo = econo.lower()
+        self._force_send_list.append('Econo')
         await self.async_send_cmd()
 
     async def async_set_turbo(self, turbo):
         """Set new target turbo mode."""
         if turbo not in ON_OFF_LIST:
             return
         self._turbo = turbo.lower()
+        self._force_send_list.append('Turbo')
         await self.async_send_cmd()
 
     async def async_set_quiet(self, quiet):
         """Set new target quiet mode."""
         if quiet not in ON_OFF_LIST:
             return
         self._quiet = quiet.lower()
+        self._force_send_list.append('Quiet')
         await self.async_send_cmd()
 
     async def async_set_light(self, light):
         """Set new target light mode."""
         if light not in ON_OFF_LIST:
             return
         self._light = light.lower()
+        self._force_send_list.append('Light')
         await self.async_send_cmd()
 
     async def async_set_filters(self, filters):
         """Set new target filters mode."""
         if filters not in ON_OFF_LIST:
             return
-        self._filters = filters.lower()
+        self._filter = filters.lower()
+        self._force_send_list.append('Filter')
         await self.async_send_cmd()
 
     async def async_set_clean(self, clean):
         """Set new target clean mode."""
         if clean not in ON_OFF_LIST:
             return
         self._clean = clean.lower()
+        self._force_send_list.append('Clean')
         await self.async_send_cmd()
 
     async def async_set_beep(self, beep):
         """Set new target beep mode."""
         if beep not in ON_OFF_LIST:
             return
         self._beep = beep.lower()
+        self._force_send_list.append('Beep')
         await self.async_send_cmd()
 
     async def async_set_sleep(self, sleep):
         """Set new target sleep mode."""
         self._sleep = sleep.lower()
+        self._force_send_list.append('Sleep')
         await self.async_send_cmd()
 
     async def async_set_swingv(self, swingv):
@@ -995,6 +1020,7 @@ async def async_set_swingv(self, swingv):
             else:
                 if SWING_VERTICAL in self._swing_list:
                     self._swing_mode = SWING_VERTICAL
+        self._force_send_list.append('SwingV')
         await self.async_send_cmd()
 
     async def async_set_swingh(self, swingh):
@@ -1014,6 +1040,7 @@ async def async_set_swingh(self, swingh):
             else:
                 if SWING_HORIZONTAL in self._swing_list:
                     self._swing_mode = SWING_HORIZONTAL
+        self._force_send_list.append('SwingH')
         await self.async_send_cmd()
 
     async def _async_power_sensor_changed(self, entity_id, old_state, new_state):
@@ -1167,19 +1194,15 @@ async def send_ir(self):
             "Celsius": self._celsius,
             "Temp": self._target_temp,
             "FanSpeed": fan_speed,
-            "SwingV": self._swingv,
-            "SwingH": self._swingh,
-            "Quiet": self._quiet,
-            "Turbo": self._turbo,
-            "Econo": self._econo,
-            "Light": self._light,
-            "Filter": self._filters,
-            "Clean": self._clean,
-            "Beep": self._beep,
-            "Sleep": self._sleep,
             "Clock": int(_min),
             "Weekday": int(_dt.weekday()),
         }
+        for key in DNT_SEND_ALL_LIST:
+            if key in self._force_send_list or key not in self._dnt_send_list:
+                payload_data[key] = getattr(self, '_' + key.lower())
+
+        self._force_send_list = []
+
         payload = (json.dumps(payload_data))
         # Publish mqtt message
         await mqtt.async_publish(self.hass, self.topic, payload)

custom_components/tasmota_irhvac/const.py

@@ -90,6 +90,7 @@
 CONF_KEEP_MODE = "keep_mode_when_off"
 CONF_SWINGV = "default_swingv"
 CONF_SWINGH = "default_swingh"
+CONF_DNT_SEND_LIST = "dnt_send_list"
 
 # Platform specific default values
 DEFAULT_NAME = "IR AirConditioner"
@@ -150,7 +151,7 @@
     ATTR_TURBO: 'turbo',
     ATTR_QUIET: 'quiet',
     ATTR_LIGHT: 'light',
-    ATTR_FILTERS: 'filters',
+    ATTR_FILTERS: 'filter',
     ATTR_CLEAN: 'clean',
     ATTR_BEEP: 'beep',
     ATTR_SLEEP: 'sleep',
@@ -169,3 +170,16 @@
     'on',
     'off'
 ]
+
+DNT_SEND_ALL_LIST = [
+    'SwingV',
+    'SwingH',
+    'Quiet',
+    'Turbo',
+    'Econo',
+    'Light',
+    'Filter',
+    'Clean',
+    'Beep',
+    'Sleep',
+]

examples/configuration.yaml

@@ -65,3 +65,18 @@ climate:
     default_swingv: "high" #optional - default "" string value
     default_swingh: "left" #optional - default "" string value 
     keep_mode_when_off: True #optional - default False boolean value : Must be True for MITSUBISHI_AC, ECOCLIM, etc.
+    dnt_send_list: #optional - default []
+      # This is a setting that does not transmit the data if the On transmission
+      # is a toggle operation of a function, and if tasmota_irhvac sends On each
+      # time it does not result in normal operation. If you want a toggle
+      # action, it can be sent by the service.
+      #  - Beep
+      #  - Clean
+      #  - Econo
+      #  - Filter
+      #  - Light
+      #  - Quiet
+      #  - Sleep
+      #  - SwingH
+      #  - SwingV
+      #  - Turbo