Bug 14001: Inventory has bad $_ references

[koha.git] / tools / showdiffmarc.pl

diff --git a/tools/showdiffmarc.pl b/tools/showdiffmarc.pl
index 3b12795..488f39e 100755 (executable)
--- 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,59 @@ 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, 'embed_items');
+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;
+if( $importid ) {
+    $recordImportid = C4::ImportBatch::GetRecordFromImportBiblio( $importid, 'embed_items' );
+    $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;