Sonos binding, partly implemented getFavoriteRadios getFavorites

[BasvanH]

Hello,

Currently, the Sonos Binding does not have a channel for getting an list of the favorites. I would like to use this list in an HabPanel widget so you can choose an favorite to play.

The SonosPlayeHandler does have some parts of code towards getting the favorites, but it doesn't have it ported to an channel. Could this be implemented?

SonosPlayerHandler.java
https://github.com/eclipse/smarthome/blob/master/extensions/binding/org.eclipse.smarthome.binding.sonos/src/main/java/org/eclipse/smarthome/binding/sonos/internal/handler/ZonePlayerHandler.java

[triller-telekom]

This sounds like a good use case for #4942.

You could have a dynamic "SelectionOptionProvider" for a favorites channel and show it as a selection element in a basic ui sitemap or whatever kind of widget in HabPanel.

[BasvanH]

Kindly requesting an update.

[maggu2810]

As long as there is no volunteer that would like to implement that feature...

[lolodomo]

I will have a look as soon as my changes in Classic UI and Basic UI are merged.
A similar thing was apparently already implemented in the Kodi binding (building options dynamically).

[lolodomo]

In fact, it could be applied to 3 channels: favorite, playlist and radio.

[lolodomo]

I implemented that in the Sonos binding as a POC. Works well.
I have a problem to solve before proposing a PR: when triggering a refresh ? I was expecting a notification through the UPnP ContentDirectory subnscription but it does not happen and I don't understand why (I know that it was happening in the Vera plugin).

I have one worry: what will happen with very big lists ? Should we limit to a certain max list size ?

[lolodomo]

Unfortunately there is first an issue to fix in the jupnp library
jupnp/jupnp#97

[sjsf]

I just merged the PR - nevertheless, the above statement wrt the dependency on a yet open change in jUPnP remains valid. See also #5682 (comment).

[lolodomo]

I have not yet checked but the new feature should work starting with the snapshot 1305.

[lolodomo]

I can now confirm that it is available.