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

Fix source mapping in Onkyo (#129716)

Browse Source 
* Fix source mapping

* Fix copy paste
This commit is contained in:
Artur Pragacz 2024-11-05 03:56:47 +01:00 committed by GitHub
parent dafd54ba2b
commit 617e87e02c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 39 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

54
homeassistant/components/onkyo/media_player.py
View File

@ -128,13 +128,27 @@ ISSUE_URL_PLACEHOLDER = "/config/integrations/dashboard/add?domain=onkyo"
type InputLibValue = str | tuple[str, ...] type InputLibValue = str | tuple[str, ...]
_cmds: dict[str, InputLibValue] = {
k: v["name"] def _input_lib_cmds(zone: str) -> dict[InputSource, InputLibValue]:
for k, v in { match zone:
**PYEISCP_COMMANDS["main"]["SLI"]["values"], case "main":
**PYEISCP_COMMANDS["zone2"]["SLZ"]["values"], cmds = PYEISCP_COMMANDS["main"]["SLI"]
}.items() case "zone2":
} cmds = PYEISCP_COMMANDS["zone2"]["SLZ"]
case "zone3":
cmds = PYEISCP_COMMANDS["zone3"]["SL3"]
case "zone4":
cmds = PYEISCP_COMMANDS["zone4"]["SL4"]
result: dict[InputSource, InputLibValue] = {}
for k, v in cmds["values"].items():
try:
source = InputSource(k)
except ValueError:
continue
result[source] = v["name"]
return result
async def async_setup_platform( async def async_setup_platform(
@ -147,11 +161,8 @@ async def async_setup_platform(
host = config.get(CONF_HOST) host = config.get(CONF_HOST)
source_mapping: dict[str, InputSource] = {} source_mapping: dict[str, InputSource] = {}
for value, source_lib in _cmds.items(): for zone in ZONES:
try: for source, source_lib in _input_lib_cmds(zone).items():
source = InputSource(value)
except ValueError:
continue
if isinstance(source_lib, str): if isinstance(source_lib, str):
source_mapping.setdefault(source_lib, source) source_mapping.setdefault(source_lib, source)
else: else:
@ -340,9 +351,12 @@ class OnkyoMediaPlayer(MediaPlayerEntity):
self._volume_resolution = volume_resolution self._volume_resolution = volume_resolution
self._max_volume = max_volume self._max_volume = max_volume
self._source_mapping = sources self._name_mapping = sources
self._reverse_mapping = {value: key for key, value in sources.items()} self._reverse_name_mapping = {value: key for key, value in sources.items()}
self._lib_mapping = {_cmds[source.value]: source for source in InputSource} self._lib_mapping = _input_lib_cmds(zone)
self._reverse_lib_mapping = {
value: key for key, value in self._lib_mapping.items()
}
self._attr_source_list = list(sources.values()) self._attr_source_list = list(sources.values())
self._attr_extra_state_attributes = {} self._attr_extra_state_attributes = {}
@ -414,7 +428,7 @@ class OnkyoMediaPlayer(MediaPlayerEntity):
async def async_select_source(self, source: str) -> None: async def async_select_source(self, source: str) -> None:
"""Select input source.""" """Select input source."""
if self.source_list and source in self.source_list: if self.source_list and source in self.source_list:
source_lib = _cmds[self._reverse_mapping[source].value] source_lib = self._lib_mapping[self._reverse_name_mapping[source]]
if isinstance(source_lib, str): if isinstance(source_lib, str):
source_lib_single = source_lib source_lib_single = source_lib
else: else:
@ -432,7 +446,7 @@ class OnkyoMediaPlayer(MediaPlayerEntity):
) -> None: ) -> None:
"""Play radio station by preset number.""" """Play radio station by preset number."""
if self.source is not None: if self.source is not None:
source = self._reverse_mapping[self.source] source = self._reverse_name_mapping[self.source]
if media_type.lower() == "radio" and source in DEFAULT_PLAYABLE_SOURCES: if media_type.lower() == "radio" and source in DEFAULT_PLAYABLE_SOURCES:
self._update_receiver("preset", media_id) self._update_receiver("preset", media_id)
@ -505,9 +519,9 @@ class OnkyoMediaPlayer(MediaPlayerEntity):
@callback @callback
def _parse_source(self, source_lib: InputLibValue) -> None: def _parse_source(self, source_lib: InputLibValue) -> None:
source = self._lib_mapping[source_lib] source = self._reverse_lib_mapping[source_lib]
if source in self._source_mapping: if source in self._name_mapping:
self._attr_source = self._source_mapping[source] self._attr_source = self._name_mapping[source]
return return
source_meaning = source.value_meaning source_meaning = source.value_meaning