From b7b94b7012704d99a91ce32723404095a68fe6f1 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=utf8 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 Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com --- tools/modborrowers.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/modborrowers.pl b/tools/modborrowers.pl index 95d26de398..99379931b0 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.20.1