From 0a79c85d06c24a663919c4467aa051cc40c3f05b Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 25 Sep 2014 13:59:44 +0200 Subject: [PATCH] Bug 11876: (qa-followup) some minor changes - GetImportBiblios: prepare the query is useless here - use Modern::Perl - Remove useless indentation level in pl file and redirect to 404 if required params are not given. Signed-off-by: Jonathan Druart --- C4/ImportBatch.pm | 10 +++--- tools/showdiffmarc.pl | 98 +++++++++++++++++++++++++-------------------------- 2 files changed, 54 insertions(+), 54 deletions(-) diff --git a/C4/ImportBatch.pm b/C4/ImportBatch.pm index 51a0585..b8d2560 100644 --- a/C4/ImportBatch.pm +++ b/C4/ImportBatch.pm @@ -1029,11 +1029,11 @@ sub GetImportBiblios { my $dbh = C4::Context->dbh; my $query = "SELECT * FROM import_biblios WHERE import_record_id = ?"; - my $sth = $dbh->prepare_cached($query); - $sth->execute($import_record_id); - my $results = $sth->fetchall_arrayref({}); - $sth->finish(); - return $results; + return $dbh->selectall_arrayref( + $query, + { Slice => {} }, + $import_record_id + ); } diff --git a/tools/showdiffmarc.pl b/tools/showdiffmarc.pl index 3b12795..33881ba 100755 --- a/tools/showdiffmarc.pl +++ b/tools/showdiffmarc.pl @@ -19,8 +19,7 @@ # with Koha; if not, write to the Free Software Foundation, Inc., # 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. -use strict; -use warnings; +use Modern::Perl; # standard or CPAN modules used use CGI qw(:standard); @@ -43,59 +42,60 @@ my $importid = $input->param('importid'); my $batchid = $input->param('batchid'); -if ( $biblionumber and $importid ) { - - # Init vars - my ($recordBiblionumber, $recordImportid, $biblioTitle, $importTitle, $formatted1, $formatted2, - $errorFormatted1, $errorFormatted2); - - - # Prepare template - my ( $template, $loggedinuser, $cookie ) = get_template_and_user( - { - template_name => "tools/showdiffmarc.tt", - query => $input, - type => "intranet", - authnotrequired => 0, - flagsrequired => { tools => 'manage_staged_marc' }, - debug => 1, - } - ); +if ( not $biblionumber or not $importid ) { + print $input->redirect("/cgi-bin/koha/errors/404.pl"); + exit; +} +# Init vars +my ($recordBiblionumber, $recordImportid, $biblioTitle, $importTitle, $formatted1, $formatted2, + $errorFormatted1, $errorFormatted2); - $recordBiblionumber =GetMarcBiblio($biblionumber); - if( $recordBiblionumber ) { - $formatted1 = $recordBiblionumber->as_formatted; - my $data = GetBiblioData($biblionumber); - $biblioTitle = $data->{title}; - } else { - $errorFormatted1 = 1; - } - my ($marc,$encoding) = GetImportRecordMarc($importid); - if( $marc ) { - $recordImportid = MARC::Record->new_from_usmarc($marc) ; - $formatted2 = $recordImportid->as_formatted; - my $biblio = GetImportBiblios($importid); - $importTitle = $biblio->[0]->{'title'}; - } else { - $errorFormatted2 = 1; +# Prepare template +my ( $template, $loggedinuser, $cookie ) = get_template_and_user( + { + template_name => "tools/showdiffmarc.tt", + query => $input, + type => "intranet", + authnotrequired => 0, + flagsrequired => { tools => 'manage_staged_marc' }, + debug => 1, } +); - $template->param( SCRIPT_NAME => $ENV{'SCRIPT_NAME'}, - BIBLIONUMBER => $biblionumber, - IMPORTID => $importid, - BIBLIOTITLE => $biblioTitle, - IMPORTTITLE => $importTitle, - MARC_FORMATTED1 => $formatted1, - MARC_FORMATTED2 => $formatted2, - ERROR_FORMATTED1 => $errorFormatted1, - ERROR_FORMATTED2 => $errorFormatted2, - batchid => $batchid - ); +$recordBiblionumber =GetMarcBiblio($biblionumber); +if( $recordBiblionumber ) { + $formatted1 = $recordBiblionumber->as_formatted; + my $data = GetBiblioData($biblionumber); + $biblioTitle = $data->{title}; +} else { + $errorFormatted1 = 1; +} - output_html_with_http_headers $input, $cookie, $template->output; +my ($marc,$encoding) = GetImportRecordMarc($importid); +if( $marc ) { + $recordImportid = MARC::Record->new_from_usmarc($marc) ; + $formatted2 = $recordImportid->as_formatted; + my $biblio = GetImportBiblios($importid); + $importTitle = $biblio->[0]->{'title'}; } else { - exit; + $errorFormatted2 = 1; } + + +$template->param( + SCRIPT_NAME => $ENV{'SCRIPT_NAME'}, + BIBLIONUMBER => $biblionumber, + IMPORTID => $importid, + BIBLIOTITLE => $biblioTitle, + IMPORTTITLE => $importTitle, + MARC_FORMATTED1 => $formatted1, + MARC_FORMATTED2 => $formatted2, + ERROR_FORMATTED1 => $errorFormatted1, + ERROR_FORMATTED2 => $errorFormatted2, + batchid => $batchid +); + +output_html_with_http_headers $input, $cookie, $template->output; -- 2.1.0