erland wrote: 
> 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 confusing 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.

Why do you think there would be issues? Material will *only* set ratings
via TackStat. However, due to speed reasons, it will only -read- ratings
via LMS's songinfo "tags:R" flag. If TrackStat's setrating updates LMS
(as it appears to for me) then there is no issue.



*Material debug:* 1. Launch via http: //SERVER:9000/material/?debug=json
(Use http: //SERVER:9000/material/?debug=json,cometd to also see update
messages, e.g. play queue) 2. Open browser's developer tools 3. Open
console tab in developer tools 4. REQ/RESP messages sent to/from LMS
will be logged here.
------------------------------------------------------------------------
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