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
HumHumXX made changes - 18/Jul/12 03:44 PM
Field Original Value New Value
Description Release: http://musicbrainz.org/release/72a1e1c2-3df4-4c89-84b5-4c9f62a0ceff

Edit: Edit #14463045 		Release: http://musicbrainz.org/release/72a1e1c2-3df4-4c89-84b5-4c9f62a0ceff

Edit: http://musicbrainz.org/edit/14463045
[ Permalink | « Hide ]
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)


[ Permalink | « Hide ]
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 made changes - 23/Jul/12 09:56 AM
Status Open [ 1 ] Review Submitted [ 5 ]
[ Permalink | « Hide ]
Oliver Charles added a comment - 23/Jul/12 09:56 AM

This is in review at http://codereview.musicbrainz.org/r/2079/


Oliver Charles made changes - 23/Jul/12 09:56 AM
Fix Version/s Bug fixes, 2012-08-06 [ 10149 ]
Oliver Charles made changes - 23/Jul/12 09:56 AM
Assignee Oliver Charles [ acid2 ]
[ Permalink | « Hide ]
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.


Oliver Charles made changes - 31/Jul/12 04:49 PM
Fix Version/s Bug fixes, 2012-08-20 [ 10150 ]
Fix Version/s Bug fixes, 2012-08-06 [ 10149 ]
Oliver Charles made changes - 06/Aug/12 10:14 AM
Status Review Submitted [ 5 ] Reopened [ 4 ]
Oliver Charles made changes - 06/Aug/12 10:15 AM
Fix Version/s Bug fixes, 2012-09-03 [ 10151 ]
Fix Version/s Bug fixes, 2012-08-20 [ 10150 ]
[ Permalink | « Hide ]
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.


[ Permalink | « Hide ]
Oliver Charles added a comment - 06/Aug/12 01:06 PM

Excellent, thanks for looking into this more!


Oliver Charles made changes - 06/Aug/12 01:06 PM
Fix Version/s Bug fixes, 2012-08-20 [ 10150 ]
Fix Version/s Bug fixes, 2012-09-03 [ 10151 ]
Oliver Charles made changes - 06/Aug/12 04:13 PM
Status Reopened [ 4 ] Review Submitted [ 5 ]
[ Permalink | « Hide ]
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 made changes - 10/Aug/12 10:12 AM
Status Review Submitted [ 5 ] In Beta Testing [ 10002 ]
Oliver Charles made changes - 20/Aug/12 09:44 AM
Status In Beta Testing [ 10002 ] Reopened [ 4 ]
[ Permalink | « Hide ]
Oliver Charles added a comment - 20/Aug/12 09:45 AM

This causes http://tickets.musicbrainz.org/browse/MBS-5136


Oliver Charles made changes - 20/Aug/12 09:45 AM
Fix Version/s Bug fixes, 2012-09-03 [ 10151 ]
Fix Version/s Bug fixes, 2012-08-20 [ 10150 ]
Oliver Charles made changes - 20/Aug/12 02:20 PM
Status Reopened [ 4 ] Review Submitted [ 5 ]
[ Permalink | « Hide ]
Oliver Charles added a comment - 20/Aug/12 02:20 PM

In review with http://codereview.musicbrainz.org/r/2111/ again


[ Permalink | « Hide ]
Oliver Charles added a comment - 29/Aug/12 10:12 AM

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


Oliver Charles made changes - 29/Aug/12 10:12 AM
Fix Version/s 2012-09-17 [ 10152 ]
Fix Version/s Bug fixes, 2012-09-03 [ 10151 ]
Oliver Charles made changes - 03/Sep/12 11:23 AM
Status Review Submitted [ 5 ] In Beta Testing [ 10002 ]
Oliver Charles made changes - 17/Sep/12 10:29 AM
Status In Beta Testing [ 10002 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Atlassian JIRA (v4.0.1#471) | Report a problem | Request a feature | Contact administrators
Powered by a free Atlassian JIRA open source license for MetaBrainz Foundation. Try JIRA - bug tracking software for your team.