Bug 16670

Summary: CGI->param used in list context
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: abl, f.demians, julian.maurice, kyle, m.de.rooy, nick
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 15823    
Bug Blocks:    
Attachments: Bug 16670: (bug 15823 follow-up) CGI->param should not be called in list context
[3.22.x] Bug 16670: (bug 15823 follow-up) CGI->param should not be called in list context
[3.22.x] Bug 16670: (bug 15823 follow-up) CGI->param should not be called in list context
Bug 16670: (bug 15823 follow-up) CGI->param should not be called in list context
Bug 16670: (bug 15823 follow-up) CGI->param should not be called in list context

Description Jonathan Druart 2016-06-06 12:20:58 UTC

    
Comment 1 Jonathan Druart 2016-06-06 12:21:39 UTC
Introduced by bug 15823
Comment 2 Jonathan Druart 2016-06-06 12:32:02 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2016-06-06 12:32:06 UTC
*** Bug 16646 has been marked as a duplicate of this bug. ***
Comment 4 Jonathan Druart 2016-06-06 12:36:22 UTC Comment hidden (obsolete)
Comment 5 Jacek Ablewicz 2016-06-06 15:18:41 UTC
*** Bug 16673 has been marked as a duplicate of this bug. ***
Comment 6 Nick Clemens 2016-06-06 15:53:03 UTC
Created attachment 52089 [details] [review]
[3.22.x] Bug 16670: (bug 15823 follow-up) CGI->param should not be called in list context

This patch will remove the following warning "Staff members
are not allowed to discharge borrowers, nor borrowers to request a
discharge." in 3.22.x branches

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 7 Nick Clemens 2016-06-06 16:02:49 UTC
I can't recreate the issue on v16.05.00
Comment 8 Jacek Ablewicz 2016-06-06 16:53:00 UTC Comment hidden (obsolete)
Comment 9 Marcel de Rooy 2016-06-06 18:36:36 UTC
Created attachment 52093 [details] [review]
Bug 16670: (bug 15823 follow-up) CGI->param should not be called in list context

Signed-off-by: Jacek Ablewicz <abl@biblos.pk.edu.pl>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 10 Jonathan Druart 2016-06-06 19:15:19 UTC
(In reply to Nick Clemens from comment #7)
> I can't recreate the issue on v16.05.00

Because the param 'nopermission' is the last one passed through $template->param, but the problem might appear later.
Comment 11 Kyle M Hall 2016-06-17 14:36:24 UTC
Pushed to master for 16.11, thanks Jonathan!
Comment 12 Frédéric Demians 2016-06-24 05:44:47 UTC
Pushed in 16.05. Will be in 16.05.01.
Comment 13 Julian Maurice 2016-07-01 06:44:28 UTC
Patch pushed to 3.22.x, will be in 3.22.9