Issue Details (XML | Word | Printable)

Key: MBS-4476
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Kuno Woudt
Reporter: nikki
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
MusicBrainz Server

Provide indication of non-front cover art

Created: 23/Mar/12 09:20 PM   Updated: 17/Apr/12 04:26 PM   Resolved: 17/Apr/12 04:26 PM
Component/s: None
Affects Version/s: None
Fix Version/s: Bug fixes, 2012-04-23


 Description  « Hide

quoting what I said on MBS-4462 "We should also indicate that we do have other covers. Otherwise there's no way of distinguishing a release with no cover art from a release with tons of cover art except none of it has been marked as front."



Sort Order: Ascending order - Click to sort in descending order
Kuno Woudt added a comment - 28/Mar/12 08:58 PM

I see "cover_art_presence" in release_meta, which would be perfect for this. Except it is always sent "absent", there doesn't seem to be code setting it anywhere. Should cover_art_presence be set in the cover art edits and did we not implement this yet? Or is this field some old relic which shouldn't be used?


Kuno Woudt added a comment - 28/Mar/12 08:58 PM

(assigning to oliver as he should be able to answer my question above


Oliver Charles added a comment - 29/Mar/12 10:31 AM - edited

Hmm, you don't have the update_release_coverart trigger on cover_art_archive.cover_art? That's in 20120216-caa-metadata.sql.


Kuno Woudt added a comment - 29/Mar/12 02:25 PM - edited

fixed in commit 6f98a3d56db1dc125724a87231446b7be1991e26

but the messages I used may be a bit subtle, please suggest improvements:

https://github.com/metabrainz/musicbrainz-server/blob/caa/root/release/sidebar.tt


nikki added a comment - 29/Mar/12 06:55 PM

I would suggest a "View other images" link to the cover art tab after "No front cover image available"


Kuno Woudt added a comment - 03/Apr/12 03:13 PM

reopened to make sure the previous comment from nikki is implemented.


Oliver Charles added a comment - 17/Apr/12 04:26 PM

Fixed in 940e338.