From 725e605f8e2831195a34c5b3ca99d14b5374fe63 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 4 Aug 2020 13:54:28 +0200 Subject: [PATCH] Bug 26133: Remove GetMarc* calls in detail.pl There are several calls in catalogue/detail.pl that can be removed: GetMarcISBN, GetMarcAuthors, GetMarcSubjects, GetMarcSeries, GetMarcUrls and GetMarcHosts They pass a variable to the template that is never used. Test plan: Confirm that this TT variable is never used. Signed-off-by: Martin Renvoize Signed-off-by: Jonathan Druart (cherry picked from commit 5c190388ae1b69889e28c2dae80e0d9ea9baff1a) Signed-off-by: Lucas Gass --- catalogue/detail.pl | 12 ------------ opac/opac-detail.pl | 2 -- 2 files changed, 14 deletions(-) diff --git a/catalogue/detail.pl b/catalogue/detail.pl index c968e33dc9..b0f25d9b03 100755 --- a/catalogue/detail.pl +++ b/catalogue/detail.pl @@ -175,12 +175,6 @@ $template->param( ); my $marcnotesarray = GetMarcNotes( $record, $marcflavour ); -my $marcisbnsarray = GetMarcISBN( $record, $marcflavour ); -my $marcauthorsarray = GetMarcAuthors( $record, $marcflavour ); -my $marcsubjctsarray = GetMarcSubjects( $record, $marcflavour ); -my $marcseriesarray = GetMarcSeries($record,$marcflavour); -my $marcurlsarray = GetMarcUrls ($record,$marcflavour); -my $marchostsarray = GetMarcHosts($record,$marcflavour); my $itemtypes = { map { $_->{itemtype} => $_ } @{ Koha::ItemTypes->search->unblessed } }; @@ -417,12 +411,6 @@ if (scalar(@itemloop) == 0 || scalar(@otheritemloop) == 0) { $template->param( norequests => $norequests ); $template->param( MARCNOTES => $marcnotesarray, - MARCSUBJCTS => $marcsubjctsarray, - MARCAUTHORS => $marcauthorsarray, - MARCSERIES => $marcseriesarray, - MARCURLS => $marcurlsarray, - MARCISBNS => $marcisbnsarray, - MARCHOSTS => $marchostsarray, itemdata_ccode => $itemfields{ccode}, itemdata_enumchron => $itemfields{enumchron}, itemdata_uri => $itemfields{uri}, diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl index 2b238949e1..b6a5b0b1ff 100755 --- a/opac/opac-detail.pl +++ b/opac/opac-detail.pl @@ -783,7 +783,6 @@ if (!C4::Context->preference("OPACXSLTDetailsDisplay") ) { my $marcsubjctsarray = GetMarcSubjects($record,$marcflavour); my $marcseriesarray = GetMarcSeries ($record,$marcflavour); my $marcurlsarray = GetMarcUrls ($record,$marcflavour); - my $marchostsarray = GetMarcHosts($record,$marcflavour); $template->param( MARCSUBJCTS => $marcsubjctsarray, @@ -791,7 +790,6 @@ if (!C4::Context->preference("OPACXSLTDetailsDisplay") ) { MARCSERIES => $marcseriesarray, MARCURLS => $marcurlsarray, MARCISBNS => $marcisbnsarray, - MARCHOSTS => $marchostsarray, ); } -- 2.39.5