|
Hm. I'm not sure I agree. We used to return all the information about the work except for the type. I don't know why that was the case and since I never use that field, I never even noticed. Since we are (or rather, given that we broke it, should be) already returning almost everything about the work (and would have to continue doing so for backwards compatibility), it seems overkill to me to add a new option just to add the type and language. We return all information inside the work table, without joining any other tables. I'm not really sure what to do here, the options are:
I agree both options have pros and cons... so why not control it with `inc`-options? A decision has been reached, work language will always be present. In review with http://codereview.musicbrainz.org/r/2126/ |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
If I get it right no property of works is returned (also not the type), just the relations. E.g. this is the request Picard makes:
http://musicbrainz.org/ws/2/release/0eaeb5bf-e766-424c-b0e9-d27798d722bd?inc=release-groups%2Bmedia%2Brecordings%2Bpuids%2Bartist-credits%2Bartists%2Baliases%2Blabels%2Bisrcs%2Bartist-rels%2Brelease-rels%2Burl-rels%2Brecording-rels%2Bwork-rels%2Brecording-level-rels%2Bwork-level-rels
And this is the XML request for the work of the first track:
http://musicbrainz.org/ws/2/work/36c09693-86e9-4d25-9d82-c1b2e0057582
What would be needed in my opinion is a "works" include parameter for releases, e.g.:
http://musicbrainz.org/ws/2/release/0eaeb5bf-e766-424c-b0e9-d27798d722bd?inc=works
This would be very useful, because
PICARD-242could be implemented then.EDIT: The only property returned is the title.