@@ -, +, @@ --- cataloguing/merge.pl | 7 ++-- .../prog/en/modules/cataloguing/merge.tt | 33 +++++++++++++++----- 2 files changed, 28 insertions(+), 12 deletions(-) --- a/cataloguing/merge.pl +++ a/cataloguing/merge.pl @@ -81,7 +81,7 @@ if ($merge) { # If some items could not be moved : if (scalar(@notmoveditems) > 0) { my $itemlist = join(' ',@notmoveditems); - push @errors, "The following items could not be moved from the old record to the new one: $itemlist"; + push @errors, { code => "CANNOT_MOVE", value => $itemlist }; } # Moving subscriptions from the other record to the reference record @@ -123,7 +123,7 @@ if ($merge) { my $biblionumber = $input->param('biblionumber'); if (scalar(@biblionumber) != 2) { - push @errors, "An unexpected number of records was provided for merging. Currently only two records at a time can be merged."; + push @errors, { code => "WRONG_COUNT", value => scalar(@biblionumber) }; } else { my $data1 = GetBiblioData($biblionumber[0]); @@ -206,8 +206,7 @@ if ($merge) { if (@errors) { # Errors - my @errors_loop = map{{error => $_}}@errors; - $template->param( errors => \@errors_loop ); + $template->param( errors => \@errors ); } output_html_with_http_headers $input, $cookie, $template->output; --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt @@ -183,15 +183,24 @@ function changeFramework(fw) {
The merging was successful. Click here to see the merged record.
+ [% IF error.code == 'CANNOT_MOVE' %] + The following items could not be moved from the old record to the new one: [% error.value %] + [% ELSE %] + [% error %] + [% END %] + +The merging was successful. Click here to see the merged record.
+ [% END %] + [% ELSE %] [% IF ( choosereference ) %] @@ -228,9 +237,16 @@ function changeFramework(fw) { [% ELSE %] [% IF ( errors ) %][% error.error %]
- [% END %] + [% FOREACH error IN errors %] ++ [% IF error.code == 'WRONG_COUNT' %] + Number of records provided for merging: [% error.value %]. Currently only 2 records can be merged at a time. + [% ELSE %] + [% error %] + [% END %] + +
+ [% END %]