Bug 32842: Fix cataloguing/value_builder/unimarc_field_123j.pl

[koha-ffzg.git] / svc / bib

diff --git a/svc/bib b/svc/bib
index d7cd169..d25d071 100755 (executable)
--- a/svc/bib
+++ b/svc/bib
@@ -5,30 +5,30 @@
 #
 # This file is part of Koha.
 #
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 2 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
 #
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along
-# with Koha; if not, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
 #
 
-use strict;
-use warnings;
+use Modern::Perl;
 
-use CGI;
-use C4::Auth qw/check_api_auth/;
-use C4::Biblio;
+use CGI qw ( -utf8 );
+use C4::Auth qw( check_api_auth );
+use C4::Biblio qw( GetFrameworkCode );
 use C4::Items;
 use XML::Simple;
+use Koha::Biblios;
 
-my $query = new CGI;
+my $query = CGI->new;
 binmode STDOUT, ':encoding(UTF-8)';
 
 my ($status, $cookie, $sessionID) = check_api_auth($query, { editcatalogue => 'edit_catalogue'} );
@@ -66,9 +66,10 @@ exit 0;
 sub fetch_bib {
     my $query = shift;
     my $biblionumber = shift;
-    my $record = GetMarcBiblio( $biblionumber, $query->url_param('items') );
+    my $biblio = Koha::Biblios->find( $biblionumber );
+    my $record = $biblio->metadata->record({ embed_items => scalar $query->param('items') });
     if  (defined $record) {
-        print $query->header(-type => 'text/xml');
+        print $query->header(-type => 'text/xml',-charset => 'utf-8',);
         print $record->as_xml_record();
     } else {
         print $query->header(-type => 'text/xml', -status => '404 Not Found');
@@ -78,7 +79,9 @@ sub fetch_bib {
 sub update_bib {
     my $query = shift;
     my $biblionumber = shift;
-    my $old_record = GetMarcBiblio($biblionumber);
+    my $biblio = Koha::Biblios->find( $biblionumber );
+    my $old_record = $biblio->metadata->record;
+    my $frameworkcode = $query->url_param('frameworkcode') // GetFrameworkCode($biblionumber);
     unless  (defined $old_record) {
         print $query->header(-type => 'text/xml', -status => '404 Not Found');
         return;
@@ -86,9 +89,9 @@ sub update_bib {
 
     my $result = {};
     my $inxml = $query->param('POSTDATA');
-    print $query->header(-type => 'text/xml');
+    print $query->header(-type => 'text/xml', -charset => 'utf-8');
 
-    my $record = eval {MARC::Record::new_from_xml( $inxml, "utf8", C4::Context->preference('marcflavour'))};
+    my $record = eval {MARC::Record::new_from_xml( $inxml, "UTF-8", C4::Context->preference('marcflavour'))};
     my $do_not_escape = 0;
     if ($@) {
         $result->{'status'} = "failed";
@@ -96,7 +99,7 @@ sub update_bib {
     } else {
         my $fullrecord = $record->clone();
         my ( $itemtag, $itemsubfield ) =
-          GetMarcFromKohaField( "items.itemnumber", '' );
+          C4::Biblio::GetMarcFromKohaField( "items.itemnumber" );
 
         # delete any item tags
         foreach my $field ( $record->field($itemtag) ) {
@@ -107,14 +110,14 @@ sub update_bib {
             foreach my $field ( $fullrecord->field($itemtag) ) {
                 my $one_item_record = $record->clone();
                 $one_item_record->add_fields($field);
-                ModItemFromMarc( $one_item_record, $biblionumber,
+                C4::Items::ModItemFromMarc( $one_item_record, $biblionumber,
                     $field->subfield($itemsubfield) );
             }
         }
 
-        ModBiblio( $record, $biblionumber, '' );
-        my $new_record =
-          GetMarcBiblio( $biblionumber, $query->url_param('items') );
+        C4::Biblio::ModBiblio( $record, $biblionumber, $frameworkcode );
+        my $biblio = Koha::Biblios->find( $biblionumber );
+        my $new_record = $biblio->metadata->record({ embed_items => scalar $query->url_param('items') });
 
         $result->{'status'} = "ok";
         $result->{'biblionumber'} = $biblionumber;
@@ -130,7 +133,7 @@ sub update_bib {
 sub delete_bib {
     my $query = shift;
     my $biblionumber = shift;
-    my $error = DelBiblio($biblionumber);
+    my $error = C4::Biblio::DelBiblio($biblionumber);
 
     if (defined $error) {
         print $query->header(-type => 'text/xml', -status => '400 Bad request');