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)
committerChris Cormack <chrisc@catalyst.net.nz>
Wed, 31 Aug 2016 09:22:05 +0000 (21:22 +1200)
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: Chris Cormack <chrisc@catalyst.net.nz>

opac/opac-memberentry.pl

index a6e39b7..0d0e2c0 100755 (executable)
@@ -299,7 +299,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($_) );
         }
     }