use Modern::Perl;
use C4::Context;
-use C4::Auth qw(checkauth);
+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 = new CGI;
+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 => C4::Context->preference('TrackClicks') } );
-
-if ($uri) {
- if ( $tracker->trackingmethod() eq 'track'
- || $tracker->trackingmethod() eq 'anonymous' )
- {
- my $borrowernumber = 0;
+ { 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};
- }
+ # 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;
- # get borrower info
+ if ( defined($userenv)
+ && ref($userenv) eq 'HASH'
+ && $userenv->{number} )
+ {
+ $borrowernumber = $userenv->{number};
}
- my $biblionumber = $cgi->param('biblionumber') || 0;
- my $itemnumber = $cgi->param('itemnumber') || 0;
- my $record = C4::Biblio::GetMarcBiblio({ biblionumber => $biblionumber });
- my $marc_urls = C4::Biblio::GetMarcUrls($record, C4::Context->preference('marcflavour'));
- if ( grep { /^$uri$/ } map { $_->{MARCURL} } @$marc_urls ) {
- $tracker->trackclick(
- {
- uri => $uri,
- biblionumber => $biblionumber,
- borrowernumber => $borrowernumber,
- itemnumber => $itemnumber
- }
- );
- print $cgi->redirect($uri);
- exit;
- }
+ # get borrower info
}
- else {
- # We have a valid url, but we shouldn't track it, just redirect
+ 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;
}
}
-print $cgi->redirect("/cgi-bin/koha/errors/404.pl"); # escape early
+output_error( $cgi, '404' );
exit;