From 62182ea460e3d4f334805b0cf63336a442afbf3a Mon Sep 17 00:00:00 2001 From: Joakim Plate Date: Sun, 11 Apr 2021 08:42:32 +0200 Subject: [PATCH] Bump ha-philipsjs to 2.7.0 (#49008) This has some improvements to not consider the TV off due to some exceptions that is related to API being buggy rather than off. --- homeassistant/components/philips_js/__init__.py | 6 +++++- homeassistant/components/philips_js/manifest.json | 2 +- requirements_all.txt | 2 +- requirements_test_all.txt | 2 +- 4 files changed, 8 insertions(+), 4 deletions(-) diff --git a/homeassistant/components/philips_js/__init__.py b/homeassistant/components/philips_js/__init__.py index 7be5efeaf2f..b585451cdb0 100644 --- a/homeassistant/components/philips_js/__init__.py +++ b/homeassistant/components/philips_js/__init__.py @@ -134,8 +134,12 @@ class PhilipsTVDataUpdateCoordinator(DataUpdateCoordinator[None]): async def _notify_task(self): while self.api.on and self.api.notify_change_supported: - if await self.api.notifyChange(130): + res = await self.api.notifyChange(130) + if res: self.async_set_updated_data(None) + elif res is None: + LOGGER.debug("Aborting notify due to unexpected return") + break @callback def _async_notify_stop(self): diff --git a/homeassistant/components/philips_js/manifest.json b/homeassistant/components/philips_js/manifest.json index ad591ad330b..36e01d8f3c8 100644 --- a/homeassistant/components/philips_js/manifest.json +++ b/homeassistant/components/philips_js/manifest.json @@ -3,7 +3,7 @@ "name": "Philips TV", "documentation": "https://www.home-assistant.io/integrations/philips_js", "requirements": [ - "ha-philipsjs==2.3.2" + "ha-philipsjs==2.7.0" ], "codeowners": [ "@elupus" diff --git a/requirements_all.txt b/requirements_all.txt index 65e72b42535..63935c4b0ae 100644 --- a/requirements_all.txt +++ b/requirements_all.txt @@ -720,7 +720,7 @@ guppy3==3.1.0 ha-ffmpeg==3.0.2 # homeassistant.components.philips_js -ha-philipsjs==2.3.2 +ha-philipsjs==2.7.0 # homeassistant.components.habitica habitipy==0.2.0 diff --git a/requirements_test_all.txt b/requirements_test_all.txt index c8868b00b0a..662398d0259 100644 --- a/requirements_test_all.txt +++ b/requirements_test_all.txt @@ -393,7 +393,7 @@ guppy3==3.1.0 ha-ffmpeg==3.0.2 # homeassistant.components.philips_js -ha-philipsjs==2.3.2 +ha-philipsjs==2.7.0 # homeassistant.components.habitica habitipy==0.2.0