Attachment 45880 Details for Bug 15367 – Bug 15367: Do not display repeatable patron attributes in the batch patron modification

[patch] Bug 15367: Do not display repeatable patron attributes in the batch patron modification

Bug-15367-Do-not-display-repeatable-patron-attribu.patch (text/plain), 1.47 KB, created by Jonathan Druart on 2015-12-21 12:28:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2015-12-21 12:28:54 UTC

Size: 1.47 KB

patch

obsolete

>From 0e735a81f79e6b78c8ddaf1f426d73f394de0cf1 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 21 Dec 2015 12:24:46 +0000
>Subject: [PATCH] Bug 15367: Do not display repeatable patron attributes in the
> batch patron modification
>
>The repeatable patron attributes are not correctly managed and can cause
>data lost. To avoid that, the easier way is not to display them in the
>batch patron modification tool.
>This should be implemented, as a new enhancement.
>
>Test plan:
>Create some patron attributes, some should be repeatable.
>Use the batch patron modification tool to modify patrons.
>With this patch, the patron attributes marked as repeatable won't be
>display anymore.
>---
> tools/modborrowers.pl | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/tools/modborrowers.pl b/tools/modborrowers.pl
>index 4172bc3..36610b1 100755
>--- a/tools/modborrowers.pl
>+++ b/tools/modborrowers.pl
>@@ -106,6 +106,9 @@ if ( $op eq 'show' ) {
>     my $patron_categories = C4::Members::GetBorrowercategoryList;
>     for ( values %$patron_attribute_types ) {
>         my $attr_type = C4::Members::AttributeTypes->fetch( $_->{code} );
>+        # TODO Repeatable attributes are not correctly managed and can cause data lost.
>+        # This should be implemented.
>+        next if $attr_type->{repeatable};
>         my $options = $attr_type->authorised_value_category
>             ? GetAuthorisedValues( $attr_type->authorised_value_category )
>             : undef;
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 15367: 45880 | 45883 | 46409