|
voiceinsideyou made changes - 26/Jul/11 04:59 PM
If this gets implemented, I would suggest that the webservice be updated to treat works/iswcs precisely analogously to recordings/isrcs. A work would have an iswc-list element, which would itself contain iswc elements. It would be possible to lookup an iswc directly, e.g. /ws/2/iswc/ABCD1234, which would return an iswc element, which would itself contain a work-list element.
Christopher Key made changes - 15/Aug/11 06:19 PM
Oliver Charles made changes - 18/Nov/11 05:19 PM
Oliver Charles made changes - 20/Feb/12 01:13 PM
Johannes Weißl made changes - 19/Mar/12 12:07 PM
This looks really straight-forward, since we already have multiple ISRC codes for recordings, so this should work quite symmetrical. Is there anyone who objects to this feature? I agree with this server feature, but I would like to see a style guideline enforcing the definition of a work, so we know when 2 works can be merged. Right now 2 different ISWC codes for one work is a strong indication that 2 different works should exist in MB. This ticket is under consideration for the May 15h schema change release. Please do the following:
Robert Kaye made changes - 19/Mar/12 09:17 PM
Johannes Weißl made changes - 20/Mar/12 03:32 AM
Johannes Weißl made changes - 20/Mar/12 03:39 AM
Johannes Weißl made changes - 20/Mar/12 03:50 AM
Johannes Weißl made changes - 20/Mar/12 04:22 AM
Johannes Weißl made changes - 20/Mar/12 04:31 AM
Johannes Weißl made changes - 20/Mar/12 04:49 AM
Thanks for all you great work. I like all the pics except I think it's a pity if we loose the ISWC in work creation page (mbs-2885-edit-work.png We could instead match multiple ISWC in that field (.match(new RegExp(iswc_pattern, "gi"))). I also don't want to lose the ability to add an ISWC while creating or editing a work (and I don't need multiple ISWCs, so it seems the proposal currently has nothing but downsides for me). I've attached my suggestion for the interface. It's basically the same as what's used in the release editor for multiple labels. This suggestion would also mean that no "Add ISWC" link would be needed in the sidebar, and no "(remove)" link would be needed next to the ISWCs in the sidebar, because we would do it all via the edit tab.
nikki made changes - 20/Mar/12 11:16 AM
Nikki, I believe addiswc.png The iswc table looks good, but I'm curious about unique constraints. Presumably you want the id column to be the primary key, but what about the iswc column? Is there ever a case where a ISWC is used by >1 works? If not, could we use iswc as the primary key, and if we need that then we could extend it to work, iswc. Also, all columns are missing NOT NULL and foreign key constraints, and presumably we want a CHECK constraint on iswc to make sure it's stored consistently. I'm not sure whether the source column needs to be there, because we don't use source on isrc either. Yes, there are cases where an ISWC is used by more than one work. For me, they're ones where JASRAC has not made a distinction between original and translated versions of a song. Here's an example: This ticket's schema changes are still not complete. Nudge! Please finish updating: http://wiki.musicbrainz.org/User:RobertKaye/Schema_Change_Release_May_2012 The schema changes for this ticket are complete, and think the UI mockups clearly show how the UI will be updated. The changes for the web service need to be discussed for this ticket. Are we going to display these ISWCs in a new <iswc-list> element? Sorry for not commenting earlier:
The source I guess is to tell whether ISRC comes from CD or online DB. For WS, <iswc-list> seems good for me. @hrglgrmpf: I do agree that they should eventually be the same, but they are already handled differently (ISWCs are added/edited via the edit tab, ISRCs aren't), so I don't think it's a big problem for them to continue to being different for a bit longer, until the UI for ISRCs is updated. I don't really like the idea of having just one ISWC field on the "Create Work" page (what about when editing a work?), that sounds more confusing than having all the editing for ISWCs in one place and it would still mean that the UI for ISRCs and ISWCs is different. Ollie also already said on IRC at some point that he would rather add more JS (i.e. add label-like handling) than add more pages (i.e. ISRC-like handling), so it seems like the devs are fine with the suggestion if other people agree.
As long as we do not add extra steps (clicks, waits) to enter work. No extra steps like the label lookup for instance. nikki is right, I was mostly saying that we already require JS for the release editor, so we might as well embrace it and add a much more pleasing UI.
Oliver Charles made changes - 10/Apr/12 01:04 PM
Oliver Charles made changes - 10/Apr/12 01:04 PM
Oliver Charles made changes - 16/Apr/12 04:11 PM
Oliver Charles made changes - 16/Apr/12 04:53 PM
Oliver Charles made changes - 15/May/12 07:38 PM
The change made to the webservice for this ticket breaks backwards-compatibility with existing applications that use the webservice to lookup iswcs, I've opened http://tickets.musicbrainz.org/browse/MBS-4752
Calvin Walton made changes - 20/May/12 08:50 PM
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
+1, for examples use tag "multiple iswc", http://musicbrainz.org/tag/multiple%20iswc