Issue Details (XML | Word | Printable)

Key: MBS-5029
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Oliver Charles
Reporter: HumHumXX
Votes: 0
Watchers: 0
Operations

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

Cover art from downvoted ASIN still being displayed in the sidebar

Created: 18/Jul/12 03:43 PM   Updated: 17/Sep/12 10:29 AM   Resolved: 17/Sep/12 10:29 AM
Component/s: None
Affects Version/s: None
Fix Version/s: 2012-09-17



Sort Order: Ascending order - Click to sort in descending order
nikki added a comment - 18/Jul/12 10:47 PM

I think this is probably caused by the same problem as MBS-4669 (that the ASIN doesn't get removed from the release_meta table)


Oliver Charles added a comment - 23/Jul/12 08:44 AM

This edit was created/rejected after Wed Jul 27 15:34:41 2011, which is when I added commit c9565ddaa85 to prevent this from happening. However, it still shows artwork because a clean up script was never ran. I'll leave this ticket open and repurpose it to remove those invalid entries.


Oliver Charles added a comment - 23/Jul/12 09:56 AM

Oliver Charles added a comment - 31/Jul/12 04:49 PM

Moving to 2012-08-20 release as we are now in freeze for 2012-08-06.


nikki added a comment - 06/Aug/12 01:01 PM

I couldn't reproduce this with a rejected or cancelled ASIN on test, but I can reproduce it by editing a relationship:

http://test.musicbrainz.org/edit/17593018 moved an ASIN to the release.
http://test.musicbrainz.org/edit/17593019 moved it away again.
http://test.musicbrainz.org/release/cb94bb1c-3c16-48b8-b498-ca4cead8abe0 still has cover art from that ASIN.


Oliver Charles added a comment - 06/Aug/12 01:06 PM

Excellent, thanks for looking into this more!


Oliver Charles added a comment - 06/Aug/12 04:14 PM

This is in review with http://codereview.musicbrainz.org/r/2111/, and available for testing on http://test.musicbrainz.org/


Oliver Charles added a comment - 20/Aug/12 09:45 AM

Oliver Charles added a comment - 20/Aug/12 02:20 PM

Oliver Charles added a comment - 29/Aug/12 10:12 AM

Moving to the next release as we're in freeze now.