Needs review
Project:
Chaos Tool Suite (ctools)
Version:
8.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
9 Feb 2016 at 22:01 UTC
Updated:
26 Jul 2022 at 18:06 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
tim.plunkettComment #3
eclipsegc commentedSo my first response here is to ask for a permission autocomplete for the configuration field. I can't see how someone would successfully use this without something along these lines.
Eclipse
Comment #4
tim.plunkettThis needs web tests.
Comment #6
tim.plunkettAdded a browser test instead!
Comment #7
rjg commentedThis issue is still around 2 years later, and I've found that this functionality would be desirable across all modules, including core ones like block (for block visibility).
I created a new module to add this functionality, inspired by the patches here, hoping that it will end up in core. Source is at: https://github.com/rjgwiz/user_permission_condition
Comment #8
manuel.adanIMHO, it is preferable to work on the ctools patch rather than create a new module for this. It appears to be an old issue, not sure if exists an alternative way to have conditional access by permissions. It is a basic feature working with page_manager.
Comment #9
manuel.adanI made a mix of #6 and the proposed module by @rjg:
Some tests needed.
Comment #10
manuel.adanUpdated patch to accommodate changes introduced at #3014949: Deprecate 'context' on Block and Condition plugin annotations in favor of 'context_definitions'.
In the meantime, @geek-merlin published the User Permission Condition module that is almost a copy of this patch. The module is marked as stable.
I've been using this patch in some projects without issues until they were recently updated to D9. Patch is actually being tested at the mentioned module, so I think it can be marked as RTBC for that reason and also to gain attention from maintainers to finally decide about adding this condition plugin into the Chaos Tools or not.
Comment #11
japerryMade a few changes based on the other contrib module. We should add this to ctools, pretty sure tests will pass since this doesn't have any meaningful tests with it.