#!/usr/bin/perl
-
# Copyright 2009 BibLibre
# Parts Copyright Catalyst IT 2011
#
# 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 Modern::Perl;
+use CGI qw ( -utf8 );
-use strict;
-#use warnings; FIXME - Bug 2505
-use CGI;
use C4::Output;
use C4::Auth;
use C4::Items;
use Koha::MetadataRecord;
my $input = new CGI;
-my @biblionumber = $input->param('biblionumber');
+my @biblionumbers = $input->param('biblionumber');
my $merge = $input->param('merge');
my @errors;
if ($merge) {
my $dbh = C4::Context->dbh;
- my $sth;
# Creating a new record from the html code
- my $record = TransformHtmlToMarc( $input );
- my $tobiblio = $input->param('biblio1');
- my $frombiblio = $input->param('biblio2');
+ my $record = TransformHtmlToMarc( $input, 1 );
+ my $ref_biblionumber = $input->param('ref_biblionumber');
+ @biblionumbers = grep { $_ != $ref_biblionumber } @biblionumbers;
+
+ # prepare report
+ my @report_records;
+ my $report_fields_str = $input->param('report_fields');
+ $report_fields_str ||= C4::Context->preference('MergeReportFields');
+ my @report_fields;
+ foreach my $field_str (split /,/, $report_fields_str) {
+ if ($field_str =~ /(\d{3})([0-9a-z]*)/) {
+ my ($field, $subfields) = ($1, $2);
+ push @report_fields, {
+ tag => $field,
+ subfields => [ split //, $subfields ]
+ }
+ }
+ }
# Rewriting the leader
- $record->leader(GetMarcBiblio($tobiblio)->leader());
+ $record->leader(GetMarcBiblio($ref_biblionumber)->leader());
my $frameworkcode = $input->param('frameworkcode');
my @notmoveditems;
# Modifying the reference record
- ModBiblio($record, $tobiblio, $frameworkcode);
+ ModBiblio($record, $ref_biblionumber, $frameworkcode);
# Moving items from the other record to the reference record
- # Also moving orders from the other record to the reference record, only if the order is linked to an item of the other record
- my $itemnumbers = get_itemnumbers_of($frombiblio);
- foreach my $itloop ($itemnumbers->{$frombiblio}) {
- foreach my $itemnumber (@$itloop) {
- my $res = MoveItemFromBiblio($itemnumber, $frombiblio, $tobiblio);
+ foreach my $biblionumber (@biblionumbers) {
+ my $itemnumbers = get_itemnumbers_of($biblionumber);
+ foreach my $itemnumber (@{ $itemnumbers->{$biblionumber} }) {
+ my $res = MoveItemFromBiblio($itemnumber, $biblionumber, $ref_biblionumber);
if (not defined $res) {
push @notmoveditems, $itemnumber;
}
push @errors, { code => "CANNOT_MOVE", value => $itemlist };
}
- # Moving subscriptions from the other record to the reference record
- my $subcount = CountSubscriptionFromBiblionumber($frombiblio);
- if ($subcount > 0) {
- $sth = $dbh->prepare("UPDATE subscription SET biblionumber = ? WHERE biblionumber = ?");
- $sth->execute($tobiblio, $frombiblio);
-
- $sth = $dbh->prepare("UPDATE subscriptionhistory SET biblionumber = ? WHERE biblionumber = ?");
- $sth->execute($tobiblio, $frombiblio);
-
+ my $sth_subscription = $dbh->prepare("
+ UPDATE subscription SET biblionumber = ? WHERE biblionumber = ?
+ ");
+ my $sth_subscriptionhistory = $dbh->prepare("
+ UPDATE subscriptionhistory SET biblionumber = ? WHERE biblionumber = ?
+ ");
+ my $sth_serial = $dbh->prepare("
+ UPDATE serial SET biblionumber = ? WHERE biblionumber = ?
+ ");
+
+ my $report_header = {};
+ foreach my $biblionumber ($ref_biblionumber, @biblionumbers) {
+ # build report
+ my $marcrecord = GetMarcBiblio($biblionumber);
+ my %report_record = (
+ biblionumber => $biblionumber,
+ fields => {},
+ );
+ foreach my $field (@report_fields) {
+ my @marcfields = $marcrecord->field($field->{tag});
+ foreach my $marcfield (@marcfields) {
+ my $tag = $marcfield->tag();
+ if (scalar @{$field->{subfields}}) {
+ foreach my $subfield (@{$field->{subfields}}) {
+ my @values = $marcfield->subfield($subfield);
+ $report_header->{ $tag . $subfield } = 1;
+ push @{ $report_record{fields}->{$tag . $subfield} }, @values;
+ }
+ } elsif ($field->{tag} gt '009') {
+ my @marcsubfields = $marcfield->subfields();
+ foreach my $marcsubfield (@marcsubfields) {
+ my ($code, $value) = @$marcsubfield;
+ $report_header->{ $tag . $code } = 1;
+ push @{ $report_record{fields}->{ $tag . $code } }, $value;
+ }
+ } else {
+ $report_header->{ $tag . '@' } = 1;
+ push @{ $report_record{fields}->{ $tag .'@' } }, $marcfield->data();
+ }
+ }
+ }
+ push @report_records, \%report_record;
}
- # Moving serials
- $sth = $dbh->prepare("UPDATE serial SET biblionumber = ? WHERE biblionumber = ?");
- $sth->execute($tobiblio, $frombiblio);
+ foreach my $biblionumber (@biblionumbers) {
+ # Moving subscriptions from the other record to the reference record
+ my $subcount = CountSubscriptionFromBiblionumber($biblionumber);
+ if ($subcount > 0) {
+ $sth_subscription->execute($ref_biblionumber, $biblionumber);
+ $sth_subscriptionhistory->execute($ref_biblionumber, $biblionumber);
+ }
- # TODO : Moving reserves
+ # Moving serials
+ $sth_serial->execute($ref_biblionumber, $biblionumber);
# Moving orders (orders linked to items of frombiblio have already been moved by MoveItemFromBiblio)
- my @allorders = GetOrdersByBiblionumber($frombiblio);
- my @tobiblioitem = GetBiblioItemByBiblioNumber ($tobiblio);
+ my @allorders = GetOrdersByBiblionumber($biblionumber);
+ my @tobiblioitem = GetBiblioItemByBiblioNumber ($ref_biblionumber);
my $tobiblioitem_biblioitemnumber = $tobiblioitem [0]-> {biblioitemnumber };
foreach my $myorder (@allorders) {
- $myorder->{'biblionumber'} = $tobiblio;
+ $myorder->{'biblionumber'} = $ref_biblionumber;
ModOrder ($myorder);
# TODO : add error control (in ModOrder?)
}
- # Deleting the other record
+ # Deleting the other records
if (scalar(@errors) == 0) {
- # Move holds
- MergeHolds($dbh,$tobiblio,$frombiblio);
- my $error = DelBiblio($frombiblio);
- push @errors, $error if ($error);
+ # Move holds
+ MergeHolds($dbh, $ref_biblionumber, $biblionumber);
+ my $error = DelBiblio($biblionumber);
+ push @errors, $error if ($error);
}
+}
# Parameters
$template->param(
- result => 1,
- biblio1 => $input->param('biblio1')
+ result => 1,
+ report_records => \@report_records,
+ report_header => $report_header,
+ ref_biblionumber => $input->param('ref_biblionumber')
);
#-------------------------
# Show records to merge
#-------------------------
} else {
- my $mergereference = $input->param('mergereference');
- my $biblionumber = $input->param('biblionumber');
-
- if (scalar(@biblionumber) != 2) {
- push @errors, { code => "WRONG_COUNT", value => scalar(@biblionumber) };
- }
- else {
- my $data1 = GetBiblioData($biblionumber[0]);
- my $record1 = GetMarcBiblio($biblionumber[0]);
-
- my $data2 = GetBiblioData($biblionumber[1]);
- my $record2 = GetMarcBiblio($biblionumber[1]);
-
- # Checks if both records use the same framework
- my $frameworkcode1 = &GetFrameworkCode($biblionumber[0]);
- my $frameworkcode2 = &GetFrameworkCode($biblionumber[1]);
-
-
- my $subtitle1 = GetRecordValue('subtitle', $record1, $frameworkcode1);
- my $subtitle2 = GetRecordValue('subtitle', $record2, $frameworkcode1);
-
- if ($mergereference) {
-
- my $framework;
- if ($frameworkcode1 ne $frameworkcode2) {
- $framework = $input->param('frameworkcode')
- or push @errors, "Famework not selected.";
+ my $ref_biblionumber = $input->param('ref_biblionumber');
+
+ if ($ref_biblionumber) {
+ my $framework = $input->param('frameworkcode');
+ $framework //= GetFrameworkCode($ref_biblionumber);
+
+ # Getting MARC Structure
+ my $tagslib = GetMarcStructure(1, $framework);
+
+ my $marcflavour = lc(C4::Context->preference('marcflavour'));
+
+ # Creating a loop for display
+ my @records;
+ foreach my $biblionumber (@biblionumbers) {
+ my $marcrecord = GetMarcBiblio($biblionumber);
+ my $frameworkcode = GetFrameworkCode($biblionumber);
+ my $recordObj = new Koha::MetadataRecord({'record' => $marcrecord, schema => $marcflavour});
+ my $record = {
+ recordid => $biblionumber,
+ record => $marcrecord,
+ frameworkcode => $frameworkcode,
+ display => $recordObj->createMergeHash($tagslib),
+ };
+ if ($ref_biblionumber and $ref_biblionumber == $biblionumber) {
+ $record->{reference} = 1;
+ $template->param(ref_record => $record);
+ unshift @records, $record;
} else {
- $framework = $frameworkcode1;
+ push @records, $record;
}
-
- # Getting MARC Structure
- my $tagslib = GetMarcStructure(1, $framework);
-
- my $notreference = ($biblionumber[0] == $mergereference) ? $biblionumber[1] : $biblionumber[0];
-
- # Creating a loop for display
-
- my $recordObj1 = new Koha::MetadataRecord({ 'record' => GetMarcBiblio($mergereference), 'schema' => lc C4::Context->preference('marcflavour') });
- my $recordObj2 = new Koha::MetadataRecord({ 'record' => GetMarcBiblio($notreference), 'schema' => lc C4::Context->preference('marcflavour') });
-
- my @record1 = $recordObj1->createMergeHash($tagslib);
- my @record2 = $recordObj2->createMergeHash($tagslib);
-
- # Parameters
- $template->param(
- biblio1 => $mergereference,
- biblio2 => $notreference,
- mergereference => $mergereference,
- record1 => @record1,
- record2 => @record2,
- framework => $framework,
- );
}
- else {
+ my ($biblionumbertag) = GetMarcFromKohaField('biblio.biblionumber');
+
+ # Parameters
+ $template->param(
+ ref_biblionumber => $ref_biblionumber,
+ records => \@records,
+ ref_record => $records[0],
+ framework => $framework,
+ biblionumbertag => $biblionumbertag,
+ MergeReportFields => C4::Context->preference('MergeReportFields'),
+ );
+ } else {
+ my @records;
+ foreach my $biblionumber (@biblionumbers) {
+ my $frameworkcode = GetFrameworkCode($biblionumber);
+ my $record = {
+ biblionumber => $biblionumber,
+ data => GetBiblioData($biblionumber),
+ frameworkcode => $frameworkcode,
+ };
+ push @records, $record;
+ }
# Ask the user to choose which record will be the kept
- $template->param(
- choosereference => 1,
- biblio1 => $biblionumber[0],
- biblio2 => $biblionumber[1],
- title1 => $data1->{'title'},
- subtitle1 => $subtitle1,
- title2 => $data2->{'title'},
- subtitle2 => $subtitle2
+ $template->param(
+ choosereference => 1,
+ records => \@records,
+ );
+
+ my $frameworks = getframeworks;
+ my @frameworkselect;
+ foreach my $thisframeworkcode ( keys %$frameworks ) {
+ my %row = (
+ value => $thisframeworkcode,
+ frameworktext => $frameworks->{$thisframeworkcode}->{'frameworktext'},
);
- if ($frameworkcode1 ne $frameworkcode2) {
- my $frameworks = getframeworks;
- my @frameworkselect;
- foreach my $thisframeworkcode ( keys %$frameworks ) {
- my %row = (
- value => $thisframeworkcode,
- frameworktext => $frameworks->{$thisframeworkcode}->{'frameworktext'},
- );
- if ($frameworkcode1 eq $thisframeworkcode){
- $row{'selected'} = 1;
- }
- push @frameworkselect, \%row;
- }
- $template->param(
- frameworkselect => \@frameworkselect,
- frameworkcode1 => $frameworkcode1,
- frameworkcode2 => $frameworkcode2,
- );
- }
+ push @frameworkselect, \%row;
}
+ $template->param(
+ frameworkselect => \@frameworkselect,
+ );
}
}
output_html_with_http_headers $input, $cookie, $template->output;
exit;
-
-=head1 FUNCTIONS
-
-=cut
-
-# ------------------------
-# Functions
-# ------------------------