chamilo / chamilo-lms

main/course_home/vertical_activity.php

Braces +2 added lines, -1 removed lines

@@ -19,7 +19,8 @@
 
 if (api_is_allowed_to_edit(null, true)) {
     // HIDE
-    if (!empty($_GET['hide'])) { // visibility 1 -> 0
+    if (!empty($_GET['hide'])) {
+// visibility 1 -> 0
         $sql = "UPDATE $tool_table SET visibility=0 WHERE c_id = $course_id AND id='".$id."'";
         Database::query($sql);
         $show_message = Display::return_message(get_lang('ToolIsNowHidden'), 'confirmation');

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/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/wiki/diff.inc.php

Braces +2 added lines, -1 removed lines

@@ -38,7 +38,8 @@
     $new,
     $show_equals = false,
     $format_line_function = 'format_line'
-) {
+)
+{
     $oldArr = str_split_on_new_line($old);
     $newArr = str_split_on_new_line($new);
 

main/exercise/exercise_submit_modal.php

Braces +4 added lines, -2 removed lines

@@ -244,7 +244,8 @@ 
                 $poly_answer = convert_coordinates($answer_question, '|');
                 $max_coord = poly_get_max($poly_user, $poly_answer);
 
-                if (empty($_GET['hotspot'])) { //no user response
+                if (empty($_GET['hotspot'])) {
+//no user response
                     $overlap = -2;
                 } else {
                     $poly_user_compiled = poly_compile($poly_user, $max_coord);
@@ -346,7 +347,8 @@ 
                     $test = false;
                     // if ($answerId == 2 ){$test = true;} for test oars
 
-                    if (empty($_GET['hotspot'])) { //no user response
+                    if (empty($_GET['hotspot'])) {
+//no user response
                         $overlap = false;
                     } else {
                         //	poly_compile really works tested with gnuplot