? potx-cli.php
? potx.inc
? tablehearder-height.patch.txt
? theme
? misc/untitled.html
? modules/user/access_control.js
? profiles/default/translations
? sites/all/themes
Index: modules/upload/upload.admin.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/upload/upload.admin.inc,v
retrieving revision 1.2
diff -u -d -F^\s*function -r1.2 upload.admin.inc
--- modules/upload/upload.admin.inc	7 Aug 2007 08:39:36 -0000	1.2
+++ modules/upload/upload.admin.inc	17 Sep 2007 22:29:31 -0000
@@ -109,7 +109,7 @@ function upload_admin_settings() {
 
   $form['settings_general']['upload_max_size'] = array('#value' => '<p>'. t('Your PHP settings limit the maximum file size per upload to %size.', array('%size' => format_size(file_upload_max_size()))) .'</p>');
 
-  $roles = user_roles(0, 'upload files');
+  $roles = user_roles(FALSE, 'upload files');
   $form['roles'] = array('#type' => 'value', '#value' => $roles);
 
   foreach ($roles as $rid => $role) {
Index: modules/user/user.admin.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/user/user.admin.inc,v
retrieving revision 1.3
diff -u -d -F^\s*function -r1.3 user.admin.inc
--- modules/user/user.admin.inc	12 Sep 2007 18:29:32 -0000	1.3
+++ modules/user/user.admin.inc	17 Sep 2007 22:29:38 -0000
@@ -166,7 +166,7 @@ function user_admin_account() {
   $destination = drupal_get_destination();
 
   $status = array(t('blocked'), t('active'));
-  $roles = user_roles(1);
+  $roles = user_roles(TRUE);
   $accounts = array();
   while ($account = db_fetch_object($result)) {
     $accounts[$account->uid] = '';
@@ -508,15 +508,11 @@ function user_admin_perm($form_state, $r
   }
 
   if (is_numeric($rid)) {
-    $result = db_query('SELECT rid, name FROM {role} r WHERE r.rid = %d ORDER BY name', $rid);
+    $roles = user_roles();
+    $role_names = array($rid => $roles[$rid]);
   }
   else {
-    $result = db_query('SELECT rid, name FROM {role} ORDER BY name');
-  }
-
-  $role_names = array();
-  while ($role = db_fetch_object($result)) {
-    $role_names[$role->rid] = $role->name;
+    $role_names = user_roles();
   }
 
   // Render role/permission overview:
@@ -617,9 +613,9 @@ function theme_user_admin_perm($form) {
  * @see theme_user_admin_new_role().
  */
 function user_admin_role() {
-  $id = arg(4);
-  if ($id) {
-    if (DRUPAL_ANONYMOUS_RID == $id || DRUPAL_AUTHENTICATED_RID == $id) {
+  $rid = arg(4);
+  if ($rid) {
+    if ($rid == DRUPAL_ANONYMOUS_RID || $rid == DRUPAL_AUTHENTICATED_RID) {
       drupal_goto('admin/user/roles');
     }
     // Display the edit role form.
@@ -635,7 +631,7 @@ function user_admin_role() {
     );
     $form['rid'] = array(
       '#type' => 'value',
-      '#value' => $id,
+      '#value' => $rid,
     );
     $form['submit'] = array(
       '#type' => 'submit',
Index: modules/user/user.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/user/user.module,v
retrieving revision 1.848
diff -u -d -F^\s*function -r1.848 user.module
--- modules/user/user.module	14 Sep 2007 17:38:08 -0000	1.848
+++ modules/user/user.module	17 Sep 2007 22:29:52 -0000
@@ -1314,7 +1314,7 @@ function user_edit_form(&$form_state, $u
     $form['account']['status'] = array('#type' => 'radios', '#title' => t('Status'), '#default_value' => isset($edit['status']) ? $edit['status'] : 1, '#options' => array(t('Blocked'), t('Active')));
   }
   if (user_access('administer access control')) {
-    $roles = user_roles(1);
+    $roles = user_roles(TRUE);
     unset($roles[DRUPAL_AUTHENTICATED_RID]);
     if ($roles) {
       $default = empty($edit['roles']) ? array() : array_keys($edit['roles']);
@@ -1499,15 +1499,16 @@ function _user_mail_text($key, $language
 /**
  * Retrieve an array of roles matching specified conditions.
  *
- * @param $membersonly
- *   Set this to TRUE to exclude the 'anonymous' role.
+ * @param $only_members
+ *   (Optional) Set this to TRUE to exclude the 'anonymous user' role.
  * @param $permission
- *   A string containing a permission. If set, only roles containing that permission are returned.
+ *   (Optional) A string containing a permission. If set, only roles containing
+ *   that permission are returned.
  *
  * @return
  *   An associative array with the role id as the key and the role name as value.
  */
-function user_roles($membersonly = 0, $permission = 0) {
+function user_roles($only_members = FALSE, $permission = NULL) {
   $roles = array();
 
   if ($permission) {
@@ -1517,8 +1518,17 @@ function user_roles($membersonly = 0, $p
     $result = db_query('SELECT * FROM {role} ORDER BY name');
   }
   while ($role = db_fetch_object($result)) {
-    if (!$membersonly || ($membersonly && $role->rid != DRUPAL_ANONYMOUS_RID)) {
-      $roles[$role->rid] = $role->name;
+    if ($only_members == FALSE || $role->rid != DRUPAL_ANONYMOUS_RID) {
+      switch ($role->rid) {
+        case DRUPAL_ANONYMOUS_RID:
+          $roles[$role->rid] = t('Anonymous user');
+          break;
+        case DRUPAL_AUTHENTICATED_RID:
+          $roles[$role->rid] = t('Authenticated user');
+          break;
+        default:
+          $roles[$role->rid] = $role->name;
+      }
     }
   }
   return $roles;
@@ -1543,7 +1553,7 @@ function user_user_operations($form_stat
   );
 
   if (user_access('administer access control')) {
-    $roles = user_roles(1);
+    $roles = user_roles(TRUE);
     unset($roles[DRUPAL_AUTHENTICATED_RID]);  // Can't edit authenticated role.
 
     $add_roles = array();
@@ -1739,7 +1749,7 @@ function _user_sort($a, $b) {
 function user_filters() {
   // Regular filters
   $filters = array();
-  $roles = user_roles(1);
+  $roles = user_roles(TRUE);
   unset($roles[DRUPAL_AUTHENTICATED_RID]); // Don't list authorized role.
   if (count($roles)) {
     $filters['role'] = array(