Bug 1220 - Now Playing list in web interface doesn't always update correctly
: Now Playing list in web interface doesn't always update correctly
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.0.0
: PC Windows XP
: P2 normal with 1 vote (vote)
: ---
Assigned To: Vidur Apparao
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-03-28 11:17 UTC by Vidur Apparao
Modified: 2008-08-18 10:54 UTC (History)
4 users (show)

See Also:
Category: ---


Attachments
update after skin change (556 bytes, patch)
2005-03-29 13:13 UTC, KDF
Details | Diff
update on track start (501 bytes, patch)
2005-03-31 21:51 UTC, KDF
Details | Diff
--d_source and --d_playlist log (32.51 KB, text/plain)
2005-04-05 07:25 UTC, Steve Bernard
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Vidur Apparao 2005-03-28 11:17:22 UTC
There seem to be two cases of this:
1) The currently playing song isn't updated for skins that highlight this song.
2) If you change skins in Server Settings (not by overriding using a
skin-specific URL), the playlist uses the old skin's HTML.
Comment 1 KDF 2005-03-29 13:13:02 UTC
Created attachment 384 [details]
update after skin change

updates playlist mod time on a skin change so that the playlist render will
happen.  not many places were we are NOT updating playlist time now.  only case
where we really get any benefit is the META refresh, and only when it doesn't
happen near a song change.
Comment 2 KDF 2005-03-29 13:24:29 UTC
as for the other part of this bug, fishbone playlist seems to update ok, default
doesn't.  this is odd, since teh code to control the update should be happening
in the server.
Comment 3 KDF 2005-03-29 13:54:28 UTC
aha...seems intermittent.  I guess the skip to the next song isn't always
happening in just one part of the code.
Comment 4 Philip Meyer 2005-03-30 13:30:48 UTC
*** Bug 1253 has been marked as a duplicate of this bug. ***
Comment 5 KDF 2005-03-31 19:32:45 UTC
*** Bug 1273 has been marked as a duplicate of this bug. ***
Comment 6 KDF 2005-03-31 21:51:51 UTC
Created attachment 390 [details]
update on track start

trigger playlist modified time on a trackStartEvent to catch all song changes. 
should allow all skins to update the highlighted track to 'currentsong'.
Comment 7 Steve Bernard 2005-04-01 08:44:08 UTC
I just updated SVN to get this fix, and it seemed to work for a few tracks (in
Fishbone), but since then it's been back to the old not-updating.  Like, I
listened to a four song EP, and it looked to be refreshing properly until it
just stuck on highlighting track 3, even after it was completely finished
playing.  I'm listening to a completely different album, and the web interface
is still highlighting track 1 even though I'm listening to track 10.

The only funny business I can think of during that is I did pause and then
resume track 3 of that first EP.
Comment 8 Dan Sully 2005-04-03 15:38:17 UTC
*** Bug 1260 has been marked as a duplicate of this bug. ***
Comment 9 KDF 2005-04-04 16:41:26 UTC
how about d_source and d_playlist log output for a case where this is happening.
 d_playlist will show the playlist caching status, and d_source will hopefully
give a clue as to which playback command path is still being missed.  I would
have thought trackStartEvent covered all new track starts.  This is a bit
worrying if it isn't.  I wonder what else this will cause us to trip over.
Comment 10 Steve Bernard 2005-04-04 18:12:40 UTC
I'll get some logs of this tomorrow, but let me give you the exact 
circumstances of what I'm doing right now as it misbehaves.  I've got a 
SqueezeBoxG at home, and I've got an SSH tunnel from work to the home server 
where I listen with Softsqueeze.  When I have both connected, so that the 
SlimServer web interface gives me a choice between which player to control, I 
find that the work Softsqueeze has its track highlighting update properly, but 
the indicators for the SqueezeBox at home don't change.  This is even the case 
if only one player is playing music at the time.  It's never transpired since 
it's been acting up this way that the work one has not updated while the home 
one has, but I don't know if that's really significant.

When I'm at work tomorrow, I'll play some tracks there and at control the home 
player and try to get some debugging logs.
Comment 11 Steve Bernard 2005-04-05 07:25:08 UTC
Created attachment 401 [details]
--d_source and --d_playlist log

Okay, I've attached a log of my SlimServer as I played tracks from work just as
I described in my comment last night.  To reiterate: I have two players
connected to the server: Stereo (my home stereo) and Work (SoftSqueeze).  I had
both players on and running and was using SlimServer's web interface from work.


I started playing an album on Stereo and I let it play for four songs, stopping
the player in the middle of the fourth.  That should be enough data to see what
happens at track change time.  Throughout the whole time, the first track
remained highlighted in the web interface in Fishbone.

The browser I'm using at work is Firefox 1.02 on WinXP.  The home server is
Ubuntu Linux 4.  The SlimServer version was the newest trunk version in SVN at
the moment.
Comment 12 Blackketter Dean 2005-04-20 14:57:24 UTC
Vidur believes that this is fixed in the latest nightly.  Please confirm and reopen if it's not.  Thanks.
Comment 13 Steve Bernard 2005-04-20 16:13:16 UTC
I don't seem to have an option to reopen this bug, but I'm not seeing 
difference in behavior as of right now.  The only player active in the 
interface is my SBG player attached to my stereo.  I'm playing an album right 
now, and it's playing track 4, but track 1 is still highlighted.

I'd reopen it if I could but the only option by the Commit button is "Leave as 
RESOLVED FIXED" and there's no place I see to change it anywhere else.
Comment 14 Vidur Apparao 2005-04-21 08:47:51 UTC
Kevin, seems like the Source.pm patch sets the playlistChangeTime in response to
a trackStart event (reported by SB2), but not in openSong (for SB1 and older).
Seems like the right thing to do would be to add:

               $client->currentPlaylistChangeTime(time());

inside the if (!$client->reportsTrackStart()) clause in openSong (before line
1399 of r3007). I'd post a patch, but my tree has several changes in it.

What do you think?
Comment 15 Steve Bernard 2005-04-21 09:07:50 UTC
That would make sense as to why my SoftSqueeze is fine (it emulates SB2 in its
newest versions, if I'm not mistaken) but my SBG doesn't update.
Comment 16 KDF 2005-04-21 09:20:59 UTC
That does look suspiciously like the hole I was looking for.  I can merge that
in tonight.
Comment 17 Vidur Apparao 2005-04-21 17:18:01 UTC
Beat you to the punch. Fixed in trunk. Will move to the 6.0.x branch shortly.
Comment 18 Vidur Apparao 2005-04-21 17:23:05 UTC
And in the branch (change 3032). The trunk revision is change 3031.
Comment 19 Steve Bernard 2005-04-21 17:57:39 UTC
Woot, that's the stuff.  Thanks, guys.