Koha/opac/tracklinks.pl
David Cook 5fe49c7db1
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 <magnus@libriotech.no>
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 <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
2022-08-31 08:46:11 -03:00

90 lines
2.9 KiB
Perl
Executable file

#!/usr/bin/perl
# script to log clicks on links to external urls
# Copyright 2012 Catalyst IT
# This file is part of Koha.
#
# Koha is free software; you can redistribute it and/or modify it
# under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 3 of the License, or
# (at your option) any later version.
#
# Koha is distributed in the hope that it will be useful, but
# WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with Koha; if not, see <http://www.gnu.org/licenses>.
use Modern::Perl;
use C4::Context;
use C4::Auth qw( checkauth );
use C4::Biblio;
use C4::Output qw( output_error );
use Koha::Biblios;
use Koha::Items;
use Koha::Linktracker;
use CGI qw ( -utf8 );
use List::MoreUtils qw( any );
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 ) {
output_error( $cgi, '404' );
exit;
}
my $tracker = Koha::Linktracker->new(
{ trackingmethod => $tracking_method } );
if ($uri && ($biblionumber || $itemnumber) ) {
my $borrowernumber;
# we have a uri and we want to track
if ( $tracker->trackingmethod() eq 'track' ) {
my ( $user, $cookie, $sessionID, $flags ) =
checkauth( $cgi, 1, {}, 'opac' );
my $userenv = C4::Context->userenv;
if ( defined($userenv)
&& ref($userenv) eq 'HASH'
&& $userenv->{number} )
{
$borrowernumber = $userenv->{number};
}
# get borrower info
}
my $biblio = Koha::Biblios->find($biblionumber);
my $record = eval{ $biblio->metadata->record };
my $marc_urls = $record ? C4::Biblio::GetMarcUrls($record, C4::Context->preference('marcflavour')) : [];
my $search_crit = { uri => { -like => "%$uri%" } };
if( $itemnumber ) { # itemnumber is leading over biblionumber
$search_crit->{itemnumber} = $itemnumber;
} elsif( $biblionumber ) {
$search_crit->{biblionumber} = $biblionumber;
}
if ( ( any { $_ eq $uri } map { $_->{MARCURL} } @$marc_urls )
|| Koha::Items->search( $search_crit )->count )
{
$tracker->trackclick(
{
uri => $uri,
biblionumber => $biblionumber,
borrowernumber => $borrowernumber,
itemnumber => $itemnumber
}
) if ( $tracker->trackingmethod() eq 'track' || $tracker->trackingmethod() eq 'anonymous' );
print $cgi->redirect($uri);
exit;
}
}
output_error( $cgi, '404' );
exit;