Attachment 53668 Details for Bug 16969 – Bug 16969 cgi->param used in list context in opac-memberentry.pl

[patch] Bug 16969 cgi->param used in list context in opac-memberentry.pl

Bug-16969-cgi-param-used-in-list-context-in-opac-m.patch (text/plain), 1.04 KB, created by Marc Véron on 2016-07-24 13:49:20 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2016-07-24 13:49:20 UTC

Size: 1.04 KB

patch

obsolete

>From ffc06b0759edf8bc5d7696ae86c178bc06ae76f2 Mon Sep 17 00:00:00 2001
>From: Chris Cormack <chrisc@catalyst.net.nz>
>Date: Sun, 24 Jul 2016 19:18:51 +1200
>Subject: [PATCH] Bug 16969 cgi->param used in list context in
> opac-memberentry.pl
>
>To test
>1/ Hit the page, notice the warning in the log
>2/ Apply patch
>3/ Hit page, notice no warning in the log
>4/ Test functionality all still works
>
>Works as expected. (Note: See Bug 16960 for updating patron details).
>Signed-off-by: Marc <veron@veron.ch>
>---
> opac/opac-memberentry.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/opac/opac-memberentry.pl b/opac/opac-memberentry.pl
>index b73ee1b..86f3850 100755
>--- a/opac/opac-memberentry.pl
>+++ b/opac/opac-memberentry.pl
>@@ -360,7 +360,7 @@ sub ParseCgiForBorrower {
>     foreach ( $cgi->param ) {
>         if ( $_ =~ '^borrower_' ) {
>             my ($key) = substr( $_, 9 );
>-            $borrower{$key} = $scrubber->scrub( $cgi->param($_) );
>+            $borrower{$key} = $scrubber->scrub( scalar $cgi->param($_) );
>         }
>     }
> 
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16969: 53667 | 53668 | 53669