chamilo / chamilo-lms

main/cron/lang/list_undefined_langvars.php

Braces +2 added lines, -1 removed lines

@@ -69,7 +69,8 @@
 echo "</table>\n";
 
 
-function get_all_php_files($base_path) {
+function get_all_php_files($base_path)
+{
     $list = scandir($base_path);
     $files = array();
     foreach ($list as $item) {

Indentation +9 added lines, -9 removed lines

@@ -19,7 +19,7 @@ 
     $file = $path.'/'.$entry;
     if (is_file($file)) {
         $terms = array_merge($terms, SubLanguageManager::get_all_language_variable_in_file($file, true));
-	}
+    }
 }
 // get only the array keys (the language variables defined in language files)
 $defined_terms = array_flip(array_keys($terms));
@@ -32,14 +32,14 @@ 
 foreach ($files as $file) {
     //echo 'Analyzing '.$file."<br />";
     $shortfile = substr($file, $l);
-	$lines = file($file);
+    $lines = file($file);
     foreach ($lines as $line) {
-    	$myterms = array();
+        $myterms = array();
         $res = preg_match_all('/get_lang\(\'(\\w*)\'\)/', $line, $myterms);
         if ($res > 0) {
             foreach ($myterms[1] as $term) {
                 if (!isset($defined_terms[$term]) && !isset($defined_terms['lang'.$term])) {
-                	$undefined_terms[$term] = $shortfile;
+                    $undefined_terms[$term] = $shortfile;
                     //echo "Undefined: $term<br />";
                 }
             }
@@ -49,7 +49,7 @@ 
         if ($res > 0) {
             foreach ($myterms[1] as $term) {
                 if (!isset($defined_terms[$term]) && !isset($defined_terms['lang'.$term])) {
-                	$undefined_terms[$term] = $shortfile;
+                    $undefined_terms[$term] = $shortfile;
                     //echo "Undefined: $term<br />";
                 }
             }
@@ -73,17 +73,17 @@ 
     $list = scandir($base_path);
     $files = array();
     foreach ($list as $item) {
-    	if (substr($item, 0, 1) == '.') {continue; }
+        if (substr($item, 0, 1) == '.') {continue; }
         $special_dirs = array(api_get_path(SYS_TEST_PATH), api_get_path(SYS_COURSE_PATH), api_get_path(SYS_LANG_PATH), api_get_path(SYS_ARCHIVE_PATH));
         if (in_array($base_path.$item.'/', $special_dirs)) {continue; }
         if (is_dir($base_path.$item)) {
-        	$files = array_merge($files, get_all_php_files($base_path.$item.'/'));
+            $files = array_merge($files, get_all_php_files($base_path.$item.'/'));
         } else {
             //only analyse php files
                 $sub = substr($item, -4);
-        	if ($sub == '.php' or $sub == '.tpl') {
+            if ($sub == '.php' or $sub == '.tpl') {
                     $files[] = $base_path.$item;
-        	}
+            }
         }
     }
     $list = null;

main/cron/lang/langstats.class.php

Braces +2 added lines, -1 removed lines

@@ -67,7 +67,8 @@
                     return false; //cannot use if sqlite not installed
                 }
                 $err = Database::query('SELECT * FROM lang_freq');
-                if ($err === false) { //the database probably does not exist, create it
+                if ($err === false) {
+//the database probably does not exist, create it
                     $err = Database::query(
                         'CREATE TABLE lang_freq ('
                         . ' id int PRIMARY KEY AUTO_INCREMENT, '

main/cron/import_csv.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -2021,7 +2021,7 @@ 
                 );
 
                 $this->logger->addInfo(
-                    "User $userId added to course ".$courseInfo['code']." with status '$status' with course category: '$userCourseCategory'"
+                    "user $userId added to course ".$courseInfo['code']." with status '$status' with course category: '$userCourseCategory'"
                 );
             }
         }
@@ -2230,7 +2230,7 @@ 
         );
 
         foreach ($truncateTables as $table) {
-            $sql = "TRUNCATE $table";
+            $sql = "truncate $table";
             Database::query($sql);
             echo $sql.PHP_EOL;
         }

Doc Comments +1 added lines, -1 removed lines

@@ -2240,7 +2240,7 @@
      * @param bool $moveFile
      * @param array $teacherBackup
      * @param array $groupBackup
-     * @return bool
+     * @return false|null
      */
     private function importCareers(
         $file,

Spacing +1 added lines, -1 removed lines

@@ -2267,7 +2267,7 @@
 
             foreach ($data as $row) {
                 foreach ($row as $key => $value) {
-                    $key = (string)trim($key);
+                    $key = (string) trim($key);
                     // Remove utf8 bom
                     $key = preg_replace('/[\x00-\x1F\x80-\xFF]/', '', $key);
                     $row[$key] = $value;

main/upload/form.scorm.php

Braces +2 added lines, -1 removed lines

@@ -11,7 +11,8 @@
 /**
  * Small function to list files in archive/
  */
-function get_zip_files_in_garbage() {
+function get_zip_files_in_garbage()
+{
     $list = array();
     $dh = opendir(api_get_path(SYS_ARCHIVE_PATH));
     if ($dh === false) {

main/cron/lang/list_unused_langvars.php

Braces +2 added lines, -1 removed lines

@@ -69,7 +69,8 @@
 echo "</table>\n";
 
 
-function get_all_php_files($base_path) {
+function get_all_php_files($base_path)
+{
     $list = scandir($base_path);
     $files = array();
     foreach ($list as $item) {

Indentation +10 added lines, -10 removed lines

@@ -68,18 +68,18 @@ 
 // defined terms, and this should prove the concept that there are much
 // more variables than what we really use
 if (count($usedTerms) < 1) {
-  die("No used terms<br />\n");
+    die("No used terms<br />\n");
 } else {
-  echo "The following terms were defined but never used: <br />\n<table>";
+    echo "The following terms were defined but never used: <br />\n<table>";
 }
 $i = 1;
 foreach ($defined_terms as $term => $file) {
-  // remove "lang" prefix just in case
-  if (substr($term, 0, 4) == 'lang') { $term = substr($term, 4); }
-  if (!isset($usedTerms[$term])) {
+    // remove "lang" prefix just in case
+    if (substr($term, 0, 4) == 'lang') { $term = substr($term, 4); }
+    if (!isset($usedTerms[$term])) {
     echo "<tr><td>$i</td><td>$term</td></tr>\n";
     $i++;
-  }
+    }
 }
 echo "</table>\n";
 
@@ -88,17 +88,17 @@ 
     $list = scandir($base_path);
     $files = array();
     foreach ($list as $item) {
-    	if (substr($item, 0, 1) == '.') {continue; }
+        if (substr($item, 0, 1) == '.') {continue; }
         $special_dirs = array(api_get_path(SYS_TEST_PATH), api_get_path(SYS_COURSE_PATH), api_get_path(SYS_LANG_PATH), api_get_path(SYS_ARCHIVE_PATH));
         if (in_array($base_path.$item.'/', $special_dirs)) {continue; }
         if (is_dir($base_path.$item)) {
-        	$files = array_merge($files, get_all_php_files($base_path.$item.'/'));
+            $files = array_merge($files, get_all_php_files($base_path.$item.'/'));
         } else {
             //only analyse php files
                 $sub = substr($item, -4);
-        	if ($sub == '.php' or $sub == '.tpl') {
+            if ($sub == '.php' or $sub == '.tpl') {
                     $files[] = $base_path.$item;
-        	}
+            }
         }
     }
     $list = null;

custompages/language.php

Indentation +3 added lines, -3 removed lines

@@ -24,16 +24,16 @@
 $lang_match = $chamilo_langs[get_preferred_language($available_langs)];
 // recover previous value ...
 if (isset($_SESSION['user_language_choice'])) {
-	$lang_match = $_SESSION['user_language_choice'];
+    $lang_match = $_SESSION['user_language_choice'];
 }
 
 // Chamilo parameter, on logout
 if (isset($_REQUEST['language']) && !empty($_REQUEST['language']) && in_array($_REQUEST['language'], $chamilo_langs)) {
-	$lang_match = $_REQUEST['language'];
+    $lang_match = $_REQUEST['language'];
 }
 // Incoming link parameter
 if (isset($_REQUEST['lang']) && !empty($_REQUEST['lang']) && in_array($_REQUEST['lang'], $available_langs)) {
-	$lang_match = $chamilo_langs[$_REQUEST['lang']];
+    $lang_match = $chamilo_langs[$_REQUEST['lang']];
 }
 
 $detect = api_get_setting('auto_detect_language_custom_pages');

main/gradebook/lib/fe/usertable.class.php

Braces +2 added lines, -1 removed lines

@@ -87,7 +87,8 @@
         // generate the data to display
         $sortable_data = array();
         foreach ($data_array as $data) {
-            if ($data[2] != "") {//filter by course removed
+            if ($data[2] != "") {
+//filter by course removed
                 $row = array();
                 $row[] = $this->build_type_column($data[0]);
                 $row[] = $this->build_name_link($data[0]);

main/exercise/overview.php

Indentation +1 added lines, -1 removed lines

@@ -267,7 +267,7 @@
             break;
         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'));
+                $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'));
             } else {
                 $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'), get_lang('Details'));
             }

main/admin/course_request_rejected.php

Braces +2 added lines, -1 removed lines

@@ -150,7 +150,8 @@
 /**
  * Actions in the list: edit, accept, delete, request additional information.
  */
-function modify_filter($id) {
+function modify_filter($id)
+{
     $code = CourseRequestManager::get_course_request_code($id);
     $result = '<a href="course_request_edit.php?id='.$id.'&caller=2">'.Display::return_icon('edit.png', get_lang('Edit'), array('style' => 'vertical-align: middle;')).'</a>'.
         '&nbsp;<a href="?accept_course_request='.$id.'">'.Display::return_icon('accept.png', get_lang('AcceptThisCourseRequest'), array('style' => 'vertical-align: middle;', 'onclick' => 'javascript: if (!confirm(\''.addslashes(api_htmlentities(sprintf(get_lang('ANewCourseWillBeCreated'), $code), ENT_QUOTES)).'\')) return false;')).'</a>';