mherger wrote: 
> > Possible bug in M&AI implementation in Material (which is getting
> very
> > good btw)
> 
> I'm seeing that Material is sending
> 
> ["musicartistinfo","albumreview","artist:Gary Moore","album:Dark days in
> 
> Paradise"]
> 
> It would be preferable if it sent "album_id:123" when possible. And then
> 
> I need to eliminate a small bug in MAI...

I was just looking at the MAI code for this. If I used "album_id:", MAI
does a lookup on the DB to get the album name. So, I save it some time
here :-) However, it will also call "cleanupAlbumName", which removes
the part in parenthesis. if MAI is called with "album:" it does not
cleanup the name - hence the error. So, I guess MAI needs to be fixed to
call the cleanup in this "album:" case.

The reason for not using IDs was so that it would work with streams,
etc. And seeing as MAI just maps the ID -> name anyway, I see no benefit
(as the name is that supplied by LMS anyway).


------------------------------------------------------------------------
cpd73's Profile: http://forums.slimdevices.com/member.php?userid=66686
View this thread: http://forums.slimdevices.com/showthread.php?t=109624

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

Reply via email to