From 5127681603ed91064e98b49bf27bf540b63ae839 Mon Sep 17 00:00:00 2001
From: Jon Knight <J.P.Knight@lboro.ac.uk>
Date: Wed, 23 Aug 2017 09:03:38 +0000
Subject: [PATCH] Added error if there are multiple MARC templates with the
same name.
https://bugs.koha-community.org/show_bug.cgi?id=18389
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
---
misc/migration_tools/bulkmarcimport.pl | 16 +++++++++++-----
1 file changed, 11 insertions(+), 5 deletions(-)
diff --git a/misc/migration_tools/bulkmarcimport.pl b/misc/migration_tools/bulkmarcimport.pl
index f71d885..e444558 100755
--- a/misc/migration_tools/bulkmarcimport.pl
+++ b/misc/migration_tools/bulkmarcimport.pl
@@ -122,13 +122,19 @@ if(defined $localcust) { #local customize module
if($marc_mod_template ne '') {
my @templates = GetModificationTemplates();
foreach my $this_template (@templates) {
- if($this_template->{'name'} eq $marc_mod_template) {
- $marc_mod_template_id = $this_template->{'template_id'};
- last;
- }
+ if($this_template->{'name'} eq $marc_mod_template) {
+ if($marc_mod_template_id < 0) {
+ $marc_mod_template_id = $this_template->{'template_id'};
+ } else {
+ print "WARNING: MARC modification template name " .
+ "'$marc_mod_template' matches multiple templates. " .
+ "Please rename these templates\n";
+ exit 1;
+ }
+ }
}
if($marc_mod_template_id < 0) {
- die "Can't located MARC modification template '$marc_mod_template'\n";
+ die "Can't located MARC modification template '$marc_mod_template'\n";
}
}
--
2.1.4