chamilo / chamilo-lms

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

main/admin/legal_list.php

Braces +3 added lines, -2 removed lines

@@ -24,8 +24,9 @@
 
 // Actions
 if (isset ($_GET['action'])) {
-	if ($_GET['action'] == 'show_message')
-		Display :: display_normal_message(Security::remove_XSS(stripslashes($_GET['message'])));
+	if ($_GET['action'] == 'show_message') {
+			Display :: display_normal_message(Security::remove_XSS(stripslashes($_GET['message'])));
+	}
 	Security::clear_token();
 }
 

main/admin/add_sessions_to_promotion.php

Braces +4 added lines, -1 removed lines

@@ -144,7 +144,10 @@
 echo '</div>';
 ?>
 
-<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if(!empty($_GET['add'])) echo '&add=true' ; ?>" style="margin:0px;" <?php if($ajax_search){echo ' onsubmit="valide();"';}?>>
+<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if(!empty($_GET['add'])) {
+    echo '&add=true' ;
+}
+?>" style="margin:0px;" <?php if($ajax_search){echo ' onsubmit="valide();"';}?>>
 <?php echo '<legend>'.$tool_name.' '.$promotion_data['name'].'</legend>';
 
 if ($add_type=='multiple') {

main/admin/access_url_edit_usergroup_to_url.php

Braces +5 added lines, -4 removed lines

@@ -254,10 +254,11 @@
 	<td colspan="3" align="center">
 		<br />
 		<?php
-		if(isset($_GET['add']))
-			echo '<button class="save" onclick="valide()" >'.get_lang('AddUserGroupToURL').'</button>';
-		else
-			echo '<button class="save" onclick="valide()" >'.get_lang('EditUserGroupToURL').'</button>';
+		if(isset($_GET['add'])) {
+					echo '<button class="save" onclick="valide()" >'.get_lang('AddUserGroupToURL').'</button>';
+		} else {
+					echo '<button class="save" onclick="valide()" >'.get_lang('EditUserGroupToURL').'</button>';
+		}
 		?>
 	</td>
 </tr>

main/admin/settings.lib.php

Braces +15 added lines, -10 removed lines

@@ -471,8 +471,9 @@ 
                     $extraction_path = $cssToUpload.$style_name.'/';
                     for ($i = 0; $i < $num_files; $i++) {
                         $entry = $zip->getNameIndex($i);
-                        if (substr($entry, -1) == '/')
-                            continue;
+                        if (substr($entry, -1) == '/') {
+                                                    continue;
+                        }
 
                         $pos_slash = strpos($entry, '/');
                         $entry_without_first_dir = substr($entry, $pos_slash + 1);
@@ -1231,17 +1232,21 @@ 
                 $hideme = array('disabled');
             } elseif ($url_info['active'] == 1) {
                 // We show the elements.
-                if (empty($row['variable']))
-                    $row['variable'] = 0;
-                if (empty($row['subkey']))
-                    $row['subkey'] = 0;
-                if (empty($row['category']))
-                    $row['category'] = 0;
+                if (empty($row['variable'])) {
+                                    $row['variable'] = 0;
+                }
+                if (empty($row['subkey'])) {
+                                    $row['subkey'] = 0;
+                }
+                if (empty($row['category'])) {
+                                    $row['category'] = 0;
+                }
 
                 if (is_array($settings_by_access_list[ $row['variable'] ] [ $row['subkey'] ] [ $row['category'] ])) {
                     // We are sure that the other site have a selected value.
-                    if ($settings_by_access_list[ $row['variable'] ] [ $row['subkey'] ] [ $row['category'] ]['selected_value'] != '')
-                        $row['selected_value'] = $settings_by_access_list[$row['variable']] [$row['subkey']] [$row['category']]['selected_value'];
+                    if ($settings_by_access_list[ $row['variable'] ] [ $row['subkey'] ] [ $row['category'] ]['selected_value'] != '') {
+                                            $row['selected_value'] = $settings_by_access_list[$row['variable']] [$row['subkey']] [$row['category']]['selected_value'];
+                    }
                 }
                 // There is no else{} statement because we load the default $row['selected_value'] of the main Chamilo site.
             }