chamilo / chamilo-lms

main/survey/ch_score.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
             $options = array(
                 '--' => '--'
             );
-            for ($i=1; $i <= $questionData['maximum_score']; $i++) {
+            for ($i = 1; $i <= $questionData['maximum_score']; $i++) {
                 $options[$i] = $i;
             }
 

main/survey/survey_list.php

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
     if (api_is_allowed_to_edit()) {
         SurveyManager::copy_survey($_GET['survey_id']);
         $message = get_lang('SurveyCopied');
-        header('Location: ' . api_get_path(WEB_CODE_PATH) . 'survey/survey_list.php?' . api_get_cidreq());
+        header('Location: '.api_get_path(WEB_CODE_PATH).'survey/survey_list.php?'.api_get_cidreq());
         exit;
     }
 }
@@ -162,10 +162,10 @@ 
 if (!api_is_course_coach() || $extend_rights_for_coachs == 'true') {
     // Action links
     echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/create_new_survey.php?'.api_get_cidreq().'&amp;action=add">'.
-        Display::return_icon('new_survey.png', get_lang('CreateNewSurvey'),'',ICON_SIZE_MEDIUM).'</a> ';
+        Display::return_icon('new_survey.png', get_lang('CreateNewSurvey'), '', ICON_SIZE_MEDIUM).'</a> ';
 }
 echo '<a href="'.api_get_self().'?'.api_get_cidreq().'&amp;search=advanced">'.
-    Display::return_icon('search.png', get_lang('Search'),'',ICON_SIZE_MEDIUM).'</a>';
+    Display::return_icon('search.png', get_lang('Search'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 
 // Load main content

main/session/session_course_user_list.php

Indentation +7 added lines, -7 removed lines

@@ -50,8 +50,8 @@ 
 
 $result = Database::query($sql);
 if (!list($session_name,$course_title) = Database::fetch_row($result)) {
-	header('Location: session_course_list.php?id_session='.$id_session);
-	exit();
+    header('Location: session_course_list.php?id_session='.$id_session);
+    exit();
 }
 
 switch ($action) {
@@ -122,7 +122,7 @@ 
 <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page-1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous');?></a>
 <?php
 } else {
-	echo get_lang('Previous');
+    echo get_lang('Previous');
 }
 ?>
 |
@@ -132,7 +132,7 @@ 
 <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page+1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next');?></a>
 <?php
 } else {
-	echo get_lang('Next');
+    echo get_lang('Next');
 }
 ?>
 </div>
@@ -154,9 +154,9 @@ 
 $i=0;
 foreach ($users as $key => $enreg) {
 
-	if ($key == $limit) {
-		break;
-	}
+    if ($key == $limit) {
+        break;
+    }
     ?>
     <tr class="<?php echo $i?'row_odd':'row_even'; ?>">
         <td><input type="checkbox" name="idChecked[]" value="<?php echo $enreg['user_id']; ?>"></td>

Spacing +35 added lines, -35 removed lines

@@ -18,7 +18,7 @@ 
 $id_session = intval($_GET['id_session']);
 SessionManager::protectSession($id_session);
 
-if (empty($id_session )) {
+if (empty($id_session)) {
     api_not_allowed();
 }
 
@@ -28,16 +28,16 @@ 
 
 $page           = isset($_GET['page']) ? intval($_GET['page']) : null;
 $action         = isset($_REQUEST['action']) ? $_REQUEST['action'] : null;
-$default_sort   = api_sort_by_first_name() ? 'firstname':'lastname';
-$sort           = isset($_GET['sort']) && in_array($_GET['sort'], array('lastname','firstname','username')) ? $_GET['sort'] : $default_sort;
+$default_sort   = api_sort_by_first_name() ? 'firstname' : 'lastname';
+$sort           = isset($_GET['sort']) && in_array($_GET['sort'], array('lastname', 'firstname', 'username')) ? $_GET['sort'] : $default_sort;
 $idChecked      = isset($_GET['idChecked']) && is_array($_GET['idChecked']) ? $_GET['idChecked'] : (isset($_POST['idChecked']) && is_array($_POST['idChecked']) ? $_POST['idChecked'] : null);
-$direction      = isset($_GET['direction']) && in_array($_GET['direction'], array('desc','asc')) ? $_GET['direction'] : 'desc';
+$direction      = isset($_GET['direction']) && in_array($_GET['direction'], array('desc', 'asc')) ? $_GET['direction'] : 'desc';
 
 if (is_array($idChecked)) {
     $my_temp = array();
     foreach ($idChecked as $id) {
         // forcing the intval
-        $my_temp[]= intval($id);
+        $my_temp[] = intval($id);
     }
     $idChecked = $my_temp;
 }
@@ -49,14 +49,14 @@ 
 		WHERE src.session_id='$id_session' AND src.c_id='$courseId' ";
 
 $result = Database::query($sql);
-if (!list($session_name,$course_title) = Database::fetch_row($result)) {
+if (!list($session_name, $course_title) = Database::fetch_row($result)) {
 	header('Location: session_course_list.php?id_session='.$id_session);
 	exit();
 }
 
 switch ($action) {
     case 'delete':
-        if (is_array($idChecked) && count($idChecked)>0) {
+        if (is_array($idChecked) && count($idChecked) > 0) {
             array_map('intval', $idChecked);
             $idChecked = implode(',', $idChecked);
         }
@@ -91,7 +91,7 @@ 
          ON (s.session_id = scru.session_id AND s.user_id = scru.user_id AND scru.c_id = '".$courseId."' )
          WHERE s.session_id='$id_session'
          ORDER BY $sort $direction
-         LIMIT $from,".($limit+1);
+         LIMIT $from,".($limit + 1);
 
 if ($direction == 'desc') {
     $direction = 'asc';
@@ -107,8 +107,8 @@ 
 $tool_name = get_lang('Session').': '.$session_name.' - '.get_lang('Course').': '.$course_title;
 
 //$interbreadcrumb[] = array("url" => "index.php","name" => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array("url" => "session_list.php","name" => get_lang('SessionList'));
-$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".$id_session,"name" => get_lang('SessionOverview'));
+$interbreadcrumb[] = array("url" => "session_list.php", "name" => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".$id_session, "name" => get_lang('SessionOverview'));
 
 Display::display_header($tool_name);
 
@@ -117,9 +117,9 @@ 
 <form method="post" action="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=<?php echo $sort; ?>" onsubmit="javascript:if(!confirm('<?php echo get_lang('ConfirmYourChoice'); ?>')) return false;">
 <div align="right">
 <?php
-if($page) {
+if ($page) {
 ?>
-<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page-1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous');?></a>
+<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page - 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous'); ?></a>
 <?php
 } else {
 	echo get_lang('Previous');
@@ -127,9 +127,9 @@ 
 ?>
 |
 <?php
-if($nbr_results > $limit) {
+if ($nbr_results > $limit) {
 ?>
-<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page+1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next');?></a>
+<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page + 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next'); ?></a>
 <?php
 } else {
 	echo get_lang('Next');
@@ -141,39 +141,39 @@ 
 <tr>
   <th>&nbsp;</th>
   <?php if ($is_western_name_order) { ?>
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName');?></a></th>
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName');?></a></th>
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName'); ?></a></th>
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName'); ?></a></th>
   <?php } else { ?>
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName');?></a></th>
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName');?></a></th>
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName'); ?></a></th>
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName'); ?></a></th>
   <?php } ?>
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=username&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('Login');?></a></th>
-  <th><?php echo get_lang('Actions');?></th>
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=username&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('Login'); ?></a></th>
+  <th><?php echo get_lang('Actions'); ?></th>
 </tr>
 <?php
-$i=0;
+$i = 0;
 foreach ($users as $key => $enreg) {
 
 	if ($key == $limit) {
 		break;
 	}
     ?>
-    <tr class="<?php echo $i?'row_odd':'row_even'; ?>">
+    <tr class="<?php echo $i ? 'row_odd' : 'row_even'; ?>">
         <td><input type="checkbox" name="idChecked[]" value="<?php echo $enreg['user_id']; ?>"></td>
         <?php if ($is_western_name_order) { ?>
-        <td><?php echo api_htmlentities($enreg['firstname'],ENT_QUOTES,$charset); ?></td>
-        <td><?php echo api_htmlentities($enreg['lastname'],ENT_QUOTES,$charset); ?></td>
+        <td><?php echo api_htmlentities($enreg['firstname'], ENT_QUOTES, $charset); ?></td>
+        <td><?php echo api_htmlentities($enreg['lastname'], ENT_QUOTES, $charset); ?></td>
         <?php } else { ?>
-        <td><?php echo api_htmlentities($enreg['lastname'],ENT_QUOTES,$charset); ?></td>
-        <td><?php echo api_htmlentities($enreg['firstname'],ENT_QUOTES,$charset); ?></td>
+        <td><?php echo api_htmlentities($enreg['lastname'], ENT_QUOTES, $charset); ?></td>
+        <td><?php echo api_htmlentities($enreg['firstname'], ENT_QUOTES, $charset); ?></td>
         <?php } ?>
-        <td><?php echo api_htmlentities($enreg['username'],ENT_QUOTES,$charset); ?></td>
+        <td><?php echo api_htmlentities($enreg['username'], ENT_QUOTES, $charset); ?></td>
         <td>
         <?php if ($enreg['is_subscribed']) { ?>
             <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=<?php echo $sort; ?>&action=delete&idChecked[]=<?php echo $enreg['user_id']; ?>" onclick="javascript:if(!confirm('<?php echo get_lang('ConfirmYourChoice'); ?>')) return false;">
                 <?php Display::display_icon('delete.png', get_lang('Delete')); ?>
             </a>
-        <?php } else  { ?>
+        <?php } else { ?>
             <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=<?php echo $sort; ?>&action=add&idChecked[]=<?php echo $enreg['user_id']; ?>" onclick="javascript:if(!confirm('<?php echo get_lang('ConfirmYourChoice'); ?>')) return false;">
                 <?php Display::display_icon('add.png', get_lang('Add'), array(), ICON_SIZE_SMALL); ?>
             </a>
@@ -182,7 +182,7 @@ 
         </td>
     </tr>
     <?php
-        $i=$i ? 0 : 1;
+        $i = $i ? 0 : 1;
 }
 unset($users);
 ?>
@@ -190,9 +190,9 @@ 
 <br />
 <div align="left">
     <?php
-    if($page) {
+    if ($page) {
     ?>
-    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page-1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous'); ?></a>
+    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page - 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous'); ?></a>
     <?php
     } else {
         echo get_lang('Previous');
@@ -200,9 +200,9 @@ 
     ?>
     |
     <?php
-    if($nbr_results > $limit) {
+    if ($nbr_results > $limit) {
     ?>
-    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page+1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next'); ?></a>
+    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page + 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next'); ?></a>
     <?php
     } else {
         echo get_lang('Next');
@@ -211,8 +211,8 @@ 
 </div>
 <br />
 <select name="action">
-<option value="delete"><?php echo get_lang('UnsubscribeSelectedUsersFromSession');?></option>
-<option value="add"><?php echo get_lang('AddUsers');?></option>
+<option value="delete"><?php echo get_lang('UnsubscribeSelectedUsersFromSession'); ?></option>
+<option value="add"><?php echo get_lang('AddUsers'); ?></option>
 </select>
 <button class="save" type="submit"> <?php echo get_lang('Ok'); ?></button>
 </form>

main/session/session_course_user.php

Spacing +12 added lines, -12 removed lines

@@ -16,7 +16,7 @@ 
 SessionManager::protectSession($id_session);
 
 // setting breadcrumbs
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 $interbreadcrumb[] = array(
     'url' => 'resume_session.php?id_session='.$id_session,
     'name' => get_lang('SessionOverview'),
@@ -43,7 +43,7 @@ 
             FROM '.Database :: get_main_table(TABLE_MAIN_SESSION).'
             WHERE id='.$id_session;
     $rs = Database::query($sql);
-    if (Database::result($rs,0,0)!=$_user['user_id']) {
+    if (Database::result($rs, 0, 0) != $_user['user_id']) {
         api_not_allowed(true);
     }
 }
@@ -88,9 +88,9 @@ 
 
     foreach ($CourseList as $enreg_course) {
         $exists = false;
-        foreach($existingCourses as $existingCourse) {
+        foreach ($existingCourses as $existingCourse) {
             if ($enreg_course == $existingCourse['id']) {
-                $exists=true;
+                $exists = true;
             }
         }
 
@@ -110,7 +110,7 @@ 
     }
 
     foreach ($existingCourses as $existingCourse) {
-        if (!in_array($existingCourse['id'], $CourseList)){
+        if (!in_array($existingCourse['id'], $CourseList)) {
             $existingCourse = Database::escape_string($existingCourse['id']);
             $sql = "INSERT IGNORE INTO $tbl_session_rel_course_rel_user (session_id,c_id,user_id)
                     VALUES ('$id_session','$existingCourse','$id_user')";
@@ -146,7 +146,7 @@ 
         WHERE srcru.user_id = $id_user AND session_id = $id_session";
 
 //all
-$sql_all="SELECT course.id, code, title, visual_code, src.session_id
+$sql_all = "SELECT course.id, code, title, visual_code, src.session_id
         FROM $tbl_course course
         INNER JOIN $tbl_session_rel_course  as src
         ON course.id = src.c_id AND session_id = $id_session";
@@ -163,9 +163,9 @@ 
 
 foreach ($CoursesAll as $course) {
     if (in_array($course['id'], $course_temp)) {
-        $nosessionCourses[$course['id']] = $course ;
+        $nosessionCourses[$course['id']] = $course;
     } else {
-        $sessionCourses[$course['id']] = $course ;
+        $sessionCourses[$course['id']] = $course;
     }
 }
 unset($Courses);
@@ -184,9 +184,9 @@ 
             <td width="45%" align="center">
                 <div id="ajax_list_courses_multiple">
                     <select id="origin" name="NoSessionCoursesList[]" multiple="multiple" size="20" style="width:320px;"> <?php
-                        foreach($nosessionCourses as $enreg) {
+                        foreach ($nosessionCourses as $enreg) {
                             ?>
-                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')',ENT_QUOTES).'"'; if(in_array($enreg['code'],$CourseList)) echo 'selected="selected"'; ?>><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
+                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')', ENT_QUOTES).'"'; if (in_array($enreg['code'], $CourseList)) echo 'selected="selected"'; ?>><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
                         <?php
                         }
                         ?> </select></div> <?php
@@ -211,9 +211,9 @@ 
             <td width="45%" align="center">
                 <select id='destination' name="SessionCoursesList[]" multiple="multiple" size="20" style="width:320px;">
             <?php
-            foreach($sessionCourses as $enreg) {
+            foreach ($sessionCourses as $enreg) {
                 ?>
-                <option value="<?php echo $enreg['id']; ?>" title="<?php echo htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')',ENT_QUOTES); ?>"><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
+                <option value="<?php echo $enreg['id']; ?>" title="<?php echo htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')', ENT_QUOTES); ?>"><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
             <?php
             }
             unset($sessionCourses);

Braces +4 added lines, -1 removed lines

@@ -186,7 +186,10 @@
                     <select id="origin" name="NoSessionCoursesList[]" multiple="multiple" size="20" style="width:320px;"> <?php
                         foreach($nosessionCourses as $enreg) {
                             ?>
-                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')',ENT_QUOTES).'"'; if(in_array($enreg['code'],$CourseList)) echo 'selected="selected"'; ?>><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
+                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')',ENT_QUOTES).'"'; if(in_array($enreg['code'],$CourseList)) {
+    echo 'selected="selected"';
+}
+?>><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
                         <?php
                         }
                         ?> </select></div> <?php

main/session/session_course_list.php

Indentation +26 added lines, -26 removed lines

@@ -30,26 +30,26 @@ 
 $result = Database::query("SELECT name FROM $tbl_session WHERE id='$id_session'");
 
 if (!list($session_name)=Database::fetch_row($result)) {
-	header('Location: session_list.php');
-	exit;
+    header('Location: session_list.php');
+    exit;
 }
 
 if ($action == 'delete') {
-	$idChecked = $_REQUEST['idChecked'];
-	if (is_array($idChecked) && count($idChecked)>0) {
-		$my_temp = array();
-		foreach ($idChecked as $id){
-			$my_temp[]= Database::escape_string($id);// forcing the escape_string
-		}
-		$idChecked = $my_temp;
-		$idChecked="'".implode("','", $idChecked)."'";
-		$result = Database::query("DELETE FROM $tbl_session_rel_course WHERE session_id='$id_session' AND c_id IN($idChecked)");
-		$nbr_affected_rows=Database::affected_rows($result);
-		Database::query("DELETE FROM $tbl_session_rel_course_rel_user WHERE session_id='$id_session' AND c_id IN($idChecked)");
-		Database::query("UPDATE $tbl_session SET nbr_courses=nbr_courses-$nbr_affected_rows WHERE id='$id_session'");
-	}
-	header('Location: '.api_get_self().'?id_session='.$id_session.'&sort='.$sort);
-	exit();
+    $idChecked = $_REQUEST['idChecked'];
+    if (is_array($idChecked) && count($idChecked)>0) {
+        $my_temp = array();
+        foreach ($idChecked as $id){
+            $my_temp[]= Database::escape_string($id);// forcing the escape_string
+        }
+        $idChecked = $my_temp;
+        $idChecked="'".implode("','", $idChecked)."'";
+        $result = Database::query("DELETE FROM $tbl_session_rel_course WHERE session_id='$id_session' AND c_id IN($idChecked)");
+        $nbr_affected_rows=Database::affected_rows($result);
+        Database::query("DELETE FROM $tbl_session_rel_course_rel_user WHERE session_id='$id_session' AND c_id IN($idChecked)");
+        Database::query("UPDATE $tbl_session SET nbr_courses=nbr_courses-$nbr_affected_rows WHERE id='$id_session'");
+    }
+    header('Location: '.api_get_self().'?id_session='.$id_session.'&sort='.$sort);
+    exit();
 }
 
 $limit  = 20;
@@ -82,17 +82,17 @@ 
 $tableCourses = array();
 
 foreach ($Courses as $key=>$enreg) {
-	$course = array();
-	$course[] = '<input type="checkbox" name="idChecked[]" value="'.$enreg['id'].'">';
-	$course[] = api_htmlentities($enreg['title'],ENT_QUOTES,$charset);
-	$course[] = '<a href="session_course_user_list.php?id_session='.$id_session.'&course_code='.$enreg['code'].'">'.$enreg['nbr_users'].' '.get_lang('Users').'</a>';
-	$course[] = '<a href="'.api_get_path(WEB_COURSE_PATH).$enreg['code'].'/?id_session='.$id_session.'">'.
-		Display::return_icon('course_home.gif', get_lang('Course')).'</a>
+    $course = array();
+    $course[] = '<input type="checkbox" name="idChecked[]" value="'.$enreg['id'].'">';
+    $course[] = api_htmlentities($enreg['title'],ENT_QUOTES,$charset);
+    $course[] = '<a href="session_course_user_list.php?id_session='.$id_session.'&course_code='.$enreg['code'].'">'.$enreg['nbr_users'].' '.get_lang('Users').'</a>';
+    $course[] = '<a href="'.api_get_path(WEB_COURSE_PATH).$enreg['code'].'/?id_session='.$id_session.'">'.
+        Display::return_icon('course_home.gif', get_lang('Course')).'</a>
 			<a href="session_course_edit.php?id_session='.$id_session.'&page=session_course_list.php&course_code='.$enreg['code'].'">'.
-		Display::return_icon('edit.png', get_lang('Edit')).'</a>
+        Display::return_icon('edit.png', get_lang('Edit')).'</a>
 			<a href="'.api_get_self().'?id_session='.$id_session.'&sort='.$sort.'&action=delete&idChecked[]='.$enreg['id'].'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)).'\')) return false;">'.
-		Display::return_icon('delete.png', get_lang('Delete')).'</a>';
-	$tableCourses[] = $course;
+        Display::return_icon('delete.png', get_lang('Delete')).'</a>';
+    $tableCourses[] = $course;
 }
 echo '<form method="post" action="'.api_get_self().'">';
 Display :: display_sortable_table($tableHeader, $tableCourses, array (), array ());

Spacing +19 added lines, -19 removed lines

@@ -8,7 +8,7 @@ 
 require_once '../inc/global.inc.php';
 
 // setting the section (for the tabs)
-$this_section=SECTION_PLATFORM_ADMIN;
+$this_section = SECTION_PLATFORM_ADMIN;
 
 $id_session = intval($_GET['id_session']);
 SessionManager::protectSession($id_session);
@@ -16,8 +16,8 @@ 
 // Database Table Definitions
 $tbl_course							= Database::get_main_table(TABLE_MAIN_COURSE);
 $tbl_session						= Database::get_main_table(TABLE_MAIN_SESSION);
-$tbl_session_rel_course				= Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
-$tbl_session_rel_course_rel_user	= Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
+$tbl_session_rel_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
+$tbl_session_rel_course_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
 
 if (empty($id_session)) {
     api_not_allowed();
@@ -25,26 +25,26 @@ 
 
 $page   = intval($_GET['page']);
 $action = $_REQUEST['action'];
-$sort   = in_array($_GET['sort'],array('title','nbr_users'))?$_GET['sort']:'title';
+$sort   = in_array($_GET['sort'], array('title', 'nbr_users')) ? $_GET['sort'] : 'title';
 
 $result = Database::query("SELECT name FROM $tbl_session WHERE id='$id_session'");
 
-if (!list($session_name)=Database::fetch_row($result)) {
+if (!list($session_name) = Database::fetch_row($result)) {
 	header('Location: session_list.php');
 	exit;
 }
 
 if ($action == 'delete') {
 	$idChecked = $_REQUEST['idChecked'];
-	if (is_array($idChecked) && count($idChecked)>0) {
+	if (is_array($idChecked) && count($idChecked) > 0) {
 		$my_temp = array();
-		foreach ($idChecked as $id){
-			$my_temp[]= Database::escape_string($id);// forcing the escape_string
+		foreach ($idChecked as $id) {
+			$my_temp[] = Database::escape_string($id); // forcing the escape_string
 		}
 		$idChecked = $my_temp;
-		$idChecked="'".implode("','", $idChecked)."'";
+		$idChecked = "'".implode("','", $idChecked)."'";
 		$result = Database::query("DELETE FROM $tbl_session_rel_course WHERE session_id='$id_session' AND c_id IN($idChecked)");
-		$nbr_affected_rows=Database::affected_rows($result);
+		$nbr_affected_rows = Database::affected_rows($result);
 		Database::query("DELETE FROM $tbl_session_rel_course_rel_user WHERE session_id='$id_session' AND c_id IN($idChecked)");
 		Database::query("UPDATE $tbl_session SET nbr_courses=nbr_courses-$nbr_affected_rows WHERE id='$id_session'");
 	}
@@ -59,14 +59,14 @@ 
 		FROM $tbl_session_rel_course, $tbl_course c
 		WHERE c_id = c.id AND session_id='$id_session'
 		ORDER BY $sort
-		LIMIT $from,".($limit+1);
-$result=Database::query($sql);
-$Courses=Database::store_result($result);
-$tool_name = api_htmlentities($session_name,ENT_QUOTES,$charset).' : '.get_lang('CourseListInSession');
+		LIMIT $from,".($limit + 1);
+$result = Database::query($sql);
+$Courses = Database::store_result($result);
+$tool_name = api_htmlentities($session_name, ENT_QUOTES, $charset).' : '.get_lang('CourseListInSession');
 
 //$interbreadcrumb[]=array('url' => 'index.php',"name" => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array('url' => "session_list.php","name" => get_lang('SessionList'));
-$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".Security::remove_XSS($_REQUEST['id_session']),"name" => get_lang('SessionOverview'));
+$interbreadcrumb[] = array('url' => "session_list.php", "name" => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".Security::remove_XSS($_REQUEST['id_session']), "name" => get_lang('SessionOverview'));
 
 Display::display_header($tool_name);
 echo Display::page_header($tool_name);
@@ -84,18 +84,18 @@ 
 foreach ($Courses as $key=>$enreg) {
 	$course = array();
 	$course[] = '<input type="checkbox" name="idChecked[]" value="'.$enreg['id'].'">';
-	$course[] = api_htmlentities($enreg['title'],ENT_QUOTES,$charset);
+	$course[] = api_htmlentities($enreg['title'], ENT_QUOTES, $charset);
 	$course[] = '<a href="session_course_user_list.php?id_session='.$id_session.'&course_code='.$enreg['code'].'">'.$enreg['nbr_users'].' '.get_lang('Users').'</a>';
 	$course[] = '<a href="'.api_get_path(WEB_COURSE_PATH).$enreg['code'].'/?id_session='.$id_session.'">'.
 		Display::return_icon('course_home.gif', get_lang('Course')).'</a>
 			<a href="session_course_edit.php?id_session='.$id_session.'&page=session_course_list.php&course_code='.$enreg['code'].'">'.
 		Display::return_icon('edit.png', get_lang('Edit')).'</a>
-			<a href="'.api_get_self().'?id_session='.$id_session.'&sort='.$sort.'&action=delete&idChecked[]='.$enreg['id'].'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)).'\')) return false;">'.
+			<a href="'.api_get_self().'?id_session='.$id_session.'&sort='.$sort.'&action=delete&idChecked[]='.$enreg['id'].'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;">'.
 		Display::return_icon('delete.png', get_lang('Delete')).'</a>';
 	$tableCourses[] = $course;
 }
 echo '<form method="post" action="'.api_get_self().'">';
-Display :: display_sortable_table($tableHeader, $tableCourses, array (), array ());
+Display :: display_sortable_table($tableHeader, $tableCourses, array(), array());
 echo '<select name="action">
 	<option value="delete">'.get_lang('UnsubscribeCoursesFromSession').'</option>
 	</select>

main/session/session_category_list.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 }
 
 //$interbreadcrumb[] = array("url" => "index.php", "name" => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 
 if (isset($_GET['search']) && $_GET['search'] == 'advanced') {
     $interbreadcrumb[] = array("url" => 'session_category_list.php', "name" => get_lang('ListSessionCategory'));
@@ -76,7 +76,7 @@ 
         $where .= " AND access_url_id = ".api_get_current_access_url_id()." ";
     }
 
-    $table_access_url_rel_session= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
+    $table_access_url_rel_session = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
     $query = "SELECT sc.*, (
                 SELECT count(id) FROM $tbl_session s
                 INNER JOIN $table_access_url_rel_session us

main/session/add_students_to_session.php

Spacing +2 added lines, -2 removed lines

@@ -14,12 +14,12 @@
 
 // setting breadcrumbs
 //$interbreadcrumb[] = array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 
 // Setting the name of the tool
 $tool_name = get_lang('SubscribeStudentsToSession');
 $add_type = 'multiple';
-if (isset($_REQUEST['add_type']) && $_REQUEST['add_type']!='') {
+if (isset($_REQUEST['add_type']) && $_REQUEST['add_type'] != '') {
     $add_type = Security::remove_XSS($_REQUEST['add_type']);
 }
 $form_sent  = 0;

Indentation +7 added lines, -7 removed lines

@@ -70,13 +70,13 @@
         <tr>
             <td align="center">
                 <?php
-                 echo Display::select(
-                     'sessions[]',
-                     $sessionList,
-                     '',
-                     array('style'=>'width:100%', 'multiple'=>'multiple', 'id'=>'sessions', 'size'=>'15px'),
-                     false
-                 );
+                    echo Display::select(
+                        'sessions[]',
+                        $sessionList,
+                        '',
+                        array('style'=>'width:100%', 'multiple'=>'multiple', 'id'=>'sessions', 'size'=>'15px'),
+                        false
+                    );
                 ?>
             </td>
             <td align="center">

main/session/session_import_drh.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
 $tool_name = get_lang('ImportSessionDrhList');
 
 //$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[]=array('url' => 'session_list.php','name' => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 
 set_time_limit(0);
 
@@ -27,7 +27,7 @@ 
 
 echo '<div class="actions">';
 echo '<a href="../session/session_list.php">'.
-    Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('PlatformAdmin'),'',ICON_SIZE_MEDIUM).'</a>';
+    Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('PlatformAdmin'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 
 if (!empty($error_message)) {

main/session/resume_session.php

Indentation +20 added lines, -20 removed lines

@@ -155,7 +155,7 @@ 
     $sessionRepository = Database::getManager()->getRepository('ChamiloCoreBundle:Session');
     $courses = $sessionRepository->getCoursesOrderedByPosition($session);
 
-	foreach ($courses as $course) {
+    foreach ($courses as $course) {
         //select the number of users
         $sql = "SELECT count(*)
                 FROM $tbl_session_rel_user sru,
@@ -167,37 +167,37 @@ 
                     sru.relation_type <> ".SESSION_RELATION_TYPE_RRHH." AND
                     srcru.session_id = '".intval($sessionId)."'";
 
-		$rs = Database::query($sql);
+        $rs = Database::query($sql);
         $numberOfUsers = Database::result($rs, 0, 0);
 
-		// Get coachs of the courses in session
+        // Get coachs of the courses in session
 
-		$sql = "SELECT user.lastname, user.firstname, user.username
+        $sql = "SELECT user.lastname, user.firstname, user.username
                 FROM $tbl_session_rel_course_rel_user session_rcru, $tbl_user user
 				WHERE
 				    session_rcru.user_id = user.user_id AND
 				    session_rcru.session_id = '".intval($sessionId)."' AND
 				    session_rcru.c_id ='".intval($course->getId())."' AND
 				    session_rcru.status=2";
-		$rs = Database::query($sql);
+        $rs = Database::query($sql);
 
-		$coachs = array();
-		if (Database::num_rows($rs) > 0) {
-			while($info_coach = Database::fetch_array($rs)) {
+        $coachs = array();
+        if (Database::num_rows($rs) > 0) {
+            while($info_coach = Database::fetch_array($rs)) {
                 $coachs[] = api_get_person_name(
                         $info_coach['firstname'],
                         $info_coach['lastname']
                     ).' ('.$info_coach['username'].')';
-			}
-		} else {
-			$coach = get_lang('None');
-		}
+            }
+        } else {
+            $coach = get_lang('None');
+        }
 
-		if (count($coachs) > 0) {
-			$coach = implode('<br />',$coachs);
-		} else {
-			$coach = get_lang('None');
-		}
+        if (count($coachs) > 0) {
+            $coach = implode('<br />',$coachs);
+        } else {
+            $coach = get_lang('None');
+        }
 
         $orderButtons = null;
 
@@ -233,8 +233,8 @@ 
 
         $courseUrl = api_get_course_url($course->getCode(), $sessionId);
 
-		// hide_course_breadcrumb the parameter has been added to hide the name
-		// of the course, that appeared in the default $interbreadcrumb
+        // hide_course_breadcrumb the parameter has been added to hide the name
+        // of the course, that appeared in the default $interbreadcrumb
         $courseItem .= '
 		<tr>
 			<td class="title">'.Display::url(
@@ -260,7 +260,7 @@ 
 			</td>
 		</tr>';
         $count++;
-	}
+    }
     $courseListToShow .= $courseItem;
 }
 $courseListToShow .= '</table><br />';

Spacing +6 added lines, -6 removed lines

@@ -29,7 +29,7 @@ 
 $tool_name = get_lang('SessionOverview');
 
 //$interbreadcrumb[] = array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 
 $orig_param = '&origin=resume_session';
 
@@ -183,7 +183,7 @@ 
 
 		$coachs = array();
 		if (Database::num_rows($rs) > 0) {
-			while($info_coach = Database::fetch_array($rs)) {
+			while ($info_coach = Database::fetch_array($rs)) {
                 $coachs[] = api_get_person_name(
                         $info_coach['firstname'],
                         $info_coach['lastname']
@@ -194,7 +194,7 @@ 
 		}
 
 		if (count($coachs) > 0) {
-			$coach = implode('<br />',$coachs);
+			$coach = implode('<br />', $coachs);
 		} else {
 			$coach = get_lang('None');
 		}
@@ -217,7 +217,7 @@ 
         $downIcon = 'down.png';
         $downUrl = api_get_self().'?id_session='.$sessionId.'&course_id='.$course->getId().'&action=move_down';
 
-        if ($count +1 == count($courses)) {
+        if ($count + 1 == count($courses)) {
             $downIcon = 'down_na.png';
             $downUrl = '#';
         }
@@ -244,7 +244,7 @@ 
 			<td>'.$coach.'</td>
 			<td>'.$numberOfUsers.'</td>
 			<td>
-                <a href="'. $courseUrl . '">'.
+                <a href="'. $courseUrl.'">'.
                 Display::return_icon('course_home.gif', get_lang('Course')).'</a>
                 '.$orderButtons.'
                 <a href="session_course_user_list.php?id_session='.$sessionId.'&course_code='.$course->getCode().'">'.
@@ -309,7 +309,7 @@ 
             array('onclick' => "javascript:if(!confirm('".get_lang('ConfirmYourChoice')."')) return false;")
         );
 
-        $addUserToUrlLink= '';
+        $addUserToUrlLink = '';
         if ($multiple_url_is_on) {
             if ($user['access_url_id'] != $url_id) {
                 $userLink .= ' '.Display::return_icon(