Issue Details (XML | Word | Printable)

Key: MBS-2548
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Johannes Weißl
Reporter: Nicolás Tamargo
Votes: 7
Watchers: 2

If you were logged in you would be able to see more operations.
MusicBrainz Server

Client-Side Discogs URL validation (artist vs. label vs. release vs. master)

Created: 04/Jun/11 09:55 PM   Updated: 26/Jul/11 04:55 PM   Resolved: 25/Jul/11 11:52 AM
Component/s: Edit system, User interface
Affects Version/s: NGS - Final
Fix Version/s: Bug fixes, 2011-07-25

Issue Links:

 Description  « Hide

Import of

"Due to some quirks in the UI, it's very easy and common for Discogs URLs for artists to be added to releases, labels to releases, etc etc. I've done it myself before, users will continue to do it.

It would seem to me to be relatively easy to add either JavaScript or server-side validation of these URLs based on the "/artist/" , "/label/", "/release/" parts of the URL with respect to the entity in MB the AR is being added to."

I would add that, in NGS, it might also be really useful to block anything but /master/ at RG level.

Sort Order: Ascending order - Click to sort in descending order
Johannes Weißl added a comment - 27/Jun/11 01:03 AM

I think both server side and javascript verification is necessary... the javascript doesn't always work...

Johannes Weißl added a comment - 21/Jul/11 02:04 AM

I submitted a patch for review... it is only client side so far, but maybe this helps already?