chamilo / chamilo-lms

main/course_home/course_home.php

Braces +2 added lines, -1 removed lines

@@ -176,7 +176,8 @@
 $auto_launch = api_get_course_setting('enable_lp_auto_launch');
 $session_id = api_get_session_id();
 if (!empty($auto_launch)) {
-    if ($auto_launch == 2) { //LP list
+    if ($auto_launch == 2) {
+//LP list
         if (api_is_platform_admin() || api_is_allowed_to_edit()) {
             $show_autolaunch_lp_warning = true;
         } else {

main/auth/external_login/updateUser.php

Braces +2 added lines, -1 removed lines

@@ -25,7 +25,8 @@
 $user['country_bu'] = $new_user['country_bu'];
 $user['extra'] = $new_user['extra'];
 
-if ($new_user !== false) { //User can login
+if ($new_user !== false) {
+//User can login
     external_update_user($user);
     $loginFailed = false;
     $_user['user_id'] = $user['user_id'];

main/auth/sso/sso_server_test.php

Braces +6 added lines, -3 removed lines

@@ -84,15 +84,18 @@
     //echo do something to logout
 }
 
-function validate_user($user, $pass) {
+function validate_user($user, $pass)
+{
     return true;
 }
-function filter_xss($val) {
+function filter_xss($val)
+{
     //do some cleaning
     return $val;
 }
 
-function chamilo_sso_protocol() {
+function chamilo_sso_protocol()
+{
     //get the sso_protocol from chamilo using webservices
     return 'http://';
 }

main/lp/storageapi.php

Braces +30 added lines, -20 removed lines

@@ -45,8 +45,9 @@ 
         }
         break;
     case "stackgetall":
-        if (storage_can_set($_REQUEST['svuser']))
-            print storage_stack_getall($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
+        if (storage_can_set($_REQUEST['svuser'])) {
+                    print storage_stack_getall($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey']);
+        }
         break;
     case "getposition":
         print storage_get_position($_REQUEST['svuser'], $_REQUEST['svcourse'], $_REQUEST['svsco'], $_REQUEST['svkey'], $_REQUEST['svasc']);
@@ -63,7 +64,8 @@ 
         // Do nothing
 }
 
-function storage_can_set($sv_user) {
+function storage_can_set($sv_user)
+{
     // platform admin can change any user's stored values, other users can only change their own values
     $allowed = ((api_is_platform_admin()) || ($sv_user == api_get_user_id()));
     if (!$allowed) {
@@ -72,7 +74,8 @@ 
     return $allowed;
 }
 
-function storage_get($sv_user, $sv_course, $sv_sco, $sv_key) {
+function storage_get($sv_user, $sv_course, $sv_sco, $sv_key)
+{
     $sql = "select sv_value
         from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES)."
         where user_id= '$sv_user'
@@ -87,13 +90,13 @@ 
         } else {
             return $row['sv_value'];
         }
-    }
-    else {
+    } else {
         return null;
     }
 }
 
-function storage_get_leaders($sv_user, $sv_course, $sv_sco, $sv_key, $sv_asc, $sv_length) {
+function storage_get_leaders($sv_user, $sv_course, $sv_sco, $sv_key, $sv_asc, $sv_length)
+{
 
     // get leaders
     $sql_leaders = "select u.user_id, firstname, lastname, email, username, sv_value as value
@@ -127,7 +130,8 @@ 
     return json_encode($result);
 }
 
-function storage_get_position($sv_user, $sv_course, $sv_sco, $sv_key, $sv_asc, $sv_length) {
+function storage_get_position($sv_user, $sv_course, $sv_sco, $sv_key, $sv_asc, $sv_length)
+{
     $sql = "select count(list.user_id) as position
         from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES)." search,
             ".Database::get_main_table(TABLE_TRACK_STORED_VALUES)." list
@@ -144,13 +148,13 @@ 
     if (Database::num_rows($res) > 0) {
         $row = Database::fetch_assoc($res);
         return $row['position'];
-    }
-    else {
+    } else {
         return null;
     }
 }
 
-function storage_set($sv_user, $sv_course, $sv_sco, $sv_key, $sv_value) {
+function storage_set($sv_user, $sv_course, $sv_sco, $sv_key, $sv_value)
+{
     $sv_value = Database::escape_string($sv_value);
     $sql = "replace into ".Database::get_main_table(TABLE_TRACK_STORED_VALUES)."
         (user_id, sco_id, course_id, sv_key, sv_value)
@@ -160,7 +164,8 @@ 
     return Database::affected_rows($res);
 }
 
-function storage_getall($sv_user, $sv_course, $sv_sco) {
+function storage_getall($sv_user, $sv_course, $sv_sco)
+{
     $sql = "select sv_key, sv_value
         from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES)."
         where user_id= '$sv_user'
@@ -177,7 +182,8 @@ 
     return json_encode($data);
 }
 
-function storage_stack_push($sv_user, $sv_course, $sv_sco, $sv_key, $sv_value) {
+function storage_stack_push($sv_user, $sv_course, $sv_sco, $sv_key, $sv_value)
+{
     $sv_value = Database::escape_string($sv_value);
     Database::query("start transaction");
     $sqlorder = "select ifnull((select max(stack_order)
@@ -198,14 +204,14 @@ 
     if ($resorder && $resinsert) {
         Database::query("commit");
         return 1;
-    }
-    else {
+    } else {
         Database::query("rollback");
         return 0;
     }
 }
 
-function storage_stack_pop($sv_user, $sv_course, $sv_sco, $sv_key) {
+function storage_stack_pop($sv_user, $sv_course, $sv_sco, $sv_key)
+{
     Database::query("start transaction");
     $sqlselect = "select sv_value, stack_order
         from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES_STACK)."
@@ -239,7 +245,8 @@ 
     }
 }
 
-function storage_stack_length($sv_user, $sv_course, $sv_sco, $sv_key) {
+function storage_stack_length($sv_user, $sv_course, $sv_sco, $sv_key)
+{
     $sql = "select count(*) as length
         from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES_STACK)."
         where user_id= '$sv_user'
@@ -251,7 +258,8 @@ 
     return $row['length'];
 }
 
-function storage_stack_clear($sv_user, $sv_course, $sv_sco, $sv_key) {
+function storage_stack_clear($sv_user, $sv_course, $sv_sco, $sv_key)
+{
     $sql = "delete
         from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES_STACK)."
         where user_id= '$sv_user'
@@ -262,7 +270,8 @@ 
     return Database::num_rows($res);
 }
 
-function storage_stack_getall($sv_user, $sv_course, $sv_sco, $sv_key) {
+function storage_stack_getall($sv_user, $sv_course, $sv_sco, $sv_key)
+{
     $sql = "select stack_order as stack_order, sv_value as value
         from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES_STACK)."
         where user_id= '$sv_user'
@@ -280,7 +289,8 @@ 
     return json_encode($results);
 }
 
-function storage_get_all_users() {
+function storage_get_all_users()
+{
     $sql = "select user_id, username, firstname, lastname
         from ".Database::get_main_table(TABLE_MAIN_USER)."
         order by user_id asc";

main/lp/learnpathItem.class.php

Braces +9 added lines, -6 removed lines

@@ -752,7 +752,8 @@ 
     public function get_lesson_mode()
     {
         $mode = 'normal';
-        if ($this->get_prevent_reinit() != 0) { // If prevent_reinit == 0
+        if ($this->get_prevent_reinit() != 0) {
+// If prevent_reinit == 0
             $my_status = $this->get_status();
             if ($my_status != $this->possible_status[0]
                     && $my_status != $this->possible_status[1]) {
@@ -2044,11 +2045,13 @@ 
         $restart = 1;
         $mystatus = $this->get_status(true);
         if ($this->get_prevent_reinit() > 0
-        ) { // If prevent_reinit == 1 (or more)
+        ) {
+// If prevent_reinit == 1 (or more)
             // If status is not attempted or incomplete, authorize retaking (of the same) anyway. Otherwise:
             if ($mystatus != $this->possible_status[0] && $mystatus != $this->possible_status[1]) {
                 $restart = -1;
-            } else { //status incompleted or not attempted
+            } else {
+//status incompleted or not attempted
                 $restart = 0;
             }
         } else {
@@ -2989,7 +2992,8 @@ 
                     // Do nothing, just let the local attributes be used.
                 }
             }
-        } else { // If not SCO, such messages should not be expected.
+        } else {
+// If not SCO, such messages should not be expected.
             $type = strtolower($this->type);
             switch ($type) {
                 case 'asset':
@@ -3429,8 +3433,7 @@ 
         }
         if ($scorm_time == '0'
                 and ($this->type != 'sco')
-                and $this->current_start_time != 0)
-        {
+                and $this->current_start_time != 0) {
             $my_time = time() - $this->current_start_time;
             if ($my_time > 0) {
                 $this->update_time($my_time);

main/lp/lp_ajax_save_item.php

Braces +4 added lines, -2 removed lines

@@ -55,7 +55,8 @@ 
     $lmsFinish = 0,
     $userNavigatesAway = 0,
     $statusSignalReceived = 0
-) {
+)
+{
     //global $debug;
     $debug = 0;
     $return = null;
@@ -477,7 +478,8 @@ 
     if (is_array($_REQUEST['interact'])) {
         foreach ($_REQUEST['interact'] as $idx => $interac) {
             $interactions[$idx] = preg_split('/,/', substr($interac, 1, -1));
-            if (!isset($interactions[$idx][7])) { // Make sure there are 7 elements.
+            if (!isset($interactions[$idx][7])) {
+// Make sure there are 7 elements.
                 $interactions[$idx][7] = '';
             }
         }

main/tracking/exams.php

Braces +2 added lines, -1 removed lines

@@ -380,7 +380,8 @@
  * @param $b
  * @return int
  */
-function sort_user($a, $b) {
+function sort_user($a, $b)
+{
     if (is_numeric($a['score']) && is_numeric($b['score'])) {
         if ($a['score'] < $b['score']) {
             return 1;

main/tracking/userlogCSV.php

Braces +4 added lines, -2 removed lines

@@ -163,7 +163,8 @@ 
             // just sum $i up
         }
 
-    } else { // if uInfo is set
+    } else {
+// if uInfo is set
         /*
          * Informations about student uInfo
          */
@@ -333,6 +334,7 @@ 
     echo api_html_entity_decode($title_line, ENT_QUOTES, $charset);
     echo api_html_entity_decode($line, ENT_QUOTES, $charset);
     exit;
-} else { // not allowed
+} else {
+// not allowed
     api_not_allowed();
 }

main/gradebook/lib/be/exerciselink.class.php

Braces +2 added lines, -1 removed lines

@@ -419,7 +419,8 @@
         return false;
     }
 
-    public function is_allowed_to_change_name() {
+    public function is_allowed_to_change_name()
+    {
         return false;
     }