jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-21 20:27:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix alexa fails reporting the state in specific cases (#108743)

Browse Source 
* Fix alexa fails reporting the state in specific cases

* More cases
This commit is contained in:
Jan Bouwhuis 2024-01-23 23:36:44 +01:00 committed by GitHub
parent 823f268054
commit c725238c20
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 25 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
homeassistant/components/alexa/capabilities.py
View File

@ -860,8 +860,8 @@ class AlexaInputController(AlexaCapability):
def inputs(self) -> list[dict[str, str]] | None: def inputs(self) -> list[dict[str, str]] | None:
"""Return the list of valid supported inputs.""" """Return the list of valid supported inputs."""
source_list: list[Any] = self.entity.attributes.get( source_list: list[Any] = (
media_player.ATTR_INPUT_SOURCE_LIST, [] self.entity.attributes.get(media_player.ATTR_INPUT_SOURCE_LIST) or []
) )
return AlexaInputController.get_valid_inputs(source_list) return AlexaInputController.get_valid_inputs(source_list)
@ -1196,7 +1196,7 @@ class AlexaThermostatController(AlexaCapability):
return None return None
supported_modes: list[str] = [] supported_modes: list[str] = []
hvac_modes = self.entity.attributes.get(climate.ATTR_HVAC_MODES, []) hvac_modes = self.entity.attributes.get(climate.ATTR_HVAC_MODES) or []
for mode in hvac_modes: for mode in hvac_modes:
if thermostat_mode := API_THERMOSTAT_MODES.get(mode): if thermostat_mode := API_THERMOSTAT_MODES.get(mode):
supported_modes.append(thermostat_mode) supported_modes.append(thermostat_mode)
@ -1422,18 +1422,22 @@ class AlexaModeController(AlexaCapability):
# Humidifier mode # Humidifier mode
if self.instance == f"{humidifier.DOMAIN}.{humidifier.ATTR_MODE}": if self.instance == f"{humidifier.DOMAIN}.{humidifier.ATTR_MODE}":
mode = self.entity.attributes.get(humidifier.ATTR_MODE, None) mode = self.entity.attributes.get(humidifier.ATTR_MODE)
if mode in self.entity.attributes.get(humidifier.ATTR_AVAILABLE_MODES, []): modes: list[str] = (
self.entity.attributes.get(humidifier.ATTR_AVAILABLE_MODES) or []
)
if mode in modes:
return f"{humidifier.ATTR_MODE}.{mode}" return f"{humidifier.ATTR_MODE}.{mode}"
# Water heater operation mode # Water heater operation mode
if self.instance == f"{water_heater.DOMAIN}.{water_heater.ATTR_OPERATION_MODE}": if self.instance == f"{water_heater.DOMAIN}.{water_heater.ATTR_OPERATION_MODE}":
operation_mode = self.entity.attributes.get( operation_mode = self.entity.attributes.get(
water_heater.ATTR_OPERATION_MODE, None water_heater.ATTR_OPERATION_MODE
) )
if operation_mode in self.entity.attributes.get( operation_modes: list[str] = (
water_heater.ATTR_OPERATION_LIST, [] self.entity.attributes.get(water_heater.ATTR_OPERATION_LIST) or []
): )
if operation_mode in operation_modes:
return f"{water_heater.ATTR_OPERATION_MODE}.{operation_mode}" return f"{water_heater.ATTR_OPERATION_MODE}.{operation_mode}"
# Cover Position # Cover Position
@ -1492,7 +1496,7 @@ class AlexaModeController(AlexaCapability):
self._resource = AlexaModeResource( self._resource = AlexaModeResource(
[AlexaGlobalCatalog.SETTING_PRESET], False [AlexaGlobalCatalog.SETTING_PRESET], False
) )
preset_modes = self.entity.attributes.get(fan.ATTR_PRESET_MODES, []) preset_modes = self.entity.attributes.get(fan.ATTR_PRESET_MODES) or []
for preset_mode in preset_modes: for preset_mode in preset_modes:
self._resource.add_mode( self._resource.add_mode(
f"{fan.ATTR_PRESET_MODE}.{preset_mode}", [preset_mode] f"{fan.ATTR_PRESET_MODE}.{preset_mode}", [preset_mode]
@ -1508,7 +1512,7 @@ class AlexaModeController(AlexaCapability):
# Humidifier modes # Humidifier modes
if self.instance == f"{humidifier.DOMAIN}.{humidifier.ATTR_MODE}": if self.instance == f"{humidifier.DOMAIN}.{humidifier.ATTR_MODE}":
self._resource = AlexaModeResource([AlexaGlobalCatalog.SETTING_MODE], False) self._resource = AlexaModeResource([AlexaGlobalCatalog.SETTING_MODE], False)
modes = self.entity.attributes.get(humidifier.ATTR_AVAILABLE_MODES, []) modes = self.entity.attributes.get(humidifier.ATTR_AVAILABLE_MODES) or []
for mode in modes: for mode in modes:
self._resource.add_mode(f"{humidifier.ATTR_MODE}.{mode}", [mode]) self._resource.add_mode(f"{humidifier.ATTR_MODE}.{mode}", [mode])
# Humidifiers or Fans with a single mode completely break Alexa discovery, # Humidifiers or Fans with a single mode completely break Alexa discovery,
@ -1522,8 +1526,8 @@ class AlexaModeController(AlexaCapability):
# Water heater operation modes # Water heater operation modes
if self.instance == f"{water_heater.DOMAIN}.{water_heater.ATTR_OPERATION_MODE}": if self.instance == f"{water_heater.DOMAIN}.{water_heater.ATTR_OPERATION_MODE}":
self._resource = AlexaModeResource([AlexaGlobalCatalog.SETTING_MODE], False) self._resource = AlexaModeResource([AlexaGlobalCatalog.SETTING_MODE], False)
operation_modes = self.entity.attributes.get( operation_modes = (
water_heater.ATTR_OPERATION_LIST, [] self.entity.attributes.get(water_heater.ATTR_OPERATION_LIST) or []
) )
for operation_mode in operation_modes: for operation_mode in operation_modes:
self._resource.add_mode( self._resource.add_mode(
@ -2368,7 +2372,7 @@ class AlexaEqualizerController(AlexaCapability):
"""Return the sound modes supported in the configurations object.""" """Return the sound modes supported in the configurations object."""
configurations = None configurations = None
supported_sound_modes = self.get_valid_inputs( supported_sound_modes = self.get_valid_inputs(
self.entity.attributes.get(media_player.ATTR_SOUND_MODE_LIST, []) self.entity.attributes.get(media_player.ATTR_SOUND_MODE_LIST) or []
) )
if supported_sound_modes: if supported_sound_modes:
configurations = {"modes": {"supported": supported_sound_modes}} configurations = {"modes": {"supported": supported_sound_modes}}

5
homeassistant/components/alexa/entities.py
View File

@ -478,7 +478,7 @@ class ClimateCapabilities(AlexaEntity):
if ( if (
self.entity.domain == climate.DOMAIN self.entity.domain == climate.DOMAIN
and climate.HVACMode.OFF and climate.HVACMode.OFF
in self.entity.attributes.get(climate.ATTR_HVAC_MODES, []) in (self.entity.attributes.get(climate.ATTR_HVAC_MODES) or [])
or self.entity.domain == water_heater.DOMAIN or self.entity.domain == water_heater.DOMAIN
and (supported_features & water_heater.WaterHeaterEntityFeature.ON_OFF) and (supported_features & water_heater.WaterHeaterEntityFeature.ON_OFF)
): ):
@ -742,7 +742,8 @@ class MediaPlayerCapabilities(AlexaEntity):
and domain != "denonavr" and domain != "denonavr"
): ):
inputs = AlexaEqualizerController.get_valid_inputs( inputs = AlexaEqualizerController.get_valid_inputs(
self.entity.attributes.get(media_player.const.ATTR_SOUND_MODE_LIST, []) self.entity.attributes.get(media_player.const.ATTR_SOUND_MODE_LIST)
or []
) )
if len(inputs) > 0: if len(inputs) > 0:
yield AlexaEqualizerController(self.entity) yield AlexaEqualizerController(self.entity)

8
homeassistant/components/alexa/handlers.py
View File

@ -570,7 +570,7 @@ async def async_api_select_input(
# Attempt to map the ALL UPPERCASE payload name to a source. # Attempt to map the ALL UPPERCASE payload name to a source.
# Strips trailing 1 to match single input devices. # Strips trailing 1 to match single input devices.
source_list = entity.attributes.get(media_player.const.ATTR_INPUT_SOURCE_LIST, []) source_list = entity.attributes.get(media_player.const.ATTR_INPUT_SOURCE_LIST) or []
for source in source_list: for source in source_list:
formatted_source = ( formatted_source = (
source.lower().replace("-", "").replace("_", "").replace(" ", "") source.lower().replace("-", "").replace("_", "").replace(" ", "")
@ -987,7 +987,7 @@ async def async_api_set_thermostat_mode(
ha_preset = next((k for k, v in API_THERMOSTAT_PRESETS.items() if v == mode), None) ha_preset = next((k for k, v in API_THERMOSTAT_PRESETS.items() if v == mode), None)
if ha_preset: if ha_preset:
presets = entity.attributes.get(climate.ATTR_PRESET_MODES, []) presets = entity.attributes.get(climate.ATTR_PRESET_MODES) or []
if ha_preset not in presets: if ha_preset not in presets:
msg = f"The requested thermostat mode {ha_preset} is not supported" msg = f"The requested thermostat mode {ha_preset} is not supported"
@ -997,7 +997,7 @@ async def async_api_set_thermostat_mode(
data[climate.ATTR_PRESET_MODE] = ha_preset data[climate.ATTR_PRESET_MODE] = ha_preset
elif mode == "CUSTOM": elif mode == "CUSTOM":
operation_list = entity.attributes.get(climate.ATTR_HVAC_MODES, []) operation_list = entity.attributes.get(climate.ATTR_HVAC_MODES) or []
custom_mode = directive.payload["thermostatMode"]["customName"] custom_mode = directive.payload["thermostatMode"]["customName"]
custom_mode = next( custom_mode = next(
(k for k, v in API_THERMOSTAT_MODES_CUSTOM.items() if v == custom_mode), (k for k, v in API_THERMOSTAT_MODES_CUSTOM.items() if v == custom_mode),
@ -1013,7 +1013,7 @@ async def async_api_set_thermostat_mode(
data[climate.ATTR_HVAC_MODE] = custom_mode data[climate.ATTR_HVAC_MODE] = custom_mode
else: else:
operation_list = entity.attributes.get(climate.ATTR_HVAC_MODES, []) operation_list = entity.attributes.get(climate.ATTR_HVAC_MODES) or []
ha_modes: dict[str, str] = { ha_modes: dict[str, str] = {
k: v for k, v in API_THERMOSTAT_MODES.items() if v == mode k: v for k, v in API_THERMOSTAT_MODES.items() if v == mode
} }