@@ -, +, @@ --- 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) {

Merging records

[% IF ( result ) %] [% IF ( errors ) %] + [% FOREACH error IN errors %] -
[% error.error %].
Therefore, the record to be merged has not been deleted.
- [% END %] +
- [% ELSE %] - -

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 %] + +
Therefore, the record to be merged has not been deleted.
[% END %] + [% ELSE %] + +

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 ) %]
- [% FOREACH error IN 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 %]
[% ELSE %]
@@ -377,3 +393,4 @@ function changeFramework(fw) { [% INCLUDE 'intranet-bottom.inc' %] +[A --