Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Reference: Drupal 7 Information Architecture handbook page.
According to this document:
Content
Drupal path:
admin/contentThe Content section lists all nodes. No module should add its configuration here. Only lists related to node objects should be included here.
Examples: Lists of comments
Based on this, since the module's configuration is not a list of comments, but rather comment access control, it might be better to add it under admin/config/people instead. Per the handbook page:
People
Drupal path:
admin/config/peopleThe People category should hold all modules that would fit under the old "User Management" category in Drupal 6. (Note: Do not confuse this category with the People section above.)
Examples: Access control modules
In the case of this particular module I am not sure if this is best, but it is worth considering.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | commentaccess-1200544.patch | 1.48 KB | rschwab |
| #1 | commentaccess-config-path-1200544.patch | 1.16 KB | xjm |
Comments
Comment #1
xjmHere is a patch that would apply this change.
Comment #2
rschwab CreditAttribution: rschwab commentedThank you for your patch! I think you're correct I have the configuration in the wrong place. However, I think admin/config/content would be the best spot for it. Even though the example above includes "access control modules" and Commentaccess is one... I think the configuration options are more about setting defaults on content authoring, rather than user management. I'll adapt your patch and commit it asap.
Thanks again! I'm always happy to see people care enough about the module to contribute to the issue queue.
Comment #3
rschwab CreditAttribution: rschwab commentedMade the updates as per 1 & 2 and also added a configuration link on the modules admin page. Committing to dev.