Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display "Locations list" and "Locations map" links for read only users #3755

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AguirreIF
Copy link

Hi, for users with Read permission in the module Locations, the Locations map link is not visible because it is inserted (from app/tools/menu.php) only for users with Write permission, with this check in app/tools/all-locations-list.php:

<?php
if($User->get_module_permissions ("locations")>=User::ACCESS_RW) {
include('menu.php');
...

So in this PR I extracted those links from app/tools/menu.php and added them to the app/tools/all-locations-list.php and app/tools/all-locations-map.php files.

Hope it makes sense :)

@AguirreIF AguirreIF marked this pull request as ready for review February 9, 2023 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant