Changeset 1085 for trunk/admin/user_list.php

Timestamp:

Mar 16, 2006, 11:58:16 PM (18 years ago)

Author:

rub

Message:

Step 7 improvement issue 0000301:

o can attribute status <= current user
o define mode adviser

=> buttons disabled (gray on IE, not on FF)
=> truncated actions
=> display info mode adviser

File:

• trunk/admin/user_list.php (modified) (8 diffs)

trunk/admin/user_list.php

@@ -98 +98 @@
                 u.'.$conf['user_fields']['email'].' AS email,
                 ui.status,
+                ui.adviser,
                 ui.enabled_high
   FROM '.USERS_TABLE.' AS u
@@ -338 +339 @@
     array('nb_image_line', 'nb_line_page', 'template', 'language',
           'recent_period', 'maxwidth', 'expand', 'show_nb_comments',
-          'maxheight', 'status', 'enabled_high');
-  
-  $true_false_fields = array('expand', 'show_nb_comments', 'enabled_high');
+          'maxheight', 'status', 'adviser', 'enabled_high');
+  
+  $true_false_fields = array('expand', 'show_nb_comments', 'adviser', 'enabled_high');
   
   foreach ($formfields as $formfield)
@@ -381 +382 @@
         }
       }
-      
+
       // Webmaster status must not be changed
       if ($conf['webmaster_id'] == $user_id and isset($data['status']))
@@ -387 +388 @@
         $data['status'] = 'webmaster';
       }
-      
+
+      // Webmaster and guest adviser must not be changed
+      if ((($conf['webmaster_id'] == $user_id) or ($conf['guest_id'] == $user_id)) and isset($data['adviser']))
+      {
+        $data['adviser'] = 'false';
+      }
+
       array_push($datas, $data);
     }
-    
-//       echo '<pre>';
-//       print_r($datas);
-//       echo '</pre>';
     
     mass_updates(USER_INFOS_TABLE, $dbfields, $datas);
@@ -573 +576 @@
   $template->assign_vars(
     array(
+      'ADVISER_YES' => 'true' == $_POST['adviser'] ? 'checked="checked"' : '',
+      'ADVISER_NO' => 'false' == $_POST['adviser'] ? 'checked="checked"' : '',
       'NB_IMAGE_LINE' => $_POST['nb_image_line'],
       'NB_LINE_PAGE' => $_POST['nb_line_page'],
@@ -583 +588 @@
         'true' == $_POST['show_nb_comments'] ? 'checked="checked"' : '',
       'SHOW_NB_COMMENTS_NO' =>
-        'false' == $_POST['show_nb_comments'] ? 'checked="checked"' : ''
+        'false' == $_POST['show_nb_comments'] ? 'checked="checked"' : '',
+      'ENABLED_HIGH_YES' => 'true' == $_POST['enabled_high'] ? 'checked="checked"' : '',
+      'ENABLED_HIGH_NO' => 'false' == $_POST['enabled_high'] ? 'checked="checked"' : '',
       ));
 }
@@ -666 +673 @@
     $selected = '';
   }
-  
-  $template->assign_block_vars(
-    $blockname,
-    array(
-      'VALUE' => $status,
-      'CONTENT' => $lang['user_status_'.$status],
-      'SELECTED' => $selected
-      ));
+
+  // Only status <= can be assign
+  if (is_autorize_status(get_access_type_status($status)))
+  {
+    $template->assign_block_vars(
+      $blockname,
+      array(
+        'VALUE' => $status,
+        'CONTENT' => $lang['user_status_'.$status],
+        'SELECTED' => $selected
+        ));
+  }
 }
 
@@ -801 +812 @@
       'U_PERM' => $perm_url.$local_user['id'],
       'USERNAME' => $local_user['username'],
-      'STATUS' => $lang['user_status_'.$local_user['status']],
+      'STATUS' => $lang['user_status_'.$local_user['status']].(($local_user['adviser'] == 'true') ? ' ['.$lang['adviser'].']' : ''),
       'EMAIL' => isset($local_user['email']) ? $local_user['email'] : '',
       'GROUPS' => $groups_string,