Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Onkyo] Fix various issues #1529

Merged
merged 3 commits into from
Dec 7, 2016
Merged

[Onkyo] Fix various issues #1529

merged 3 commits into from
Dec 7, 2016

Conversation

marcelrv
Copy link
Contributor

@marcelrv marcelrv commented Dec 5, 2016

Fixes various issues for Onkyo binding

  • Fix lookup of listening modes & audio sources below Custom jetty config #10
  • Adds several missing listening modes
  • Adds several missing sources
  • Fix Exceptions statusUpdateReceived in case of N/A
  • Fix issue when looking up sources for zone2

Signed-off-by: Marcel Verpaalen marcel@verpaalen.com

Fixes various issues for Onkyo binding
* Fix lookup of listening modes & audio sources below #10
* Adds several missing listening modes
* Adds several missing sources
* Fix Exceptions statusUpdateReceived in case of N/A
* Fix issue when looking up sources for zone2

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
@kaikreuzer
Copy link
Member

@pail23, Could you please review?

@hfanieng
Copy link

hfanieng commented Dec 5, 2016

Fix Exceptions statusUpdateReceived in case of N/A

Does this mean i can now use the binding even when my receiver is switched to standby without flooding the log?

@NorbertHD
Copy link

@hfanieng
No, because the Onkyo sends LMDN/A when in standby and there is a bug in OnkyoHandler.java.

                    case LISTEN_MODE_SET: 
                         String listenModeStr = data.substring(3, 5); 
                         // update only when listen mode is supported 
                         if (listenModeStr != "N/") { 
                             int listenMode = Integer.parseInt(listenModeStr, 16); 
                             updateState(CHANNEL_LISTENMODE, new DecimalType(listenMode)); 
                         } 
                         break; 

if (listenModeStr != "N/")
should be
if ( ! listenModeStr.contentEquals("N/"))

@hfanieng
Copy link

hfanieng commented Dec 5, 2016

No, because the Onkyo sends LMDN/A when in standby and there is a bug in OnkyoHandler.java.

Thank you for pointing me on that @NorbertHD 👍 If this bug is fixed the Onkyo-Binding is one thing i would like to test whit my hardware and oH2.

Mentioned by @hfanieng

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
@marcelrv
Copy link
Contributor Author

marcelrv commented Dec 5, 2016

@NorbertHD also included the fix you proposed.

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
@pail23
Copy link
Contributor

pail23 commented Dec 6, 2016

@marcelrv Thanks for the fixes! The changes lgtm.
@kaikreuzer From my point of view this ready to be merged.

@hfanieng
Copy link

hfanieng commented Dec 6, 2016

@marcelrv Thanks for the quick reaction!

@kaikreuzer
Copy link
Member

Thank you all!

@kaikreuzer kaikreuzer merged commit 8f10cf4 into openhab:master Dec 7, 2016
@marcelrv marcelrv deleted the OnkyoFixes2 branch December 7, 2016 10:27
Jamstah pushed a commit to Jamstah/openhab-addons that referenced this pull request Dec 12, 2016
Fixes various issues for Onkyo binding
* Fix lookup of listening modes & audio sources below openhab#10
* Adds several missing listening modes
* Adds several missing sources
* Fix Exceptions statusUpdateReceived in case of N/A
* Fix issue when looking up sources for zone2

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
@kaikreuzer kaikreuzer modified the milestone: 2.0.0 Jan 17, 2017
fharni pushed a commit to fharni/openhab2-addons that referenced this pull request Feb 10, 2017
Fixes various issues for Onkyo binding
* Fix lookup of listening modes & audio sources below openhab#10
* Adds several missing listening modes
* Adds several missing sources
* Fix Exceptions statusUpdateReceived in case of N/A
* Fix issue when looking up sources for zone2

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants