From 66606751ab14ce97fb7b4e1e2013888cb4b00ac1 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Sun, 25 Aug 2024 13:32:35 -1000 Subject: [PATCH] Move supports_subscribe to AbstractPairing We have a check to see if subscribe is broken but checking it depends on the pairing type. Make sure all pairings support this. related issue https://github.com/home-assistant/core/issues/124099 --- aiohomekit/controller/abstract.py | 1 + aiohomekit/controller/ip/pairing.py | 2 -- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/aiohomekit/controller/abstract.py b/aiohomekit/controller/abstract.py index 6fd48c77..b14f892d 100644 --- a/aiohomekit/controller/abstract.py +++ b/aiohomekit/controller/abstract.py @@ -83,6 +83,7 @@ def __init__( self.config_changed_listeners: set[Callable[[int], None]] = set() self._accessories_state: AccessoriesState | None = None self._shutdown = False + self.supports_subscribe = True self.id = pairing_data["AccessoryPairingID"] self._pairing_data = pairing_data diff --git a/aiohomekit/controller/ip/pairing.py b/aiohomekit/controller/ip/pairing.py index 0278dc9f..b5c9da09 100644 --- a/aiohomekit/controller/ip/pairing.py +++ b/aiohomekit/controller/ip/pairing.py @@ -86,8 +86,6 @@ def __init__( """ self.pairing_data = pairing_data self.connection = SecureHomeKitConnection(self, self.pairing_data) - self.supports_subscribe = True - super().__init__(controller, pairing_data) @property