Since Baidu Map configuration is kind of special, we could add a specific permission for the module to control the access to the Baidu Map administration settings page.
Currently, it is controlled by 'administer site configuration' which is already used by many other modules, but maybe we could add a more granular permission to control the access to this page.

We would need to modify the access property of the menu item in hook_menu and add an implementation of hook_permission in the module file.

Please let me know if you would have any questions, objections, comments, suggestions, recommendations or concerns on any aspects of this task, I would be glad to provide more information or explain in more details.

Any questions, feedback, testing, changes, ideas or recommendations would be highly appreciated.
Thanks to all in advance.

Comments

xiukun.zhou’s picture

xiukun.zhou commented
Status: Active » Fixed

Quick follow-up on this feature request:

Added an an implementation of hook_permission, in the module file, as suggested in the issue summary and modified the access property of the Baidu Map administration settings page menu item.

I went ahead and committed the changes against the 7.x-1.x branch at 5ac680e.

I allowed myself to mark this issue as fixed for now, but feel free to re-open it, or post a new ticket, at any time if you have any further objections with this ticket or related commit 5ac680e (we would surely be happy to hear your feedback).

Please let me know if you would have any further comments, feedback, questions, issues, objections, suggestions or concerns on the commit or this feature request in general, I would be glad to provide more information or explain in more details.

Thanks in advance to everyone for your testing, reviews, feedback and comments on this issue.
Cheers!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.