chamilo / chamilo-lms

main/blog/download.php

Braces +3 added lines, -1 removed lines

@@ -40,7 +40,9 @@
 //if the rewrite rule asks for a directory, we redirect to the course view
 if (is_dir($full_file_name)) {
 	//remove last slash if present
-	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/blog/blog.php

Braces +11 added lines, -9 removed lines

@@ -379,9 +379,10 @@ 
 	$result = Database::query($sql);
 	$row = Database::fetch_array($result);
 
-	if ($row['number'] == 1)
-		$user_task = true;
-}
+	if ($row['number'] == 1) {
+			$user_task = true;
+	}
+	}
 
 switch ($action) {
 	case 'new_post':
@@ -468,16 +469,17 @@ 
 			echo '<br /><br />';
 			Blog :: display_assigned_task_list($blog_id);
 			echo '<br /><br />';
+		} else {
+					api_not_allowed();
 		}
-		else
-			api_not_allowed();
 
 		break;
 	case 'execute_task' :
-		if (isset ($_GET['post_id']))
-			Blog :: display_post($blog_id, intval($_GET['post_id']));
-		else
-			Blog :: display_select_task_post($blog_id, intval($_GET['task_id']));
+		if (isset ($_GET['post_id'])) {
+					Blog :: display_post($blog_id, intval($_GET['post_id']));
+		} else {
+					Blog :: display_select_task_post($blog_id, intval($_GET['task_id']));
+		}
 
 		break;
 	case 'view_search_result' :

main/exercice/exercise_report.php

Braces +2 added lines, -1 removed lines

@@ -400,8 +400,9 @@
 $extra .= $form->return_form();
 $extra .= '</div>';
 
-if ($is_allowedToEdit)
+if ($is_allowedToEdit) {
     echo $extra;
+}
 
 echo $actions;
 

main/exercice/exercise.class.php

Braces +51 added lines, -24 removed lines

@@ -538,10 +538,11 @@ 
      */
     public function isInList($questionId)
     {
-        if (is_array($this->questionList))
-            return in_array($questionId,$this->questionList);
-        else
-            return false;
+        if (is_array($this->questionList)) {
+                    return in_array($questionId,$this->questionList);
+        } else {
+                    return false;
+        }
     }
 
     /**
@@ -1227,10 +1228,11 @@ 
 
             $var = Exercise::selectTimeLimit();
 
-            if (($this->start_time != '0000-00-00 00:00:00'))
-                $form->addElement('html','<div id="start_date_div" style="display:block;">');
-            else
-                $form->addElement('html','<div id="start_date_div" style="display:none;">');
+            if (($this->start_time != '0000-00-00 00:00:00')) {
+                            $form->addElement('html','<div id="start_date_div" style="display:block;">');
+            } else {
+                            $form->addElement('html','<div id="start_date_div" style="display:none;">');
+            }
 
             $form->addElement('date_time_picker', 'start_time');
 
@@ -1238,10 +1240,11 @@ 
 
             $form->addElement('checkbox', 'activate_end_date_check', null , get_lang('EnableEndTime'), array('onclick' => 'activate_end_date()'));
 
-            if (($this->end_time != '0000-00-00 00:00:00'))
-                $form->addElement('html','<div id="end_date_div" style="display:block;">');
-            else
-                $form->addElement('html','<div id="end_date_div" style="display:none;">');
+            if (($this->end_time != '0000-00-00 00:00:00')) {
+                            $form->addElement('html','<div id="end_date_div" style="display:block;">');
+            } else {
+                            $form->addElement('html','<div id="end_date_div" style="display:none;">');
+            }
 
             $form->addElement('date_time_picker', 'end_time');
             $form->addElement('html','</div>');
@@ -2268,7 +2271,9 @@ 
         $organs_at_risk_hit = 0;
         $questionScore = 0;
 
-        if ($debug) error_log('Start answer loop ');
+        if ($debug) {
+            error_log('Start answer loop ');
+        }
 
         $answer_correct_array = array();
 
@@ -2383,7 +2388,9 @@ 
                     }
                     $totalScore += $answerWeighting;
 
-                    if ($debug) error_log("studentChoice: $studentChoice");
+                    if ($debug) {
+                        error_log("studentChoice: $studentChoice");
+                    }
                     break;
                 case GLOBAL_MULTIPLE_ANSWER:
                     if ($from_database) {
@@ -2408,7 +2415,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) {
@@ -3125,9 +3134,13 @@ 
             } // end switch Answertype
 
             if ($show_result) {
-                if ($debug) error_log('show result '.$show_result);
+                if ($debug) {
+                    error_log('show result '.$show_result);
+                }
                 if ($from == 'exercise_result') {
-                    if ($debug) error_log('Showing questions $from '.$from);
+                    if ($debug) {
+                        error_log('Showing questions $from '.$from);
+                    }
                     //display answers (if not matching type, or if the answer is correct)
                     if (
                         !in_array(
@@ -3415,7 +3428,9 @@ 
                         }
                     }
                 } else {
-                    if ($debug) error_log('Showing questions $from '.$from);
+                    if ($debug) {
+                        error_log('Showing questions $from '.$from);
+                    }
 
                     switch ($answerType) {
                         case UNIQUE_ANSWER:
@@ -3747,10 +3762,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;
 
@@ -3823,7 +3842,9 @@ 
             //  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;
@@ -3999,8 +4020,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;
@@ -4102,7 +4127,9 @@ 
             $sql = 'UPDATE ' . $stat_table . ' SET
                         exe_result = exe_result + ' . floatval($questionScore) . '
                     WHERE exe_id = ' . $exeId;
-            if ($debug) error_log($sql);
+            if ($debug) {
+                error_log($sql);
+            }
             Database::query($sql);
         }
 

main/exercice/overview.php

Braces +1 added lines, -2 removed lines

@@ -250,8 +250,7 @@
         case RESULT_DISABLE_SHOW_SCORE_ONLY:
             if ($objExercise->feedback_type != EXERCISE_FEEDBACK_TYPE_END) {
                $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'));
-            }
-            else {
+            } else {
                 $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'), get_lang('Details'));
             }
             break;

main/exercice/TestCategory.php

Braces +1 added lines, -2 removed lines

@@ -401,8 +401,7 @@
 				$nbQuestionInThisCat = TestCategory::getNumberOfQuestionsInCategoryForTest($exerciseId, $tabcatid[$i]);
 				if ($nbQuestionInThisCat > $in_nbrandom) {
 					$nbquestionresult += $in_nbrandom;
-				}
-				else {
+				} else {
 					$nbquestionresult += $nbQuestionInThisCat;
 				}
 			}

main/exercice/hotspot_lang_conversion.php

Braces +5 added lines, -4 removed lines

@@ -18,10 +18,11 @@
 	$lang = urldecode($_GET['lang']);
 	if (preg_match('/^[a-zA-Z0-9\._-]+$/', $lang)) {
 		//$lang = str_replace($search,$replace,urldecode($_GET['lang']));
-		if(file_exists($hotspot_lang_file . $lang . '/hotspot.inc.php'))
-			$hotspot_lang_file .= $lang . '/hotspot.inc.php';
-		else
-			$hotspot_lang_file .= 'english/hotspot.inc.php';
+		if(file_exists($hotspot_lang_file . $lang . '/hotspot.inc.php')) {
+					$hotspot_lang_file .= $lang . '/hotspot.inc.php';
+		} else {
+					$hotspot_lang_file .= 'english/hotspot.inc.php';
+		}
 	} else {
 		$hotspot_lang_file .= 'english/hotspot.inc.php';
 	}

main/exercice/hotspot_savescore.inc.php

Braces +4 added lines, -3 removed lines

@@ -15,12 +15,13 @@
 $exerciseId   = $objExcercise->selectId();
 // Save clicking order
 $answerOrderId = count($_SESSION['exerciseResult'][$questionId]['ids'])+1;
-if ($_GET['answerId'] == "0") // click is NOT on a hotspot
+if ($_GET['answerId'] == "0") {
+    // click is NOT on a hotspot
 {
 	$hit = 0;
-	$answerId = NULL;
 }
-else // user clicked ON a hotspot
+	$answerId = NULL;
+} else // user clicked ON a hotspot
 {
 	$hit = 1;
 	$answerId = api_substr($_GET['answerId'],22,2);

main/exercice/exercise_admin.php

Braces +3 added lines, -2 removed lines

@@ -171,10 +171,11 @@
     }
     echo '</div>';
 
-    if ($objExercise->feedback_type == 1)
-        Display::display_normal_message(
+    if ($objExercise->feedback_type == 1) {
+            Display::display_normal_message(
             get_lang('DirectFeedbackCantModifyTypeQuestion')
         );
+    }
 
     if (api_get_setting('search_enabled')=='true' && !extension_loaded('xapian')) {
         Display::display_error_message(get_lang('SearchXapianModuleNotInstalled'));