Remove deprecated uses of zigpy objects (#2262)

* Remove deprecated `is_reply` key * Remove deprecated use of attribute definition
zigpy · Mar 8, 2023 · c9c9e37 · c9c9e37
1 parent a39deae
commit c9c9e37
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 10 deletions.
diff --git a/zhaquirks/adeo/color_controller.py b/zhaquirks/adeo/color_controller.py
@@ -71,8 +71,8 @@ class AdeoManufacturerCluster(EventableCluster):
         0x00: foundation.ZCLCommandDef(
             "preset",
             {"param1": t.uint8_t, "param2": t.uint8_t},
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
-            is_reply=False,
         )
     }
 

diff --git a/zhaquirks/inovelli/__init__.py b/zhaquirks/inovelli/__init__.py
@@ -159,7 +159,7 @@ class Inovelli_VZM31SN_Cluster(CustomCluster):
         0x00: foundation.ZCLCommandDef(
             "button_event",
             {"button_pressed": t.uint8_t, "press_type": t.uint8_t},
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
         ),
         0x01: foundation.ZCLCommandDef(
@@ -170,13 +170,13 @@ class Inovelli_VZM31SN_Cluster(CustomCluster):
                 "led_level": t.uint8_t,
                 "led_duration": t.uint8_t,
             },
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
         ),
         0x02: foundation.ZCLCommandDef(
             "reset_energy_meter",
             {},
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
         ),
         0x03: foundation.ZCLCommandDef(
@@ -188,15 +188,15 @@ class Inovelli_VZM31SN_Cluster(CustomCluster):
                 "led_level": t.uint8_t,
                 "led_duration": t.uint8_t,
             },
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
         ),
         0x24: foundation.ZCLCommandDef(
             "led_effect_complete",
             {
                 "notification_type": t.uint8_t,
             },
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
         ),
     }

diff --git a/zhaquirks/philips/rdm001.py b/zhaquirks/philips/rdm001.py
@@ -84,7 +84,7 @@ class PhilipsRemoteCluster(CustomCluster):
                 "param6": t.uint8_t,
             },
             is_manufacturer_specific=True,
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
         )
     }
     BUTTONS = {

diff --git a/zhaquirks/sengled/e1e_g7f.py b/zhaquirks/sengled/e1e_g7f.py
@@ -77,7 +77,7 @@ class SengledE1EG7FManufacturerSpecificCluster(CustomCluster):
                 "param3": t.uint8_t,
                 "param4": t.uint8_t,
             },
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
         )
     }

diff --git a/zhaquirks/siglis/zigfred.py b/zhaquirks/siglis/zigfred.py
@@ -60,7 +60,7 @@ class ZigfredCluster(CustomCluster):
         ZIGFRED_CLUSTER_COMMAND_BUTTON_EVENT: foundation.ZCLCommandDef(
             "button_event",
             {"param1": t.uint32_t},
-            is_reply=False,
+            direction=foundation.Direction.Server_to_Client,
             is_manufacturer_specific=True,
         ),
     }

diff --git a/zhaquirks/tuya/mcu/__init__.py b/zhaquirks/tuya/mcu/__init__.py
@@ -109,7 +109,7 @@ async def write_attributes(self, attributes, manufacturer=None):
             cluster_data = TuyaClusterData(
                 endpoint_id=self.endpoint.endpoint_id,
                 cluster_name=self.ep_attribute,
-                cluster_attr=self.attributes[record.attrid][0],
+                cluster_attr=self.attributes[record.attrid].name,
                 attr_value=record.value.value,
                 expect_reply=False,
                 manufacturer=manufacturer,