chamilo / chamilo-lms

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);
+    }
 }

plugin/dashboard/block_evaluation_graph/block_evaluation_graph.class.php

Indentation +102 added lines, -102 removed lines

@@ -23,42 +23,42 @@ 
 class BlockEvaluationGraph extends Block
 {
     private $user_id;
-	private $courses;
-	private $sessions;
-	private $path;
-	private $permission = array(DRH, SESSIONADMIN);
-
-	/**
-	 * Constructor
-	 */
+    private $courses;
+    private $sessions;
+    private $path;
+    private $permission = array(DRH, SESSIONADMIN);
+
+    /**
+     * Constructor
+     */
     public function __construct($user_id)
     {
-    	$this->path = 'block_evaluation_graph';
-    	$this->user_id 	= $user_id;
-    	$this->bg_width = 450;
-    	$this->bg_height = 350;
-    	if ($this->is_block_visible_for_user($user_id)) {
+        $this->path = 'block_evaluation_graph';
+        $this->user_id 	= $user_id;
+        $this->bg_width = 450;
+        $this->bg_height = 350;
+        if ($this->is_block_visible_for_user($user_id)) {
             if (!api_is_session_admin()) {
                 $this->courses  = CourseManager::get_courses_followed_by_drh($user_id);
             }
             $this->sessions = SessionManager::get_sessions_followed_by_drh($user_id);
-    	}
+        }
     }
 
-	/**
-	 * 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;
     }
 
     /**
@@ -68,14 +68,14 @@ 
      */
     public function get_block()
     {
-		global $charset;
-    	$column = 1;
-    	$data   = array();
+        global $charset;
+        $column = 1;
+        $data   = array();
 
-		$evaluations_base_courses_graph         = $this->get_evaluations_base_courses_graph();
-		$evaluations_courses_in_sessions_graph  = $this->get_evaluations_courses_in_sessions_graph();
+        $evaluations_base_courses_graph         = $this->get_evaluations_base_courses_graph();
+        $evaluations_courses_in_sessions_graph  = $this->get_evaluations_courses_in_sessions_graph();
 
-		$html = '<div class="panel panel-default" id="intro">
+        $html = '<div class="panel panel-default" id="intro">
                     <div class="panel-heading">
                         '.get_lang('EvaluationsGraph').'
                         <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.'">
@@ -105,43 +105,43 @@ 
                 }
             }
         }
-		$html .= '</div>
+        $html .= '</div>
 			     </div>';
 
-    	$data['column'] = $column;
-    	$data['content_html'] = $html;
+        $data['column'] = $column;
+        $data['content_html'] = $html;
 
-    	return $data;
-	}
+        return $data;
+    }
 
     /**
- 	 * This method return a graph containing informations about evaluations
+     * This method return a graph containing informations about evaluations
      * inside base courses, it's used inside get_block method for showing
      * it inside dashboard interface
- 	 * @return string  img html
- 	 */
+     * @return string  img html
+     */
     public function get_evaluations_base_courses_graph()
     {
-		$graphs = array();
-		if (!empty($this->courses)) {
-			$courses_code = array_keys($this->courses);
-			foreach ($courses_code as $course_code) {
-				$cats = Category::load(null, null, $course_code, null, null, null, false);
-
-				if (isset($cats) && isset($cats[0])) {
-					$alleval = $cats[0]->get_evaluations(null, true, $course_code);
-					$alllinks = $cats[0]->get_links(null, true);
-					$users = GradebookUtils::get_all_users($alleval, $alllinks);
-					$datagen = new FlatViewDataGenerator ($users, $alleval, $alllinks);
-					$evaluation_sumary = $datagen->get_evaluation_sumary_results();
-					if (!empty($evaluation_sumary)) {
-						$items = array_keys($evaluation_sumary);
-						$max = $min = $avg = array();
-						foreach ($evaluation_sumary as $evaluation) {
-							$max[] = $evaluation['max'];
+        $graphs = array();
+        if (!empty($this->courses)) {
+            $courses_code = array_keys($this->courses);
+            foreach ($courses_code as $course_code) {
+                $cats = Category::load(null, null, $course_code, null, null, null, false);
+
+                if (isset($cats) && isset($cats[0])) {
+                    $alleval = $cats[0]->get_evaluations(null, true, $course_code);
+                    $alllinks = $cats[0]->get_links(null, true);
+                    $users = GradebookUtils::get_all_users($alleval, $alllinks);
+                    $datagen = new FlatViewDataGenerator ($users, $alleval, $alllinks);
+                    $evaluation_sumary = $datagen->get_evaluation_sumary_results();
+                    if (!empty($evaluation_sumary)) {
+                        $items = array_keys($evaluation_sumary);
+                        $max = $min = $avg = array();
+                        foreach ($evaluation_sumary as $evaluation) {
+                            $max[] = $evaluation['max'];
                             $min[] = !empty($evaluation['min']) ? $evaluation['min'] : 0;
-							$avg[] = $evaluation['avg'];
-						}
+                            $avg[] = $evaluation['avg'];
+                        }
                         // Dataset definition
                         $dataSet = new pData();
                         $dataSet->addPoints($min, 'Serie3');
@@ -283,43 +283,43 @@ 
                         if (!empty($imgPath)) {
                             $courses_graph[$course_code] = '<img src="' . $imgPath . '">';
                         }
-					}
-				}
-			} // end for
-		}
-    	return $graphs;
- 	}
-
-	/**
- 	 * This method return a graph containing information about evaluations
+                    }
+                }
+            } // end for
+        }
+        return $graphs;
+        }
+
+    /**
+     * This method return a graph containing information about evaluations
      * inside courses in sessions, it's used inside get_block method for
      * showing it inside dashboard interface
- 	 * @return string  img html
- 	 */
+     * @return string  img html
+     */
     public function get_evaluations_courses_in_sessions_graph()
     {
-		$graphs = array();
-		if (!empty($this->sessions)) {
-			$session_ids = array_keys($this->sessions);
-			foreach ($session_ids as $session_id) {
-				$courses_code = array_keys(Tracking::get_courses_list_from_session($session_id));
-				$courses_graph = array();
-				foreach ($courses_code as $course_code) {
-					$cats = Category::load(null, null, $course_code, null, null, $session_id);
-					if (isset($cats) && isset($cats[0])) {
-						$alleval = $cats[0]->get_evaluations(null, true, $course_code);
-						$alllinks = $cats[0]->get_links(null, true);
-						$users = GradebookUtils::get_all_users($alleval, $alllinks);
-						$datagen = new FlatViewDataGenerator ($users, $alleval, $alllinks);
-						$evaluation_sumary = $datagen->get_evaluation_sumary_results();
-						if (!empty($evaluation_sumary)) {
-							$items = array_keys($evaluation_sumary);
-							$max = $min = $avg = array();
-							foreach ($evaluation_sumary as $evaluation) {
-								$max[] = $evaluation['max'];
-								$min[] = $evaluation['min'];
-								$avg[] = $evaluation['avg'];
-							}
+        $graphs = array();
+        if (!empty($this->sessions)) {
+            $session_ids = array_keys($this->sessions);
+            foreach ($session_ids as $session_id) {
+                $courses_code = array_keys(Tracking::get_courses_list_from_session($session_id));
+                $courses_graph = array();
+                foreach ($courses_code as $course_code) {
+                    $cats = Category::load(null, null, $course_code, null, null, $session_id);
+                    if (isset($cats) && isset($cats[0])) {
+                        $alleval = $cats[0]->get_evaluations(null, true, $course_code);
+                        $alllinks = $cats[0]->get_links(null, true);
+                        $users = GradebookUtils::get_all_users($alleval, $alllinks);
+                        $datagen = new FlatViewDataGenerator ($users, $alleval, $alllinks);
+                        $evaluation_sumary = $datagen->get_evaluation_sumary_results();
+                        if (!empty($evaluation_sumary)) {
+                            $items = array_keys($evaluation_sumary);
+                            $max = $min = $avg = array();
+                            foreach ($evaluation_sumary as $evaluation) {
+                                $max[] = $evaluation['max'];
+                                $min[] = $evaluation['min'];
+                                $avg[] = $evaluation['avg'];
+                            }
                             // Dataset definition
                             $dataSet = new pData();
                             $dataSet->addPoints($min, 'Serie3');
@@ -455,14 +455,14 @@ 
                             if (!empty($imgPath)) {
                                 $courses_graph[$course_code] = '<img src="' . $imgPath . '">';
                             }
-						}
-					}
-				}
-				if (!empty($courses_graph)) {
-					$graphs[$session_id] = $courses_graph;
-				}
-			}
-		}
-    	return $graphs;
- 	}
+                        }
+                    }
+                }
+                if (!empty($courses_graph)) {
+                    $graphs[$session_id] = $courses_graph;
+                }
+            }
+        }
+        return $graphs;
+        }
 }

Spacing +19 added lines, -19 removed lines

@@ -39,7 +39,7 @@ 
     	$this->bg_height = 350;
     	if ($this->is_block_visible_for_user($user_id)) {
             if (!api_is_session_admin()) {
-                $this->courses  = CourseManager::get_courses_followed_by_drh($user_id);
+                $this->courses = CourseManager::get_courses_followed_by_drh($user_id);
             }
             $this->sessions = SessionManager::get_sessions_followed_by_drh($user_id);
     	}
@@ -78,13 +78,13 @@ 
 		$html = '<div class="panel panel-default" id="intro">
                     <div class="panel-heading">
                         '.get_lang('EvaluationsGraph').'
-                        <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>
                     <div class="panel-body">';
         if (empty($evaluations_base_courses_graph) && empty($evaluations_courses_in_sessions_graph)) {
-            $html .= '<p>'.api_convert_encoding(get_lang('GraphicNotAvailable'),'UTF-8').'</p>';
+            $html .= '<p>'.api_convert_encoding(get_lang('GraphicNotAvailable'), 'UTF-8').'</p>';
         } else {
             // display evaluations base courses graph
             if (!empty($evaluations_base_courses_graph)) {
@@ -132,7 +132,7 @@ 
 					$alleval = $cats[0]->get_evaluations(null, true, $course_code);
 					$alllinks = $cats[0]->get_links(null, true);
 					$users = GradebookUtils::get_all_users($alleval, $alllinks);
-					$datagen = new FlatViewDataGenerator ($users, $alleval, $alllinks);
+					$datagen = new FlatViewDataGenerator($users, $alleval, $alllinks);
 					$evaluation_sumary = $datagen->get_evaluation_sumary_results();
 					if (!empty($evaluation_sumary)) {
 						$items = array_keys($evaluation_sumary);
@@ -158,16 +158,16 @@ 
 
                         $dataSet->normalize(100, '%');
 
-                        $dataSet->loadPalette(api_get_path(SYS_CODE_PATH) . 'palettes/pchart/default.color', true);
+                        $dataSet->loadPalette(api_get_path(SYS_CODE_PATH).'palettes/pchart/default.color', true);
 
                         // Cache definition
                         $cachePath = api_get_path(SYS_ARCHIVE_PATH);
                         $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 {
                             /* Create the pChart object */
                             $widthSize = $this->bg_width;
@@ -196,7 +196,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
                                 )
                             );
@@ -276,12 +276,12 @@ 
                             /* 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;
                         }
                         if (!empty($imgPath)) {
-                            $courses_graph[$course_code] = '<img src="' . $imgPath . '">';
+                            $courses_graph[$course_code] = '<img src="'.$imgPath.'">';
                         }
 					}
 				}
@@ -310,7 +310,7 @@ 
 						$alleval = $cats[0]->get_evaluations(null, true, $course_code);
 						$alllinks = $cats[0]->get_links(null, true);
 						$users = GradebookUtils::get_all_users($alleval, $alllinks);
-						$datagen = new FlatViewDataGenerator ($users, $alleval, $alllinks);
+						$datagen = new FlatViewDataGenerator($users, $alleval, $alllinks);
 						$evaluation_sumary = $datagen->get_evaluation_sumary_results();
 						if (!empty($evaluation_sumary)) {
 							$items = array_keys($evaluation_sumary);
@@ -336,16 +336,16 @@ 
 
                             $dataSet->normalize(100, '%');
 
-                            $dataSet->loadPalette(api_get_path(SYS_CODE_PATH) . 'palettes/pchart/default.color', true);
+                            $dataSet->loadPalette(api_get_path(SYS_CODE_PATH).'palettes/pchart/default.color', true);
 
                             // Cache definition
                             $cachePath = api_get_path(SYS_ARCHIVE_PATH);
                             $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 {
                                 /* Create the pChart object */
                                 $widthSize = $this->bg_width;
@@ -374,7 +374,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
                                     )
                                 );
@@ -448,12 +448,12 @@ 
 
                                 /* 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;
                             }
                             if (!empty($imgPath)) {
-                                $courses_graph[$course_code] = '<img src="' . $imgPath . '">';
+                                $courses_graph[$course_code] = '<img src="'.$imgPath.'">';
                             }
 						}
 					}

plugin/dashboard/block_session/block_session.class.php

Spacing +4 added lines, -4 removed lines

@@ -22,9 +22,9 @@ 
 	/**
 	 * Constructor
 	 */
-    public function __construct ($user_id)
+    public function __construct($user_id)
     {
-    	$this->user_id 	= $user_id;
+    	$this->user_id = $user_id;
     	$this->path = 'block_session';
     	if ($this->is_block_visible_for_user($user_id)) {
             $this->sessions = SessionManager::get_sessions_followed_by_drh($user_id);
@@ -66,7 +66,7 @@ 
 			            <div class="panel panel-default" id="intro">
 			                <div class="panel-heading">
 			                    '.get_lang('SessionsInformation').'
-			                    <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>
@@ -116,7 +116,7 @@ 
 
 	 			$count_courses_in_session = count(Tracking::get_courses_list_from_session($session_id));
 
-				if ($i%2 == 0) $class_tr = 'row_odd';
+				if ($i % 2 == 0) $class_tr = 'row_odd';
 	    		else $class_tr = 'row_even';
 
 				$sessions_table .= '<tr class="'.$class_tr.'">

Braces +5 added lines, -2 removed lines

@@ -116,8 +116,11 @@
 
 	 			$count_courses_in_session = count(Tracking::get_courses_list_from_session($session_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';
+	    		}
 
 				$sessions_table .= '<tr class="'.$class_tr.'">
 										<td>'.$title.'</td>

Indentation +72 added lines, -72 removed lines

@@ -15,36 +15,36 @@ 
 class BlockSession extends Block
 {
     private $user_id;
-	private $sessions;
-	private $path;
-	private $permission = array(DRH, SESSIONADMIN);
+    private $sessions;
+    private $path;
+    private $permission = array(DRH, SESSIONADMIN);
 
-	/**
-	 * Constructor
-	 */
+    /**
+     * Constructor
+     */
     public function __construct ($user_id)
     {
-    	$this->user_id 	= $user_id;
-    	$this->path = 'block_session';
-    	if ($this->is_block_visible_for_user($user_id)) {
+        $this->user_id 	= $user_id;
+        $this->path = 'block_session';
+        if ($this->is_block_visible_for_user($user_id)) {
             $this->sessions = SessionManager::get_sessions_followed_by_drh($user_id);
-    	}
+        }
     }
 
-	/**
-	 * 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;
     }
 
     /**
@@ -55,14 +55,14 @@ 
     public function get_block()
     {
 
-		global $charset;
+        global $charset;
 
-    	$column = 2;
-    	$data   = array();
+        $column = 2;
+        $data   = array();
 
-		$content = $this->get_content_html();
+        $content = $this->get_content_html();
 
-		$content_html = '
+        $content_html = '
 			            <div class="panel panel-default" id="intro">
 			                <div class="panel-heading">
 			                    '.get_lang('SessionsInformation').'
@@ -76,78 +76,78 @@ 
 			            </div>
 				';
 
-    	$data['column'] = $column;
-    	$data['content_html'] = $content_html;
+        $data['column'] = $column;
+        $data['content_html'] = $content_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_content_html()
     {
 
- 		$content = '';
-		$sessions = $this->sessions;
+            $content = '';
+        $sessions = $this->sessions;
 
-		//$content = '<div style="margin:10px;">';
-		$content .= '<h4>'.get_lang('YourSessionsList').'</h4>';
+        //$content = '<div style="margin:10px;">';
+        $content .= '<h4>'.get_lang('YourSessionsList').'</h4>';
 
-		if (count($sessions) > 0) {
-			$sessions_table = '<table class="data_table" width:"95%">';
- 			$sessions_table .= '<tr>
+        if (count($sessions) > 0) {
+            $sessions_table = '<table class="data_table" width:"95%">';
+                $sessions_table .= '<tr>
 									<th >'.get_lang('Title').'</th>
 									<th >'.get_lang('Date').'</th>
 									<th width="100px">'.get_lang('NbCoursesPerSession').'</th>
 								</tr>';
-			$i = 1;
-			foreach ($sessions as $session) {
+            $i = 1;
+            foreach ($sessions as $session) {
 
-				$session_id = intval($session['id']);
-				$title = $session['name'];
+                $session_id = intval($session['id']);
+                $title = $session['name'];
 
-				if ($session['access_start_date'] != '0000-00-00 00:00:00' && $session['access_end_date'] != '0000-00-00 00:00:00') {
-					$date = get_lang('From').' '.api_convert_and_format_date($session['access_start_date'], DATE_FORMAT_SHORT, date_default_timezone_get()).' '.get_lang('To').' '.api_convert_and_format_date($session['access_end_date'], DATE_FORMAT_SHORT, date_default_timezone_get());
-				} else {
-					$date = ' - ';
-				}
+                if ($session['access_start_date'] != '0000-00-00 00:00:00' && $session['access_end_date'] != '0000-00-00 00:00:00') {
+                    $date = get_lang('From').' '.api_convert_and_format_date($session['access_start_date'], DATE_FORMAT_SHORT, date_default_timezone_get()).' '.get_lang('To').' '.api_convert_and_format_date($session['access_end_date'], DATE_FORMAT_SHORT, date_default_timezone_get());
+                } else {
+                    $date = ' - ';
+                }
 
-	 			$count_courses_in_session = count(Tracking::get_courses_list_from_session($session_id));
+                    $count_courses_in_session = count(Tracking::get_courses_list_from_session($session_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';
 
-				$sessions_table .= '<tr class="'.$class_tr.'">
+                $sessions_table .= '<tr class="'.$class_tr.'">
 										<td>'.$title.'</td>
 										<td align="center">'.$date.'</td>
 										<td align="center">'.$count_courses_in_session.'</td>
 								   </tr>';
-				$i++;
-			}
-			$sessions_table .= '</table>';
-			$content .= $sessions_table;
-		} else {
-			$content .= get_lang('ThereIsNoInformationAboutYourSessions');
-		}
+                $i++;
+            }
+            $sessions_table .= '</table>';
+            $content .= $sessions_table;
+        } else {
+            $content .= get_lang('ThereIsNoInformationAboutYourSessions');
+        }
 
-		if (count($sessions) > 0) {
-			$content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/session.php">'.get_lang('SeeMore').'</a></div>';
-		}
+        if (count($sessions) > 0) {
+            $content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/session.php">'.get_lang('SeeMore').'</a></div>';
+        }
 
-		//$content .= '</div>';
+        //$content .= '</div>';
 
- 		return $content;
- 	}
+            return $content;
+        }
 
     /**
-	 * Get number of sessions
-	 * @return int
-	 */
-	function get_number_of_sessions()
+     * Get number of sessions
+     * @return int
+     */
+    function get_number_of_sessions()
     {
-		return count($this->sessions);
-	}
+        return count($this->sessions);
+    }
 
 }

plugin/dashboard/block_global_info/block_global_info.class.php

Indentation +47 added lines, -47 removed lines

@@ -45,13 +45,13 @@ 
      */
     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;
     }
 
 
@@ -62,11 +62,11 @@ 
      */
     public function get_block()
     {
-    	global $charset;
-    	$column = 2;
-    	$data   = array();
+        global $charset;
+        $column = 2;
+        $data   = array();
         $content = $this->get_content_html();
-    	$html = '
+        $html = '
     	            <div class="panel panel-default" id="intro">
     	                <div class="panel-heading">'.get_lang('GlobalPlatformInformation').'
     	                    <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.'">
@@ -78,46 +78,46 @@ 
     	                </div>
     	            </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
- 	 */
-     public function get_content_html()
-     {
-         $global_data = $this->get_global_information_data();
-         //$content = '<div style="margin:10px;">';
-         $content = '<h4>'.get_lang('GlobalPlatformInformation').'</h4>';
-         $data_table = null;
-         if (!empty($global_data)) {
-             $data_table = '<table class="table table-bordered" width:"95%">';
-             $i = 1;
-             foreach ($global_data as $data) {
-                 if ($i%2 == 0) {
-                     $class_tr = 'row_odd';
-                 } else {
-                     $class_tr = 'row_even';
-                 }
-                 $data_table .= '<tr class="'.$class_tr.'">';
-                 foreach ($data as $cell) {
-                     $data_table .= '<td align="right">'.$cell.'</td>';
-                 }
-                 $data_table .= '</tr>';
-                 $i++;
-             }
-             $data_table .= '</table>';
-         } else {
-             $data_table .= get_lang('ThereIsNoInformationAboutThePlatform');
-         }
-         $content .= $data_table;
-         //$content .= '</div>';
+        /**
+         * 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_content_html()
+        {
+            $global_data = $this->get_global_information_data();
+            //$content = '<div style="margin:10px;">';
+            $content = '<h4>'.get_lang('GlobalPlatformInformation').'</h4>';
+            $data_table = null;
+            if (!empty($global_data)) {
+                $data_table = '<table class="table table-bordered" width:"95%">';
+                $i = 1;
+                foreach ($global_data as $data) {
+                    if ($i%2 == 0) {
+                        $class_tr = 'row_odd';
+                    } else {
+                        $class_tr = 'row_even';
+                    }
+                    $data_table .= '<tr class="'.$class_tr.'">';
+                    foreach ($data as $cell) {
+                        $data_table .= '<td align="right">'.$cell.'</td>';
+                    }
+                    $data_table .= '</tr>';
+                    $i++;
+                }
+                $data_table .= '</table>';
+            } else {
+                $data_table .= get_lang('ThereIsNoInformationAboutThePlatform');
+            }
+            $content .= $data_table;
+            //$content .= '</div>';
 
-         return $content;
+            return $content;
     }
 
     /**

Spacing +5 added lines, -5 removed lines

@@ -29,10 +29,10 @@ 
     /**
      * Constructor
      */
-    public function __construct ($user_id)
+    public function __construct($user_id)
     {
-        $this->user_id 		= $user_id;
-        $this->path 		= 'block_global_info';
+        $this->user_id = $user_id;
+        $this->path = 'block_global_info';
         if ($this->is_block_visible_for_user($user_id)) {
             //$this->courses = CourseManager::get_courses_followed_by_drh($user_id);
         }
@@ -69,7 +69,7 @@ 
     	$html = '
     	            <div class="panel panel-default" id="intro">
     	                <div class="panel-heading">'.get_lang('GlobalPlatformInformation').'
-    	                    <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>
@@ -98,7 +98,7 @@ 
              $data_table = '<table class="table table-bordered" width:"95%">';
              $i = 1;
              foreach ($global_data as $data) {
-                 if ($i%2 == 0) {
+                 if ($i % 2 == 0) {
                      $class_tr = 'row_odd';
                  } else {
                      $class_tr = 'row_even';

plugin/dashboard/block_teacher_graph/block_teacher_graph.class.php

Spacing +14 added lines, -14 removed lines

@@ -29,10 +29,10 @@ 
 	/**
 	 * Controller
 	 */
-    public function __construct ($user_id)
+    public function __construct($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_teacher_graph';
+    	$this->user_id = $user_id;
+    	$this->path = 'block_teacher_graph';
     	if ($this->is_block_visible_for_user($user_id)) {
             $this->teachers = UserManager::get_users_followed_by_drh($user_id, COURSEMANAGER);
     	}
@@ -68,7 +68,7 @@ 
 		$html = '
                 <div class="panel panel-default" id="intro">
                     <div class="panel-heading">'.get_lang('TeachersInformationsGraph').'
-                        <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>
@@ -107,7 +107,7 @@ 
 				foreach ($a_last_week as $day) {
 					// day is received as y-m-d 12:00:00
 					$start_date = api_get_utc_datetime($day);
-					$end_date = api_get_utc_datetime($day+(3600*24-1));
+					$end_date = api_get_utc_datetime($day + (3600 * 24 - 1));
 
 					$time_on_platform_by_day = Tracking::get_time_spent_on_the_platform($user_id, 'custom', $start_date, $end_date);
 					$hours = floor($time_on_platform_by_day / 3600);
@@ -117,26 +117,26 @@ 
 				$dataSet->addPoints($time_by_days, $username);
 			}
 
-			$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]);
 			$days_on_week = array();
 			foreach ($a_last_week as $weekday) {
-				$days_on_week[] = date('d/m',$weekday);
+				$days_on_week[] = date('d/m', $weekday);
 			}
 
 			$dataSet->addPoints($days_on_week, 'Days');
 			$dataSet->setAbscissaName($last_week);
 			$dataSet->setAxisName(0, get_lang('Minutes'));
             $dataSet->setAbscissa('Days');
-            $dataSet->loadPalette(api_get_path(SYS_CODE_PATH) . 'palettes/pchart/default.color', true);
+            $dataSet->loadPalette(api_get_path(SYS_CODE_PATH).'palettes/pchart/default.color', true);
 
             // Cache definition
             $cachePath = api_get_path(SYS_ARCHIVE_PATH);
             $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 {
 
                 /* Create the pChart object */
@@ -153,7 +153,7 @@ 
                 $myPicture->drawRectangle(0, 0, $widthSize - 1, $heightSize - 1, array('R' => 0, 'G' => 0, 'B' => 0));
 
                 /* Set the default font */
-                $myPicture->setFontProperties(array('FontName' => api_get_path(SYS_FONTS_PATH) . 'opensans/OpenSans-Regular.ttf', 'FontSize' => 10));
+                $myPicture->setFontProperties(array('FontName' => api_get_path(SYS_FONTS_PATH).'opensans/OpenSans-Regular.ttf', 'FontSize' => 10));
 
                 /* Do NOT Write the chart title */
 
@@ -189,11 +189,11 @@ 
 
                 /* 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>';
 		}

Indentation +74 added lines, -74 removed lines

@@ -26,32 +26,32 @@ 
     private $path;
     private $permission = array(DRH);
 
-	/**
-	 * Controller
-	 */
+    /**
+     * Controller
+     */
     public function __construct ($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_teacher_graph';
-    	if ($this->is_block_visible_for_user($user_id)) {
+        $this->user_id  = $user_id;
+        $this->path 	= 'block_teacher_graph';
+        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;
     }
 
     /**
@@ -61,11 +61,11 @@ 
      */
     public function get_block()
     {
-    	global $charset;
-    	$column = 1;
-    	$data   = array();
-		$teacher_information_graph = $this->get_teachers_information_graph();
-		$html = '
+        global $charset;
+        $column = 1;
+        $data   = array();
+        $teacher_information_graph = $this->get_teachers_information_graph();
+        $html = '
                 <div class="panel panel-default" id="intro">
                     <div class="panel-heading">'.get_lang('TeachersInformationsGraph').'
                         <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.'">
@@ -79,53 +79,53 @@ 
                 </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_information_graph()
     {
-	 	$teachers = $this->teachers;
-		$graph = '';
-
- 		$user_ids = array_keys($teachers);
- 		$a_last_week = get_last_week();
-
-		if (is_array($user_ids) && count($user_ids) > 0) {
-			$dataSet = new pData;
-			foreach ($user_ids as $user_id) {
-				$teacher_info = api_get_user_info($user_id);
-				$username = $teacher_info['username'];
-				$time_by_days = array();
-				foreach ($a_last_week as $day) {
-					// day is received as y-m-d 12:00:00
-					$start_date = api_get_utc_datetime($day);
-					$end_date = api_get_utc_datetime($day+(3600*24-1));
-
-					$time_on_platform_by_day = Tracking::get_time_spent_on_the_platform($user_id, 'custom', $start_date, $end_date);
-					$hours = floor($time_on_platform_by_day / 3600);
-					$min = floor(($time_on_platform_by_day - ($hours * 3600)) / 60);
-					$time_by_days[] = $min;
-				}
-				$dataSet->addPoints($time_by_days, $username);
-			}
-
-			$last_week 	 = date('Y-m-d',$a_last_week[0]).' '.get_lang('To').' '.date('Y-m-d', $a_last_week[6]);
-			$days_on_week = array();
-			foreach ($a_last_week as $weekday) {
-				$days_on_week[] = date('d/m',$weekday);
-			}
-
-			$dataSet->addPoints($days_on_week, 'Days');
-			$dataSet->setAbscissaName($last_week);
-			$dataSet->setAxisName(0, get_lang('Minutes'));
+            $teachers = $this->teachers;
+        $graph = '';
+
+            $user_ids = array_keys($teachers);
+            $a_last_week = get_last_week();
+
+        if (is_array($user_ids) && count($user_ids) > 0) {
+            $dataSet = new pData;
+            foreach ($user_ids as $user_id) {
+                $teacher_info = api_get_user_info($user_id);
+                $username = $teacher_info['username'];
+                $time_by_days = array();
+                foreach ($a_last_week as $day) {
+                    // day is received as y-m-d 12:00:00
+                    $start_date = api_get_utc_datetime($day);
+                    $end_date = api_get_utc_datetime($day+(3600*24-1));
+
+                    $time_on_platform_by_day = Tracking::get_time_spent_on_the_platform($user_id, 'custom', $start_date, $end_date);
+                    $hours = floor($time_on_platform_by_day / 3600);
+                    $min = floor(($time_on_platform_by_day - ($hours * 3600)) / 60);
+                    $time_by_days[] = $min;
+                }
+                $dataSet->addPoints($time_by_days, $username);
+            }
+
+            $last_week 	 = date('Y-m-d',$a_last_week[0]).' '.get_lang('To').' '.date('Y-m-d', $a_last_week[6]);
+            $days_on_week = array();
+            foreach ($a_last_week as $weekday) {
+                $days_on_week[] = date('d/m',$weekday);
+            }
+
+            $dataSet->addPoints($days_on_week, 'Days');
+            $dataSet->setAbscissaName($last_week);
+            $dataSet->setAxisName(0, get_lang('Minutes'));
             $dataSet->setAbscissa('Days');
             $dataSet->loadPalette(api_get_path(SYS_CODE_PATH) . 'palettes/pchart/default.color', true);
 
@@ -194,20 +194,20 @@ 
                 $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 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_daily/block_daily.class.php

Spacing +21 added lines, -21 removed lines

@@ -72,19 +72,19 @@ 
         $data = array();
         $content = $this->get_content_html();
         $html = '<div class="panel panel-default" id="intro">
-		                <div class="panel-heading">' . get_lang('GradebookAndAttendances') . '
+		                <div class="panel-heading">' . get_lang('GradebookAndAttendances').'
 		                    <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>
 				';
@@ -102,17 +102,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('Evaluation') . '</th>
-	    						<th width="20%">' . get_lang('ToolAttendance') . '</th>
+	    						<th>' . get_lang('CourseTitle').'</th>
+	    						<th width="20%">' . get_lang('NbStudents').'</th>
+	    						<th width="20%">' . get_lang('Evaluation').'</th>
+	    						<th width="20%">' . get_lang('ToolAttendance').'</th>
 	    					</tr>';
             $i = 1;
             foreach ($course_data as $course) {
@@ -121,12 +121,12 @@ 
                 } else {
                     $class_tr = 'row_even';
                 }
-                $data_table .= '<tr class="' . $class_tr . '">';
+                $data_table .= '<tr class="'.$class_tr.'">';
                 if (!isset($course[3])) {
                     $course[3] = get_lang('NotAvailable');
                 }
                 foreach ($course as $cell) {
-                    $data_table .= '<td align="right">' . $cell . '</td>';
+                    $data_table .= '<td align="right">'.$cell.'</td>';
                 }
                 $data_table .= '</tr>';
                 $i++;
@@ -138,7 +138,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">' . get_lang('SeeMore') . '</a></div>';
+            <a href="' . api_get_path(WEB_CODE_PATH).'mySpace/course.php">'.get_lang('SeeMore').'</a></div>';
         }
         //$content .= '</div>';
         return $content;
@@ -176,7 +176,7 @@ 
             $table_course = Database::get_course_table(TABLE_ATTENDANCE);
 
             $sql = "SELECT id, name, attendance_qualify_max FROM $table_course
-                    WHERE c_id = " . $course_info['real_id'] . " AND active = 1 AND session_id = 0";
+                    WHERE c_id = ".$course_info['real_id']." AND active = 1 AND session_id = 0";
             $rs = Database::query($sql);
             $attendance = array();
             $attendances = array();
@@ -188,7 +188,7 @@ 
                 $attendance['course_code'] = $course_info['code'];
 
                 if ($attendance['done'] != '0') {
-                    $attendances[] = '<a href="' . api_get_path(WEB_PATH).'main/attendance/index.php?cidReq=' . $attendance['course_code'] . '&action=attendance_sheet_print&attendance_id=' . $attendance['id']. '">' . Display::return_icon('printmgr.gif', get_lang('Print')).'</a>';
+                    $attendances[] = '<a href="'.api_get_path(WEB_PATH).'main/attendance/index.php?cidReq='.$attendance['course_code'].'&action=attendance_sheet_print&attendance_id='.$attendance['id'].'">'.Display::return_icon('printmgr.gif', get_lang('Print')).'</a>';
                 } else {
                     $attendances[] = get_lang("NotAvailable");
                 }
@@ -200,7 +200,7 @@ 
             // Number of students
 
             $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)) {
@@ -211,11 +211,11 @@ 
             }
 
             if (!empty($tematic_advance)) {
-                $tematic_advance_progress = '<a title="' . get_lang(
+                $tematic_advance_progress = '<a title="'.get_lang(
                         'GoToThematicAdvance'
-                    ) . '" href="' . api_get_path(
+                    ).'" href="'.api_get_path(
                         WEB_CODE_PATH
-                    ) . 'attendance/index.php?cidReq=' . $course_code . '&action=attendance_sheet_print&attendance_id=">' . $tematic_advance . '%</a>';
+                    ).'attendance/index.php?cidReq='.$course_code.'&action=attendance_sheet_print&attendance_id=">'.$tematic_advance.'%</a>';
             } else {
                 $tematic_advance_progress = '0%';
             }
@@ -224,8 +224,8 @@ 
             $tbl_grade_categories = Database :: get_main_table(
                 TABLE_MAIN_GRADEBOOK_CATEGORY
             );
-            $sql = "SELECT id from " . $tbl_grade_categories . "
-                    WHERE course_code ='" . $course_code . "'";
+            $sql = "SELECT id from ".$tbl_grade_categories."
+                    WHERE course_code ='" . $course_code."'";
             $rs = Database::query($sql);
             $category = null;
             while ($row = Database::fetch_array($rs)) {
@@ -238,7 +238,7 @@ 
                 if (count($eval) > 0) {
                     $i = 0;
                     foreach ($eval as $item) {
-                        $score .= '<a href="' . api_get_path(WEB_PATH).'main/gradebook/gradebook_view_result.php?export=pdf&cat_code=' . $cat[0]->get_id() . '&official_code=' . $cat[0]->get_course_code() . '&selecteval=' . $item->get_id(). '">' . $item->get_name() . '</a>';
+                        $score .= '<a href="'.api_get_path(WEB_PATH).'main/gradebook/gradebook_view_result.php?export=pdf&cat_code='.$cat[0]->get_id().'&official_code='.$cat[0]->get_course_code().'&selecteval='.$item->get_id().'">'.$item->get_name().'</a>';
                         if (count($eval) - 1 != $i) {
                             $score .= ', ';
                         }