Bug 16969 cgi->param used in list context in opac-memberentry.pl
authorChris Cormack <chrisc@catalyst.net.nz>
Sun, 24 Jul 2016 07:18:51 +0000 (19:18 +1200)
committerKyle M Hall <kyle@bywatersolutions.com>
Wed, 10 Aug 2016 14:02:05 +0000 (14:02 +0000)
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>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

opac/opac-memberentry.pl

index e4945e7..592c99b 100755 (executable)
@@ -374,7 +374,7 @@ sub ParseCgiForBorrower {
     foreach ( $cgi->param ) {
         if ( $_ =~ '^borrower_' ) {
             my ($key) = substr( $_, 9 );
-            $borrower{$key} = HTML::Entities::encode( $scrubber->scrub( $cgi->param($_) ) );
+            $borrower{$key} = HTML::Entities::encode( $scrubber->scrub( scalar $cgi->param($_) ) );
         }
     }