chamilo / chamilo-lms

main/admin/user_update_import.php

Braces +4 added lines, -1 removed lines

@@ -462,7 +462,10 @@
         <blockquote>
             <pre>
                 <b>UserName</b>;LastName;FirstName;Email;NewUserName;Password;AuthSource;OfficialCode;PhoneNumber;Status;ExpiryDate;Active;Language;Courses;ClassId;
-                xxx;xxx;xxx;xxx;xxx;xxx;xxx;xxx;xxx;user/teacher/drh;0000-00-00 00:00:00;0/1;xxx;<span style="color:red;"><?php if (count($list_reponse) > 0) echo implode(';', $list_reponse).';'; ?></span>xxx1|xxx2|xxx3;1;<br />
+                xxx;xxx;xxx;xxx;xxx;xxx;xxx;xxx;xxx;user/teacher/drh;0000-00-00 00:00:00;0/1;xxx;<span style="color:red;"><?php if (count($list_reponse) > 0) {
+    echo implode(';', $list_reponse).';';
+}
+?></span>xxx1|xxx2|xxx3;1;<br />
             </pre>
         </blockquote>
     <p><?php

main/admin/sub_language.php

Braces +3 added lines, -2 removed lines

@@ -200,8 +200,9 @@
 					continue;
 				}
 
-				if (is_array($variable_value))
-					echo $lang_file;
+				if (is_array($variable_value)) {
+									echo $lang_file;
+				}
 				$founded = false;
 				if ($search_in_english && $search_in_variable) {
 					// searching the item in the parent tool

main/admin/group_list.php

Braces +3 added lines, -2 removed lines

@@ -412,8 +412,9 @@
     //$table->set_column_filter(6, 'status_filter');
     //$table->set_column_filter(7, 'active_filter');
     //$table->set_column_filter(8, 'modify_filter');
-    if (api_is_platform_admin())
-        $table->set_form_actions(array('delete' => get_lang('DeleteFromPlatform')));
+    if (api_is_platform_admin()) {
+            $table->set_form_actions(array('delete' => get_lang('DeleteFromPlatform')));
+    }
     $table->display();
 }
 Display :: display_footer();

main/admin/settings.php

Braces +3 added lines, -2 removed lines

@@ -533,8 +533,9 @@
             }
             break;
         default:
-            if (isset($form))
-                $form->display();
+            if (isset($form)) {
+                            $form->display();
+            }
     }
 }
 

main/admin/group_edit.php

Braces +1 added lines, -2 removed lines

@@ -109,8 +109,7 @@
 	$picture_uri = $group_data['picture_uri'];
 	if ($group['delete_picture']) {
 		$picture_uri = GroupPortalManager::delete_group_picture($group_id);
-		}
-	elseif (!empty($picture['name'])) {
+		} elseif (!empty($picture['name'])) {
         $picture_uri = GroupPortalManager::update_group_picture(
             $group_id,
             $_FILES['picture']['name'],

main/admin/access_url_edit_courses_to_url.php

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

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');

main/admin/subscribe_user2class.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

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