cpd73 wrote: 
> No, I do not want to request rating with a separate call - this would be
> slow for a large queue of items. I'm *only* going to obtain rating via
> the 'R' tags parameter. Why does your plugin not also set the rating in
> LMS? TrackStat appears to do this - as in if I set a rating via Material
> it is returned in the 'R' parameter.
> 
Just for information.

If I remember correctly I suspect TrackStat refresh functionality might
result in issues if it’s  enabled and ratings are set with something
else than “trackstat setrating” command. This command will write to both
TrackStat tables and core LMS tables so a rating set with TrackStat
should always be visible if you obtain ratings with the ‘R’ tags
parameter. However, if you set the rating some other way the refresh
functionality in TrackStat executed after rescans and at LMS startup
might cause issues since it will overwrite changed ratings in core LMS
with the ones in TrackStat tables. It won’t be an issue for new ratings
since those aren’t overwritten but I think it will restore changed
ratings. So if people use Material skin and that doesn’t use the
trackstat setrating command they will likely get confusions behavior.

Note, it’s really TrackStat that does the wrong thing here, I’m just
mentioning it in case someone have strange behavior due to this. I’m not
sure the ‘R’ flag existed in core LMS when TrackStat was implemented, at
least I wasn’t aware of it. Unfortunately, I’m unlikely to personally
get the time to fix it in TrackStat in the foreseeable future. So to sum
it up, if Material doesn’t use the TrackStat command, people using
Material skin and TrackStat plugin should probably verify how it works
before setting ratings with Material skin.



Erland Isaksson ('My homepage' (http://erland.isaksson.info))
Developer of 'many plugins/applets'
(https://wiki.slimdevices.com/index.php/User_Erland.html)
------------------------------------------------------------------------
erland's Profile: http://forums.slimdevices.com/member.php?userid=3124
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