Bug 5376

Summary: Batch Edit and Delete need Superlibrarian Permissions
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: ToolsAssignee: Ian Walls <koha.sekjal>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: critical    
Priority: PATCH-Sent (DO NOT USE) CC: brendan, chris, koha.sekjal
Version: Main   
Hardware: All   
OS: All   
URL: batchMod.pl
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: patch

Description Nicole C. Engard 2010-11-08 14:12:41 UTC
When anyone with simply batch edit or batch delete permissions tries to perform either of these actions they are told they don't have the right permissions.  

In batchMod.pl if $op isn't defined, then we get a wacky '*' for permissions which means either superlibrarian or full tools permissions (i'm not sure which).  At one of the stages, the $op variable is not posted, so the permission go weird
Comment 1 Nicole C. Engard 2010-11-08 14:13:51 UTC
update - I'm assuming the '*' is superlibrarian based on further testing.
Comment 2 Nicole C. Engard 2010-12-27 13:16:32 UTC
this has become a bit of an issue for me (and I don't have the skills to fix it on my own) so I'm bumping it up to get some attention :)
Comment 3 Nicole C. Engard 2011-01-26 19:37:15 UTC
Created attachment 3027 [details] [review]
patch
Comment 4 Chris Cormack 2011-01-29 19:03:28 UTC
Patch pushed, please test
Comment 5 Ian Walls 2011-02-08 03:33:19 UTC
Patch tested and confirmed.  Closing ticket.