From d7ae9611914832954322118e2748f9c0b25728d3 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 21 Dec 2015 12:31:59 +0000 Subject: [PATCH] Bug 12636: Do not display unique patron attributes in the batch patron modification MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Batch patron modification should not offer to update patron attributes with values which are designated as unique since such attributes are required to be unique to a single patron. Test plan: Create some patron attributes, some should be unique. Use the batch patron modification tool to modify patrons. With this patch, the patron attributes marked as unique won't be display anymore. Signed-off-by: Marc VĂ©ron Signed-off-by: Katrin Fischer --- tools/modborrowers.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/modborrowers.pl b/tools/modborrowers.pl index 95d26de..9937993 100755 --- a/tools/modborrowers.pl +++ b/tools/modborrowers.pl @@ -110,6 +110,7 @@ if ( $op eq 'show' ) { # TODO Repeatable attributes are not correctly managed and can cause data lost. # This should be implemented. next if $attr_type->{repeatable}; + next if $attr_type->{unique_id}; # Don't display patron attributes that must be unqiue my $options = $attr_type->authorised_value_category ? GetAuthorisedValues( $attr_type->authorised_value_category ) : undef; -- 2.1.4