Bug 15386: Checkout / patron pages: Hide menu items leading to 404 pages
authorMarc Véron <veron@veron.ch>
Wed, 16 Dec 2015 15:00:15 +0000 (16:00 +0100)
committerJulian Maurice <julian.maurice@biblibre.com>
Thu, 31 Dec 2015 09:45:35 +0000 (10:45 +0100)
commit93b204f6b66ed9848a26accc632658f6d37e3ccd
tree109228e38d397959661105472163961587e387cc
parent638a6343f653fdf0a04504ada952d01978d4733a
Bug 15386: Checkout / patron pages: Hide menu items leading to 404 pages

Some of the menu items on patron / Checkout related pages lead to a 404 page
depending on the permissions. This patch hides them as appropriate.

To test:

- For a staff member A, set permission 'borrowers' only (+ 'catalogue')
- Log in as A
- Search a patron and display details
- Verify that none of the menu items at the left lead to the 404 page

- For a staff member B, set permission 'circulate_remaining_permission'
  only (+ catalogue)
- Log in as B
- Go to Circulation > Checkout, search a patron
- Click menu items at the left, some of them lead to the 404 page
- Verify that none of the menu items at the left lead to the 404 page

- Do the same for a staff member C with both permissions set
- Verify that menu items appear as expected

Signed-off-by: Lucio Moraes <lmoraes@catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Amended patch: I have remove the double if
CAN_user_circulate_circulate_remaining_permissions statement (and group
block into one). And also remove unecessary change ('IF a IF b' instead
of existing 'IF a && b')

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 976d715ac52a3c54cbc397d22127e773f977e0db)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc