From 7e55f0aefcddafe6bdece04be40305fcb9e65e5c Mon Sep 17 00:00:00 2001
From: Julio Montoya <gugli100@gmail.com>
Date: Tue, 9 Aug 2011 11:03:43 +0200
Subject: [PATCH] Minor - changing status icons

---
 main/admin/user_edit.php     | 21 ++++++++-------------
 main/user/subscribe_user.php | 13 +++++--------
 2 files changed, 13 insertions(+), 21 deletions(-)

diff --git a/main/admin/user_edit.php b/main/admin/user_edit.php
index 131f7b44e8..f7379f661f 100755
--- a/main/admin/user_edit.php
+++ b/main/admin/user_edit.php
@@ -33,16 +33,11 @@ function password_switch_radio_button(){
 }
 
 function display_drh_list(){
-	if(document.getElementById("status_select").value=='.COURSEMANAGER.')
-	{
+	if(document.getElementById("status_select").value=='.COURSEMANAGER.') {
 		document.getElementById("id_platform_admin").style.display="block";
-	}
-	else if (document.getElementById("status_select").value=='.STUDENT.')
-	{
+	} else if (document.getElementById("status_select").value=='.STUDENT.') {
 		document.getElementById("id_platform_admin").style.display="none";
-	}
-	else
-	{
+	} else {
 		document.getElementById("id_platform_admin").style.display="none";
 	}
 }
@@ -171,12 +166,12 @@ $form->addGroup($group, 'password', null, '', false);
 
 // Status
 $status = array();
-$status[COURSEMANAGER] = get_lang('Teacher');
-$status[STUDENT] = get_lang('Learner');
-$status[DRH] = get_lang('Drh');
-$status[SESSIONADMIN] = get_lang('SessionsAdmin');
+$status[COURSEMANAGER] 	= get_lang('Teacher');
+$status[STUDENT] 		= get_lang('Learner');
+$status[DRH] 			= get_lang('Drh');
+$status[SESSIONADMIN] 	= get_lang('SessionsAdmin');
 
-$form->addElement('select', 'status', get_lang('Profile'), $status, array('id' => 'status_select', 'onchange' => 'javascript: display_drh_list();'));
+$form->addElement('select', 'status', get_lang('Profile'), $status, array('id' => 'status_select', 'onchange' => 'javascript: display_drh_list();','class'=>'chzn-select'));
 
 //Language
 $form->addElement('select_language', 'language', get_lang('Language'));
diff --git a/main/user/subscribe_user.php b/main/user/subscribe_user.php
index fd67170742..4cc7fd91f1 100755
--- a/main/user/subscribe_user.php
+++ b/main/user/subscribe_user.php
@@ -602,15 +602,15 @@ function active_filter($active, $url_params, $row) {
 	global $_user;
 	if ($active=='1') {
 		$action='AccountActive';
-		$image='right';
+		$image='accept';
 	}
 
 	if ($active=='0') {
 		$action='AccountInactive';
-		$image='wrong';
+		$image='error';
 	}
 	if ($row['0']<>$_user['user_id']) { // you cannot lock yourself out otherwise you could disable all the accounts including your own => everybody is locked out and nobody can change it anymore.
-		$result = '<center><img src="../img/'.$image.'.gif" border="0" style="vertical-align: middle;" alt="'.get_lang(ucfirst($action)).'" title="'.get_lang(ucfirst($action)).'"/></center>';
+		$result = '<center>'.Display::return_icon($image.'.png',  get_lang(ucfirst($action)), array('border'=>'0', 'style'=>'vertical-align: middle;') , 16).'</center>';
 	}
 	return $result;
 }
@@ -622,13 +622,10 @@ $sort_by_first_name = api_sort_by_first_name();
 // Build search-form
 echo '<div class="actions">';
 
-$actions .= '<a href="user.php">'.Display::return_icon('user.png',get_lang('BackToUserList'),'','32').'</a>';
-if ($_GET['keyword'])
-{
+if ($_GET['keyword']) {
 	$actions .= '<a href="subscribe_user.php?type='.Security::remove_XSS($_REQUEST['type']).'">'.Display::return_icon('clean_group.gif').' '.get_lang('ClearSearchResults').'</a>';
 }
-if ($_GET['subscribe_user_filter_value'] AND !empty($_GET['subscribe_user_filter_value']))
-{
+if ($_GET['subscribe_user_filter_value'] AND !empty($_GET['subscribe_user_filter_value'])) {
 	$actions .= '<a href="subscribe_user.php?type='.Security::remove_XSS($_REQUEST['type']).'">'.Display::return_icon('clean_group.gif').' '.get_lang('ClearFilterResults').'</a>';
 }
 if (api_get_setting('ProfilingFilterAddingUsers') == 'true') {