Example of need : ISWC split out in 2011-06-24 16:58:39 CEST http://musicbrainz.org/edit/14461700
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 to track that issue.
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.
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.
As long as we do not add extra steps (clicks, waits) to enter work. No extra steps like the label lookup for instance.
@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.
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.
The source I guess is to tell whether ISRC comes from CD or online DB.
And yes, a simple text field would be enough for ISWC in add work.
You say the treatment should be same ISWC and ISRC. I add IPI same same too.
For WS, <iswc-list> seems good for me.
Sorry for not commenting earlier:
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?
The schema changes for this ticket are complete, and think the UI mockups clearly show how the UI will be updated.
This ticket's schema changes are still not complete. Nudge! Please finish updating: http://wiki.musicbrainz.org/User:RobertKaye/Schema_Change_Release_May_2012
I've also seen times where of top work would have one ISWC and we could have sub works in MB for convenince purpose and they will all have the same ISWC (I don't remember my example but I think I saw that once, on a CD).
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:
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.
Nikki, I believe addiswc.png is your suggestion ?
I want to state that we do need multiple ISWC for some works anyway.
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.
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), I wish we could keep one ISWC field as I almost always create works with ISWC.
We could instead match multiple ISWC in that field (.match(new RegExp(iswc_pattern, "gi"))).
Specify exactly what the user interface changes will be. Mock ups would be great. Due date: April 2
Specify exactly what the database changes will be.
CREATE TABLE iswc
created TIMESTAMP WITH TIME ZONE
This ticket is under consideration for the May 15h schema change release. Please do the following:
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 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?
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.
+1, for examples use tag "multiple iswc", http://musicbrainz.org/tag/multiple%20iswc