mherger wrote: 
> 
> The problem we're trying to resolve is that for some menu items we want
> 
> to override the globally set library_id.
> 

No, from a client perspective it's a different problem: from a client
perspective there is no "globally defined library_id" when using the
CLI. From a client perspective you are simply using an invalid
library_id in the menus.

> 
> Nope. You decided that the menus as defined for SP were not good enough
> 
> for your use.
> 
Yes. And that is probably something that doesn't only affect iPeng but
_anybody_ using the documented CLI.

> 
> You can't expect us/me to 
> implement your cherry-picked functionality here or there, depending on 
> what you want/don't want to implement on your own.
> 
I don't want you to implement cherry picked functionality, I just want
you to only use valid library_ids.

Having different "library_id" value ranges across different functions
isn't exactly a good design.

> 
> Didn't you ask me to 
> add library_id to the status query in order to allow you to implement 
> your own menus? Now please don't stop half way.
> 


> 
> We/I've taken much blame for putting too much logic eg. into the artists
> 
> query. Now I'm trying to stay away from the same problem and it's not 
> right either. Oh well...
> 

By whom? Tell me where? I can't remember anyone complaining about that
at any time.
The only thing _I_ ever complained about is that you don't expose the
model layer and obfuscate content type by using intransparent and
undocumented functions to create the menus.
It would actually make a lot of things in LMS much more transparent,
less confusing, easier to debug and less error prone if you would simply
use the same API in your menus that you expose in the documentation,
which is the CLI. "Eat you own dog food" has always been a good primary
guideline for API design. It helps to solve so many issues.
I'm still getting requests every day to make simple changes to menus
where it's not possible due to this issue.

If I now use some arbitrary logic to replace "-1" library values by
something else or removing it, in a year we will all have forgotten and
when you do the next change then it will break something else and we
need to change everything all over again.



---
learn more about iPeng, the iPhone and iPad remote for the Squeezebox
and
Logitech UE Smart Radio as well as iPeng Party, the free Party-App, 
at penguinlovesmusic.com
*New: iPeng 8, the Universal App for iOS 7 and iOS 8*
------------------------------------------------------------------------
pippin's Profile: http://forums.slimdevices.com/member.php?userid=13777
View this thread: http://forums.slimdevices.com/showthread.php?t=101701

_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/beta

Reply via email to