chamilo / chamilo-lms

plugin/ext_auth_chamilo_logout_button_behaviour/index.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
     api_get_setting('cas_activate') == 'true' &&
     $_user['auth_source'] == CAS_AUTH_SOURCE
 ) {
-    $_template['show_message']   = true;
+    $_template['show_message'] = true;
     // the link URL
     $link_url = "#";
     if (!empty($plugin_info['settings']['ext_auth_chamilo_logout_button_behaviour_eaclbb_form_link_url'])) {

plugin/ext_auth_chamilo_logout_button_behaviour/plugin.php

Spacing +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 //the plugin title
-$plugin_info['title']      = 'Enable or disable logout button';
+$plugin_info['title'] = 'Enable or disable logout button';
 
 //the comments that go with the plugin
 $plugin_info['comment']     = "If you use some other authentication than local Chamilo authentication, you may have to disable the Chamilo logout button and give users information about your SSO logout.";
@@ -23,8 +23,8 @@ 
 $form->addElement("html", "<div class='normal-message'>Fill the text boxes below to overwrite the default values used in this plugin</div>");
 $form->addElement('text', 'eaclbb_form_link_url', 'Page to load when clicking on the logout button');
 $form->addElement('text', 'eaclbb_form_link_infobulle', 'Tooltip text for the logout button (displayed on mouseover)');
-$form->addElement('checkbox', 'eaclbb_form_link_image', "Display the logout button disabled (black and white)", "",  array('checked' => true));
-$form->addElement('checkbox', 'eaclbb_form_alert_onoff', "Display an alert when clicking on the logout button", "",  array('checked' => true));
+$form->addElement('checkbox', 'eaclbb_form_link_image', "Display the logout button disabled (black and white)", "", array('checked' => true));
+$form->addElement('checkbox', 'eaclbb_form_alert_onoff', "Display an alert when clicking on the logout button", "", array('checked' => true));
 $form->addElement('text', 'eaclbb_form_alert_text', "Text displayed in the alert box when clickng on the logout button (if checkbox above has been checked).");
 
 $form->addButtonSave(get_lang('Save'), 'submit_button');
@@ -44,4 +44,4 @@ 
 $plugin_info['settings_form'] = $form;
 
 // Set the templates that are going to be used
-$plugin_info['templates']   = array('template.tpl');
+$plugin_info['templates'] = array('template.tpl');

plugin/bbb/cron.php

Spacing +9 added lines, -9 removed lines

@@ -1,16 +1,16 @@ 
 <?php
 
-require __DIR__ . '/../../vendor/autoload.php';
-if (file_exists(__DIR__ . '/config.vm.php')) {
+require __DIR__.'/../../vendor/autoload.php';
+if (file_exists(__DIR__.'/config.vm.php')) {
 
-    require_once dirname(__FILE__) . '/config.php';
+    require_once dirname(__FILE__).'/config.php';
 
-    require __DIR__ . '/lib/vm/AbstractVM.php';
-    require __DIR__ . '/lib/vm/VMInterface.php';
-    require __DIR__ . '/lib/vm/DigitalOceanVM.php';
-    require __DIR__ . '/lib/VM.php';
+    require __DIR__.'/lib/vm/AbstractVM.php';
+    require __DIR__.'/lib/vm/VMInterface.php';
+    require __DIR__.'/lib/vm/DigitalOceanVM.php';
+    require __DIR__.'/lib/VM.php';
 
-    $config = require __DIR__ . '/config.vm.php';
+    $config = require __DIR__.'/config.vm.php';
 
     $vm = new VM($config);
 
@@ -21,7 +21,7 @@ 
             if (empty($activeSessions)) {
                 $vm->runCron();
             } else {
-                echo "Can't run cron active sessions found: " . $activeSessions;
+                echo "Can't run cron active sessions found: ".$activeSessions;
             }
         }
     }

plugin/bbb/lib/VM.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 
         foreach ($vmList as $vm) {
             if (isset($vm['enabled']) && $vm['enabled'] == true) {
-                $className = $vm['name'] . 'VM';
+                $className = $vm['name'].'VM';
 
                 return new $className($vm);
                 break;

plugin/bbb/lib/vm/DigitalOceanVM.php

Spacing +5 added lines, -5 removed lines

@@ -100,7 +100,7 @@ 
                         break;
                 }
             } else {
-                throw new \Exception(" Id " . $this->vmId . " doesn't exists.");
+                throw new \Exception(" Id ".$this->vmId." doesn't exists.");
             }
         } catch (Exception $e) {
             die($e->getMessage());
@@ -118,22 +118,22 @@ 
 
         $powerOff = $droplets->powerOff($this->vmId);
 
-        $this->addMessage('Power off droplet #' . $this->vmId);
+        $this->addMessage('Power off droplet #'.$this->vmId);
 
         $this->waitForEvent($powerOff->event_id);
 
-        $this->addMessage('Current status: ' . $dropletInfo->droplet->status);
+        $this->addMessage('Current status: '.$dropletInfo->droplet->status);
 
         $resizeDroplet = $droplets->resize(
             $this->vmId,
             array('size_id' => intval($sizeId))
         );
-        $this->addMessage('Resize droplet to size id: ' . $sizeId);
+        $this->addMessage('Resize droplet to size id: '.$sizeId);
         $this->waitForEvent($resizeDroplet->event_id);
 
         $powerOn = $droplets->powerOn($this->vmId);
         $this->waitForEvent($powerOn->event_id);
-        $this->addMessage('Power on droplet #' . $this->vmId);
+        $this->addMessage('Power on droplet #'.$this->vmId);
 
     }
 

plugin/dashboard/block_course/block_course.class.php

Spacing +14 added lines, -14 removed lines

@@ -63,19 +63,19 @@ 
         $content = $this->get_content_html();
         $html = '
 		            <div class="panel panel-default" id="intro">
-		                <div class="panel-heading">' . get_lang('CoursesInformation') . '
+		                <div class="panel-heading">' . get_lang('CoursesInformation').'
 		                    <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\'' . addslashes(
                 api_htmlentities(
                     get_lang('ConfirmYourChoice'),
                     ENT_QUOTES,
                     $charset
                 )
-            ) . '\')) return false;" href="index.php?action=disable_block&path=' . $this->path . '">
+            ).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
                 <em class="fa fa-times"></em>
                 </a></div>
 		                </div>
 		                <div class="panel-body">
-		                   ' . $content . '
+		                   ' . $content.'
 		                </div>
 		            </div>
 				';
@@ -93,17 +93,17 @@ 
     {
         $course_data = $this->get_course_information_data();
         //$content = '<div style="margin:10px;">';
-        $content = '<h4>' . get_lang(
+        $content = '<h4>'.get_lang(
                 'YourCourseList'
-            ) . '</h4>';
+            ).'</h4>';
         $data_table = null;
         if (!empty($course_data)) {
             $data_table .= '<table class="data_table" width:"95%">';
             $data_table .= '<tr>
-	    						<th>' . get_lang('CourseTitle') . '</th>
-	    						<th width="20%">' . get_lang('NbStudents') . '</th>
-	    						<th width="20%">' . get_lang('AvgTimeSpentInTheCourse') . '</th>
-	    						<th width="20%">' . get_lang('ThematicAdvance') . '</th>
+	    						<th>' . get_lang('CourseTitle').'</th>
+	    						<th width="20%">' . get_lang('NbStudents').'</th>
+	    						<th width="20%">' . get_lang('AvgTimeSpentInTheCourse').'</th>
+	    						<th width="20%">' . get_lang('ThematicAdvance').'</th>
 	    					</tr>';
             $i = 1;
             foreach ($course_data as $course) {
@@ -112,12 +112,12 @@ 
                 } else {
                     $class_tr = 'row_even';
                 }
-                $data_table .= '<tr class="' . $class_tr . '">';
+                $data_table .= '<tr class="'.$class_tr.'">';
                 if (!isset($course[2])) {
                     $course[2] = '0:00:00';
                 }
                 foreach ($course as $cell) {
-                    $data_table .= '<td align="right">' . $cell . '</td>';
+                    $data_table .= '<td align="right">'.$cell.'</td>';
                 }
                 $data_table .= '</tr>';
                 $i++;
@@ -128,7 +128,7 @@ 
         }
         $content .= $data_table;
         if (!empty($course_data)) {
-            $content .= '<div style="text-align:right;margin-top:10px;"><a href="' . api_get_path(WEB_CODE_PATH) . 'mySpace/course.php?follow">' . get_lang('SeeMore') . '</a></div>';
+            $content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/course.php?follow">'.get_lang('SeeMore').'</a></div>';
         }
         //$content .= '</div>';
 
@@ -163,7 +163,7 @@ 
 
             // students directly subscribed to the course
             $sql = "SELECT user_id FROM $tbl_course_user as course_rel_user
-                    WHERE course_rel_user.status=" . STUDENT . " AND course_rel_user.c_id='$courseId'";
+                    WHERE course_rel_user.status=".STUDENT." AND course_rel_user.c_id='$courseId'";
             $rs = Database::query($sql);
             $users = array();
             while ($row = Database::fetch_array($rs)) {
@@ -182,7 +182,7 @@ 
             );
 
             if (!empty($tematic_advance)) {
-                $tematic_advance_progress = '<a title="' . get_lang('GoToThematicAdvance') . '" href="' . api_get_path(WEB_CODE_PATH) . 'course_progress/index.php?cidReq=' . $course_code . '&action=thematic_details">' . $tematic_advance . '%</a>';
+                $tematic_advance_progress = '<a title="'.get_lang('GoToThematicAdvance').'" href="'.api_get_path(WEB_CODE_PATH).'course_progress/index.php?cidReq='.$course_code.'&action=thematic_details">'.$tematic_advance.'%</a>';
             } else {
                 $tematic_advance_progress = '0%';
             }

plugin/dashboard/block_student_graph/block_student_graph.class.php

Indentation +74 added lines, -74 removed lines

@@ -22,40 +22,40 @@ 
 {
 
     private $user_id;
-	private $students;
-	private $path;
-	private $permission = array(DRH);
+    private $students;
+    private $path;
+    private $permission = array(DRH);
 
-	/**
-	 * Constructor
-	 */
+    /**
+     * Constructor
+     */
     public function __construct ($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_student_graph';
-    	if ($this->is_block_visible_for_user($user_id)) {
-    		/*if (api_is_platform_admin()) {
+        $this->user_id  = $user_id;
+        $this->path 	= 'block_student_graph';
+        if ($this->is_block_visible_for_user($user_id)) {
+            /*if (api_is_platform_admin()) {
 	    		$this->students = UserManager::get_user_list(array('status' => STUDENT));
 	    	} else if (api_is_drh()) {*/
-	    		$this->students =  UserManager::get_users_followed_by_drh($user_id, STUDENT);
-	    	//}
-    	}
+                $this->students =  UserManager::get_users_followed_by_drh($user_id, STUDENT);
+            //}
+        }
     }
 
     /**
-	 * This method check if a user is allowed to see the block inside dashboard interface
-	 * @param	int		User id
-	 * @return	bool	Is block visible for user
-	 */
+     * This method check if a user is allowed to see the block inside dashboard interface
+     * @param	int		User id
+     * @return	bool	Is block visible for user
+     */
     public function is_block_visible_for_user($user_id)
     {
-    	$user_info = api_get_user_info($user_id);
-		$user_status = $user_info['status'];
-		$is_block_visible_for_user = false;
-    	if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
-    		$is_block_visible_for_user = true;
-    	}
-    	return $is_block_visible_for_user;
+        $user_info = api_get_user_info($user_id);
+        $user_status = $user_info['status'];
+        $is_block_visible_for_user = false;
+        if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
+            $is_block_visible_for_user = true;
+        }
+        return $is_block_visible_for_user;
     }
 
     /**
@@ -67,12 +67,12 @@ 
     public function get_block()
     {
 
-    	global $charset;
-    	$column = 1;
-    	$data   = array();
-		$students_attendance_graph = $this->get_students_attendance_graph();
+        global $charset;
+        $column = 1;
+        $data   = array();
+        $students_attendance_graph = $this->get_students_attendance_graph();
 
-		$html = '<div class="panel panel-default" id="intro">
+        $html = '<div class="panel panel-default" id="intro">
 	                <div class="panel-heading">
 	                    '.get_lang('StudentsInformationsGraph').'
 	                    <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES,$charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
@@ -84,51 +84,51 @@ 
 						'.$students_attendance_graph.'
 	                </div>
 	            </div>';
-    	$data['column'] = $column;
-    	$data['content_html'] = $html;
-    	return $data;
+        $data['column'] = $column;
+        $data['content_html'] = $html;
+        return $data;
     }
 
     /**
- 	 * This method return a graph containing information about students evaluation,
+     * This method return a graph containing information about students evaluation,
      * it's used inside get_block method for showing it inside dashboard interface
- 	 * @return string  img html
- 	 */
+     * @return string  img html
+     */
     public function get_students_attendance_graph()
     {
 
-		$students = $this->students;
- 		$attendance = new Attendance();
-
- 		// get data
- 		$attendances_faults_avg = array();
- 		if (is_array($students) && count($students) > 0) {
-	 		foreach ($students as $student) {
-	 			$student_id = $student['user_id'];
-	 			//$student_info = api_get_user_info($student_id);
-				// get average of faults in attendances by student
-	 			$results_faults_avg = $attendance->get_faults_average_inside_courses($student_id);
-
-	 			if (!empty($results_faults_avg)) {
-	 				$attendances_faults_avg[$student['lastname']] = $results_faults_avg['porcent'];
-	 			} else {
-	 				$attendances_faults_avg[$student['lastname']] = 0;
-	 			}
-	 		}
- 		}
+        $students = $this->students;
+            $attendance = new Attendance();
+
+            // get data
+            $attendances_faults_avg = array();
+            if (is_array($students) && count($students) > 0) {
+                foreach ($students as $student) {
+                    $student_id = $student['user_id'];
+                    //$student_info = api_get_user_info($student_id);
+                // get average of faults in attendances by student
+                    $results_faults_avg = $attendance->get_faults_average_inside_courses($student_id);
+
+                    if (!empty($results_faults_avg)) {
+                        $attendances_faults_avg[$student['lastname']] = $results_faults_avg['porcent'];
+                    } else {
+                        $attendances_faults_avg[$student['lastname']] = 0;
+                    }
+                }
+            }
 
- 		arsort($attendances_faults_avg);
-		$usernames = array_keys($attendances_faults_avg);
+            arsort($attendances_faults_avg);
+        $usernames = array_keys($attendances_faults_avg);
 
-		$faults = array();
-		foreach ($usernames as $username) {
-			$faults[] = $attendances_faults_avg[$username];
-		}
+        $faults = array();
+        foreach ($usernames as $username) {
+            $faults[] = $attendances_faults_avg[$username];
+        }
 
-		$graph = '';
-		$img_file = '';
+        $graph = '';
+        $img_file = '';
 
-		if (is_array($usernames) && count($usernames) > 0) {
+        if (is_array($usernames) && count($usernames) > 0) {
 
             // Defining data
             $dataSet = new pData();
@@ -233,19 +233,19 @@ 
                 $imgPath = api_get_path(WEB_ARCHIVE_PATH) . $chartHash;
             }
             $graph = '<img src="' . $imgPath . '" >';
-		} else {
-			$graph = '<p>'.api_convert_encoding(get_lang('GraphicNotAvailable'),'UTF-8').'</p>';
-		}
+        } else {
+            $graph = '<p>'.api_convert_encoding(get_lang('GraphicNotAvailable'),'UTF-8').'</p>';
+        }
 
- 		return $graph;
- 	}
+            return $graph;
+        }
 
     /**
-	 * Get number of students
-	 * @return int
-	 */
-	function get_number_of_students()
+     * Get number of students
+     * @return int
+     */
+    function get_number_of_students()
     {
-		return count($this->students);
-	}
+        return count($this->students);
+    }
 }

Spacing +12 added lines, -12 removed lines

@@ -29,15 +29,15 @@ 
 	/**
 	 * Constructor
 	 */
-    public function __construct ($user_id)
+    public function __construct($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_student_graph';
+    	$this->user_id = $user_id;
+    	$this->path = 'block_student_graph';
     	if ($this->is_block_visible_for_user($user_id)) {
     		/*if (api_is_platform_admin()) {
 	    		$this->students = UserManager::get_user_list(array('status' => STUDENT));
 	    	} else if (api_is_drh()) {*/
-	    		$this->students =  UserManager::get_users_followed_by_drh($user_id, STUDENT);
+	    		$this->students = UserManager::get_users_followed_by_drh($user_id, STUDENT);
 	    	//}
     	}
     }
@@ -75,7 +75,7 @@ 
 		$html = '<div class="panel panel-default" id="intro">
 	                <div class="panel-heading">
 	                    '.get_lang('StudentsInformationsGraph').'
-	                    <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES,$charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
+	                    <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES, $charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
 	                    <em class="fa fa-times"></em>
 	                    </a></div>
 	                </div>
@@ -155,9 +155,9 @@ 
             $myCache = new pCache(array('CacheFolder' => substr($cachePath, 0, strlen($cachePath) - 1)));
             $chartHash = $myCache->getHash($dataSet);
             if ($myCache->isInCache($chartHash)) {
-                $imgPath = api_get_path(SYS_ARCHIVE_PATH) . $chartHash;
+                $imgPath = api_get_path(SYS_ARCHIVE_PATH).$chartHash;
                 $myCache->saveFromCache($chartHash, $imgPath);
-                $imgPath = api_get_path(WEB_ARCHIVE_PATH) . $chartHash;
+                $imgPath = api_get_path(WEB_ARCHIVE_PATH).$chartHash;
             } else {
 
                 $maxCounts = max(count($usernames), count($faults));
@@ -182,7 +182,7 @@ 
                 /* Set the default font */
                 $myPicture->setFontProperties(
                     array(
-                        'FontName' => api_get_path(SYS_FONTS_PATH) . 'opensans/OpenSans-Regular.ttf',
+                        'FontName' => api_get_path(SYS_FONTS_PATH).'opensans/OpenSans-Regular.ttf',
                         'FontSize' => 10
                     )
                 );
@@ -228,13 +228,13 @@ 
 
                 /* Write and save into cache */
                 $myCache->writeToCache($chartHash, $myPicture);
-                $imgPath = api_get_path(SYS_ARCHIVE_PATH) . $chartHash;
+                $imgPath = api_get_path(SYS_ARCHIVE_PATH).$chartHash;
                 $myCache->saveFromCache($chartHash, $imgPath);
-                $imgPath = api_get_path(WEB_ARCHIVE_PATH) . $chartHash;
+                $imgPath = api_get_path(WEB_ARCHIVE_PATH).$chartHash;
             }
-            $graph = '<img src="' . $imgPath . '" >';
+            $graph = '<img src="'.$imgPath.'" >';
 		} else {
-			$graph = '<p>'.api_convert_encoding(get_lang('GraphicNotAvailable'),'UTF-8').'</p>';
+			$graph = '<p>'.api_convert_encoding(get_lang('GraphicNotAvailable'), 'UTF-8').'</p>';
 		}
 
  		return $graph;

plugin/dashboard/block_teacher/block_teacher.class.php

Spacing +13 added lines, -13 removed lines

@@ -27,10 +27,10 @@ 
 	/**
 	 * Controller
 	 */
-    public function __construct ($user_id)
+    public function __construct($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_teacher';
+    	$this->user_id = $user_id;
+    	$this->path = 'block_teacher';
     	if ($this->is_block_visible_for_user($user_id)) {
 	        $this->teachers = UserManager::get_users_followed_by_drh($user_id, COURSEMANAGER);
     	}
@@ -69,7 +69,7 @@ 
                 <div class="panel panel-default" id="intro">
                     <div class="panel-heading">
                         '.get_lang('TeachersInformationsList').'
-                        <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES,$charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
+                        <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES, $charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
                         <em class="fa fa-times"></em>
                         </a></div>
                     </div>
@@ -112,18 +112,18 @@ 
 
 	 			$teacher_id = $teacher['user_id'];
 	 			$firstname 	= $teacher['firstname'];
-	 			$lastname 	= $teacher['lastname'];
-	 			$username	= $teacher['username'];
+	 			$lastname = $teacher['lastname'];
+	 			$username = $teacher['username'];
 
 	 			$time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id));
 	 			$last_connection = Tracking :: get_last_connection_date($teacher_id);
 
-				if ($i%2 == 0) $class_tr = 'row_odd';
+				if ($i % 2 == 0) $class_tr = 'row_odd';
 			    else $class_tr = 'row_even';
 
 				$teachers_table .= '
 									<tr class="'.$class_tr.'">
-										<td>'.api_get_person_name($firstname,$lastname).' ('.$username.')</td>
+										<td>'.api_get_person_name($firstname, $lastname).' ('.$username.')</td>
 										<td align="right">'.$time_on_platform.'</td>
 										<td align="right">'.$last_connection.'</td>
 									</tr>
@@ -155,7 +155,7 @@ 
         $teachers_table = null;
  		if (count($teachers) > 0) {
  			$a_last_week = get_last_week();
- 			$last_week 	 = date('Y-m-d',$a_last_week[0]).' '.get_lang('To').' '.date('Y-m-d', $a_last_week[6]);
+ 			$last_week 	 = date('Y-m-d', $a_last_week[0]).' '.get_lang('To').' '.date('Y-m-d', $a_last_week[6]);
 
 	 		$teachers_table .= '<table class="data_table" width:"95%">';
 	 		$teachers_table .= '
@@ -171,13 +171,13 @@ 
 	 			$teacher_id = $teacher['user_id'];
 	 			$firstname  = $teacher['firstname'];
 	 			$lastname   = $teacher['lastname'];
-				$username	= $teacher['username'];
-	 			$time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id,true));
+				$username = $teacher['username'];
+	 			$time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id, true));
 
-	 			if ($i%2 == 0) $class_tr = 'row_odd';
+	 			if ($i % 2 == 0) $class_tr = 'row_odd';
 	    		else $class_tr = 'row_even';
 	    		$teachers_table .= '<tr class="'.$class_tr.'">
-										<td>'.api_get_person_name($firstname,$lastname).' ('.$username.')</td>
+										<td>'.api_get_person_name($firstname, $lastname).' ('.$username.')</td>
 										<td align="right">'.$time_on_platform.'</td>
 									</tr>';
 

Braces +10 added lines, -4 removed lines

@@ -118,8 +118,11 @@ 
 	 			$time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id));
 	 			$last_connection = Tracking :: get_last_connection_date($teacher_id);
 
-				if ($i%2 == 0) $class_tr = 'row_odd';
-			    else $class_tr = 'row_even';
+				if ($i%2 == 0) {
+				    $class_tr = 'row_odd';
+				} else {
+			        $class_tr = 'row_even';
+			    }
 
 				$teachers_table .= '
 									<tr class="'.$class_tr.'">
@@ -174,8 +177,11 @@ 
 				$username	= $teacher['username'];
 	 			$time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id,true));
 
-	 			if ($i%2 == 0) $class_tr = 'row_odd';
-	    		else $class_tr = 'row_even';
+	 			if ($i%2 == 0) {
+	 			    $class_tr = 'row_odd';
+	 			} else {
+	    		    $class_tr = 'row_even';
+	    		}
 	    		$teachers_table .= '<tr class="'.$class_tr.'">
 										<td>'.api_get_person_name($firstname,$lastname).' ('.$username.')</td>
 										<td align="right">'.$time_on_platform.'</td>

Indentation +98 added lines, -98 removed lines

@@ -24,32 +24,32 @@ 
     private $path;
     private $permission = array(DRH);
 
-	/**
-	 * Controller
-	 */
+    /**
+     * Controller
+     */
     public function __construct ($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_teacher';
-    	if ($this->is_block_visible_for_user($user_id)) {
-	        $this->teachers = UserManager::get_users_followed_by_drh($user_id, COURSEMANAGER);
-    	}
+        $this->user_id  = $user_id;
+        $this->path 	= 'block_teacher';
+        if ($this->is_block_visible_for_user($user_id)) {
+            $this->teachers = UserManager::get_users_followed_by_drh($user_id, COURSEMANAGER);
+        }
     }
 
     /**
-	 * This method check if a user is allowed to see the block inside dashboard interface
-	 * @param	int		User id
-	 * @return	bool	Is block visible for user
-	 */
+     * This method check if a user is allowed to see the block inside dashboard interface
+     * @param	int		User id
+     * @return	bool	Is block visible for user
+     */
     public function is_block_visible_for_user($user_id)
     {
-    	$user_info = api_get_user_info($user_id);
-		$user_status = $user_info['status'];
-		$is_block_visible_for_user = false;
-    	if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
-    		$is_block_visible_for_user = true;
-    	}
-    	return $is_block_visible_for_user;
+        $user_info = api_get_user_info($user_id);
+        $user_status = $user_info['status'];
+        $is_block_visible_for_user = false;
+        if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
+            $is_block_visible_for_user = true;
+        }
+        return $is_block_visible_for_user;
     }
 
     /**
@@ -60,12 +60,12 @@ 
     public function get_block()
     {
 
-    	global $charset;
-    	$column = 1;
-    	$data = array();
-		$teacher_content_html = $this->get_teachers_content_html_for_drh();
+        global $charset;
+        $column = 1;
+        $data = array();
+        $teacher_content_html = $this->get_teachers_content_html_for_drh();
 
-		$html = '
+        $html = '
                 <div class="panel panel-default" id="intro">
                     <div class="panel-heading">
                         '.get_lang('TeachersInformationsList').'
@@ -79,27 +79,27 @@ 
                 </div>
 				';
 
-    	$data['column'] = $column;
-    	$data['content_html'] = $html;
+        $data['column'] = $column;
+        $data['content_html'] = $html;
 
-    	return $data;
+        return $data;
 
     }
 
     /**
- 	 * This method return a content html, it's used inside get_block method for showing it inside dashboard interface
- 	 * @return string  content html
- 	 */
+     * This method return a content html, it's used inside get_block method for showing it inside dashboard interface
+     * @return string  content html
+     */
     public function get_teachers_content_html_for_platform_admin()
     {
-	 	$teachers = $this->teachers;
-		//$content = '<div style="margin:10px;">';
-		$content = '<h4>'.get_lang('YourTeachers').'</h4>';
+            $teachers = $this->teachers;
+        //$content = '<div style="margin:10px;">';
+        $content = '<h4>'.get_lang('YourTeachers').'</h4>';
 
         $teachers_table = null;
-		if (count($teachers) > 0) {
-	 		$teachers_table .= '<table class="data_table" width:"95%">';
-	 		$teachers_table .= '
+        if (count($teachers) > 0) {
+                $teachers_table .= '<table class="data_table" width:"95%">';
+                $teachers_table .= '
 								<tr>
 									<th>'.get_lang('User').'</th>
 									<th>'.get_lang('TimeSpentOnThePlatform').'</th>
@@ -107,103 +107,103 @@ 
 								</tr>
 							';
 
-	 		$i = 1;
-	 		foreach ($teachers as $teacher) {
+                $i = 1;
+                foreach ($teachers as $teacher) {
 
-	 			$teacher_id = $teacher['user_id'];
-	 			$firstname 	= $teacher['firstname'];
-	 			$lastname 	= $teacher['lastname'];
-	 			$username	= $teacher['username'];
+                    $teacher_id = $teacher['user_id'];
+                    $firstname 	= $teacher['firstname'];
+                    $lastname 	= $teacher['lastname'];
+                    $username	= $teacher['username'];
 
-	 			$time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id));
-	 			$last_connection = Tracking :: get_last_connection_date($teacher_id);
+                    $time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id));
+                    $last_connection = Tracking :: get_last_connection_date($teacher_id);
 
-				if ($i%2 == 0) $class_tr = 'row_odd';
-			    else $class_tr = 'row_even';
+                if ($i%2 == 0) $class_tr = 'row_odd';
+                else $class_tr = 'row_even';
 
-				$teachers_table .= '
+                $teachers_table .= '
 									<tr class="'.$class_tr.'">
 										<td>'.api_get_person_name($firstname,$lastname).' ('.$username.')</td>
 										<td align="right">'.$time_on_platform.'</td>
 										<td align="right">'.$last_connection.'</td>
 									</tr>
 									';
-	 			$i++;
-	 		}
-	 		$teachers_table .= '</table>';
-		} else {
-			$teachers_table .= get_lang('ThereIsNoInformationAboutYourTeachers');
-		}
+                    $i++;
+                }
+                $teachers_table .= '</table>';
+        } else {
+            $teachers_table .= get_lang('ThereIsNoInformationAboutYourTeachers');
+        }
 
-	 	$content .= $teachers_table;
+            $content .= $teachers_table;
 
- 		if (count($teachers) > 0) {
-			$content .= '<div style="text-align:right;margin-top:10px;">
+            if (count($teachers) > 0) {
+            $content .= '<div style="text-align:right;margin-top:10px;">
 			<a href="'.api_get_path(WEB_CODE_PATH).'mySpace/index.php?view=admin">'.get_lang('SeeMore').'</a></div>';
-		}
+        }
 
-		//$content .= '</div>';
+        //$content .= '</div>';
 
- 		return $content;
-	}
+            return $content;
+    }
 
-	public function get_teachers_content_html_for_drh()
+    public function get_teachers_content_html_for_drh()
     {
-  		$teachers = $this->teachers;
- 		//$content = '<div style="margin:10px;">';
- 		$content = '<h4>'.get_lang('YourTeachers').'</h4>';
+            $teachers = $this->teachers;
+            //$content = '<div style="margin:10px;">';
+            $content = '<h4>'.get_lang('YourTeachers').'</h4>';
         $teachers_table = null;
- 		if (count($teachers) > 0) {
- 			$a_last_week = get_last_week();
- 			$last_week 	 = date('Y-m-d',$a_last_week[0]).' '.get_lang('To').' '.date('Y-m-d', $a_last_week[6]);
+            if (count($teachers) > 0) {
+                $a_last_week = get_last_week();
+                $last_week 	 = date('Y-m-d',$a_last_week[0]).' '.get_lang('To').' '.date('Y-m-d', $a_last_week[6]);
 
-	 		$teachers_table .= '<table class="data_table" width:"95%">';
-	 		$teachers_table .= '
+                $teachers_table .= '<table class="data_table" width:"95%">';
+                $teachers_table .= '
 								<tr>
 									<th>'.get_lang('User').'</th>
 									<th>'.get_lang('TimeSpentLastWeek').'<br />'.$last_week.'</th>
 								</tr>
 							';
 
-	 		$i = 1;
-	 		foreach ($teachers as $teacher) {
+                $i = 1;
+                foreach ($teachers as $teacher) {
 
-	 			$teacher_id = $teacher['user_id'];
-	 			$firstname  = $teacher['firstname'];
-	 			$lastname   = $teacher['lastname'];
-				$username	= $teacher['username'];
-	 			$time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id,true));
+                    $teacher_id = $teacher['user_id'];
+                    $firstname  = $teacher['firstname'];
+                    $lastname   = $teacher['lastname'];
+                $username	= $teacher['username'];
+                    $time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id,true));
 
-	 			if ($i%2 == 0) $class_tr = 'row_odd';
-	    		else $class_tr = 'row_even';
-	    		$teachers_table .= '<tr class="'.$class_tr.'">
+                    if ($i%2 == 0) $class_tr = 'row_odd';
+                else $class_tr = 'row_even';
+                $teachers_table .= '<tr class="'.$class_tr.'">
 										<td>'.api_get_person_name($firstname,$lastname).' ('.$username.')</td>
 										<td align="right">'.$time_on_platform.'</td>
 									</tr>';
 
-	 			$i++;
-	 		}
-	 		$teachers_table .= '</table>';
- 		} else {
- 			$teachers_table .= get_lang('ThereIsNoInformationAboutYourTeachers');
- 		}
+                    $i++;
+                }
+                $teachers_table .= '</table>';
+            } else {
+                $teachers_table .= get_lang('ThereIsNoInformationAboutYourTeachers');
+            }
 
-  		$content .= $teachers_table;
+            $content .= $teachers_table;
 
- 		if (count($teachers) > 0) {
-			$content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/teachers.php">'.get_lang('SeeMore').'</a></div>';
-		}
-		//$content .= '</div>';
+            if (count($teachers) > 0) {
+            $content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/teachers.php">'.get_lang('SeeMore').'</a></div>';
+        }
+        //$content .= '</div>';
 
-  		return $content;
-  	}
+            return $content;
+        }
 
     /**
-	 * Get number of teachers
-	 * @return int
-	 */
-	function get_number_of_teachers()
+     * Get number of teachers
+     * @return int
+     */
+    function get_number_of_teachers()
     {
-		return count($this->teachers);
-	}
+        return count($this->teachers);
+    }
 }

plugin/dashboard/block_student/block_student.class.php

Spacing +11 added lines, -11 removed lines

@@ -21,12 +21,12 @@ 
 	/**
 	 * Constructor
 	 */
-    public function __construct ($user_id)
+    public function __construct($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_student';
+    	$this->user_id = $user_id;
+    	$this->path = 'block_student';
     	if ($this->is_block_visible_for_user($user_id)) {
-            $this->students =  UserManager::get_users_followed_by_drh($user_id, STUDENT);
+            $this->students = UserManager::get_users_followed_by_drh($user_id, STUDENT);
     	}
     }
 
@@ -61,7 +61,7 @@ 
 		$html = '<div class="panel panel-default" id="intro">
                     <div class="panel-heading">
                         '.get_lang('StudentsInformationsList').'
-                        <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES,$charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
+                        <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES, $charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
                             <em class="fa fa-times"></em>
                         </a>
                         </div>
@@ -102,9 +102,9 @@ 
 
 	 			$courses_by_user = CourseManager::get_courses_list_by_user_id($student['user_id'], true);
 	 			$count_courses = count($courses_by_user);
-				$rowspan = $count_courses?$count_courses+1:2;
+				$rowspan = $count_courses ? $count_courses + 1 : 2;
 
-				if ($i%2 == 0) {
+				if ($i % 2 == 0) {
                     $style = ' style="background-color:#F2F2F2" ';
                 } else {
                     $style = ' style="background-color:#FFF" ';
@@ -169,7 +169,7 @@ 
 	 			$student_id = $student['user_id'];
 	 			$firstname  = $student['firstname'];
 	 			$lastname   = $student['lastname'];
-	 			$username	= $student['username'];
+	 			$username = $student['username'];
 
 
 				// get average of faults in attendances by student
@@ -188,7 +188,7 @@ 
 	 				$cats = Category::load(null, null, $course_code, null, null, null, false);
 	 				$scoretotal = array();
 	 				if (isset($cats) && isset($cats[0])) {
-		 				$scoretotal= $cats[0]->calc_score($student_id, null, $course_code);
+		 				$scoretotal = $cats[0]->calc_score($student_id, null, $course_code);
 	 				}
 
 	 				if (!empty($scoretotal)) {
@@ -198,10 +198,10 @@ 
 	 			}
 
 	 			if (!empty($weight)) {
-	 				$evaluations_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.round($score,2).'/'.round($weight,2).'('.round(($score / $weight) * 100,2) . ' %)</a>';
+	 				$evaluations_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.round($score, 2).'/'.round($weight, 2).'('.round(($score / $weight) * 100, 2).' %)</a>';
 	 			}
 
-	 			if ($i%2 == 0) {
+	 			if ($i % 2 == 0) {
                     $class_tr = 'row_odd';
                 } else {
                     $class_tr = 'row_even';

Indentation +141 added lines, -141 removed lines

@@ -14,36 +14,36 @@ 
 class BlockStudent extends Block
 {
     private $user_id;
-	private $students;
-	private $path;
-	private $permission = array(DRH);
+    private $students;
+    private $path;
+    private $permission = array(DRH);
 
-	/**
-	 * Constructor
-	 */
+    /**
+     * Constructor
+     */
     public function __construct ($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_student';
-    	if ($this->is_block_visible_for_user($user_id)) {
+        $this->user_id  = $user_id;
+        $this->path 	= 'block_student';
+        if ($this->is_block_visible_for_user($user_id)) {
             $this->students =  UserManager::get_users_followed_by_drh($user_id, STUDENT);
-    	}
+        }
     }
 
     /**
-	 * This method check if a user is allowed to see the block inside dashboard interface
-	 * @param	int		User id
-	 * @return	bool	Is block visible for user
-	 */
+     * This method check if a user is allowed to see the block inside dashboard interface
+     * @param	int		User id
+     * @return	bool	Is block visible for user
+     */
     public function is_block_visible_for_user($user_id)
     {
-    	$user_info = api_get_user_info($user_id);
-		$user_status = $user_info['status'];
-		$is_block_visible_for_user = false;
-    	if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
-    		$is_block_visible_for_user = true;
-    	}
-    	return $is_block_visible_for_user;
+        $user_info = api_get_user_info($user_id);
+        $user_status = $user_info['status'];
+        $is_block_visible_for_user = false;
+        if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
+            $is_block_visible_for_user = true;
+        }
+        return $is_block_visible_for_user;
     }
 
     /**
@@ -53,12 +53,12 @@ 
      */
     public function get_block()
     {
-    	global $charset;
-    	$column = 1;
-    	$data   = array();
+        global $charset;
+        $column = 1;
+        $data   = array();
 
         $student_content_html = $this->get_students_content_html_for_drh();
-		$html = '<div class="panel panel-default" id="intro">
+        $html = '<div class="panel panel-default" id="intro">
                     <div class="panel-heading">
                         '.get_lang('StudentsInformationsList').'
                         <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES,$charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
@@ -70,24 +70,24 @@ 
                         '.$student_content_html.'
                     </div>
                     </div>';
-    	$data['column'] = $column;
-    	$data['content_html'] = $html;
-    	return $data;
+        $data['column'] = $column;
+        $data['content_html'] = $html;
+        return $data;
     }
 
     /**
- 	 * This method return a content html, it's used inside get_block method for showing it inside dashboard interface
- 	 * @return string  content html
- 	 */
+     * This method return a content html, it's used inside get_block method for showing it inside dashboard interface
+     * @return string  content html
+     */
     public function get_students_content_html_for_platform_admin()
     {
- 		$students = $this->students;
- 		//$content = '<div style="margin:10px;">';
- 		$content = '<h4>'.get_lang('YourStudents').'</h4>';
+            $students = $this->students;
+            //$content = '<div style="margin:10px;">';
+            $content = '<h4>'.get_lang('YourStudents').'</h4>';
         $students_table = null;
- 		if (count($students) > 0) {
-	 		$students_table .= '<table class="data_table">';
-	 		$students_table .= '<tr>
+            if (count($students) > 0) {
+                $students_table .= '<table class="data_table">';
+                $students_table .= '<tr>
 									<th width="10%" rowspan="2">'.get_lang('FirstName').'</th>
 									<th width="10%" rowspan="2">'.get_lang('LastName').'</th>
 									<th width="30%" colspan="2">'.get_lang('CourseInformation').'</th>
@@ -97,146 +97,146 @@ 
 									<th width="10%">'.get_lang('Time').'</th>
 								</tr>';
 
-	 		$i = 1;
-	 		foreach ($students as $student) {
+                $i = 1;
+                foreach ($students as $student) {
 
-	 			$courses_by_user = CourseManager::get_courses_list_by_user_id($student['user_id'], true);
-	 			$count_courses = count($courses_by_user);
-				$rowspan = $count_courses?$count_courses+1:2;
+                    $courses_by_user = CourseManager::get_courses_list_by_user_id($student['user_id'], true);
+                    $count_courses = count($courses_by_user);
+                $rowspan = $count_courses?$count_courses+1:2;
 
-				if ($i%2 == 0) {
+                if ($i%2 == 0) {
                     $style = ' style="background-color:#F2F2F2" ';
                 } else {
                     $style = ' style="background-color:#FFF" ';
                 }
 
-	 			$students_table .= '<tr '.$style.'>
+                    $students_table .= '<tr '.$style.'>
 										<td rowspan="'.$rowspan.'">'.$student['firstname'].'</td>
 										<td rowspan="'.$rowspan.'">'.$student['lastname'].'</td>
 									</tr>';
 
-	 			// courses information about the student
-	 			if (!empty($courses_by_user)) {
-		 			foreach ($courses_by_user as $course) {
-		 				$course_code = $course['code'];
-						$courseInfo = api_get_course_info($course_code);
-						$courseId = $courseInfo['real_id'];
-		 				$course_title = $course['title'];
-		 				$time = api_time_to_hms(Tracking :: get_time_spent_on_the_course($student['user_id'], $courseId));
-		 				$students_table .= '<tr '.$style.'>
+                    // courses information about the student
+                    if (!empty($courses_by_user)) {
+                        foreach ($courses_by_user as $course) {
+                            $course_code = $course['code'];
+                        $courseInfo = api_get_course_info($course_code);
+                        $courseId = $courseInfo['real_id'];
+                            $course_title = $course['title'];
+                            $time = api_time_to_hms(Tracking :: get_time_spent_on_the_course($student['user_id'], $courseId));
+                            $students_table .= '<tr '.$style.'>
 											<td align="right">'.$course_title.'</td>
 											<td align="right">'.$time.'</td>
 											</tr>';
-		 			}
-	 			} else {
-	 				$students_table .= '<tr '.$style.'>
+                        }
+                    } else {
+                        $students_table .= '<tr '.$style.'>
 											<td align="center" colspan="2"><i>'.get_lang('Empty').'</i></td>
 										</tr>';
-	 			}
-	 			$i++;
-	 		}
-	 		$students_table .= '</table>';
- 		} else {
- 			$students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
- 		}
-
- 		$content .= $students_table;
-
- 		if (count($students) > 0) {
-			$content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/index.php?view=admin&display=useroverview">'.get_lang('SeeMore').'</a></div>';
-		}
-		//$content .= '</div>';
-
- 		return $content;
- 	}
-
-  	public function get_students_content_html_for_drh() {
-  		$attendance = new Attendance();
-  		$students = $this->students;
- 		//$content = '<div style="margin:5px;">';
- 		$content = '<h4>'.get_lang('YourStudents').'</h4>';
+                    }
+                    $i++;
+                }
+                $students_table .= '</table>';
+            } else {
+                $students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
+            }
+
+            $content .= $students_table;
+
+            if (count($students) > 0) {
+            $content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/index.php?view=admin&display=useroverview">'.get_lang('SeeMore').'</a></div>';
+        }
+        //$content .= '</div>';
+
+            return $content;
+        }
+
+        public function get_students_content_html_for_drh() {
+            $attendance = new Attendance();
+            $students = $this->students;
+            //$content = '<div style="margin:5px;">';
+            $content = '<h4>'.get_lang('YourStudents').'</h4>';
         $students_table = null;
- 		if (count($students) > 0) {
-	 		$students_table .= '<table class="data_table">';
-	 		$students_table .= '<tr>
+            if (count($students) > 0) {
+                $students_table .= '<table class="data_table">';
+                $students_table .= '<tr>
 									<th>'.get_lang('User').'</th>
 									<th>'.get_lang('AttendancesFaults').'</th>
 									<th>'.get_lang('Evaluations').'</th>
 								</tr>';
-	 		$i = 1;
-	 		foreach ($students as $student) {
-
-	 			$student_id = $student['user_id'];
-	 			$firstname  = $student['firstname'];
-	 			$lastname   = $student['lastname'];
-	 			$username	= $student['username'];
-
-
-				// get average of faults in attendances by student
-	 			$results_faults_avg = $attendance->get_faults_average_inside_courses($student_id);
-	 			if (!empty($results_faults_avg)) {
-	 				$attendances_faults_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.$results_faults_avg['faults'].'/'.$results_faults_avg['total'].' ('.$results_faults_avg['porcent'].'%)</a>';
-	 			} else {
-	 				$attendances_faults_avg = '0%';
-	 			}
-
-	 			$courses_by_user = CourseManager::get_courses_list_by_user_id($student_id, true);
-				$evaluations_avg  = 0;
-				$score = $weight = 0;
-	 			foreach ($courses_by_user as $course) {
-	 				$course_code = $course['code'];
-	 				$cats = Category::load(null, null, $course_code, null, null, null, false);
-	 				$scoretotal = array();
-	 				if (isset($cats) && isset($cats[0])) {
-		 				$scoretotal= $cats[0]->calc_score($student_id, null, $course_code);
-	 				}
-
-	 				if (!empty($scoretotal)) {
-	 					$score  += $scoretotal[0];
-	 					$weight += $scoretotal[1];
-	 				}
-	 			}
-
-	 			if (!empty($weight)) {
-	 				$evaluations_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.round($score,2).'/'.round($weight,2).'('.round(($score / $weight) * 100,2) . ' %)</a>';
-	 			}
-
-	 			if ($i%2 == 0) {
+                $i = 1;
+                foreach ($students as $student) {
+
+                    $student_id = $student['user_id'];
+                    $firstname  = $student['firstname'];
+                    $lastname   = $student['lastname'];
+                    $username	= $student['username'];
+
+
+                // get average of faults in attendances by student
+                    $results_faults_avg = $attendance->get_faults_average_inside_courses($student_id);
+                    if (!empty($results_faults_avg)) {
+                        $attendances_faults_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.$results_faults_avg['faults'].'/'.$results_faults_avg['total'].' ('.$results_faults_avg['porcent'].'%)</a>';
+                    } else {
+                        $attendances_faults_avg = '0%';
+                    }
+
+                    $courses_by_user = CourseManager::get_courses_list_by_user_id($student_id, true);
+                $evaluations_avg  = 0;
+                $score = $weight = 0;
+                    foreach ($courses_by_user as $course) {
+                        $course_code = $course['code'];
+                        $cats = Category::load(null, null, $course_code, null, null, null, false);
+                        $scoretotal = array();
+                        if (isset($cats) && isset($cats[0])) {
+                            $scoretotal= $cats[0]->calc_score($student_id, null, $course_code);
+                        }
+
+                        if (!empty($scoretotal)) {
+                            $score  += $scoretotal[0];
+                            $weight += $scoretotal[1];
+                        }
+                    }
+
+                    if (!empty($weight)) {
+                        $evaluations_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.round($score,2).'/'.round($weight,2).'('.round(($score / $weight) * 100,2) . ' %)</a>';
+                    }
+
+                    if ($i%2 == 0) {
                     $class_tr = 'row_odd';
                 } else {
                     $class_tr = 'row_even';
                 }
-	    		$students_table .= '<tr class="'.$class_tr.'">
+                $students_table .= '<tr class="'.$class_tr.'">
 										<td>'.api_get_person_name($firstname, $lastname).' ('.$username.')</td>
 										<td>'.$attendances_faults_avg.'</td>
 										<td>'.$evaluations_avg.'</td>
 									</tr>';
 
-	 			$i++;
-	 		}
-	 		$students_table .= '</table>';
- 		} else {
- 			$students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
- 		}
+                    $i++;
+                }
+                $students_table .= '</table>';
+            } else {
+                $students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
+            }
 
-  		$content .= $students_table;
+            $content .= $students_table;
 
- 		if (count($students) > 0) {
-			$content .= '<div style="text-align:right;margin-top:10px;">
+            if (count($students) > 0) {
+            $content .= '<div style="text-align:right;margin-top:10px;">
                             <a href="'.api_get_path(WEB_CODE_PATH).'mySpace/index.php?view=admin&display=yourstudents">'.get_lang('SeeMore').'</a>
                          </div>';
-		}
-		//$content .= '</div>';
+        }
+        //$content .= '</div>';
 
-  		return $content;
-  	}
+            return $content;
+        }
 
     /**
-	 * Get number of students
-	 * @return int
-	 */
-	function get_number_of_students()
+     * Get number of students
+     * @return int
+     */
+    function get_number_of_students()
     {
-		return count($this->students);
-	}
+        return count($this->students);
+    }
 }