Attachment 61136 Details for Bug 13757 – Bug 13757: Add extended attributes to the patron modification

[patch] Bug 13757: Add extended attributes to the patron modification

Bug-13757-Add-extended-attributes-to-the-patron-mo.patch (text/plain), 2.26 KB, created by Nick Clemens (kidclamp) on 2017-03-15 20:30:25 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2017-03-15 20:30:25 UTC

Size: 2.26 KB

patch

obsolete

>From 159e72d5f2f5b0e849859e84fdd28652cff89a69 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Mon, 19 Dec 2016 12:51:04 -0300
>Subject: [PATCH] Bug 13757: Add extended attributes to the patron modification
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> opac/opac-memberentry.pl | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
>diff --git a/opac/opac-memberentry.pl b/opac/opac-memberentry.pl
>index a08f65e..2100ca8 100755
>--- a/opac/opac-memberentry.pl
>+++ b/opac/opac-memberentry.pl
>@@ -20,6 +20,7 @@ use Modern::Perl;
> use CGI qw ( -utf8 );
> use Digest::MD5 qw( md5_base64 md5_hex );
> use Encode qw( encode );
>+use JSON;
> use List::MoreUtils qw( each_array uniq );
> use String::Random qw( random_string );
> 
>@@ -261,6 +262,7 @@ elsif ( $action eq 'update' ) {
>             );
> 
>             $borrower_changes{borrowernumber} = $borrowernumber;
>+            $borrower_changes{extended_attributes} = to_json($attributes);
> 
>             # FIXME update the following with
>             # Koha::Patron::Modifications->search({ borrowernumber => $borrowernumber })->delete;
>@@ -307,7 +309,7 @@ elsif ( $action eq 'edit' ) {    #Display logged in borrower's data
>         $template->param( display_patron_image => 1 ) if $patron_image;
>     }
> 
>-    $template->param( patron_attribute_classes => GeneratePatronAttributesForm( $borrower ) );
>+    $template->param( patron_attribute_classes => GeneratePatronAttributesForm( $borrowernumber ) );
> } else {
>     $template->param( patron_attribute_classes => GeneratePatronAttributesForm() );
> }
>@@ -465,7 +467,7 @@ sub DelEmptyFields {
> }
> 
> sub GeneratePatronAttributesForm {
>-    my ( $borrower, $entered_attributes ) = @_;
>+    my ( $borrowernumber, $entered_attributes ) = @_;
> 
>     # Get all attribute types and the values for this patron (if applicable)
>     my @types = C4::Members::AttributeTypes::GetAttributeTypes();
>@@ -476,7 +478,7 @@ sub GeneratePatronAttributesForm {
> 
>     my %attr_values = ();
> 
>-    if ( $borrower ) {
>+    if ( $borrowernumber ) {
>         my $attributes = C4::Members::Attributes::GetBorrowerAttributes($borrowernumber);
> 
>         # Remap the patron's attributes into a hash of arrayrefs per attribute (depends on
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 13757: 42254 | 42626 | 42627 | 47625 | 48040 | 48041 | 48042 | 48043 | 48379 | 48380 | 48381 | 48382 | 56924 | 56925 | 56926 | 56927 | 56928 | 56929 | 57041 | 57042 | 57043 | 57044 | 57045 | 57046 | 58252 | 58253 | 58254 | 58255 | 58351 | 58352 | 58353 | 58354 | 58355 | 58356 | 58357 | 58428 | 58429 | 58581 | 58703 | 58706 | 58707 | 58708 | 58709 | 58710 | 58711 | 58712 | 58713 | 58714 | 58715 | 59376 | 59737 | 59738 | 59739 | 59813 | 59896 | 59897 | 59898 | 60359 | 60360 | 61132 | 61133 | 61134 | 61135 | 61136 | 61137 | 61138 | 61139 | 61140 | 61141 | 61142 | 61143 | 61144 | 61145 | 61146 | 61147 | 61148 | 61149 | 61150 | 61492 | 61493 | 61555 | 61556 | 61557 | 61558 | 61559 | 61560 | 61561 | 61562 | 61563 | 61564 | 61565 | 61566 | 61567 | 61568 | 61569 | 61570 | 61571 | 61572 | 61573 | 61574 | 61575