From 2614e07e1e2e6386b5f91e65f127940072e54d4d Mon Sep 17 00:00:00 2001 From: Colin Campbell Date: Tue, 9 Jul 2013 11:18:49 +0100 Subject: [PATCH] Bug 10507: improve warning about duplicate patron attribute value If a warning about a duplicated patron attribute is returned, the value was being passed back for display in the warning message with the same name as the variable in errors As the message is displayed in the context of errors the error value 1 was being displayed not the value duplicated as intended. Pass the value to the template with a unique name. Signed-off-by: Owen Leonard I fixed another tiny typo while signing off: "is already in use" instead of "is already is use." Signed-off-by: Kyle M Hall Passes koha-qa.pl, works as advertised. Signed-off-by: Galen Charlton --- .../intranet-tmpl/prog/en/modules/members/memberentrygen.tt | 2 +- members/memberentry.pl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt index 1bb8e3e752..7a2d29e43f 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -176,7 +176,7 @@ [% END %] [% IF ( ERROR_extended_unique_id_failed ) %]
  • The attribute value - [% ERROR_extended_unique_id_failed %] is already is use by another patron record.
  • + [% ERROR_extended_unique_id_failed_value %] is already in use by another patron record. [% END %] diff --git a/members/memberentry.pl b/members/memberentry.pl index b7123f2380..0bf3122330 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -315,7 +315,7 @@ if ($op eq 'save' || $op eq 'insert'){ foreach my $attr (@$extended_patron_attributes) { unless (C4::Members::Attributes::CheckUniqueness($attr->{code}, $attr->{value}, $borrowernumber)) { push @errors, "ERROR_extended_unique_id_failed"; - $template->param(ERROR_extended_unique_id_failed => "$attr->{code}/$attr->{value}"); + $template->param(ERROR_extended_unique_id_failed_value => "$attr->{code}/$attr->{value}"); } } } -- 2.39.5