Bug 21251 - tracklinks.pl doesn't work with URLs containing ampersands
Summary: tracklinks.pl doesn't work with URLs containing ampersands
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-21 00:49 UTC by Alex Buckley
Modified: 2019-10-14 19:56 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Buckley 2018-08-21 00:49:12 UTC
When the $uri variable is set from accessing the cgi->param('uri') value retrieved from the URL in tracklinks.pl if the uri to be redirected to contains a ampersand the ampersand and any content after it is ignored and not assigned to the $uri value. 

This is because when values are passed via the URL  the ampersand is the delimiter between the variables and so any values after the ampersand are not considered to belong to uri. 

This results in a 404 error being displayed when track clicks is enabled and the URL in the biblio's 856$u contains an ampersand.
Comment 1 Alex Buckley 2018-08-21 02:35:41 UTC
Turned out to be an issue with the URL encoding in the XSLT