After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 560886 - Now Playing displays previous and current artist if current song has no album.
Now Playing displays previous and current artist if current song has no album.
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: User Interface
1.4.1
Other All
: Normal normal
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-11-15 02:50 UTC by Matt
Modified: 2009-03-26 01:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot (8.82 KB, image/png)
2008-11-15 02:53 UTC, Matt
  Details
Check the track info string for enough spans before splitting (3.30 KB, patch)
2009-03-21 04:53 UTC, John Millikin
committed Details | Review

Description Matt 2008-11-15 02:50:39 UTC
Please describe the problem:
When the current song doesn't have an album, (not even "Unknown") Now Playing displays the previous song's artist above the current song's artist.

Steps to reproduce:
1. Listen to a song with an album.
2. List to one without one.
3. Go to Now Playing.
(I don't know if switching 2 and 3 makes a difference. I forgot which one I did. :)
 


Actual results:


Expected results:


Does this happen every time?


Other information:
In case it's relevant, both songs were playing from Last.fm. Album-less songs don't appear too often from Last.fm, so I couldn't test it again. The next song with an album is displayed correctly.
Comment 1 Matt 2008-11-15 02:53:17 UTC
Created attachment 122711 [details]
screenshot

The previous song (with an album) was by The Maccabees and the current song (without an album) is I've Seen Enough by Cold War Kids.

Thanks.
Comment 2 John Millikin 2009-03-21 04:53:58 UTC
Created attachment 131066 [details] [review]
Check the track info string for enough spans before splitting

LargeTrackDisplayInfo does some very weird stuff to get the lines for display, apparently to work around translation-related code in the generic track info widget. Therefore, rather than fix the underlying problem, this will just sort of plaster it over.

Patch: When splitting the info line on <span>, check that there are at least three <span>s before splitting.
Comment 3 Gabriel Burt 2009-03-25 23:30:32 UTC
Looks good, thanks John.