Bug 19195

Summary: Noisy warns when creating or editing a basket
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: AcquisitionsAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: dominic.pichette, fridolin.somers, jonathan.druart, katrin.fischer, mtj
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19195: Preventing noisy warns when creating or editing a basket
Bug 19195: Preventing noisy warns when creating or editing a basket
Bug 19195: Do not explicitely force scalar context when unecessary

Description Aleisha Amohia 2017-08-29 05:02:07 UTC
To reproduce, go to Acquisitions, find or create a vendor.
Create a basket, put information in all fields
Notice warns
Edit the basket
Notice warns
CGI::param called in list context from package CGI::Compile::ROOT::home_vagrant_kohaclone_acqui_basketheader_2epl line 138, this can lead to vulnerabilities. See the warning in "Fetching the value or values of a single named parameter" at /usr/share/perl5/CGI.pm line 436.
Comment 1 Aleisha Amohia 2017-08-29 05:04:39 UTC
Created attachment 66574 [details] [review]
Bug 19195: Preventing noisy warns when creating or editing a basket

To test:
1) Open the koha intranet error log
2) Go to Acquisitions -> Find or create a vendor
3) Create a new basket, filling all fields
4) Notice warns in error log
5) Edit this basket
6) Notice warns in error log
7) Apply patch
8) Create another basket, confirm warns do not show
9) Edit this basket, confirm warns do not show

Sponsored-by: Catalyst IT
Comment 2 Dominic Pichette 2017-09-15 20:44:57 UTC
Created attachment 67166 [details] [review]
Bug 19195: Preventing noisy warns when creating or editing a basket

To test:
1) Open the koha intranet error log
2) Go to Acquisitions -> Find or create a vendor
3) Create a new basket, filling all fields
4) Notice warns in error log
5) Edit this basket
6) Notice warns in error log
7) Apply patch
8) Create another basket, confirm warns do not show
9) Edit this basket, confirm warns do not show

Sponsored-by: Catalyst IT
Comment 3 Dominic Pichette 2017-09-15 20:47:54 UTC
signed-off by : Dominic Pichette
Comment 4 Jonathan Druart 2017-09-19 14:58:30 UTC
Created attachment 67217 [details] [review]
Bug 19195: Do not explicitely force scalar context when unecessary

These ones are already called in scalar context
Comment 5 Jonathan Druart 2017-09-19 14:59:57 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 6 Fridolin Somers 2017-10-27 09:37:50 UTC
Pushed to 17.05.x, will be in 17.05.06.
Comment 7 Katrin Fischer 2017-10-29 18:18:46 UTC
These patches have been pushed to 16.11.x and will be in 16.11.14.
Comment 8 Mason James 2017-11-23 23:21:57 UTC
Pushed to 16.05.x, for 16.05.19 release