From 7e877be695ecc070ccfd7dda85049cc63534ec98 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 2 Oct 2012 08:58:40 -0400 Subject: [PATCH] Bug 8862 - merge.pl contains untranslatable strings Content-Type: text/plain; charset="utf-8" --- cataloguing/merge.pl | 7 ++-- .../prog/en/modules/cataloguing/merge.tt | 33 +++++++++++++++----- 2 files changed, 28 insertions(+), 12 deletions(-) diff --git a/cataloguing/merge.pl b/cataloguing/merge.pl index 020001d..1492f1f 100755 --- a/cataloguing/merge.pl +++ b/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; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt index 911c2f0..9895251 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt +++ b/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 \ No newline at end of file -- 1.7.2.5