[svn r15833] In User's tool is possible to change the status of a user into coach. (see FS#2701)

main/inc/lib/course.lib.php

@@ -1086,13 +1086,22 @@ class CourseManager
 		$session_id = intval($session_id);
 		$a_users = array();		
 		$table_users = Database :: get_main_table(TABLE_MAIN_USER);		
-		
 		$where = array();
 		
+		if ( $session_id == 0 ) 
+        {
+			$sql = 'SELECT DISTINCT course_rel_user.status, user.user_id ';
+		}
+		else
+		{
 			$sql = 'SELECT DISTINCT user.user_id ';
-        if ( $session_id == 0 ) {
+		}
+		
+        if ( $session_id == 0 ) 
+        {
         	$sql .= ', course_rel_user.role, course_rel_user.tutor_id ';
         }	
+        	
         $sql .= ' FROM '.$table_users.' as user ';
 
 		if(api_get_setting('use_session_mode')=='true' && $with_session)
@@ -1127,11 +1136,14 @@ class CourseManager
 		while($user = Database::fetch_array($rs))
 		{
 			$user_infos = Database :: get_user_info_from_id($user['user_id']);					
-			//$user_infos['status'] = $user['status'];
+			$user_infos['status'] = $user['status'];
-            if ( isset($user['role']) ) {
+			//$user['status']=$user_infos['status'] ; 
+            if ( isset($user['role']) ) 
+            {
                 $user_infos['role'] = $user['role'];
             }
-            if ( isset($user['tutor_id']) ) {
+            if ( isset($user['tutor_id']) ) 
+            {
                 $user_infos['tutor_id'] = $user['tutor_id'];
             }                
 			$a_users[$user['user_id']] = $user_infos; 

main/user/user.php

@@ -518,7 +518,7 @@ function get_user_data($from, $number_of_items, $column, $direction)
 				$temp[] = $o_course_user['official_code'];
 				
 				
-	/*			// deprecated feature
+				// deprecated feature
 				if(isset($o_course_user['tutor_id']) && $o_course_user['tutor_id']==1)
 				{
 					$temp[] = get_lang('Tutor');
@@ -527,7 +527,7 @@ function get_user_data($from, $number_of_items, $column, $direction)
 				{
 					$temp[] = '-';
 				}
-	*/			
+				
 				if(isset($o_course_user['status']) && $o_course_user['status']==1)
 				{
 					$temp[] = get_lang('CourseManager');
@@ -653,10 +653,10 @@ $table->set_header($header_nr++, get_lang('OfficialCode'));
 if( api_is_allowed_to_edit())
 {
 	// deprecated feature 
-	// $table->set_header($header_nr++, get_lang('Tutor'),false);
+	$table->set_header($header_nr++, get_lang('Tutor'),false);
 	$table->set_header($header_nr++, get_lang('CourseManager'),false);
 	$table->set_header($header_nr++, get_lang('Active'),false);
-	$table->set_column_filter(7,'active_filter');
+	$table->set_column_filter(8,'active_filter');
 }
 
 //actions column

main/user/userInfo.php

@@ -1,9 +1,9 @@
-<?php // $Id: userInfo.php 15793 2008-07-15 22:13:13Z juliomontoya $
+<?php // $Id: userInfo.php 15833 2008-07-21 18:57:25Z juliomontoya $
 /*
 ==============================================================================
 	Dokeos - elearning and course management software
 
-	Copyright (c) 2004-2008 Dokeos S.A.
+	Copyright (c) 2004-2008 Dokeos SPRL
 	Copyright (c) 2003 hent University (UGent)
 	Copyright (c) 2001 Universite catholique de Louvain (UCL)
 	Copyright (c) various contributors
@@ -54,7 +54,7 @@ $editMainUserInfo = Security::remove_XSS($_REQUEST['editMainUserInfo']);
 $uInfo = $editMainUserInfo; 
 $this_section = SECTION_COURSES;
 
-$nameTools = get_lang("Users");
+$nameTools = get_lang('Users');
 api_protect_course_script(true);
 if(api_is_anonymous())
 {
@@ -65,7 +65,7 @@ if(api_is_anonymous())
 $TBL_USERINFO_DEF 		= Database :: get_course_table(TABLE_USER_INFO);
 $TBL_USERINFO_CONTENT 	= Database :: get_course_table(TABLE_USER_INFO_CONTENT);
 
-$interbreadcrumb[] = array ("url" => "user.php", "name" => get_lang('Users'));
+$interbreadcrumb[] = array ('url' => 'user.php', 'name' => get_lang('Users'));
 
 if ($origin != 'learnpath')
 { //so we are not in learnpath tool
@@ -102,7 +102,6 @@ $courseCode = $currentCourseID = $_course['sysCode'];
 $tbl_coursUser = Database :: get_main_table(TABLE_MAIN_COURSE_USER);
 
 $userIdViewer = $_user['user_id']; // id fo the user currently online
-
 //$userIdViewed = $_GET['userIdViewed']; // Id of the user we want to view
 
 $allowedToEditContent = ($userIdViewer == $userIdViewed) || $is_platformAdmin;
@@ -190,7 +189,7 @@ if ($allowedToEditDef)
 				$userProperties['status'] = 1;
 			}				 	
 			
-			/* deprecated feature
+			// deprecated feature
 			 
 			// is coach
 			if (isset ($_POST['promoteTutor']))
@@ -202,7 +201,7 @@ if ($allowedToEditDef)
 					$userProperties['tutor'] = 1;
 				}							
 			}
-			*/
+			
 			// role is a string
 			if (isset ($_POST['role']))
 			{
@@ -239,7 +238,6 @@ if ($allowedToEditContent)
 	elseif (!empty($_GET['editContent']))
 	{
 		$displayMode = "viewContentEdit";	
-	
 		$userIdViewed = $userIdViewed;
 	}
 }
@@ -355,14 +353,14 @@ elseif ($displayMode == "viewMainInfoEdit")
 				"<table width=\"80%\" border=\"0\">",
 					"<tr align=\"center\" bgcolor=\"#E6E6E6\">\n",
 						"<td align=\"left\">", get_lang('Name'), "</td>\n",
-						"<td align=\"left\">", get_lang('Role'), "</td>\n",						
+						"<td width=\"100px\" align=\"left\">", get_lang('Description'), "</td>\n",	
+						"<td>", get_lang('Tutor'), "</td>\n",					
 						"<td>", get_lang('CourseManager'), "</td>\n",
 					"</tr>\n",
 					"<tr align=\"center\">",
 						"<td align=\"left\"><b>", htmlize($mainUserInfo['firstName']), " ", htmlize($mainUserInfo['lastName']), "</b></td>\n",
-						"<td align=\"left\"><input type=\"text\" name =\"role\" value=\"", $mainUserInfo['role'], "\" maxlength=\"40\" /></td>";
+						"<td align=\"left\"><input type=\"text\" name =\"role\" value=\"", $mainUserInfo['role'], "\" maxlength=\"40\" /></td>",
-						//deprecated feature
+						"<td><input class=\"checkbox\" type=\"checkbox\" name=\"promoteTutor\" value=\"1\" ", $tutorChecked, " /></td>";
-						//"<td><input class=\"checkbox\" type=\"checkbox\" name=\"promoteTutor\" value=\"1\" ", $tutorChecked, " /></td>";
 
 		if (!($is_courseAdmin && $_user['user_id'] == $userIdViewed))
 		{
@@ -388,7 +386,8 @@ elseif ($displayMode == "viewMainInfoEdit")
 				}
 
 	}
-	else{
+	else
+	{
 		Display :: display_normal_message(get_lang('ThisStudentIsSubscribeThroughASession'));
 	}
 }
@@ -408,18 +407,19 @@ elseif ($displayMode == "viewContentList") // default display
 	
 	if ($mainUserInfo)
 	{		
-		
 		$image_array=UserManager::get_user_picture_path_by_id($userIdViewed,'web',false,true);			
 		echo '<img src="'.$image_array['dir'].$image_array['file'].'" border="1">';
 		
 		//DISPLAY TABLE HEADING
 		if ($origin == 'learnpath') { $allowedToEditDef=false; $is_allowedToTrack=false; }
+		
 				//"<td>",get_lang('Tutor'),"</td>\n",
 		echo	"<table width=\"80%\" border=\"0\">",
 
 				"<tr align=\"center\" bgcolor=\"#E6E6E6\">\n",
 				"<td align=\"left\">",get_lang('Name'),"</td>\n",
-				"<td align=\"left\">",get_lang('Role'),"</td>\n",				
+				"<td width=\"100px\" align=\"left\">",get_lang('Description'),"</td>\n",
+				"<td>",get_lang('Tutor'),"</td>\n",				
 				"<td>",get_lang('CourseManager'),"</td>\n",
 				($allowedToEditDef?"<td>".get_lang('Edit')."</td>\n":""),
                 ($is_allowedToTrack?"<td>".get_lang('Tracking')."</td>\n":""),
@@ -431,7 +431,8 @@ elseif ($displayMode == "viewContentList") // default display
 				"<td  align=\"left\">",htmlize($mainUserInfo['role']),"</td>";
 
 				//DISPLAY TABLE CONTENT
-				/* deprecated feature
+				
+				// deprecated feature
 				if ($mainUserInfo['tutor_id'] == 1)
 				{
 					echo "<td>",get_lang('Tutor'),"</td>\n";
@@ -440,7 +441,6 @@ elseif ($displayMode == "viewContentList") // default display
 				{
 					echo "<td> - </td>\n";
 				}				
-				*/
 
 				if ($mainUserInfo['status'] == 1)
 				{

main/user/userInfoLib.php

@@ -266,7 +266,8 @@ function update_user_course_properties($user_id, $course_code, $properties)
     	//feature deprecated   tutor_id      	= '".Database::escape_string($properties['tutor'])."'
     	$sql = "UPDATE $tbl_coursUser
     					SET 	".$sqlChangeStatus."	           
-	                        role      		= '".Database::escape_string($properties['role'])."'	                                                     
+	                    role      		= '".Database::escape_string($properties['role'])."',
+	                    tutor_id      	= '".Database::escape_string($properties['tutor'])."' 	                                                     
 	                    WHERE   user_id	    	= '".$user_id."'
 	                    AND     course_code		= '".$course_code."'";