Bugzilla – Bug 7211
On Jive displays, don't try displaying the artwork for 'No album'
Last modified: 2009-07-31 10:17:25 UTC
Description: When the search query returns details for the searches to Jive it returns an artwork id. The artwork id is the first artwork which is valid for the resulting search (I think). Because in my case I have artwork for '2 Unlimited' and all their tracks are from 'No album' (ie no album named), *everything* that is grouped as 'No album' is given the artwork for 2 Unlimited. This makes looking at some albums a little odd. For example, looking at Madonna's albums I've got 'No album' with a picture of 2 Unlimited beside it. This patch changes the behaviour to remove the artwork IFF the album is the 'No album' entry. Implementation: Before we do the loop processing the result set we remember what the text is for the 'No album' string. When we're returning the artwork identifier we check if the item name matches the 'No album' string. If it doesn't match, we return the artwork. If it does match, we don't return any artwork. Testing: Locate Madonna in the collection. See that the 'No album' entry now has a bland 'note' icon, and not the scary '2 Unlimited' icon.
Created attachment 2911 [details] Patch for Slim/Control/Queries.pm to remove the artwork from 'No album' results.
Looks good. Please commit to trunk.
Justin: Have you had a chance to post this into Trunk yet? if so, what is the change number. Thanks :)
I don't have svn write access; if you want me to commit it I'll be happy to... if I could find out how :-) I assumed that Dean's comment was directed at someone with write access, not at me :-|
sorry for the delay, Justin, and thanks for the patch. Tested and verified as fixed, SC trunk 7.0, change 18791
Verified fixed in 7.0.1 - 19325 r2409 - r23
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1 Please try that version, if you still see the error, then reopen this bug. To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html
Reduce number of active targets for SC