Bug 15815 - Now Playing information can alternate previous-next-previous-next at track skip
: Now Playing information can alternate previous-next-previous-next at track skip
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: Now Playing
: 7.4.x
: All All
: P3 normal (vote)
: 7.5.0
Assigned To: Alan Young
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-03-02 08:18 UTC by Alan Young
Modified: 2010-04-08 17:26 UTC (History)
2 users (show)

See Also:
Category: Bug


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alan Young 2010-03-02 08:18:15 UTC
Sometimes when invoking a track skip, or a number of other user-initiated changes to what is playing, one sees (1) "<new track title> / Now Playing" on the controller (for example) and a little later this is updated with (2) "<new track title> / <new artist> - <new album>".

Occasionally. the display briefly gets reverted to "<old track title> / <old artist> - <old album>" between (1) and (2).  This looks rather odd.

After some investigation this seem to be because (1) is initiated by a displayStatus/showBriefly which may timeout before the playerstatus for (2) arrives. This is especially noticeable with TinySC. Lack of timely update (but without the flip-flop behaviour) also is a feature when dealing with remote tracks.

I looked at statusQuery_filter. I note that it does not send a playerstatus for 'playlist open', nor for 'playlist jump' and this means that the playerstatus will not happen until the 'playlist newsong' when the new song actually starts playing. I think that this is probably the right thing to do, as otherwise the server gets hit hard generating multiple playerstatus updates.

One solution might be to allow 'playlist open' and 'playlist jump' to trigger a playerstatus but with a longer delay that usual - say 1s. This would usually be overtaken by the playerstatus triggered from the 'playlist newsong' and therefore only one playerstatus would be generated. This would be particularly for remote services such as Rhapsody. I think that this ought to be combined with a longer timeout on the display of the showBriefly from the displayStatus.

I have tested this somewhat, without a change to the showBriefly timeout and it looks good.
Comment 1 SVN Bot 2010-03-03 00:02:37 UTC
 == Auto-comment from SVN commit #30307 to the slim repo by ayoung ==
 == https://svn.slimdevices.com/slim?view=revision&revision=30307 ==

Fixed bug 15815: Now Playing information can alternate previous-next-previous-next at track skip 
Keep the now-playing title show-briefly up longer to allow for delay in playerstatus update.
Increase the likelihood of a useful playerstatus update being delivered in a timely fashion if track start is slow.
Comment 2 SVN Bot 2010-03-03 00:02:42 UTC
 == Auto-comment from SVN commit #8619 to the jive repo by ayoung ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8619 ==

Fixed bug 15815: Now Playing information can alternate previous-next-previous-next at track skip 
Keep the now-playing title show-briefly up longer to allow for delay in playerstatus update.
Increase the likelihood of a useful playerstatus update being delivered in a timely fashion if track start is slow.
Comment 3 Alan Young 2010-03-03 00:04:07 UTC
The changes above will not fix the problem is all cases but it will for the vast majority.
Comment 4 SVN Bot 2010-03-03 00:13:11 UTC
 == Auto-comment from SVN commit #30308 to the slim repo by ayoung ==
 == https://svn.slimdevices.com/slim?view=revision&revision=30308 ==

bug 15815: Now Playing information can alternate previous-next-previous-next at track skip
Comment 5 Alan Young 2010-03-09 23:23:33 UTC
Bug 15868 effectively reopens this bug
Comment 6 Chris Owens 2010-04-08 17:26:22 UTC
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release.

If you are still seeing this issue, please let us know!