Hello,

Attached is a patch which populates the album Label field with label data available in the already fetched Music Brainz release data object. I hate looking up labels. Would love to see this make it in before 6.x.2.0 final release.

CommentFileSizeAuthor
station_catalog.module.patch1.02 KBthumb

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Patch (to be ported) » Needs work

I've not looked through this code in a long time, there are several things commented out that hint that this was intended, especially the commented out call to _station_catalog_musicbrainz_label().

thumb’s picture

thumb commented

Yeah, not sure whether _station_catalog_musicbrainz_label() is really needed by most users. It looks like a separate label-specific query provides much more detailed info on the label, including history, subsidiary status, etc.

http://wiki.musicbrainz.org/XMLWebService#The_Label_Class

If we don't think anyone needs/wants this, perhaps _station_catalog_musicbrainz_label() should be dropped.

refreshingapathy’s picture

refreshingapathy commented

Reviewing the MusicBrainz documentation about the sheer volume of information that the call would produce... I think that much label data would probably be overkill for most people.