Bug 13544

Summary: admin/auth_subfields_structure.pl does not compile in older perls
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: chris, martin.renvoize, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13522
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 13544: Make it explicit that getauthtypes returns a hash ref
[SIGNED OFF] Bug 13544: Make it explicit that getauthtypes returns a hash ref
[PASSED QA] Bug 13544: Make it explicit that getauthtypes returns a hash ref

Description Jonathan Druart 2015-01-09 14:53:27 UTC

    
Comment 1 Jonathan Druart 2015-01-09 15:04:17 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2015-01-11 22:13:20 UTC
Not sure how to test this, my system uses perl 5.18, but I found no regressions in editing the authority frameworks.
Comment 3 Jonathan Druart 2015-01-12 08:13:45 UTC
You can use sandboxes, they use perl 5.10.1
Comment 4 Katrin Fischer 2015-01-20 21:43:12 UTC Comment hidden (obsolete)
Comment 5 Martin Renvoize (ashimema) 2015-01-28 08:03:15 UTC
Created attachment 35570 [details] [review]
[PASSED QA] Bug 13544: Make it explicit that getauthtypes returns a hash ref

Prior to perl 5.12 keys can only operate on a hash.

Test plan:
With perl 5.10, access to admin/auth_subfields_structure.pl.
Without this patch, you get:
Type of arg 1 to keys must be hash (not subroutine entry) at
/home/koha/src/admin/auth_subfields_structure.pl line 102, near
"getauthtypes)"

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Confirmed problem and tested patch on a sandbox, signed off locally.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Martin Renvoize (ashimema) 2015-01-28 08:04:03 UTC
Passing QA on this.. simple patch that fixes the problem.
Comment 7 Tomás Cohen Arazi (tcohen) 2015-02-04 18:54:46 UTC
Patch pushed to master.

Thanks Jonathan!
Comment 8 Chris Cormack 2015-02-09 20:37:24 UTC
Pushed to 3.18.x will be in 3.18.4