chamilo / chamilo-lms

main/admin/ldap_users_list.php

Braces +15 added lines, -20 removed lines

@@ -51,8 +51,7 @@ 
 				if ($user_id != $_user['user_id'] && UserManager :: delete_user($_GET['user_id']))
 				{
 					Display :: display_normal_message(get_lang('UserDeleted'));
-				}
-				else
+				} else
 				{
 					Display :: display_error_message(get_lang('CannotDeleteUser'));
 				}
@@ -88,8 +87,7 @@ 
 						{
 							$message .= '- '.$login.'<br />';
 						}
-					}
-					else
+					} else
 					{
 						$message=get_lang('NoUserAdded');
 					}
@@ -100,13 +98,11 @@ 
 				Display :: display_header($tool_name);
 		}
 		Security::clear_token();
-	}
-	else
+	} else
 	{
 		Display::display_header($tool_name);
 	}
-}
-else
+} else
 {
 	Display::display_header($tool_name);
 }
@@ -133,8 +129,7 @@ 
 				if($number_of_selected_users == $number_of_deleted_users)
 				{
 					Display :: display_normal_message(get_lang('SelectedUsersDeleted'));
-				}
-				else
+				} else
 				{
 					Display :: display_error_message(get_lang('SomeUsersNotDeleted'));
 				}
@@ -150,13 +145,13 @@ 
 						$UserList[] = ldap_add_user($user_id);
 					}
 				}
-				if (isset($_GET['id_session']) && (trim($_GET['id_session'])!=""))
-					addUserToSession($UserList, $_GET['id_session']);
+				if (isset($_GET['id_session']) && (trim($_GET['id_session'])!="")) {
+									addUserToSession($UserList, $_GET['id_session']);
+				}
 				if(count($UserList)>0)
 				{
 					Display :: display_normal_message(count($UserList)." ".get_lang('LDAPUsersAdded'));
-				}
-				else
+				} else
 				{
 					Display :: display_normal_message(get_lang('NoUserAdded'));
 				}
@@ -173,14 +168,14 @@ 
 {
 	$form->addText('keyword_firstname', get_lang('FirstName'), false);
 	$form->addText('keyword_lastname', get_lang('LastName'), false);
-}
-else
+} else
 {
 	$form->addText('keyword_lastname',get_lang('LastName'),false);
 	$form->addText('keyword_firstname',get_lang('FirstName'),false);
 }
-if (isset($_GET['id_session']))
+if (isset($_GET['id_session'])) {
 	$form->addElement('hidden','id_session',$_GET['id_session']);
+}
 
 $type = array();
 $type["all"] = get_lang('All');
@@ -201,8 +196,9 @@ 
 $parameters['keyword_firstname'] = @$_GET['keyword_firstname'] ?: null;
 $parameters['keyword_lastname'] = @$_GET['keyword_lastname'] ?: null;
 $parameters['keyword_email'] = @$_GET['keyword_email'] ?: null;
-if (isset($_GET['id_session']))
+if (isset($_GET['id_session'])) {
 	$parameters['id_session'] = $_GET['id_session'];
+}
 // Create a sortable table with user-data
 
 $parameters['sec_token'] = Security::get_token();
@@ -214,8 +210,7 @@ 
 {
 	$table->set_header(2, get_lang('FirstName'));
 	$table->set_header(3, get_lang('LastName'));
-}
-else
+} else
 {
 	$table->set_header(2, get_lang('LastName'));
 	$table->set_header(3, get_lang('FirstName'));

main/admin/access_url_edit_course_category_to_url.php

Braces +5 added lines, -4 removed lines

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

main/admin/add_users_to_usergroup.php

Braces +8 added lines, -2 removed lines

@@ -345,7 +345,10 @@ 
 $searchForm->display();
 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;">
+<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
 echo '<legend>'.$tool_name.': '.$data['name'].'</legend>';
 
@@ -420,7 +423,10 @@ 
     ?>
     <br />
       <label class="control-label">
-          <input type="checkbox" <?php if ($user_with_any_group) echo 'checked="checked"';?> onchange="checked_in_no_group(this.checked);" name="user_with_any_group" id="user_with_any_group_id">
+          <input type="checkbox" <?php if ($user_with_any_group) {
+    echo 'checked="checked"';
+}
+?> onchange="checked_in_no_group(this.checked);" name="user_with_any_group" id="user_with_any_group_id">
           <?php echo get_lang('UsersRegisteredInAnyGroup'); ?>
       </label>
     </div>

main/announcements/download.php

Braces +3 added lines, -1 removed lines

@@ -43,7 +43,9 @@
     //remove last slash if present
     //$doc_url = ($doc_url{strlen($doc_url)-1}=='/')?substr($doc_url,0,strlen($doc_url)-1):$doc_url;
     //mod_rewrite can change /some/path/ to /some/path// in some cases, so clean them all off (René)
-    while ($doc_url{$dul = strlen($doc_url)-1}=='/') $doc_url = substr($doc_url,0,$dul);
+    while ($doc_url{$dul = strlen($doc_url)-1}=='/') {
+        $doc_url = substr($doc_url,0,$dul);
+    }
     //create the path
     $document_explorer = api_get_path(WEB_COURSE_PATH).api_get_course_path(); // home course path
     //redirect

main/survey/fillsurvey.php

Braces +1 added lines, -1 removed lines

@@ -1132,7 +1132,7 @@
                         $questions[$row['sort']]['survey_group_sec1'] = $row['survey_group_sec1'];
                         $questions[$row['sort']]['survey_group_sec2'] = $row['survey_group_sec2'];
                         $questions[$row['sort']]['survey_group_pri'] = $row['survey_group_pri'];
-                    }  else {
+                    } else {
                         // If the type is a page break we are finished loading the questions for this page
                         break;
                     }

main/survey/survey.lib.php

Braces +3 added lines, -5 removed lines

@@ -1247,7 +1247,7 @@ 
                     '".Database::escape_string($_course['id'])."')";
             Database::query($sql);
             $shared_question_id = Database::insert_id();
-        }  else {
+        } else {
             // Updating an existing question
             // adding the question to the survey_question table
             $sql = "UPDATE $tbl_survey_question SET
@@ -2776,14 +2776,12 @@ 
                                 echo $answers_of_user[$question_id][$option_id]['value'];
                                 echo "</td>";
                             }
-                        }
-                        else {
+                        } else {
                             echo '<td align="center">';
                             if (!empty($answers_of_user[$question_id][$option_id])) {
                                 if ($answers_of_user[$question_id][$option_id]['value'] != 0) {
                                     echo $answers_of_user[$question_id][$option_id]['value'];
-                                }
-                                else {
+                                } else {
                                     echo 'v';
                                 }
                             }

main/survey/survey.php

Braces +3 added lines, -1 removed lines

@@ -124,7 +124,9 @@
 	}
 }
 
-if (!empty($survey_data['survey_version'])) echo '<b>'.get_lang('Version').': '.$survey_data['survey_version'].'</b>';
+if (!empty($survey_data['survey_version'])) {
+    echo '<b>'.get_lang('Version').': '.$survey_data['survey_version'].'</b>';
+}
 
 // We exit here is the first or last question is a pagebreak (which causes errors)
 SurveyUtil::check_first_last_question($_GET['survey_id']);

main/session/session_course_user.php

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_edit.php

Braces +8 added lines, -2 removed lines

@@ -166,13 +166,19 @@
 
         <select name="id_coach[]" class="form-control" multiple>
             <option value="0">----- <?php echo get_lang("Choose") ?> -----</option>
-            <option value="0" <?php if(count($arr_infos) == 0) echo 'selected="selected"'; ?>>
+            <option value="0" <?php if(count($arr_infos) == 0) {
+    echo 'selected="selected"';
+}
+?>>
                 <?php echo get_lang('None') ?>
             </option>
             <?php
             foreach ($coaches as $enreg) {
                 ?>
-                <option value="<?php echo $enreg['user_id']; ?>" <?php if(((is_array($arr_infos) && in_array($enreg['user_id'], $arr_infos)))) echo 'selected="selected"'; ?>>
+                <option value="<?php echo $enreg['user_id']; ?>" <?php if(((is_array($arr_infos) && in_array($enreg['user_id'], $arr_infos)))) {
+    echo 'selected="selected"';
+}
+?>>
                     <?php echo api_get_person_name($enreg['firstname'], $enreg['lastname']).' ('.$enreg['username'].')'; ?>
                 </option>
             <?php