Bug 5244 – Now Playing display of internet radio station should show both station name and song metadata, if avaiable, together

Bug 5244 - Now Playing display of internet radio station should show both station name and song metadata, if avaiable, together

Summary:

Now Playing display of internet radio station should show both station name a...

Status:	RESOLVED WORKSFORME

Product:	SB Controller
Classification:	Unclassified
Component:	Browser
Version:	unspecified
Platform:	Macintosh Other

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	Squeezebox QA Team email alias

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2007-08-12 21:46 UTC by Blackketter Dean
Modified:	2007-10-30 09:36 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Screenshot of now playing screensaver while listening to lounging sound (50.23 KB, image/png) 2007-09-05 08:52 UTC, Ben Klaas	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Blackketter Dean 2007-08-12 21:46:24 UTC

On Jive, if an internet radio station is  playing and there is song metadata, both should be visible.  If there is no cover art, the placeholder for the cover art should be omitted, to free up more visible space.

Comment 1 Ben Klaas 2007-09-05 08:47:13 UTC

I'm listening to Slim Picks->Best of Internet Radio->Lounging Sound right now, which sends track metadata that is viewable from squeezebox (artist - track). Jive is failing to display it, and I'm getting this error on the console:

SDL_event_queue >  5 :  11
104340:503032 DEBUG (SlimBrowserApplet.lua:620) - _statusSink()
error in event function:
        ...ers/bklaas/jive/src/pkg/jive/share/jive/ui/Label.lua:144: assertion failed!
stack traceback:
        [C 0x167c5]: in function 'assert'
        ...ers/bklaas/jive/src/pkg/jive/share/jive/ui/Label.lua:144: in function 'setValue'
        ...g/jive/share/applets/NowPlaying/NowPlayingApplet.lua:175: in function 'setAlbum'
        ...g/jive/share/applets/NowPlaying/NowPlayingApplet.lua:218: in function 'updateTrack'
        ...g/jive/share/applets/NowPlaying/NowPlayingApplet.lua:283: in function '?'
        ...g/jive/share/applets/NowPlaying/NowPlayingApplet.lua:103: in function 'func'
        ...rs/bklaas/jive/src/pkg/jive/share/jive/net/Comet.lua:277: in function 'sink'
        ...s/bklaas/jive/src/pkg/jive/share/jive/net/Socket.lua:115: in function 'msg'
        ...s/jive/src/pkg/jive/share/jive/net/NetworkThread.lua:260: in function 'idle'
        ...s/jive/src/pkg/jive/share/jive/net/NetworkThread.lua:326: in function <...s/jive/src/pkg/jive/share/jive/net/NetworkThread.lua:325>
        [C 0x1411c]: ?
        [C 0x13b0d]: ?
        [C 0x12dee]: in function 'processEvents'
        ...ers/bklaas/jive/src/pkg/jive/share/jive/JiveMain.lua:224: in function <...ers/bklaas/jive/src/pkg/jive/share/jive/JiveMain.lua:161>
        (tail call): ?
        ...ers/bklaas/jive/src/pkg/jive/share/jive/JiveMain.lua:285: in main chunk
        [C 0x1c3d7]: ?
        [C 0x2bd8]: ?


screenshot of Now Playing screensaver to follow

Comment 2 Ben Klaas 2007-09-05 08:52:48 UTC

Created attachment 2128 [details]
Screenshot of now playing screensaver while listening to lounging sound

Comment 3 Ben Klaas 2007-10-30 09:36:09 UTC

I'm looking at Lounging Sound right now with the 6.22 kernel build from Oct 30, and don't see the issue any more

It's displaying the default cover art item, and not at the size it should be (this is a general problem on NowPlaying with tracks with now cover art). But that's not this bug.