chamilo / chamilo-lms

main/cron/lang/check_parse_lang.php

Spacing +4 added lines, -4 removed lines

@@ -9,7 +9,7 @@ 
 //die();
 require_once '../../inc/global.inc.php';
 $path = api_get_path(SYS_LANG_PATH).'english';
-ini_set('memory_limit','128M');
+ini_set('memory_limit', '128M');
 /**
  * Main code
  */
@@ -18,7 +18,7 @@ 
 $langs = scandir(api_get_path(SYS_LANG_PATH));
 foreach ($langs as $lang) {
   $dir = api_get_path(SYS_LANG_PATH).$lang;
-  if (is_dir($dir) && substr($lang,0,1) != '.' && !empty($lang)) {
+  if (is_dir($dir) && substr($lang, 0, 1) != '.' && !empty($lang)) {
     echo "$lang...";
     $ok = true;
     foreach ($list as $entry) {
@@ -26,8 +26,8 @@ 
       $out = array();
       if (is_file($file)) {
         //$terms = array_merge($terms,SubLanguageManager::get_all_language_variable_in_file($file,true));
-        @exec('php5 -l '.$file,$out);
-        if (substr($out[0],0,2)!='No') {
+        @exec('php5 -l '.$file, $out);
+        if (substr($out[0], 0, 2) != 'No') {
           echo $out[0]."\n";
           $ok = false;
         }

main/cron/lang/langstats_file_builder.php

Spacing +13 added lines, -13 removed lines

@@ -34,7 +34,7 @@ 
  */
 foreach ($terms as $row) {
   if ($words_counter > 10000) { break; }
-  $words = str_word_count(get_lang($row['term_name'],null,$orig_lang));
+  $words = str_word_count(get_lang($row['term_name'], null, $orig_lang));
   $words_counter += $words;
   $terms_in_limit[$row['term_name']] = $i;
   //echo "Term <b>".$row['term_name']."</b> is <b>'".get_lang($row['term_name'],null,$orig_lang)."'</b> which means $words words<br /><br />\n";
@@ -59,37 +59,37 @@ 
 $global_var = array(); //keep the combination of all vars
 $terms_in_limit = array_flip($terms_in_limit);
 foreach ($list_files as $file) {
-  if (substr($file,0,1) == '.') {continue;}
+  if (substr($file, 0, 1) == '.') {continue; }
   //echo "'".substr($file,0,-8)."',<br />"; //print in a PHP array format
   $vars = file($lang_dir.'/'.$orig_lang.'/'.$file);
   $local_var = array();
   $file_string = '<?php'."\n";
   foreach ($vars as $line) {
     $var = array();
-    $res = preg_match('/^(\$\w*)/',$line,$var);
-    if ($res>0) {
+    $res = preg_match('/^(\$\w*)/', $line, $var);
+    if ($res > 0) {
       //echo $var[1]."<br />";
       
-      if (in_array(substr($var[1],1),$terms_in_limit)) {
+      if (in_array(substr($var[1], 1), $terms_in_limit)) {
         //echo "Var ".$var[1]." was in the limit<br />";
         $local_var[$var[1]] = $line;
         $file_string .= $line;
-        $terms_found[] = substr($var[1],1); //e.g. store Tools
-        $words_found += str_word_count(get_lang($var[1],null,$orig_lang));
-      } elseif (in_array(substr($var[1],5),$terms_in_limit)) {
+        $terms_found[] = substr($var[1], 1); //e.g. store Tools
+        $words_found += str_word_count(get_lang($var[1], null, $orig_lang));
+      } elseif (in_array(substr($var[1], 5), $terms_in_limit)) {
         //echo "Var ".$var[1]." was in the limit<br />";
         $local_var[$var[1]] = $line;
         $file_string .= $line;
-        $terms_found[] = substr($var[1],5); //e.g. store langTools
-        $words_found += str_word_count(get_lang(substr($var[1],5),null,$orig_lang));
+        $terms_found[] = substr($var[1], 5); //e.g. store langTools
+        $words_found += str_word_count(get_lang(substr($var[1], 5), null, $orig_lang));
       } //else do not care
     }
   }
   echo "Writing ".$arch_dir.'/langstats/'.$orig_lang.'/'.$file."<br />\n";
-  file_put_contents($arch_dir.'/langstats/'.$orig_lang.'/'.$file,$file_string); 
+  file_put_contents($arch_dir.'/langstats/'.$orig_lang.'/'.$file, $file_string); 
   $global_var += $local_var;
 };
-$terms_diff = count($global_var)-count($terms_in_limit);
+$terms_diff = count($global_var) - count($terms_in_limit);
 echo count($global_var)." terms found in English files (summing up to $words_found words). Some terms ($terms_diff in this case) might have appeared in two different files<br />";
 /**
  * Display results
@@ -97,5 +97,5 @@ 
 
 echo "Difference between filtered and found in English:<br />";
 //print_r($terms_found);
-echo "<pre>".print_r(array_diff($terms_in_limit,$terms_found),1)."</pre>";
+echo "<pre>".print_r(array_diff($terms_in_limit, $terms_found), 1)."</pre>";
 echo "#";

main/cron/lang/langstats.class.php

Spacing +8 added lines, -8 removed lines

@@ -30,7 +30,7 @@ 
                     return false; //cannot use if sqlite not installed
                 }
                 if (empty($file)) {
-                    $file = api_get_path(SYS_ARCHIVE_PATH) . '/langstasdb';
+                    $file = api_get_path(SYS_ARCHIVE_PATH).'/langstasdb';
                 }
                 if (is_file($file) && is_writeable($file)) {
                     $this->db = new SQLite3($file, SQLITE3_OPEN_READWRITE);
@@ -39,7 +39,7 @@ 
                         $this->db = new SQLite3($file);
                     } catch (Exception $e) {
                         $this->error = 'DatabaseCreateError';
-                        error_log('Exception: ' . $e->getMessage());
+                        error_log('Exception: '.$e->getMessage());
                         return false;
                     }
                     $err = $this->db->exec(
@@ -104,7 +104,7 @@ 
             $num = $row[3];
             $num++;
             $res = $this->db->query(
-                'UPDATE lang_freq SET term_count = ' . $num . ' WHERE id = ' . $row[0]
+                'UPDATE lang_freq SET term_count = '.$num.' WHERE id = '.$row[0]
             );
             if ($res === false) {
                 $this->error = 'CouldNotUpdateTerm';
@@ -136,7 +136,7 @@ 
     public function get_popular_terms($num = 1000)
     {
         $res = $this->db->query(
-            'SELECT * FROM lang_freq ORDER BY term_count DESC LIMIT ' . $num
+            'SELECT * FROM lang_freq ORDER BY term_count DESC LIMIT '.$num
         );
         $list = array();
         while ($row = $res->fetchArray()) {
@@ -163,16 +163,16 @@ 
      */
     public function get_variables_origin()
     {
-        $path = api_get_path(SYS_LANG_PATH) . 'english/';
+        $path = api_get_path(SYS_LANG_PATH).'english/';
         $vars = array();
         $priority = array('trad4all');
         foreach ($priority as $file) {
             $list = SubLanguageManager::get_all_language_variable_in_file(
-                $path . $file . '.inc.php',
+                $path.$file.'.inc.php',
                 true
             );
             foreach ($list as $var => $trad) {
-                $vars[$var] = $file . '.inc.php';
+                $vars[$var] = $file.'.inc.php';
             }
         }
         $files = scandir($path);
@@ -181,7 +181,7 @@ 
                 continue;
             }
             $list = SubLanguageManager::get_all_language_variable_in_file(
-                $path . $file,
+                $path.$file,
                 true
             );
             foreach ($list as $var => $trad) {

main/cron/lang/list_unused_langvars.php

Spacing +9 added lines, -9 removed lines

@@ -43,7 +43,7 @@ 
         $res = preg_match_all('/get_lang\(\'(\\w*)\'\)/', $line, $myTerms);
         if ($res > 0) {
             foreach ($myTerms[1] as $term) {
-                if (substr($term, 0, 4)=='lang') {
+                if (substr($term, 0, 4) == 'lang') {
                     $term = substr($term, 4);
                 }
                 $usedTerms[$term] = $shortFile;
@@ -53,7 +53,7 @@ 
             $res = preg_match_all('/\{[\'"](\\w*)[\'"]\|get_lang\}/', $line, $myTerms);
             if ($res > 0) {
                 foreach ($myTerms[1] as $term) {
-                    if (substr($term, 0, 4)=='lang') {
+                    if (substr($term, 0, 4) == 'lang') {
                         $term = substr($term, 4);
                     }
                     $usedTerms[$term] = $shortFile;
@@ -67,7 +67,7 @@ 
 // Compare defined terms VS used terms. Used terms should be smaller than
 // defined terms, and this should prove the concept that there are much
 // more variables than what we really use
-if (count($usedTerms)<1) {
+if (count($usedTerms) < 1) {
   die("No used terms<br />\n");
 } else {
   echo "The following terms were defined but never used: <br />\n<table>";
@@ -75,7 +75,7 @@ 
 $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 (substr($term, 0, 4) == 'lang') { $term = substr($term, 4); }
   if (!isset($usedTerms[$term])) {
     echo "<tr><td>$i</td><td>$term</td></tr>\n";
     $i++;
@@ -88,14 +88,14 @@ 
     $list = scandir($base_path);
     $files = array();
     foreach ($list as $item) {
-    	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 (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);
+                $sub = substr($item, -4);
         	if ($sub == '.php' or $sub == '.tpl') {
                     $files[] = $base_path.$item;
         	}

main/cron/fix_online_time.php

Spacing +5 added lines, -5 removed lines

@@ -6,7 +6,7 @@ 
  * @package chamilo.cron
  * @author Imanol Losada <imanol.losada@beeznest.com>
  */
-require_once __DIR__ . '/../inc/global.inc.php';
+require_once __DIR__.'/../inc/global.inc.php';
 
 /**
  * Get ids of COURSEMANAGERs that are inside a course right now
@@ -15,9 +15,9 @@ 
 function getTeachersInCourseIds()
 {
     $table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
-    $joinStatement = ' JOIN ' . Database::get_main_table(TABLE_MAIN_USER) . ' ON login_user_id = user_id';
+    $joinStatement = ' JOIN '.Database::get_main_table(TABLE_MAIN_USER).' ON login_user_id = user_id';
     return Database::select(
-        'login_user_id', $table . $joinStatement,
+        'login_user_id', $table.$joinStatement,
         array(
             'where' => array(
                 'c_id IS NOT NULL AND status = ?' => array(
@@ -44,7 +44,7 @@ 
     $dataBaseCurrentHour = array_shift($utcResult);
     $maximumIdleTimeInCourse = date(
         'Y-m-d H:i:s',
-        strtotime($dataBaseCurrentHour . ' ' . $timeLimit)
+        strtotime($dataBaseCurrentHour.' '.$timeLimit)
     );
     $table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_COURSE_ACCESS);
     $onLineTrackTable = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
@@ -69,7 +69,7 @@ 
             array(
                 'logout_course_date' => date(
                     'Y-m-d H:i:s',
-                    strtotime($currentTeacherData['logout_course_date'] . ' ' . $extraTime)
+                    strtotime($currentTeacherData['logout_course_date'].' '.$extraTime)
                 )
             ),
             array(

main/cron/fix_fill_blank.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
 $execute = isset($_GET['execute']) ? $_GET['execute'] : null;
 $courseId = isset($_GET['c_id']) ? $_GET['c_id'] : null;
-$questionId  = isset($_GET['question_id']) ? $_GET['question_id'] : null;
+$questionId = isset($_GET['question_id']) ? $_GET['question_id'] : null;
 
 $sql2 = "SELECT * FROM $table WHERE answer LIKE '%[%]%'";
 

main/dashboard/index.php

Spacing +2 added lines, -2 removed lines

@@ -20,12 +20,12 @@
 
 // current section
 $this_section = SECTION_DASHBOARD;
-unset($_SESSION['this_section']);//for hmtl editor repository
+unset($_SESSION['this_section']); //for hmtl editor repository
 
 // get actions
 $actions = array('listing', 'store_user_block', 'disable_block');
 $action = 'listing';
-if (isset($_GET['action']) && in_array($_GET['action'],$actions)) {
+if (isset($_GET['action']) && in_array($_GET['action'], $actions)) {
 	$action = $_GET['action'];
 }
 

main/timeline/index.php

Spacing +15 added lines, -15 removed lines

@@ -23,19 +23,19 @@ 
 
 switch ($action) {
     case 'add':
-        $interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Timeline'));
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('Add'));
+        $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Add'));
         break;
     case 'edit':
-        $interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Timeline'));
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('Edit'));
+        $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Edit'));
         break;
     case 'add_item':
-        $interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Timeline'));
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('TimelineItem'));
+        $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('TimelineItem'));
         break;
     default:
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Timeline'));
 }
 
 //jqgrid will use this URL to do the selects
@@ -46,8 +46,8 @@ 
 
 //Column config
 $column_model   = array(
-                        array('name'=>'name',           'index'=>'name',        'width'=>'120',   'align'=>'left'),
-                        array('name'=>'actions',        'index'=>'actions',     'width'=>'100',  'align'=>'left', 'sortable'=>'false')
+                        array('name'=>'name', 'index'=>'name', 'width'=>'120', 'align'=>'left'),
+                        array('name'=>'actions', 'index'=>'actions', 'width'=>'100', 'align'=>'left', 'sortable'=>'false')
                        );
 //Autowidth
 $extra_params['autowidth'] = 'true';
@@ -55,11 +55,11 @@ 
 $extra_params['height'] = 'auto';
 
 //With this function we can add actions to the jgrid (edit, delete, etc)
-$htmlHeadXtra[] ='
+$htmlHeadXtra[] = '
 <script>
 $(function() {
     // grid definition see the $timeline->display() function
-    '.Display::grid_js('timelines',  $url,$columns,$column_model,$extra_params, array(), null,true).'
+    '.Display::grid_js('timelines', $url, $columns, $column_model, $extra_params, array(), null, true).'
 });
 </script>';
 
@@ -80,12 +80,12 @@ 
                 $values = $form->exportValues();
                 $res    = $timeline->save($values);
                 if ($res) {
-                    $message = Display::return_message(get_lang('ItemAdded'),'success');
+                    $message = Display::return_message(get_lang('ItemAdded'), 'success');
                 }
             }
             $content = $timeline->listing();
         } else {
-            $actions .= '<a href="'.api_get_self().'">'.Display::return_icon('back.png',get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+            $actions .= '<a href="'.api_get_self().'">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
             $form->addElement('hidden', 'sec_token');
             $form->setConstants(array('sec_token' => $token));
             $content = $form->return_form();
@@ -106,7 +106,7 @@ 
             }
             $timeline->display();
         } else {
-            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png',get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
             $form->addElement('hidden', 'sec_token');
             $form->setConstants(array('sec_token' => $token));
             $content = $form->return_form();
@@ -128,7 +128,7 @@ 
             }
             $timeline->display();
         } else {
-            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png',get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
             $form->addElement('hidden', 'sec_token');
             $form->setConstants(array('sec_token' => $token));
             $content = $form->return_form();

main/gradebook/gradebook_scoring_system.php

Spacing +13 added lines, -13 removed lines

@@ -13,7 +13,7 @@ 
     api_not_allowed();
 }
 
-$htmlHeadXtra[]= '<script>
+$htmlHeadXtra[] = '<script>
 function plusItem(item) {
     document.getElementById(item).style.display = "inline";
     document.getElementById("plus-"+item).style.display = "none";
@@ -45,10 +45,10 @@ 
 );
 
 $select_cat = intval($_GET['selectcat']);
-$displayscore= ScoreDisplay :: instance();
+$displayscore = ScoreDisplay :: instance();
 $customdisplays = $displayscore->get_custom_score_display_settings();
 
-$nr_items = (count($customdisplays) != '0' )? count($customdisplays) : '1';
+$nr_items = (count($customdisplays) != '0') ? count($customdisplays) : '1';
 $scoreform = new ScoreDisplayForm(
     'scoring_system_form',
     api_get_self().'?selectcat='.$select_cat.'&'.api_get_cidreq()
@@ -57,18 +57,18 @@ 
 if ($scoreform->validate()) {
     $value_export = '';
     $value_export = $scoreform->exportValues();
-    $value_export = isset($value_export) ? $scoreform->exportValues(): '';
+    $value_export = isset($value_export) ? $scoreform->exportValues() : '';
     $values = $value_export;
 
     // create new array of custom display settings
     // this loop also checks if all score ranges are unique
 
-    $scoringdisplay= array ();
+    $scoringdisplay = array();
     $ranges_ok = true;
-    $endscore= isset($values['endscore']) ? $values['endscore'] : null;
-    $displaytext=isset($values['displaytext']) ? $values['displaytext'] : null;
-    for ($counter= 1; $ranges_ok && $counter <= 20; $counter++) {
-        $setting= array ();
+    $endscore = isset($values['endscore']) ? $values['endscore'] : null;
+    $displaytext = isset($values['displaytext']) ? $values['displaytext'] : null;
+    for ($counter = 1; $ranges_ok && $counter <= 20; $counter++) {
+        $setting = array();
         $setting['score'] = $endscore[$counter];
         $setting['display'] = $displaytext[$counter];
         if (!empty($setting['score'])) {
@@ -77,12 +77,12 @@ 
                     $ranges_ok = false;
                 }
             }
-            $scoringdisplay[]= $setting;
+            $scoringdisplay[] = $setting;
         }
     }
 
     if (!$ranges_ok) {
-        header('Location: ' . api_get_self() . '?nouniqueranges=&selectcat=' . $select_cat.'&'.api_get_cidreq());
+        header('Location: '.api_get_self().'?nouniqueranges=&selectcat='.$select_cat.'&'.api_get_cidreq());
         exit;
     }
 
@@ -102,8 +102,8 @@ 
 $this_section = SECTION_COURSES;
 Display :: display_header(get_lang('ScoreEdit'));
 
-if (((isset($_GET['isStudentView']) && $_GET['isStudentView']=='false') ||
-    (isset($_GET['selectcat']) && ($_SESSION['studentview']=='teacherview')))
+if (((isset($_GET['isStudentView']) && $_GET['isStudentView'] == 'false') ||
+    (isset($_GET['selectcat']) && ($_SESSION['studentview'] == 'teacherview')))
 ) {
     if (isset ($_GET['scoringupdated'])) {
         Display::display_confirmation_message(get_lang('ScoringUpdated'), false);