chamilo / chamilo-lms

main/document/save_pixlr.php

Braces +3 added lines, -1 removed lines

@@ -124,7 +124,9 @@
 
     if (file_exists($saveDir.'/'.$filename.'.'.$extension)) {
         $i = 1;
-        while (file_exists($saveDir.'/'.$filename.'_'.$i.'.'.$extension)) $i++;
+        while (file_exists($saveDir.'/'.$filename.'_'.$i.'.'.$extension)) {
+            $i++;
+        }
         $paintFileName = $filename.'_'.$i.'.'.$extension;
         $title = $filename.'_'.$i.'.'.$extension;
     }

main/document/webcam_receiver.php

Braces +1 added lines, -2 removed lines

@@ -17,8 +17,7 @@
     $webcamname = $params['webcamname'];
     $webcamdir = $params['webcamdir'];
     $webcamuserid = $params['webcamuserid'];
-}
-else {
+} else {
     api_not_allowed();
     die();
 }

main/work/work.lib.php

Braces +24 added lines, -12 removed lines

@@ -442,7 +442,8 @@ 
     $sessionId,
     $userId = null,
     $onlyUserList = array()
-) {
+)
+{
     $work_table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $user_table = Database::get_main_table(TABLE_MAIN_USER);
 
@@ -1369,7 +1370,8 @@ 
     $direction,
     $where_condition,
     $getCount = false
-) {
+)
+{
     $workTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $workTableAssignment = Database::get_course_table(TABLE_STUDENT_PUBLICATION_ASSIGNMENT);
     $courseInfo = api_get_course_info();
@@ -1528,7 +1530,8 @@ 
     $direction,
     $where_condition,
     $getCount = false
-) {
+)
+{
     $workTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $workTableAssignment = Database::get_course_table(TABLE_STUDENT_PUBLICATION_ASSIGNMENT);
 
@@ -1715,7 +1718,8 @@ 
     $studentId = null,
     $whereCondition = '',
     $getCount = false
-) {
+)
+{
     if ($getCount) {
         $select1 = " SELECT count(u.user_id) as count ";
         $select2 = " SELECT count(u.user_id) as count ";
@@ -1916,7 +1920,8 @@ 
     $getCount = false,
     $courseId = 0,
     $sessionId = 0
-) {
+)
+{
     $work_table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $user_table = Database::get_main_table(TABLE_MAIN_USER);
 
@@ -2870,7 +2875,8 @@ 
     $groupId = null,
     $sessionId = null,
     $getCount = false
-) {
+)
+{
     $usersInWork = null;
     $usersInCourse = null;
 
@@ -3103,7 +3109,8 @@ 
     $parentId,
     $courseInfo = array(),
     $sessionId = 0
-) {
+)
+{
     if (empty($courseInfo)) {
         $courseInfo = api_get_course_info();
     }
@@ -3147,7 +3154,8 @@ 
     $parentId,
     $courseInfo = array(),
     $sessionId = 0
-) {
+)
+{
     if (empty($courseInfo)) {
         $courseInfo = api_get_course_info();
     }
@@ -3196,7 +3204,8 @@ 
     $parentInfo,
     $courseInfo = array(),
     $sessionId = 0
-) {
+)
+{
     if (empty($courseInfo)) {
         $courseInfo = api_get_course_info();
     }
@@ -3813,7 +3822,8 @@ 
     $file = [],
     $checkDuplicated = false,
     $showFlashMessage = true
-) {
+)
+{
     $work_table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
 
     $courseId = $courseInfo['real_id'];
@@ -4921,7 +4931,8 @@ 
     $sidx,
     $sord,
     $getCount = false
-) {
+)
+{
     $my_folder_data = get_work_data_by_id($workId);
     $workParents = array();
     if (empty($my_folder_data)) {
@@ -5202,7 +5213,8 @@ 
     $courseInfo,
     $sessionId,
     $format = 'pdf'
-) {
+)
+{
     if (empty($courseInfo)) {
         return false;
     }

main/inc/lib/link.lib.php

Braces +2 added lines, -1 removed lines

@@ -1449,7 +1449,8 @@
                         $value,
                         $regs
                     )
-                ) { // possibly in <...>
+                ) {
+// possibly in <...>
                     if (($kwlist = trim($regs[1])) != '') {
                         $kw = '<i kw="'.htmlspecialchars($kwlist).'">';
                     } else {

main/exercise/exercise.class.php

Braces +34 added lines, -12 removed lines

@@ -3292,7 +3292,9 @@ 
                 );
         }
 
-        if ($debug) error_log('Start answer loop ');
+        if ($debug) {
+            error_log('Start answer loop ');
+        }
 
         for ($answerId = 1; $answerId <= $nbrAnswers; $answerId++) {
             $answer = $objAnswerTmp->selectAnswer($answerId);
@@ -3406,7 +3408,9 @@ 
                     }
                     $totalScore += $answerWeighting;
 
-                    if ($debug) error_log("studentChoice: $studentChoice");
+                    if ($debug) {
+                        error_log("studentChoice: $studentChoice");
+                    }
                     break;
                 case GLOBAL_MULTIPLE_ANSWER:
                     if ($from_database) {
@@ -3431,7 +3435,9 @@ 
                         $real_answers[$answerId] = (bool) $studentChoice;
                     }
                     $totalScore += $answerWeighting;
-                    if ($debug) error_log("studentChoice: $studentChoice");
+                    if ($debug) {
+                        error_log("studentChoice: $studentChoice");
+                    }
                     break;
                 case MULTIPLE_ANSWER_COMBINATION_TRUE_FALSE:
                     if ($from_database) {
@@ -4245,7 +4251,9 @@ 
             } // end switch Answertype
 
             if ($show_result) {
-                if ($debug) error_log('Showing questions $from '.$from);
+                if ($debug) {
+                    error_log('Showing questions $from '.$from);
+                }
                 if ($from == 'exercise_result') {
                     //display answers (if not matching type, or if the answer is correct)
                     if (
@@ -4529,7 +4537,8 @@ 
                                         $url_hotspot = $destination_items[4];
                                     }
                                 }
-                            } else {	// the first delineation feedback
+                            } else {
+// the first delineation feedback
                                 if ($debug > 0) {
                                     error_log(__LINE__.' first', 0);
                                 }
@@ -4557,7 +4566,9 @@ 
                         }
                     }
                 } else {
-                    if ($debug) error_log('Showing questions $from '.$from);
+                    if ($debug) {
+                        error_log('Showing questions $from '.$from);
+                    }
 
                     switch ($answerType) {
                         case UNIQUE_ANSWER:
@@ -4874,7 +4885,8 @@ 
                                         $url_hotspot = $destination_items[4];
                                     }
                                 }
-                            } else {	// the first delineation feedback
+                            } else {
+// the first delineation feedback
                                 if ($debug > 0) {
                                     error_log(__LINE__.' first', 0);
                                 }
@@ -4919,10 +4931,14 @@ 
                     }
                 }
             }
-            if ($debug) error_log(' ------ ');
+            if ($debug) {
+                error_log(' ------ ');
+            }
         } // end for that loops over all answers of the current question
 
-        if ($debug) error_log('-- end answer loop --');
+        if ($debug) {
+            error_log('-- end answer loop --');
+        }
 
         $final_answer = true;
 
@@ -4994,7 +5010,9 @@ 
             // TODO Change this, because it is wrong to show the user
             //  some results that haven't been stored in the database yet
             if ($answerType == HOT_SPOT || $answerType == HOT_SPOT_ORDER || $answerType == HOT_SPOT_DELINEATION) {
-                if ($debug) error_log('$from AND this is a hotspot kind of question ');
+                if ($debug) {
+                    error_log('$from AND this is a hotspot kind of question ');
+                }
                 $my_exe_id = 0;
                 $from_database = 0;
                 if ($answerType == HOT_SPOT_DELINEATION) {
@@ -5195,8 +5213,12 @@ 
         // stored by exercise_results.php (using the session)
 
         if ($saved_results) {
-            if ($debug) error_log("Save question results $saved_results");
-            if ($debug) error_log(print_r($choice, 1));
+            if ($debug) {
+                error_log("Save question results $saved_results");
+            }
+            if ($debug) {
+                error_log(print_r($choice, 1));
+            }
 
             if (empty($choice)) {
                 $choice = 0;

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

main/webservices/additional_webservices.php

Braces +2 added lines, -1 removed lines

@@ -132,7 +132,8 @@
  */
 function pptConverterGetCommandBaseParams()
 {
-    if (IS_WINDOWS_OS) { // IS_WINDOWS_OS has been defined in main_api.lib.php
+    if (IS_WINDOWS_OS) {
+// IS_WINDOWS_OS has been defined in main_api.lib.php
         $converterPath = str_replace('/', '\\', api_get_path(SYS_PATH).'main/inc/lib/ppt2png');
         $classPath = $converterPath.';'.$converterPath.'/jodconverter-2.2.2.jar;'.$converterPath.'/jodconverter-cli-2.2.2.jar';
         $cmd = 'java -Dfile.encoding=UTF-8 -cp "'.$classPath.'" DokeosConverter';

main/admin/add_users_to_usergroup.php

Braces +8 added lines, -2 removed lines

@@ -341,7 +341,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>';
 
@@ -419,7 +422,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>

plugin/vchamilo/vcron.php

Braces +4 added lines, -2 removed lines

@@ -48,7 +48,8 @@ 
 *
 *
 */
-function fire_vhost_cron($vhost) {
+function fire_vhost_cron($vhost)
+{
     global $VCRON;
 
     if ($VCRON->TRACE_ENABLE) {
@@ -140,7 +141,8 @@ 
 *
 *
 */
-function exec_vhost_cron($vhost) {
+function exec_vhost_cron($vhost)
+{
     global $VCRON, $DB, $_configuration;
 
     if ($VCRON->TRACE_ENABLE) {