Bug 32336: (QA follow-up) Use $metadata->schema

[srvgit] / catalogue / export.pl

diff --git a/catalogue/export.pl b/catalogue/export.pl
index 3a76084..ff02cea 100755 (executable)
--- a/catalogue/export.pl
+++ b/catalogue/export.pl
@@ -1,63 +1,89 @@
 #!/usr/bin/perl
-use strict;
-#use warnings; FIXME - Bug 2505
+use Modern::Perl;
 
 use C4::Record;
-use C4::Auth;
+use C4::Auth qw( get_template_and_user );
 use C4::Output;
-use C4::Biblio;
-use CGI;
+use C4::Biblio qw( GetMarcControlnumber );
+use CGI qw ( -utf8 );
+use C4::Ris qw( marc2ris );
 
 
 
-my $query = new CGI;
+my $query = CGI->new;
 my ( $template, $loggedinuser, $cookie ) = get_template_and_user({
-                                                                     template_name   => "tools/export.tt",
-                                                                     query           => $query,
-                                                                     type            => "intranet",
-                                                                     authnotrequired => 0,
-                                                                     flagsrequired   => { tools => 'export_catalog' },
-                                                                     debug           => 1,
-                                                                     });
+    template_name   => "tools/export.tt",
+    query           => $query,
+    type            => "intranet",
+    flagsrequired   => { catalogue => 1 },
+    });
 
 my $op=$query->param("op");
 my $format=$query->param("format");
 my $error = '';
 if ($op eq "export") {
-       my $biblionumber = $query->param("bib");
-               if ($biblionumber){
-
-                       my $marc = GetMarcBiblio($biblionumber, 1);
-
-                       if ($format =~ /endnote/) {
-                               $marc = marc2endnote($marc);
-                               $format = 'endnote';
-                       }
-                       elsif ($format =~ /marcxml/) {
-                               $marc = marc2marcxml($marc);
-                       }
-                       elsif ($format=~ /mods/) {
-                               $marc = marc2modsxml($marc);
-                       }
-                       elsif ($format =~ /dc/) {
-                               ($error,$marc) = marc2dcxml($marc,1);
-                               $format = "dublin-core.xml";
-                       }
-                       elsif ($format =~ /marc8/) {
-                               $marc = changeEncoding($marc,"MARC","MARC21","MARC-8");
-                               $marc = $marc->as_usmarc();
-                       }
-                       elsif ($format =~ /utf8/) {
-                               C4::Charset::SetUTF8Flag($marc, 1);
-                               $marc = $marc->as_usmarc();
-                       }
+    my $biblionumber = $query->param("bib");
+        if ($biblionumber){
+            my $file_id = $biblionumber;
+            my $file_pre = "bib-";
+
+            my $biblio = Koha::Biblios->find($biblionumber);
+            my $marc   = $biblio->metadata->record({ embed_items => 1 });
+
+            if( C4::Context->preference('DefaultSaveRecordFileID') eq 'controlnumber' ){
+                my $marcflavour = C4::Context->preference('marcflavour'); #FIXME This option is required but does not change control num behaviour
+                my $control_num = GetMarcControlnumber( $marc, $marcflavour );
+                if( $control_num ){
+                    $file_id = $control_num;
+                    $file_pre = "record-";
+                }
+            }
+
+            if ($format =~ /endnote/) {
+                $marc = marc2endnote($marc);
+                $format = 'endnote';
+            }
+            elsif ($format =~ /marcxml/) {
+                $marc = marc2marcxml($marc);
+                $format = "marcxml";
+            }
+            elsif ($format=~ /mods/) {
+                $marc = marc2modsxml($marc);
+                $format = "mods";
+            }
+            elsif ($format =~ /ris/) {
+                $marc = marc2ris($marc);
+                $format = "ris";
+            }
+            elsif ($format =~ /bibtex/) {
+                $marc = marc2bibtex($marc);
+                $format = "bibtex";
+            }
+            elsif ($format =~ /dc$/) {
+                $marc = marc2dcxml(undef, undef, $biblionumber, $format);
+                $format = "dublin-core.xml";
+            }
+            elsif ($format =~ /marc8/) {
+                $marc = changeEncoding($marc,"MARC","MARC21","MARC-8");
+                $marc = $marc->as_usmarc();
+                $format = "marc8";
+            }
+            elsif ($format =~ /utf8/) {
+                C4::Charset::SetUTF8Flag($marc, 1);
+                $marc = $marc->as_usmarc();
+                $format = "utf8";
+            }
             elsif ($format =~ /marcstd/) {
                 C4::Charset::SetUTF8Flag($marc,1);
                 ($error, $marc) = marc2marc($marc, 'marcstd', C4::Context->preference('marcflavour'));
+                $format = "marcstd";
+            }
+            if ( $format =~ /utf8/ or $format =~ /marcstd/ ) {
+                binmode STDOUT, ':encoding(UTF-8)';
             }
-                       print $query->header(
-                               -type => 'application/octet-stream',
-                -attachment=>"bib-$biblionumber.$format");
-                       print $marc;
-               }
+            print $query->header(
+                -type => 'application/octet-stream',
+                -attachment=>"$file_pre$file_id.$format");
+            print $marc;
+        }
 }