search

ol-iver / denonavr

Automation Library for Denon AVR receivers.
MIT License
176 stars 67 forks source link

Fix bug preventing All Zone Stereo from being disabled automatically #107

Closed JeremyKennedy closed 4 years ago

JeremyKennedy commented 4 years ago

Likely related to issue #78. My receiver reports self._sound_mode_raw as All Zone Stereo, however your if in set_sound_mode:1466 expects it to be uppercase - ALL ZONE STEREO. This change simply converts it to uppercase within the if statement, allowing All Zone Stereo to be disabled when necessary. I'm guessing that some receivers report it all caps, but mine (X1500H) uses title case.

It may be better to instead set self._sound_mode_raw to uppercase automatically, rather than only in the if, however I didn't want to mess with too much.

ol-iver commented 4 years ago

Let's start with this, thank you :blush: