Issue Details (XML | Word | Printable)

Key: MBS-4536
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Ian McEwen
Reporter: nikki
Votes: 6
Watchers: 2
Operations

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

Include the presence of cover art in the webservice

Created: 09/Apr/12 01:14 PM   Updated: Monday 01:06 PM  Due: 15/Jan/13   Resolved: 28/Jan/13 10:15 AM
Component/s: None
Affects Version/s: None
Fix Version/s: 2013-01-28

Issue Links:
Depends
 
Relates
 


 Description  « Hide

We know whether a release has cover art in the cover art archive. I don't see why we shouldn't share that information with people when they request releases via the webservice, instead of making everyone do extra requests to find it out themselves.

An example where it would be useful is Picard: https://github.com/musicbrainz/picard/pull/31



Sort Order: Ascending order - Click to sort in descending order
Oliver Charles added a comment - 29/Apr/12 04:10 PM

What type of data would people like to see? There are options for:

  • has cover art: True if the release has some cover art, regardless of type
  • amount of cover art: The amount of cover art images
  • Has front: If /front will resolve
  • Has back: If /back will resolve

If we decide what information is required, I can make this happen.


nikki added a comment - 29/Apr/12 04:20 PM

IMO "has cover art" is the most important, as that allows you to completely skip the cover art step if there's no cover art.
I can imagine that "has front" would be nice for people who are only interested in the front cover though.


Robert Kaye added a comment - 01/May/12 10:40 PM

+1 to this.


nikki added a comment - 01/May/12 10:41 PM

On IRC the other day, ijabz said pretty much the same as I did.


Robert Kaye added a comment - 01/May/12 11:18 PM

Nikki: Oh good. He was the other person I wanted to get feedback from. Lets bake it so!


Oliver Charles added a comment - 04/May/12 09:59 AM

Moving improvements out of upcoming version


Oliver Charles added a comment - 15/Oct/12 07:40 PM

Agreed to fix within 3 months.


Ian McEwen added a comment - 20/Oct/12 09:20 AM

http://codereview.musicbrainz.org/r/2410/

Interested parties: please go look at that. I'd like to make sure this is sufficient for what people want/need and that I'm doing things right (this is my first patch touching the webservice); the more eyes the better!


Wieland Hoffmann added a comment - 10/Dec/12 10:49 PM

The review has been discarded but the ticket is still "Review Submitted" - what's the real status?


Ian McEwen added a comment - 10/Dec/12 10:58 PM

Review was discarded only temporarily since I was taking a bit of a hiatus from the main flow of MB last week, but I won't reopen it until I've done a bit more work and it needs further review. So, I'm reopening this for now.



Ian McEwen added a comment - 15/Jan/13 07:41 PM

Correct; the latter is a search and thus out of my control. So presumably if that's desired we need a SEARCH ticket.