Bug 22444: currencies_manage permission doesn't provide link to manage currencies...
authorOwen Leonard <oleonard@myacpl.org>
Wed, 6 Mar 2019 19:02:50 +0000 (19:02 +0000)
committerLucas Gass <lucas@bywatersolutions.com>
Fri, 19 Apr 2019 01:36:52 +0000 (01:36 +0000)
commit483a2b2c7a3471f619ea0798a81fa715e604ba6a
treea81e27d3b1314aa9371fbc8c63d1cd6787b7e874
parentb48a2b38fec71f823e6ae99338217d837f329653
Bug 22444: currencies_manage permission doesn't provide link to manage currencies when selected alone

This patch modifies the acquisitions sidebar menu so that it checks
properly for the currencies_manage permission.

This patch also removes the menu link directly to the administration
home page. Since we're removing the check on the general
"CAN_user_parameters" permission it doesn't make sense to assume the
page is accessible.

To test, apply the patch and modifiy a staff user so that they have no
"parameters" permissions but do have "currencies_manage" permission.

Go to the Acquisitions home page. There should be a link to currencies
management in the sidebar menu.

Perform the same test with a user who has "parameters" permissions but
not "currencies_manage" permission. There should be no link to
currencies in the acquisitions sidebar menu. The user should have access
to the administrative home page.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 761b410f99cbbfff8e353c538fde2b6816408795)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit f7f13511d6a74359d65d7cc2348a73eb18da6044)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/includes/acquisitions-menu.inc