Bug 23329: (follow-up) Allow item URI with a biblionumber parameter

If you pass a URI with a biblionumber without specifying the itemnumber,
tracklinks did not redirect an item URI.

Test plan:
[1] Add URI in an item.
[2] Pass this URI with the itemnumber to tracklinks. Should pass.
[3] Pass this URI with the biblionumber to tracklinks. Should pass now too.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Marcel de Rooy 2019-08-09 09:18:13 +00:00 committed by Lucas Gass
parent 3d39ee8867
commit 7f3b26bc74
1 changed files with 7 additions and 1 deletions

View File

@ -56,8 +56,14 @@ if ($uri && ($biblionumber || $itemnumber) ) {
my $record = C4::Biblio::GetMarcBiblio({ biblionumber => $biblionumber });
my $marc_urls = C4::Biblio::GetMarcUrls($record, C4::Context->preference('marcflavour'));
my $search_crit = { uri => $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( { itemnumber => $itemnumber, uri => $uri } )->count )
|| Koha::Items->search( $search_crit )->count )
{
$tracker->trackclick(
{