Posted by thumb on September 22, 2010 at 2:38am
2 followers
Jump to:
| Project: | Station |
| Version: | 6.x-2.x-dev |
| Component: | Catalog |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Issue Summary
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.
| Attachment | Size |
|---|---|
| station_catalog.module.patch | 1.02 KB |
Comments
#1
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().
#2
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.
#3
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.