From 95cae148fc2deeb923cce027e0c548a2afea66a5 Mon Sep 17 00:00:00 2001 From: Jimmy Berry Date: Wed, 23 Nov 2011 17:26:10 -0600 Subject: Patch #366950: "Administer Users" permission should be separate from "User Settings". --- modules/user/user.module | 6 +++++- modules/user/user.test | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/modules/user/user.module b/modules/user/user.module index 48b17af..a2154ed 100644 --- a/modules/user/user.module +++ b/modules/user/user.module @@ -838,6 +838,10 @@ function user_permission() { 'title' => t('Administer users'), 'restrict access' => TRUE, ), + 'administer user settings' => array( + 'title' => t('Administer user settings'), + 'restrict access' => TRUE, + ), 'access user profiles' => array( 'title' => t('View user profiles'), ), @@ -1705,7 +1709,7 @@ function user_menu() { 'description' => 'Configure default behavior of users, including registration requirements, e-mails, fields, and user pictures.', 'page callback' => 'drupal_get_form', 'page arguments' => array('user_admin_settings'), - 'access arguments' => array('administer users'), + 'access arguments' => array('administer user settings'), 'file' => 'user.admin.inc', 'weight' => -10, ); diff --git a/modules/user/user.test b/modules/user/user.test index 6ecbfac..0646f37 100644 --- a/modules/user/user.test +++ b/modules/user/user.test @@ -1106,7 +1106,7 @@ class UserPermissionsTestCase extends DrupalWebTestCase { function setUp() { parent::setUp(); - $this->admin_user = $this->drupalCreateUser(array('administer permissions', 'access user profiles', 'administer site configuration', 'administer modules', 'administer users')); + $this->admin_user = $this->drupalCreateUser(array('administer permissions', 'access user profiles', 'administer site configuration', 'administer modules', 'administer users', 'administer user settings')); // Find the new role ID - it must be the maximum. $all_rids = array_keys($this->admin_user->roles); -- 1.7.7