Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 11217: The # in accountlines descriptions makes them un-writeoffable

Browse source 
This patch adds escaping to url params in mambers/pay.pl

To test:
- Apply patch
- Go to Home > Patrons > Manual invoice and create an invoice with a
  description containing a #
- Go to tab "Pay fines"
- Write off the fine
Expected result: Fine is written off and does no longer display in pay tab.

Signed-off-by: Claire Gravely <claire_gravely@hotmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
This commit is contained in:
Marc 2016-10-07 11:03:45 +02:00 committed by Brendan Gallagher
parent 1d0d5f1398
commit 692014c817
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
members/pay.pl
View file

@ -43,6 +43,7 @@ use C4::Members::Attributes qw(GetBorrowerAttributes);
use Koha::Patron::Images;
use Koha::Patron::Categories;
use URI::Escape;
our $input = CGI->new;
@ -174,8 +175,8 @@ sub redirect_to_paycollect {
$redirect .= get_for_redirect( 'amount', "amount$line_no", 1 );
$redirect .=
get_for_redirect( 'amountoutstanding', "amountoutstanding$line_no", 1 );
$redirect .= get_for_redirect( 'description', "description$line_no", 0 );
$redirect .= get_for_redirect( 'title', "title$line_no", 0 );
$redirect .= uri_escape_utf8( get_for_redirect( 'description', "description$line_no", 0 ) );
$redirect .= uri_escape_utf8( get_for_redirect( 'title', "title$line_no", 0 ) );
$redirect .= get_for_redirect( 'itemnumber', "itemnumber$line_no", 0 );
$redirect .= get_for_redirect( 'notify_id', "notify_id$line_no", 0 );
$redirect .= get_for_redirect( 'notify_level', "notify_level$line_no", 0 );