Details
-
Type: Bug
-
Status: Done/Fixed
-
Priority: Important
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.7.25
-
Fix Version/s: 4.7.28
-
Component/s: Drupal Integration Modules
-
Labels:
-
Versioning Impact:Patch (backwards-compatible bug fixes)
-
Documentation Required?:None
-
Funding Source:Contributed Code
-
Verified?:Yes
Description
In Drupal 8, adding or removing the "CiviCRM: access CiviCRM" permission won't hide/show the "CiviCRM" tab in the Drupal toolbar.
There's actually two bugs:
- The permission name is given case with incorrect case ("access civicrm" vs "access CiviCRM") and they are checked in case sensitive way
- The cache information needs to be returned always, otherwise if a user without permission visits the site first, the version without the tab will be cached and returned for all users - we at least need to say how this data varies and that's by permission (the current code had it vary per-user but that's less efficient - different users with the same permission can share the cache)
PR coming soon!