Reference: Drupal 7 Information Architecture handbook page.

According to this document:

People

Drupal path: admin/people

The People section should only be used for the listing of users. Modules may expose other data here. For example, a module might list events related to each user.

However, no module should add its configuration here. Instead, use the People category under the Configuration section (below).

Examples: Lists of events related to users

Based on this recommendation, it would be better to add the module's configuration under admin/config/people instead. Per the handbook page:

People

Drupal path: admin/config/people

The 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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

Status: Active » Needs review
FileSize
1.69 KB

Patch implementing this menu change attached.

alduya’s picture

I agree with the change of configuration paths.
I altered the patch so that the 'Access by Taxonomy' link is shown in the configuration screen.

Dave Cohen’s picture

Status: Needs review » Fixed

Although personally I believe access control settings belong next to user permissions, I do see the drupal.org doc sites access control modules as an example for what belongs under admin/config/people.

Thanks for the patches, sorry for the delay.

Status: Fixed » Closed (fixed)

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