mherger wrote:
>
> I didn't even follow all of this discussion, as it's a lost cause, tbh.
>
> I'll be happy to read the single page summary of this all where
> everybody can agree with...
>
Michael, would an alternative be to at least make the backend query
functions work for this use case so it's possible to make a reasonably
simple plugin for it ?
For example, would it be possible to just add a way to filter by
role_ids in:
- Slim::Control::Queries::artistsQuery
- Slim::Control::Queries::albumsQuery
- Slim::Control::Queries::titlesQuery
Preferably both a way to filter by a single role or a list of roles so
it's possible to also use it for an "All Artists" menu which isn't
restricted to the LMS Settings for band, composer, conductor.
Making it optional and keep the current logic as it is if role_ids
parameter isn't specified, should ensure we don't break any existing
code.
I'm asking because I once actually spent a few hours to try to make a
BrowseByContributor plugin and the result was that I had to duplicate
all the above three methods because I wanted to get artists of a single
role, for example role_ids=5 (album artist).
This doesn't make it easy to make a plugin for it but it makes it less
hard than it is today since the plugin developer can focus on
duplicating and adjusting the _artists, _albums, _tracks methods in
Slim::Menu::BrowseLibrary to support role filtering.
I know this isn't the optimal solution, but it at least opens up for
some third party developer to do it in case you feel it's too risky to
do the actual menu change in the LMS core. Also, it's a first step to
actually do this in the core because if you ever wants to implement the
menu changes in the core, the first thing you need to do is likely to do
the above mentioned changes.
Of course, if I'm making such plugin it would probably still be a
commercial one (since I would have to spend time maintaining it), so
maybe it isn't much better than the current solution where users can use
the more advanced Custom Browse plugin ?
The only issue I'm a bit worried about in a solution like this is that
since the role_ids filtering wouldn't be used by the core, there is a
risk other changes will break the functionality since you don't have an
easy way to test it from the standard UI.
If you would do changes like above, I'd be happy to spend some time
verifying them.
Erland Isaksson ('My homepage' (http://erland.isaksson.info))
(Developer of 'many plugins/applets (both free and commercial)'
(http://wiki.slimdevices.com/index.php/User:Erland).
If you like to encourage future presence on this forum and/or third
party plugin/applet development, 'consider purchasing some plugins'
(http://license.isaksson.info))
*Interested in the future of music streaming ? 'ickStream - A world of
music at your fingertips'
(http://forums.slimdevices.com/showthread.php?98467-Pre-Announcement-ickStream&p=743516)*.
------------------------------------------------------------------------
erland's Profile: http://forums.slimdevices.com/member.php?userid=3124
View this thread: http://forums.slimdevices.com/showthread.php?t=101264
_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/beta
