From 5fe49c7db1113026d242ad081c6b100fc8bae76e Mon Sep 17 00:00:00 2001 From: David Cook Date: Thu, 10 Mar 2022 04:02:49 +0000 Subject: [PATCH] Bug 30262: Trim whitespace off tracklinks.pl URLs This patch trims whitespace off tracklinks.pl URLs so that the URL is in the same format as the URLs returned by C4::Biblio::GetMarcUrls, which is used by tracklinks.pl internally. Test plan: 0. Apply patch and koha-plack --restart kohadev 1. Go to http://localhost:8081/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=29 2. Add a 856 with $u of 'https://koha-community.org ' (note the space at the end0 3. Save record 4. Change 'TrackClicks' syspref to 'Track' 5. Go to http://localhost:8080/cgi-bin/koha/opac-detail.pl?biblionumber=29 6. Click on the online resource (eg http://localhost:8080/cgi-bin/koha/tracklinks.pl?uri=https%3A%2F%2Fkoha-community.org%20&biblionumber=29 ) 7. Note you are redirected to https://koha-community.org Signed-off-by: Magnus Enger Added the link with a space at the end. Verified that without the patch, clickin on "Click here to access online" gives a 404. After applying the patch the redirect works as expected. Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- opac/tracklinks.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/opac/tracklinks.pl b/opac/tracklinks.pl index 8f8e0ddb78..2470359821 100755 --- a/opac/tracklinks.pl +++ b/opac/tracklinks.pl @@ -33,6 +33,7 @@ my $cgi = CGI->new; my $uri = $cgi->param('uri') || ''; my $biblionumber = $cgi->param('biblionumber');; my $itemnumber = $cgi->param('itemnumber'); +$uri =~ s/^\s+|\s+$//g if $uri; # trim my $tracking_method = C4::Context->preference('TrackClicks'); unless ( $tracking_method ) { -- 2.39.5