Bug 5376 - Batch Edit and Delete need Superlibrarian Permissions
Summary: Batch Edit and Delete need Superlibrarian Permissions
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) critical (vote)
Assignee: Ian Walls
QA Contact: Bugs List
URL: batchMod.pl
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-08 14:12 UTC by Nicole C. Engard
Modified: 2019-06-27 09:24 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
patch (2.07 KB, patch)
2011-01-26 19:37 UTC, Nicole C. Engard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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.