Attachment 34524 Details for Bug 13470 – Bug 13470 - pay fines with extended borrower attributes result in application error

[patch] Bug 13470 - pay fines with extended borrower attributes result in application error

Bug-13470---pay-fines-with-extended-borrower-attri.patch (text/plain), 1.54 KB, created by Jonathan Druart on 2014-12-19 10:07:30 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2014-12-19 10:07:30 UTC

Size: 1.54 KB

patch

obsolete

>From 4f13c7779cf99d6156eca1327515deddd0896304 Mon Sep 17 00:00:00 2001
>From: Dobrica Pavlinusic <dpavlin@rot13.org>
>Date: Tue, 16 Dec 2014 12:49:40 +0100
>Subject: [PATCH] Bug 13470 - pay fines with extended borrower attributes
> result in application error
>
>Test scenario:
>
>1. ensure that extended patron attributes are turned on in system preferences
>2. find borrower which has fine to pay, and click on pay button
>3. verify application error
>4. apply patch and verify that application error is gone
>
>Signed-off-by: Aleisha <aleishaamohia@hotmail.com>
>Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
>---
> members/paycollect.pl | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
>diff --git a/members/paycollect.pl b/members/paycollect.pl
>index ca47862..6473f02 100755
>--- a/members/paycollect.pl
>+++ b/members/paycollect.pl
>@@ -149,6 +149,7 @@ $template->param(
>     total         => $total_due,
>     activeBorrowerRelationship => (C4::Context->preference('borrowerRelationship') ne ''),
>     RoutingSerials => C4::Context->preference('RoutingSerials'),
>+    ExtendedPatronAttributes => C4::Context->preference('ExtendedPatronAttributes'),
> );
> 
> output_html_with_http_headers $input, $cookie, $template->output;
>@@ -179,9 +180,6 @@ sub borrower_add_additional_fields {
> 
>     if (C4::Context->preference('ExtendedPatronAttributes')) {
>         $b_ref->{extendedattributes} = GetBorrowerAttributes($borrowernumber);
>-        $template->param(
>-            ExtendedPatronAttributes => 1,
>-        );
>     }
> 
>     # Computes full borrower address
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 13470: 34451 | 34501 | 34524