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)
committerNick Clemens <nick@bywatersolutions.com>
Thu, 11 Apr 2019 13:34:10 +0000 (13:34 +0000)
commit761b410f99cbbfff8e353c538fde2b6816408795
treef2d3a947b092760af491262b6285478798c425b9
parent275dee5f9b9a26b3490bf3646c71c78a2d8647eb
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>
koha-tmpl/intranet-tmpl/prog/en/includes/acquisitions-menu.inc