chamilo / chamilo-lms

main/admin/career_dashboard.php

Spacing +13 added lines, -13 removed lines

@@ -18,12 +18,12 @@ 
 $htmlHeadXtra[] = api_get_jqgrid_js();
 
 // setting breadcrumbs
-$interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[]=array('url' => 'career_dashboard.php','name' => get_lang('CareersAndPromotions'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'career_dashboard.php', 'name' => get_lang('CareersAndPromotions'));
 
 Display :: display_header(null);
 
-$form = new FormValidator('filter_form','GET', api_get_self());
+$form = new FormValidator('filter_form', 'GET', api_get_self());
 
 $career = new Career();
 
@@ -54,11 +54,11 @@ 
 // action links
 echo '<div class="actions" style="margin-bottom:20px">';
     echo  '<a href="../admin/index.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 '<a href="careers.php">'.
-            Display::return_icon('career.png',get_lang('Careers'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('career.png', get_lang('Careers'), '', ICON_SIZE_MEDIUM).'</a>';
     echo '<a href="promotions.php">'.
-            Display::return_icon('promotion.png',get_lang('Promotions'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('promotion.png', get_lang('Promotions'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 
 $form->display();
@@ -119,22 +119,22 @@ 
     foreach ($career_array as $career_id => $data) {
         $career = $data['name'];
         $promotions = $data['promotions'];
-        $career = Display::url($career, 'careers.php?action=edit&id=' . $career_id);
+        $career = Display::url($career, 'careers.php?action=edit&id='.$career_id);
         $career = Display::tag('h4', $career);
-        echo '<tr><td style="background-color:#ECF0F1" colspan="3">' . $career . '</td></tr>';
+        echo '<tr><td style="background-color:#ECF0F1" colspan="3">'.$career.'</td></tr>';
         if (!empty($promotions)) {
             foreach ($promotions as $promotion_id => $promotion) {
                 $promotion_name = $promotion['name'];
-                $promotion_url = Display::url($promotion_name, 'promotions.php?action=edit&id=' . $promotion_id);
+                $promotion_url = Display::url($promotion_name, 'promotions.php?action=edit&id='.$promotion_id);
                 $sessions = $promotion['sessions'];
                 echo '<tr>';
                 $count = count($sessions);
                 $rowspan = '';
                 if (!empty($count)) {
                     $count++;
-                    $rowspan = 'rowspan="' . $count . '"';
+                    $rowspan = 'rowspan="'.$count.'"';
                 }
-                echo '<td ' . $rowspan . '>';
+                echo '<td '.$rowspan.'>';
                 echo Display::tag('h5', $promotion_url);
                 echo '</td>';
                 echo '</tr>';
@@ -144,7 +144,7 @@ 
                         $course_list = $session['courses'];
 
                         $url = Display::url($session['data']['name'],
-                            '../session/resume_session.php?id_session=' . $session['data']['id']);
+                            '../session/resume_session.php?id_session='.$session['data']['id']);
                         echo '<tr>';
                         //Session name
                         echo Display::tag('td', $url);
@@ -157,7 +157,7 @@ 
 
                                 $url = Display::url(
                                     $course['title'],
-                                    api_get_path(WEB_COURSE_PATH) . $course['directory'] . '/index.php?id_session=' . $session['data']['id']
+                                    api_get_path(WEB_COURSE_PATH).$course['directory'].'/index.php?id_session='.$session['data']['id']
                                 );
                                 echo Display::tag('td', $url);
                                 echo '</tr>';

main/admin/access_url_check_user_session.php

Spacing +6 added lines, -6 removed lines

@@ -15,18 +15,18 @@ 
 
 $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' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 
 // Database Table Definitions
-$tbl_user							= Database::get_main_table(TABLE_MAIN_USER);
-$tbl_session_rel_user				= Database::get_main_table(TABLE_MAIN_SESSION_USER);
+$tbl_user = Database::get_main_table(TABLE_MAIN_USER);
+$tbl_session_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_USER);
 $table_access_url_user              = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
 $url_id                             = api_get_current_access_url_id();
 
 $action = $_GET['action'];
 
-switch($action) {
+switch ($action) {
     case 'add_user_to_url':
         $user_id = $_REQUEST['user_id'];
         $result = UrlManager::add_user_to_url($user_id, $url_id);
@@ -58,7 +58,7 @@ 
 }
 
 
-foreach($session_list  as $session_item) {
+foreach ($session_list  as $session_item) {
     $session_id = $session_item['id'];
     $html .= '<h3>'.$session_item['name'].'</h3>';
 

main/admin/access_url_edit_courses_to_url.php

Indentation +91 added lines, -91 removed lines

@@ -11,7 +11,7 @@ 
 
 $xajax = new xajax();
 $xajax->registerFunction(
-	array('search_courses', 'Accessurleditcoursestourl', 'search_courses')
+    array('search_courses', 'Accessurleditcoursestourl', 'search_courses')
 );
 
 // setting the section (for the tabs)
@@ -20,8 +20,8 @@ 
 // Access restrictions
 api_protect_global_admin_script();
 if (!api_get_multiple_access_url()) {
-	header('Location: index.php');
-	exit;
+    header('Location: index.php');
+    exit;
 }
 
 // Database Table Definitions
@@ -36,12 +36,12 @@ 
 
 $add_type = 'multiple';
 if (isset($_REQUEST['add_type']) && $_REQUEST['add_type']!='') {
-	$add_type = Security::remove_XSS($_REQUEST['add_type']);
+    $add_type = Security::remove_XSS($_REQUEST['add_type']);
 }
 
 $access_url_id = 1;
 if (isset($_REQUEST['access_url_id']) && $_REQUEST['access_url_id']!='') {
-	$access_url_id = Security::remove_XSS($_REQUEST['access_url_id']);
+    $access_url_id = Security::remove_XSS($_REQUEST['access_url_id']);
 }
 
 $xajax -> processRequests();
@@ -83,21 +83,21 @@ 
 $users = $sessions = array();
 
 if (isset($_POST['form_sent']) && $_POST['form_sent']) {
-	$form_sent = $_POST['form_sent'];
-	$course_list = $_POST['course_list'];
-
-	if (!is_array($course_list)) {
-		$course_list = array();
-	}
-
-	if ($form_sent == 1) {
-		if ($access_url_id==0) {
-			header('Location: access_url_edit_users_to_url.php?action=show_message&message='.get_lang('SelectURL'));
-		} elseif (is_array($course_list)) {
-			UrlManager::update_urls_rel_course($course_list, $access_url_id);
-			header('Location: access_urls.php?action=show_message&message='.get_lang('CoursesWereEdited'));
-		}
-	}
+    $form_sent = $_POST['form_sent'];
+    $course_list = $_POST['course_list'];
+
+    if (!is_array($course_list)) {
+        $course_list = array();
+    }
+
+    if ($form_sent == 1) {
+        if ($access_url_id==0) {
+            header('Location: access_url_edit_users_to_url.php?action=show_message&message='.get_lang('SelectURL'));
+        } elseif (is_array($course_list)) {
+            UrlManager::update_urls_rel_course($course_list, $access_url_id);
+            header('Location: access_urls.php?action=show_message&message='.get_lang('CoursesWereEdited'));
+        }
+    }
 }
 
 Display::display_header($tool_name);
@@ -109,45 +109,45 @@ 
 api_display_tool_title($tool_name);
 
 if (isset($_GET['action']) && $_GET['action'] == 'show_message')
-	Display :: display_normal_message(Security::remove_XSS(stripslashes($_GET['message'])));
+    Display :: display_normal_message(Security::remove_XSS(stripslashes($_GET['message'])));
 
 $no_course_list = $course_list = array();
 $ajax_search = $add_type == 'unique' ? true : false;
 
 if($ajax_search) {
-	$courses = UrlManager::get_url_rel_course_data($access_url_id);
-	foreach ($courses as $course) {
-		$course_list[$course['c_id']] = $course;
-	}
+    $courses = UrlManager::get_url_rel_course_data($access_url_id);
+    foreach ($courses as $course) {
+        $course_list[$course['c_id']] = $course;
+    }
 } else {
-	$courses = UrlManager::get_url_rel_course_data();
+    $courses = UrlManager::get_url_rel_course_data();
 
-	foreach ($courses as $course) {
-		if ($course['access_url_id'] == $access_url_id) {
-			$course_list[$course['c_id']] = $course;
-		}
-	}
+    foreach ($courses as $course) {
+        if ($course['access_url_id'] == $access_url_id) {
+            $course_list[$course['c_id']] = $course;
+        }
+    }
 
-	$tbl_course = Database :: get_main_table(TABLE_MAIN_COURSE);
-	$sql="SELECT id, code, title
+    $tbl_course = Database :: get_main_table(TABLE_MAIN_COURSE);
+    $sql="SELECT id, code, title
 	  	  	FROM $tbl_course u
 			ORDER BY title, code";
-	$result = Database::query($sql);
-	$courses = Database::store_result($result);
-	$course_list_leys = array_keys($course_list);
-	foreach ($courses as $course) {
-		if (!in_array($course['id'], $course_list_leys)) {
-			$no_course_list[$course['id']] = $course;
-		}
-	}
+    $result = Database::query($sql);
+    $courses = Database::store_result($result);
+    $course_list_leys = array_keys($course_list);
+    foreach ($courses as $course) {
+        if (!in_array($course['id'], $course_list_leys)) {
+            $no_course_list[$course['id']] = $course;
+        }
+    }
 }
 
 if($add_type == 'multiple') {
-	$link_add_type_unique = '<a href="'.api_get_self().'?add_type=unique&access_url_id='.$access_url_id.'">'.get_lang('SessionAddTypeUnique').'</a>';
-	$link_add_type_multiple = get_lang('SessionAddTypeMultiple');
+    $link_add_type_unique = '<a href="'.api_get_self().'?add_type=unique&access_url_id='.$access_url_id.'">'.get_lang('SessionAddTypeUnique').'</a>';
+    $link_add_type_multiple = get_lang('SessionAddTypeMultiple');
 } else {
-	$link_add_type_unique = get_lang('SessionAddTypeUnique');
-	$link_add_type_multiple = '<a href="'.api_get_self().'?add_type=multiple&access_url_id='.$access_url_id.'">'.get_lang('SessionAddTypeMultiple').'</a>';
+    $link_add_type_unique = get_lang('SessionAddTypeUnique');
+    $link_add_type_multiple = '<a href="'.api_get_self().'?add_type=multiple&access_url_id='.$access_url_id.'">'.get_lang('SessionAddTypeMultiple').'</a>';
 }
 $url_list = UrlManager::get_url_data();
 ?>
@@ -160,31 +160,31 @@ 
 		<select name="access_url_id" onchange="javascript:send();">
 			<option value="0">-- <?php echo get_lang('SelectUrl')?> -- </option>
 			<?php
-			$url_selected='';
-			foreach ($url_list as $url_obj) {
-				$checked = '';
-				if (!empty($access_url_id)) {
-					if ($url_obj[0]==$access_url_id) {
-						$checked = 'selected=true';
-						$url_selected=$url_obj[1];
-					}
-				}
-				if ($url_obj['active']==1) {
-					?>
+            $url_selected='';
+            foreach ($url_list as $url_obj) {
+                $checked = '';
+                if (!empty($access_url_id)) {
+                    if ($url_obj[0]==$access_url_id) {
+                        $checked = 'selected=true';
+                        $url_selected=$url_obj[1];
+                    }
+                }
+                if ($url_obj['active']==1) {
+                    ?>
 					<option <?php echo $checked;?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?></option>
 				<?php
-				}
-			}
-			?>
+                }
+            }
+            ?>
 		</select>
 		<br /><br />
 		<input type="hidden" name="form_sent" value="1" />
 		<input type="hidden" name="add_type" value = "<?php echo $add_type ?>" />
 		<?php
-		if(!empty($errorMsg)) {
-			Display::display_normal_message($errorMsg); //main API
-		}
-		?>
+        if(!empty($errorMsg)) {
+            Display::display_normal_message($errorMsg); //main API
+        }
+        ?>
 		<table border="0" cellpadding="5" cellspacing="0" width="100%">
 
 			<!-- Users -->
@@ -199,38 +199,38 @@ 
 				<td align="center">
 					<div id="content_source">
 						<?php
-						if($ajax_search) {
-							?>
+                        if($ajax_search) {
+                            ?>
 							<input type="text" id="course_to_add" onkeyup="xajax_search_courses(this.value,document.formulaire.access_url_id.options[document.formulaire.access_url_id.selectedIndex].value)" />
 							<div id="ajax_list_courses"></div>
 						<?php
-						} else {
-							?>
+                        } else {
+                            ?>
 							<select id="origin_users" name="no_course_list[]" multiple="multiple" size="15" style="width:380px;">
 								<?php
-								foreach($no_course_list as $no_course) {
-									?>
+                                foreach($no_course_list as $no_course) {
+                                    ?>
 									<option value="<?php echo $no_course['id']; ?>"><?php echo $no_course['title'].' ('.$no_course['code'].')'; ?></option>
 								<?php
-								}
-								unset($no_course_list);
-								?>
+                                }
+                                unset($no_course_list);
+                                ?>
 							</select>
 						<?php
-						}
-						?>
+                        }
+                        ?>
 					</div>
 				</td>
 				<td width="10%" valign="middle" align="center">
 					<?php
-					if($ajax_search) {
-						?>
+                    if($ajax_search) {
+                        ?>
 						<button class="btn btn-default" type="button" onclick="remove_item(document.getElementById('destination_users'))" >
                             <em class="fa fa-arrow-left"></em>
 						</button>
 					<?php
-					} else {
-						?>
+                    } else {
+                        ?>
 						<button class="btn btn-default" type="button" onclick="moveItem(document.getElementById('origin_users'), document.getElementById('destination_users'))" >
                             <em class="fa fa-arrow-right"></em>
 						</button>
@@ -239,21 +239,21 @@ 
                             <em class="fa fa-arrow-left"></em>
 						</button>
 					<?php
-					}
-					?>
+                    }
+                    ?>
 					<br /><br /><br /><br /><br /><br />
 				</td>
 				<td align="center">
 					<select id="destination_users" name="course_list[]" multiple="multiple" size="15" style="width:380px;">
 						<?php
-						foreach($course_list as $course) {
-							$courseInfo = api_get_course_info_by_id($course['id']);
-							?>
+                        foreach($course_list as $course) {
+                            $courseInfo = api_get_course_info_by_id($course['id']);
+                            ?>
 							<option value="<?php echo $course['id']; ?>"><?php echo $course['title'].' ('.$courseInfo['code'].')'; ?></option>
 						<?php
-						}
-						unset($course_list);
-						?>
+                        }
+                        unset($course_list);
+                        ?>
 
 					</select></td>
 			</tr>
@@ -261,11 +261,11 @@ 
 				<td colspan="3" align="center">
 					<br />
 					<?php
-					if(isset($_GET['add']))
-						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('AddCoursesToURL').'</button>';
-					else
-						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('EditCoursesToURL').'</button>';
-					?>
+                    if(isset($_GET['add']))
+                        echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('AddCoursesToURL').'</button>';
+                    else
+                        echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('EditCoursesToURL').'</button>';
+                    ?>
 				</td>
 			</tr>
 		</table>

Spacing +19 added lines, -19 removed lines

@@ -35,12 +35,12 @@ 
 $interbreadcrumb[] = array('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs'));
 
 $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']);
 }
 
 $access_url_id = 1;
-if (isset($_REQUEST['access_url_id']) && $_REQUEST['access_url_id']!='') {
+if (isset($_REQUEST['access_url_id']) && $_REQUEST['access_url_id'] != '') {
 	$access_url_id = Security::remove_XSS($_REQUEST['access_url_id']);
 }
 
@@ -91,7 +91,7 @@ 
 	}
 
 	if ($form_sent == 1) {
-		if ($access_url_id==0) {
+		if ($access_url_id == 0) {
 			header('Location: access_url_edit_users_to_url.php?action=show_message&message='.get_lang('SelectURL'));
 		} elseif (is_array($course_list)) {
 			UrlManager::update_urls_rel_course($course_list, $access_url_id);
@@ -103,7 +103,7 @@ 
 Display::display_header($tool_name);
 
 echo '<div class="actions">';
-echo Display::url( Display::return_icon('view_more_stats.gif', get_lang('AddUserToURL'),''), api_get_path(WEB_CODE_PATH).'admin/access_url_add_courses_to_url.php');
+echo Display::url(Display::return_icon('view_more_stats.gif', get_lang('AddUserToURL'), ''), api_get_path(WEB_CODE_PATH).'admin/access_url_add_courses_to_url.php');
 echo '</div>';
 
 api_display_tool_title($tool_name);
@@ -114,7 +114,7 @@ 
 $no_course_list = $course_list = array();
 $ajax_search = $add_type == 'unique' ? true : false;
 
-if($ajax_search) {
+if ($ajax_search) {
 	$courses = UrlManager::get_url_rel_course_data($access_url_id);
 	foreach ($courses as $course) {
 		$course_list[$course['c_id']] = $course;
@@ -129,7 +129,7 @@ 
 	}
 
 	$tbl_course = Database :: get_main_table(TABLE_MAIN_COURSE);
-	$sql="SELECT id, code, title
+	$sql = "SELECT id, code, title
 	  	  	FROM $tbl_course u
 			ORDER BY title, code";
 	$result = Database::query($sql);
@@ -142,7 +142,7 @@ 
 	}
 }
 
-if($add_type == 'multiple') {
+if ($add_type == 'multiple') {
 	$link_add_type_unique = '<a href="'.api_get_self().'?add_type=unique&access_url_id='.$access_url_id.'">'.get_lang('SessionAddTypeUnique').'</a>';
 	$link_add_type_multiple = get_lang('SessionAddTypeMultiple');
 } else {
@@ -155,23 +155,23 @@ 
 		<?php echo $link_add_type_unique ?>&nbsp;|&nbsp;<?php echo $link_add_type_multiple ?>
 	</div>
 	<br /><br />
-	<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>" style="margin:0px;" <?php if($ajax_search){echo ' onsubmit="valide();"';}?> >
+	<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>" style="margin:0px;" <?php if ($ajax_search) {echo ' onsubmit="valide();"'; }?> >
 		<?php echo get_lang('SelectUrl').' : '; ?>
 		<select name="access_url_id" onchange="javascript:send();">
 			<option value="0">-- <?php echo get_lang('SelectUrl')?> -- </option>
 			<?php
-			$url_selected='';
+			$url_selected = '';
 			foreach ($url_list as $url_obj) {
 				$checked = '';
 				if (!empty($access_url_id)) {
-					if ($url_obj[0]==$access_url_id) {
+					if ($url_obj[0] == $access_url_id) {
 						$checked = 'selected=true';
-						$url_selected=$url_obj[1];
+						$url_selected = $url_obj[1];
 					}
 				}
-				if ($url_obj['active']==1) {
+				if ($url_obj['active'] == 1) {
 					?>
-					<option <?php echo $checked;?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?></option>
+					<option <?php echo $checked; ?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?></option>
 				<?php
 				}
 			}
@@ -181,7 +181,7 @@ 
 		<input type="hidden" name="form_sent" value="1" />
 		<input type="hidden" name="add_type" value = "<?php echo $add_type ?>" />
 		<?php
-		if(!empty($errorMsg)) {
+		if (!empty($errorMsg)) {
 			Display::display_normal_message($errorMsg); //main API
 		}
 		?>
@@ -199,7 +199,7 @@ 
 				<td align="center">
 					<div id="content_source">
 						<?php
-						if($ajax_search) {
+						if ($ajax_search) {
 							?>
 							<input type="text" id="course_to_add" onkeyup="xajax_search_courses(this.value,document.formulaire.access_url_id.options[document.formulaire.access_url_id.selectedIndex].value)" />
 							<div id="ajax_list_courses"></div>
@@ -208,7 +208,7 @@ 
 							?>
 							<select id="origin_users" name="no_course_list[]" multiple="multiple" size="15" style="width:380px;">
 								<?php
-								foreach($no_course_list as $no_course) {
+								foreach ($no_course_list as $no_course) {
 									?>
 									<option value="<?php echo $no_course['id']; ?>"><?php echo $no_course['title'].' ('.$no_course['code'].')'; ?></option>
 								<?php
@@ -223,7 +223,7 @@ 
 				</td>
 				<td width="10%" valign="middle" align="center">
 					<?php
-					if($ajax_search) {
+					if ($ajax_search) {
 						?>
 						<button class="btn btn-default" type="button" onclick="remove_item(document.getElementById('destination_users'))" >
                             <em class="fa fa-arrow-left"></em>
@@ -246,7 +246,7 @@ 
 				<td align="center">
 					<select id="destination_users" name="course_list[]" multiple="multiple" size="15" style="width:380px;">
 						<?php
-						foreach($course_list as $course) {
+						foreach ($course_list as $course) {
 							$courseInfo = api_get_course_info_by_id($course['id']);
 							?>
 							<option value="<?php echo $course['id']; ?>"><?php echo $course['title'].' ('.$courseInfo['code'].')'; ?></option>
@@ -261,7 +261,7 @@ 
 				<td colspan="3" align="center">
 					<br />
 					<?php
-					if(isset($_GET['add']))
+					if (isset($_GET['add']))
 						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('AddCoursesToURL').'</button>';
 					else
 						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('EditCoursesToURL').'</button>';

Braces +7 added lines, -5 removed lines

@@ -108,8 +108,9 @@ 
 
 api_display_tool_title($tool_name);
 
-if (isset($_GET['action']) && $_GET['action'] == 'show_message')
+if (isset($_GET['action']) && $_GET['action'] == 'show_message') {
 	Display :: display_normal_message(Security::remove_XSS(stripslashes($_GET['message'])));
+}
 
 $no_course_list = $course_list = array();
 $ajax_search = $add_type == 'unique' ? true : false;
@@ -261,10 +262,11 @@ 
 				<td colspan="3" align="center">
 					<br />
 					<?php
-					if(isset($_GET['add']))
-						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('AddCoursesToURL').'</button>';
-					else
-						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('EditCoursesToURL').'</button>';
+					if(isset($_GET['add'])) {
+											echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('AddCoursesToURL').'</button>';
+					} else {
+											echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('EditCoursesToURL').'</button>';
+					}
 					?>
 				</td>
 			</tr>

main/admin/archive_cleanup.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
 api_protect_admin_script(true);
 
 // setting breadcrumbs
-$interbreadcrumb[] = array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
 
 $form = new FormValidator('archive_cleanup_form', 'post', '', '', array(), FormValidator::LAYOUT_BOX);
 $form->addButtonSend(get_lang('ArchiveDirCleanupProceedButton'));
@@ -50,7 +50,7 @@ 
 
 if (isset($_GET['msg']) && isset($_GET['type'])) {
 	if (in_array($_GET['msg'], array('ArchiveDirCleanupSucceeded', 'ArchiveDirCleanupFailed')))
-	switch($_GET['type']) {
+	switch ($_GET['type']) {
 		case 'error':
 			$message = Display::return_message(get_lang($_GET['msg']), 'error');
 			break;

Braces +3 added lines, -2 removed lines

@@ -49,10 +49,11 @@
 Display::display_warning_message(get_lang('ArchiveDirCleanupDescr'));
 
 if (isset($_GET['msg']) && isset($_GET['type'])) {
-	if (in_array($_GET['msg'], array('ArchiveDirCleanupSucceeded', 'ArchiveDirCleanupFailed')))
-	switch($_GET['type']) {
+	if (in_array($_GET['msg'], array('ArchiveDirCleanupSucceeded', 'ArchiveDirCleanupFailed'))) {
+		switch($_GET['type']) {
 		case 'error':
 			$message = Display::return_message(get_lang($_GET['msg']), 'error');
+	}
 			break;
 		case 'confirmation':
 			$message = Display::return_message(get_lang($_GET['msg']), 'confirm');

Indentation +24 added lines, -24 removed lines

@@ -24,39 +24,39 @@
 $message = null;
 
 if ($form->validate()) {
-	$archive_path = api_get_path(SYS_ARCHIVE_PATH);
-	$htaccess = @file_get_contents($archive_path.'.htaccess');
-	$result = rmdirr($archive_path, true, true);
+    $archive_path = api_get_path(SYS_ARCHIVE_PATH);
+    $htaccess = @file_get_contents($archive_path.'.htaccess');
+    $result = rmdirr($archive_path, true, true);
 
-	\Chamilo\CoreBundle\Composer\ScriptHandler::dumpCssFiles();
+    \Chamilo\CoreBundle\Composer\ScriptHandler::dumpCssFiles();
 
-	if (!empty($htaccess)) {
-		@file_put_contents($archive_path.'/.htaccess', $htaccess);
-	}
-	if ($result) {
-		$message = 'ArchiveDirCleanupSucceeded';
-		$type = 'confirmation';
-	} else {
-		$message = 'ArchiveDirCleanupFailed';
-		$type = 'error';
-	}
+    if (!empty($htaccess)) {
+        @file_put_contents($archive_path.'/.htaccess', $htaccess);
+    }
+    if ($result) {
+        $message = 'ArchiveDirCleanupSucceeded';
+        $type = 'confirmation';
+    } else {
+        $message = 'ArchiveDirCleanupFailed';
+        $type = 'error';
+    }
 
-	header('Location: '.api_get_self().'?msg='.$message.'&type='.$type);
-	exit;
+    header('Location: '.api_get_self().'?msg='.$message.'&type='.$type);
+    exit;
 }
 
 Display::display_header(get_lang('ArchiveDirCleanup'));
 Display::display_warning_message(get_lang('ArchiveDirCleanupDescr'));
 
 if (isset($_GET['msg']) && isset($_GET['type'])) {
-	if (in_array($_GET['msg'], array('ArchiveDirCleanupSucceeded', 'ArchiveDirCleanupFailed')))
-	switch($_GET['type']) {
-		case 'error':
-			$message = Display::return_message(get_lang($_GET['msg']), 'error');
-			break;
-		case 'confirmation':
-			$message = Display::return_message(get_lang($_GET['msg']), 'confirm');
-	}
+    if (in_array($_GET['msg'], array('ArchiveDirCleanupSucceeded', 'ArchiveDirCleanupFailed')))
+    switch($_GET['type']) {
+        case 'error':
+            $message = Display::return_message(get_lang($_GET['msg']), 'error');
+            break;
+        case 'confirmation':
+            $message = Display::return_message(get_lang($_GET['msg']), 'confirm');
+    }
 }
 
 if (!empty($message)) {

main/admin/course_import.php

Spacing +4 added lines, -4 removed lines

@@ -12,13 +12,13 @@
  */
 function validate_data($courses)
 {
-    $errors = array ();
-    $coursecodes = array ();
+    $errors = array();
+    $coursecodes = array();
     foreach ($courses as $index => $course) {
-        $course['line'] = $index +1;
+        $course['line'] = $index + 1;
 
         // 1. Check whether mandatory fields are set.
-        $mandatory_fields = array ('Code', 'Title', 'CourseCategory');
+        $mandatory_fields = array('Code', 'Title', 'CourseCategory');
         foreach ($mandatory_fields as $field) {
             if (!isset($course[$field]) || strlen($course[$field]) == 0) {
                 $course['error'] = get_lang($field.'Mandatory');

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
 /**
  * Read the CSV-file
  * @param string $file Path to the CSV-file
- * @return array All course-information read from the file
+ * @return Ddeboer\DataImport\Reader\CsvReader All course-information read from the file
  */
 function parse_csv_data($file)
 {

main/admin/subscribe_user2class.php

Spacing +9 added lines, -9 removed lines

@@ -32,16 +32,16 @@ 
 $result = Database::query($sql);
 
 if (!list ($class_name) = Database::fetch_row($result)) {
-    header('Location: class_list.php?filtreCours=' . urlencode($course));
+    header('Location: class_list.php?filtreCours='.urlencode($course));
     exit();
 }
 
 $noPHP_SELF = true;
 
-$tool_name = get_lang('AddUsersToAClass') . ' (' . $class_name . ')';
+$tool_name = get_lang('AddUsersToAClass').' ('.$class_name.')';
 
 $interbreadcrumb[] = array("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array("url" => "class_list.php?filtreCours=" . urlencode($course), "name" => get_lang('AdminClasses'));
+$interbreadcrumb[] = array("url" => "class_list.php?filtreCours=".urlencode($course), "name" => get_lang('AdminClasses'));
 
 if ($_POST['formSent']) {
     $form_sent = $_POST['formSent'];
@@ -59,7 +59,7 @@ 
                 foreach ($left_user_list as $user_id) {
                     ClassManager :: add_user($user_id, $class_id);
                 }
-                header('Location: class_list.php?filtreCours=' . urlencode($course));
+                header('Location: class_list.php?filtreCours='.urlencode($course));
                 exit();
             }
         } elseif ($remove_from_class) {
@@ -69,7 +69,7 @@ 
                 foreach ($right_user_list as $index => $user_id) {
                     ClassManager :: unsubscribe_user($user_id, $class_id);
                 }
-                header('Location: class_list.php?filtreCours=' . urlencode($course));
+                header('Location: class_list.php?filtreCours='.urlencode($course));
                 exit();
             }
         }
@@ -78,10 +78,10 @@ 
 Display :: display_header($tool_name);
 //api_display_tool_title($tool_name);
 $target_name = api_sort_by_first_name() ? 'firstname' : 'lastname';
-$sql = "SELECT u.user_id,lastname,firstname,username FROM $tbl_user u LEFT JOIN $tbl_class_user cu ON u.user_id=cu.user_id AND class_id='$class_id' WHERE " . $target_name . " LIKE '" . $first_letter_left . "%' AND class_id IS NULL ORDER BY " . (count($left_user_list) > 0 ? "(user_id IN(" . implode(',', $left_user_list) . ")) DESC," : "") . " " . $target_name;
+$sql = "SELECT u.user_id,lastname,firstname,username FROM $tbl_user u LEFT JOIN $tbl_class_user cu ON u.user_id=cu.user_id AND class_id='$class_id' WHERE ".$target_name." LIKE '".$first_letter_left."%' AND class_id IS NULL ORDER BY ".(count($left_user_list) > 0 ? "(user_id IN(".implode(',', $left_user_list).")) DESC," : "")." ".$target_name;
 $result = Database::query($sql);
 $left_users = Database::store_result($result);
-$sql = "SELECT u.user_id,lastname,firstname,username FROM $tbl_user u,$tbl_class_user cu WHERE cu.user_id=u.user_id AND class_id='$class_id' AND " . $target_name . " LIKE '" . $first_letter_right . "%' ORDER BY " . (count($right_user_list) > 0 ? "(user_id IN(" . implode(',', $right_user_list) . ")) DESC," : "") . " " . $target_name;
+$sql = "SELECT u.user_id,lastname,firstname,username FROM $tbl_user u,$tbl_class_user cu WHERE cu.user_id=u.user_id AND class_id='$class_id' AND ".$target_name." LIKE '".$first_letter_right."%' ORDER BY ".(count($right_user_list) > 0 ? "(user_id IN(".implode(',', $right_user_list).")) DESC," : "")." ".$target_name;
 $result = Database::query($sql);
 $right_users = Database::store_result($result);
 if (!empty($error_message)) {
@@ -122,7 +122,7 @@ 
                 <?php
                 foreach ($left_users as $user) {
                     ?>
-                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $left_user_list)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($user['firstname'], $user['lastname']) . ' (' . $user['username'] . ')'; ?></option>
+                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $left_user_list)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].')'; ?></option>
                         <?php
                     }
                     ?>
@@ -138,7 +138,7 @@ 
                     <?php
                     foreach ($right_users as $user) {
                         ?>
-                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $right_user_list)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($user['firstname'], $user['lastname']) . ' (' . $user['username'] . ')'; ?></option>
+                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $right_user_list)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].')'; ?></option>
     <?php
 }
 ?>

Braces +11 added lines, -5 removed lines

@@ -63,9 +63,9 @@ 
                 exit();
             }
         } elseif ($remove_from_class) {
-            if (count($right_user_list) == 0)
-                $error_message = get_lang('AtLeastOneUser');
-            else {
+            if (count($right_user_list) == 0) {
+                            $error_message = get_lang('AtLeastOneUser');
+            } else {
                 foreach ($right_user_list as $index => $user_id) {
                     ClassManager :: unsubscribe_user($user_id, $class_id);
                 }
@@ -122,7 +122,10 @@ 
                 <?php
                 foreach ($left_users as $user) {
                     ?>
-                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $left_user_list)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($user['firstname'], $user['lastname']) . ' (' . $user['username'] . ')'; ?></option>
+                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $left_user_list)) {
+    echo 'selected="selected"';
+}
+?>><?php echo api_get_person_name($user['firstname'], $user['lastname']) . ' (' . $user['username'] . ')'; ?></option>
                         <?php
                     }
                     ?>
@@ -138,7 +141,10 @@ 
                     <?php
                     foreach ($right_users as $user) {
                         ?>
-                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $right_user_list)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($user['firstname'], $user['lastname']) . ' (' . $user['username'] . ')'; ?></option>
+                        <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $right_user_list)) {
+    echo 'selected="selected"';
+}
+?>><?php echo api_get_person_name($user['firstname'], $user['lastname']) . ' (' . $user['username'] . ')'; ?></option>
     <?php
 }
 ?>

main/admin/subscribe_user2course.php

Indentation +4 added lines, -4 removed lines

@@ -263,7 +263,7 @@ 
       <option value="">--</option>
       <?php
         echo Display :: get_alphabet_options($first_letter_user);
-      ?>
+        ?>
      </select>
     </td>
     <td width="20%">&nbsp;</td>
@@ -274,8 +274,8 @@ 
      <select name="firstLetterCourse" onchange="javascript:document.formulaire.form_sent.value='2'; document.formulaire.submit();">
       <option value="">--</option>
       <?php
-      echo Display :: get_alphabet_options($first_letter_course);
-      ?>
+        echo Display :: get_alphabet_options($first_letter_course);
+        ?>
      </select>
     </td>
    </tr>
@@ -295,7 +295,7 @@ 
         }
 
         echo $userName;
-      ?>
+        ?>
       </option>
 <?php
 }

Spacing +32 added lines, -32 removed lines

@@ -23,7 +23,7 @@ 
 $form_sent = 0;
 $first_letter_user = '';
 $first_letter_course = '';
-$courses = array ();
+$courses = array();
 $users = array();
 
 $tbl_course = Database :: get_main_table(TABLE_MAIN_COURSE);
@@ -31,7 +31,7 @@ 
 
 /* Header */
 $tool_name = get_lang('AddUsersToACourse');
-$interbreadcrumb[] = array ("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
 
 $htmlHeadXtra[] = '
 <script type="text/javascript">
@@ -44,7 +44,7 @@ 
 // displaying the header
 Display :: display_header($tool_name);
 
-$link_add_group = '<a href="usergroups.php">'.Display::return_icon('multiple.gif',get_lang('RegistrationByUsersGroups')).get_lang('RegistrationByUsersGroups').'</a>';
+$link_add_group = '<a href="usergroups.php">'.Display::return_icon('multiple.gif', get_lang('RegistrationByUsersGroups')).get_lang('RegistrationByUsersGroups').'</a>';
 echo '<div class="actions">'.$link_add_group.'</div>';
 
 $form = new FormValidator('subscribe_user2course');
@@ -52,12 +52,12 @@ 
 $form->display();
 
 //checking for extra field with filter on
-$extra_field_list= UserManager::get_extra_fields();
+$extra_field_list = UserManager::get_extra_fields();
 $new_field_list = array();
 if (is_array($extra_field_list)) {
     foreach ($extra_field_list as $extra_field) {
         //if is enabled to filter and is a "<select>" field type
-        if ($extra_field[8]==1 && $extra_field[2]==4 ) {
+        if ($extra_field[8] == 1 && $extra_field[2] == 4) {
             $new_field_list[] = array('name'=> $extra_field[3], 'variable'=>$extra_field[1], 'data'=> $extra_field[9]);
         }
     }
@@ -66,8 +66,8 @@ 
 /* React on POSTed request */
 if (isset($_POST['form_sent']) && $_POST['form_sent']) {
     $form_sent = $_POST['form_sent'];
-    $users = isset($_POST['UserList']) && is_array($_POST['UserList']) ? $_POST['UserList'] : array() ;
-    $courses = isset($_POST['CourseList']) && is_array($_POST['CourseList']) ? $_POST['CourseList'] : array() ;
+    $users = isset($_POST['UserList']) && is_array($_POST['UserList']) ? $_POST['UserList'] : array();
+    $courses = isset($_POST['CourseList']) && is_array($_POST['CourseList']) ? $_POST['CourseList'] : array();
     $first_letter_user = $_POST['firstLetterUser'];
     $first_letter_course = $_POST['firstLetterCourse'];
 
@@ -105,7 +105,7 @@ 
     $sql = "SELECT count(*) as nb_users FROM $tbl_user";
     $result = Database::query($sql);
     $num_row = Database::fetch_array($result);
-    if  ($num_row['nb_users']>1000) {
+    if ($num_row['nb_users'] > 1000) {
         //if there are too much users to gracefully handle with the HTML select list,
         // assign a default filter on users names
         $first_letter_user = 'A';
@@ -119,14 +119,14 @@ 
 //Filter by Extra Fields
 $use_extra_fields = false;
 if (is_array($extra_field_list)) {
-    if (is_array($new_field_list) && count($new_field_list)>0 ) {
-        $result_list=array();
+    if (is_array($new_field_list) && count($new_field_list) > 0) {
+        $result_list = array();
         foreach ($new_field_list as $new_field) {
             $varname = 'field_'.$new_field['variable'];
             if (UserManager::is_extra_field_available($new_field['variable'])) {
-                if (isset($_POST[$varname]) && $_POST[$varname]!='0') {
+                if (isset($_POST[$varname]) && $_POST[$varname] != '0') {
                     $use_extra_fields = true;
-                    $extra_field_result[]= UserManager::get_extra_user_data_by_value(
+                    $extra_field_result[] = UserManager::get_extra_user_data_by_value(
                         $new_field['variable'],
                         $_POST[$varname]
                     );
@@ -138,10 +138,10 @@ 
 
 if ($use_extra_fields) {
     $final_result = array();
-    if (count($extra_field_result)>1) {
-        for($i=0;$i<count($extra_field_result)-1;$i++) {
-            if (is_array($extra_field_result[$i+1])) {
-                $final_result  = array_intersect($extra_field_result[$i],$extra_field_result[$i+1]);
+    if (count($extra_field_result) > 1) {
+        for ($i = 0; $i < count($extra_field_result) - 1; $i++) {
+            if (is_array($extra_field_result[$i + 1])) {
+                $final_result = array_intersect($extra_field_result[$i], $extra_field_result[$i + 1]);
             }
         }
     } else {
@@ -149,15 +149,15 @@ 
     }
 
     if (api_is_multiple_url_enabled()) {
-        if (is_array($final_result) && count($final_result)>0) {
-            $where_filter = " AND u.user_id IN  ('".implode("','",$final_result)."') ";
+        if (is_array($final_result) && count($final_result) > 0) {
+            $where_filter = " AND u.user_id IN  ('".implode("','", $final_result)."') ";
         } else {
             //no results
             $where_filter = " AND u.user_id  = -1";
         }
     } else {
-        if (is_array($final_result) && count($final_result)>0) {
-            $where_filter = " AND user_id IN  ('".implode("','",$final_result)."') ";
+        if (is_array($final_result) && count($final_result) > 0) {
+            $where_filter = " AND user_id IN  ('".implode("','", $final_result)."') ";
         } else {
             //no results
             $where_filter = " AND user_id  = -1";
@@ -177,14 +177,14 @@ 
 $sql = "SELECT user_id, lastname, firstname, username, official_code
         FROM $tbl_user
         WHERE user_id<>2 AND ".$target_name." LIKE '".$first_letter_user."%' $where_filter
-        ORDER BY ". (count($users) > 0 ? "(user_id IN(".implode(',', $users).")) DESC," : "")." ".$orderBy;
+        ORDER BY ".(count($users) > 0 ? "(user_id IN(".implode(',', $users).")) DESC," : "")." ".$orderBy;
 
 if (api_is_multiple_url_enabled()) {
-    $tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
+    $tbl_user_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
     $access_url_id = api_get_current_access_url_id();
-    if ($access_url_id != -1){
+    if ($access_url_id != -1) {
         $sql = "SELECT u.user_id,lastname,firstname,username, official_code
-                FROM ".$tbl_user ." u
+                FROM ".$tbl_user." u
                 INNER JOIN $tbl_user_rel_access_url user_rel_url
                 ON (user_rel_url.user_id = u.user_id)
                 WHERE
@@ -192,7 +192,7 @@ 
                     access_url_id =  $access_url_id AND
                     (".$target_name." LIKE '".$first_letter_user."%' )
                     $where_filter
-                ORDER BY ". (count($users) > 0 ? "(u.user_id IN(".implode(',', $users).")) DESC," : "")." ".$orderBy;
+                ORDER BY ".(count($users) > 0 ? "(u.user_id IN(".implode(',', $users).")) DESC," : "")." ".$orderBy;
     }
 }
 
@@ -208,7 +208,7 @@ 
 if (api_is_multiple_url_enabled()) {
     $tbl_course_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_COURSE);
     $access_url_id = api_get_current_access_url_id();
-    if ($access_url_id != -1){
+    if ($access_url_id != -1) {
         $sql = "SELECT code, visual_code, title
                 FROM $tbl_course as course
                 INNER JOIN $tbl_course_rel_access_url course_rel_url
@@ -228,17 +228,17 @@ 
 <form name="formulaire" method="post" action="<?php echo api_get_self(); ?>" style="margin:0px;">
 <?php
 if (is_array($extra_field_list)) {
-    if (is_array($new_field_list) && count($new_field_list)>0 ) {
+    if (is_array($new_field_list) && count($new_field_list) > 0) {
         echo '<h3>'.get_lang('FilterUsers').'</h3>';
         foreach ($new_field_list as $new_field) {
             echo $new_field['name'];
             $varname = 'field_'.$new_field['variable'];
             echo '&nbsp;<select name="'.$varname.'">';
             echo '<option value="0">--'.get_lang('Select').'--</option>';
-            foreach	($new_field['data'] as $option) {
-                $checked='';
+            foreach ($new_field['data'] as $option) {
+                $checked = '';
                 if (isset($_POST[$varname])) {
-                    if ($_POST[$varname]==$option[1]) {
+                    if ($_POST[$varname] == $option[1]) {
                         $checked = 'selected="true"';
                     }
                 }
@@ -285,7 +285,7 @@ 
 <?php
     foreach ($db_users as $user) {
 ?>
-      <option value="<?php echo $user['user_id']; ?>" <?php if(in_array($user['user_id'],$users)) echo 'selected="selected"'; ?>>
+      <option value="<?php echo $user['user_id']; ?>" <?php if (in_array($user['user_id'], $users)) echo 'selected="selected"'; ?>>
       <?php
         $userName = api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].')';
         if ($showOfficialCode) {
@@ -312,7 +312,7 @@ 
 <?php
     foreach ($db_courses as $course) {
 ?>
-     <option value="<?php echo $course['code']; ?>" <?php if(in_array($course['code'],$courses)) echo 'selected="selected"'; ?>>
+     <option value="<?php echo $course['code']; ?>" <?php if (in_array($course['code'], $courses)) echo 'selected="selected"'; ?>>
          <?php echo '('.$course['visual_code'].') '.$course['title']; ?>
      </option>
 <?php

Braces +8 added lines, -2 removed lines

@@ -285,7 +285,10 @@ 
 <?php
     foreach ($db_users as $user) {
 ?>
-      <option value="<?php echo $user['user_id']; ?>" <?php if(in_array($user['user_id'],$users)) echo 'selected="selected"'; ?>>
+      <option value="<?php echo $user['user_id']; ?>" <?php if(in_array($user['user_id'],$users)) {
+    echo 'selected="selected"';
+}
+?>>
       <?php
         $userName = api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].')';
         if ($showOfficialCode) {
@@ -312,7 +315,10 @@ 
 <?php
     foreach ($db_courses as $course) {
 ?>
-     <option value="<?php echo $course['code']; ?>" <?php if(in_array($course['code'],$courses)) echo 'selected="selected"'; ?>>
+     <option value="<?php echo $course['code']; ?>" <?php if(in_array($course['code'],$courses)) {
+    echo 'selected="selected"';
+}
+?>>
          <?php echo '('.$course['visual_code'].') '.$course['title']; ?>
      </option>
 <?php

main/admin/teacher_time_report.php

Spacing +1 added lines, -1 removed lines

@@ -295,7 +295,7 @@
 if (isset($_GET['export'])) {
     $dataToExport = $timeReport->prepareDataToExport($withFilter);
 
-    $fileName = get_lang('TeacherTimeReport') . ' ' . api_get_local_time();
+    $fileName = get_lang('TeacherTimeReport').' '.api_get_local_time();
 
     switch ($_GET['export']) {
         case 'pdf':

main/admin/skills_gradebook.php

Indentation +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
                         array('name'=>'certificate',    'index'=>'certificate', 'width'=>'25', 'align'=>'left', 'sortable'=>'false'),
                         array('name'=>'skills',         'index'=>'skills',      'width'=>'300', 'align'=>'left', 'sortable'=>'false'),
                         array('name'=>'actions',        'index'=>'actions',     'width'=>'30', 'align'=>'left','formatter'=>'action_formatter','sortable'=>'false')
-                       );
+                        );
 //Autowidth
 $extra_params['autowidth'] = 'true';
 //height auto

Spacing +11 added lines, -11 removed lines

@@ -25,9 +25,9 @@ 
 // setting breadcrumbs
 
 $tool_name = get_lang('SkillsAndGradebooks');
-$interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
 if ($action == 'add_skill') {
-    $interbreadcrumb[]=array('url' => 'skills_gradebook.php','name' => get_lang('SkillsAndGradebooks'));
+    $interbreadcrumb[] = array('url' => 'skills_gradebook.php', 'name' => get_lang('SkillsAndGradebooks'));
     $tool_name = get_lang('Add');
 }
 
@@ -46,10 +46,10 @@ 
 
 //Column config
 $column_model   = array(
-                        array('name'=>'name',           'index'=>'name',        'width'=>'150', 'align'=>'left'),
-                        array('name'=>'certificate',    'index'=>'certificate', 'width'=>'25', 'align'=>'left', 'sortable'=>'false'),
-                        array('name'=>'skills',         'index'=>'skills',      'width'=>'300', 'align'=>'left', 'sortable'=>'false'),
-                        array('name'=>'actions',        'index'=>'actions',     'width'=>'30', 'align'=>'left','formatter'=>'action_formatter','sortable'=>'false')
+                        array('name'=>'name', 'index'=>'name', 'width'=>'150', 'align'=>'left'),
+                        array('name'=>'certificate', 'index'=>'certificate', 'width'=>'25', 'align'=>'left', 'sortable'=>'false'),
+                        array('name'=>'skills', 'index'=>'skills', 'width'=>'300', 'align'=>'left', 'sortable'=>'false'),
+                        array('name'=>'actions', 'index'=>'actions', 'width'=>'30', 'align'=>'left', 'formatter'=>'action_formatter', 'sortable'=>'false')
                        );
 //Autowidth
 $extra_params['autowidth'] = 'true';
@@ -60,9 +60,9 @@ 
 $action_links = 'function action_formatter(cellvalue, options, rowObject) {
                         //certificates
                         if (rowObject[4] == 1) {
-                            return \'<a href="?action=add_skill&id=\'+options.rowId+\'">'.Display::return_icon('add.png', get_lang('AddSkill'),'',ICON_SIZE_SMALL).'</a>'.'\';
+                            return \'<a href="?action=add_skill&id=\'+options.rowId+\'">'.Display::return_icon('add.png', get_lang('AddSkill'), '', ICON_SIZE_SMALL).'</a>'.'\';
                         } else {
-                            return \''.Display::return_icon('add_na.png', get_lang('YourGradebookFirstNeedsACertificateInOrderToBeLinkedToASkill'),'',ICON_SIZE_SMALL).''.'\';
+                            return \''.Display::return_icon('add_na.png', get_lang('YourGradebookFirstNeedsACertificateInOrderToBeLinkedToASkill'), '', ICON_SIZE_SMALL).''.'\';
                         }
                  }';
 ?>
@@ -70,14 +70,14 @@ 
 $(function() {
 <?php
     // grid definition see the $career->display() function
-    echo Display::grid_js('gradebooks', $url, $columns, $column_model, $extra_params, array(), $action_links,true);
+    echo Display::grid_js('gradebooks', $url, $columns, $column_model, $extra_params, array(), $action_links, true);
 ?>
 });
 </script>
 <?php
 $gradebook = new Gradebook();
 
-switch($action) {
+switch ($action) {
     case 'display':
         $gradebook->display();
         break;
@@ -85,7 +85,7 @@ 
         $id = isset($_REQUEST['id']) ? $_REQUEST['id'] : null;
         $gradebook_info = $gradebook->get($id);
         $url  = api_get_self().'?action='.$action.'&id='.$id;
-        $form =  $gradebook->show_skill_form($id, $url, $gradebook_info['name']);
+        $form = $gradebook->show_skill_form($id, $url, $gradebook_info['name']);
         if ($form->validate()) {
             $values = $form->exportValues();
             $res    = $gradebook->update_skills_to_gradebook($values['id'], $values['skill']);