Issue Details (XML | Word | Printable)

Key: MBS-5304
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Nicolás Tamargo
Reporter: Jacob Brett
Votes: 0
Watchers: 0
Operations

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

"Other Databases" whitelist: "Trove" ammendment

Created: 13/Sep/12 12:57 PM   Updated: 28/Jan/13 10:15 AM   Resolved: 28/Jan/13 10:15 AM
Component/s: User interface
Affects Version/s: None
Fix Version/s: 2013-01-28


 Description  « Hide

As mentioned in MBS-4888 [1]:

Currently, query strings are stripped out of all relationship types (instead of excepting Release-URL relationships, as indicated in the issue description). This in itself is a bug, though there would generally still be an issue with Release-URL relationships:

If one, for example, navigated to the NLA work page http://trove.nla.gov.au/work/9438679 (to us, a Release Group), then navigated to the first listed edition (to us, a Release), the edition URL would look like this: http://trove.nla.gov.au/work/9438679?q&sort=holdings+desc&_=1346311653315&versionId=44854390 - "selectedversion=NBD11701020" won't actually be appended to the URL, unless it is the only edition of the work. Aside from that, it doesn't seem to be a regular format (see: http://trove.nla.gov.au/work/163710586?versionId=178480216 ).

Thus, I think it's better to stick to the persistent citation URLs, e.g. http://nla.gov.au/anbd.bib-an10936546 for all entities (except works, which don't have a persistent ID).

[1] http://tickets.musicbrainz.org/browse/STYLE-118?focusedCommentId=24013&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_24013

URL formats:

Label, Artist: http://nla.gov.au/nla.party-[integer]
e.g. http://nla.gov.au/nla.party-1448035

Release Group: http://trove.nla.gov.au/work/[integer]
e.g. http://trove.nla.gov.au/work/9438679

Release: http://nla.gov.au/anbd.bib-an[integer]
e.g. http://nla.gov.au/anbd.bib-an11701020



Sort Order: Ascending order - Click to sort in descending order
Jacob Brett added a comment - 13/Sep/12 01:00 PM

You may want to hold reviewing this ticket for a day or two, so I can double-check the URL syntax (Trove is currently down).


Jacob Brett added a comment - 05/Oct/12 02:23 AM

It all checks out for the above examples--carry on. Thanks!