Change History (4)

comment:1 Changed 10 years ago by DefaultCC Plugin

Cc: teckart@informatik.uni-leipzig.de added

comment:2 Changed 10 years ago by Twan Goosen

This may not be completely feasible because of the changes in the way docId's are generated (see #490)

comment:3 in reply to:  2 Changed 10 years ago by Twan Goosen

Replying to twan.goosen@…:

This may not be completely feasible because of the changes in the way docId's are generated (see #490)

r5202 adds checking against selflink field when looking up document by ID, which should resolve this

comment:4 Changed 10 years ago by Twan Goosen

Resolution: fixed
Status: newclosed

Unfortunately the old VLO does double encoding of docId (once by wicket, and once in ShowResultPage.createBookMarkableLink()), which we probably should not try to detect and resolve (it's messy). We do not and should not double decode the docId parameter so the affected ID's will not lead to a match on ID or SelfLink.

Example case

http://catalog.clarin.eu/vlobeta/?wicket:bookmarkablePage=:eu.clarin.cmdi.vlo.pages.ShowResultPage&fq=resourceClass:Data+Provider&docId=11858/00-229C-0000-0003-1751-5@type%253Ddataprovider%2526id%253D1009

->

http://catalog-clarin.esc.rzg.mpg.de/vlo/record?fq=resourceClass:Data+Provider&docId=11858/00-229C-0000-0003-1751-5@type%253Ddataprovider%2526id%253D1009

->

Document with ID 11858/00-229C-0000-0003-1751-5@type%3Ddataprovider%26id%3D1009 could not be found

intended record:

http://catalog-clarin.esc.rzg.mpg.de/vlo/record?fq=resourceClass:Data+Provider&docId=11858/00-229C-0000-0003-1751-5@type%3Ddataprovider%26id%3D1009

No further action needed, providing the rewrite rules will do their job.

Last edited 10 years ago by Twan Goosen (previous) (diff)
Note: See TracTickets for help on using tickets.