Bugzilla – Attachment 56585 Details for
Bug 5670
Housebound Readers Module
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 5670: [QA Followup] Avoid CGI::param in list context.
Bug-5670-QA-Followup-Avoid-CGIparam-in-list-contex.patch (text/plain), 1013 bytes, created by
Jonathan Druart
on 2016-10-17 16:40:44 UTC
(
hide
)
Description:
Bug 5670: [QA Followup] Avoid CGI::param in list context.
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2016-10-17 16:40:44 UTC
Size:
1013 bytes
patch
obsolete
>From 1367b94654196ad7c938af46eea26941638e236e Mon Sep 17 00:00:00 2001 >From: Alex Sassmannshausen <alex.sassmannshausen@ptfs-europe.com> >Date: Mon, 19 Sep 2016 13:40:46 +0200 >Subject: [PATCH] Bug 5670: [QA Followup] Avoid CGI::param in list context. > >* members/housebound.pl: Force `borrowernumber` to scalar context. > >Signed-off-by: Claire Gravely <claire_gravely@hotmail.com> >--- > members/housebound.pl | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > >diff --git a/members/housebound.pl b/members/housebound.pl >index a2d5062..df7e058 100755 >--- a/members/housebound.pl >+++ b/members/housebound.pl >@@ -55,7 +55,8 @@ my $visit_id = $input->param('visit_id') // q{}; > > # Get patron > my $patron = eval { >- return Koha::Patrons->new->find($input->param('borrowernumber')); >+ my $borrowernumber = $input->param('borrowernumber') // q{}; >+ return Koha::Patrons->new->find($borrowernumber); > }; > push @messages, { type => 'error', code => 'error_on_patron_load' } > if ( $@ or !$patron ); >-- >2.8.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 5670
:
3038
|
22162
|
22163
|
22164
|
22165
|
26816
|
27288
|
50930
|
51377
|
51875
|
54384
|
54385
|
55326
|
55334
|
55335
|
55336
|
55734
|
55735
|
55736
|
55737
|
55738
|
55739
|
55865
|
55866
|
55867
|
55890
|
55891
|
56298
|
56299
|
56300
|
56301
|
56302
|
56308
|
56337
|
56341
|
56346
|
56347
|
56348
|
56349
|
56350
|
56351
|
56352
|
56353
|
56354
|
56355
|
56356
|
56357
|
56358
|
56359
|
56360
|
56361
|
56362
|
56363
|
56364
|
56365
|
56366
|
56367
|
56368
|
56369
|
56580
|
56581
|
56582
|
56583
|
56584
| 56585 |
56586
|
56587
|
56588
|
56589
|
56590
|
56591
|
56592
|
56593
|
56594
|
56595
|
56596
|
56597
|
56598
|
56599
|
56600
|
56601
|
56602
|
56603
|
56604
|
56605
|
56606