chamilo / chamilo-lms

main/dropbox/dropbox_functions.inc.php

Braces +17 added lines, -8 removed lines

@@ -192,7 +192,8 @@ 
     $viewReceivedCategory,
     $viewSentCategory,
     $view
-) {
+)
+{
     $form = new FormValidator(
         'form1',
         'post',
@@ -621,7 +622,8 @@ 
                 || $allowStudentToStudent == 'true'
                 || $current_user['status'] != 5                         // Always allow teachers.
                 || $current_user['is_tutor'] == 1                       // Always allow tutors.
-                ) && $current_user['user_id'] != $_user['user_id']) {   // Don't include yourself.
+                ) && $current_user['user_id'] != $_user['user_id']) {
+// Don't include yourself.
             if ($current_user['user_id'] == $current_user_id) {
                 continue;
             }
@@ -792,10 +794,15 @@ 
             ";
     $result = Database::query($sql);
 
-    if (!($res = Database::fetch_array($result)))
-        die(get_lang('GeneralError').' (code 901)');
-    if ($owner == 0) return $res['uploader_id'];
-    if ($res['uploader_id'] == $owner) return true;
+    if (!($res = Database::fetch_array($result))) {
+            die(get_lang('GeneralError').' (code 901)');
+    }
+    if ($owner == 0) {
+        return $res['uploader_id'];
+    }
+    if ($res['uploader_id'] == $owner) {
+        return true;
+    }
     die(get_lang('GeneralError').' (code '.$or_die.')');
 }
 
@@ -936,7 +943,8 @@ 
     }
 
     // check if the file is actually uploaded
-    if (!is_uploaded_file($dropbox_filetmpname)) { // check user fraud : no clean error msg.
+    if (!is_uploaded_file($dropbox_filetmpname)) {
+// check user fraud : no clean error msg.
         Display::addFlash(Display::return_message(get_lang('TheFileIsNotUploaded'), 'warning'));
 
         return false;
@@ -992,7 +1000,8 @@ 
                 break;
             }
         }
-    } else {  // rename file to login_filename_uniqueId format
+    } else {
+// rename file to login_filename_uniqueId format
         $dropbox_filename = $_user['username']."_".$dropbox_filename."_".uniqid('');
     }
 

main/lp/lp_final_item.php

Braces +2 added lines, -1 removed lines

@@ -208,7 +208,8 @@
     $sessionId = 0,
     $downloadCertificateLink = '',
     $badgeLink = ''
-) {
+)
+{
     $documentInfo = DocumentManager::get_document_data_by_id(
         $lpItemId,
         $courseCode,

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

Braces +2 added lines, -1 removed lines

@@ -262,7 +262,8 @@
             return null;
         }
 
-        if (api_is_allowed_to_edit()) { // Let students make access only through "Surveys" tool.
+        if (api_is_allowed_to_edit()) {
+// Let students make access only through "Surveys" tool.
             $tbl_name = $this->get_survey_table();
             $session_id = api_get_session_id();
             if ($tbl_name != '') {

main/inc/lib/sessionmanager.lib.php

Braces +6 added lines, -3 removed lines

@@ -4691,7 +4691,8 @@ 
                     if ($debug) {
                         if ($session_id) {
                             foreach ($enreg as $key => $value) {
-                                if (substr($key, 0, 6) == 'extra_') { //an extra field
+                                if (substr($key, 0, 6) == 'extra_') {
+//an extra field
                                     self::update_session_extra_field_value($session_id, substr($key, 6), $value);
                                 }
                             }
@@ -4738,7 +4739,8 @@ 
 
                         if ($session_id) {
                             foreach ($enreg as $key => $value) {
-                                if (substr($key, 0, 6) == 'extra_') { //an extra field
+                                if (substr($key, 0, 6) == 'extra_') {
+//an extra field
                                     self::update_session_extra_field_value($session_id, substr($key, 6), $value);
                                 }
                             }
@@ -4836,7 +4838,8 @@ 
                             Database::update($tbl_session, $params, array('id = ?' => $session_id));
 
                             foreach ($enreg as $key => $value) {
-                                if (substr($key, 0, 6) == 'extra_') { //an extra field
+                                if (substr($key, 0, 6) == 'extra_') {
+//an extra field
                                     self::update_session_extra_field_value($session_id, substr($key, 6), $value);
                                 }
                             }

main/lp/lp_controller.php

Braces +75 added lines, -30 removed lines

@@ -15,7 +15,9 @@ 
 $use_anonymous = true;
 
 $debug = 0;
-if ($debug > 0) error_log('New LP -+- Entered lp_controller.php -+- (action: '.$_REQUEST['action'].')', 0);
+if ($debug > 0) {
+    error_log('New LP -+- Entered lp_controller.php -+- (action: '.$_REQUEST['action'].')', 0);
+}
 
 // Language files that needs to be included.
 if (isset($_GET['action'])) {
@@ -211,10 +213,14 @@ 
     // If refresh is set, we regenerate the oLP object from the database (kind of flush).
     Session::erase('refresh');
     $myrefresh = 1;
-    if ($debug > 0) error_log('New LP - Refresh asked', 0);
-}
+    if ($debug > 0) {
+        error_log('New LP - Refresh asked', 0);
+    }
+    }
 
-if ($debug > 0) error_log('New LP - Passed refresh check', 0);
+if ($debug > 0) {
+    error_log('New LP - Passed refresh check', 0);
+}
 
 if (!empty($_REQUEST['dialog_box'])) {
     $dialog_box = stripslashes(urldecode($_REQUEST['dialog_box']));
@@ -224,17 +230,23 @@ 
 $lp_found = false;
 $lpObject = Session::read('lpobject');
 if (!empty($lpObject)) {
-    if ($debug > 0) error_log('New LP - SESSION[lpobject] is defined', 0);
+    if ($debug > 0) {
+        error_log('New LP - SESSION[lpobject] is defined', 0);
+    }
     $oLP = unserialize($lpObject);
     if (isset($oLP) && is_object($oLP)) {
-        if ($debug > 0) error_log('New LP - oLP is object', 0);
+        if ($debug > 0) {
+            error_log('New LP - oLP is object', 0);
+        }
         if ($myrefresh == 1 ||
             empty($oLP->cc) ||
             $oLP->cc != api_get_course_id() ||
             $oLP->lp_view_session_id != $session_id ||
             $oLP->scorm_debug == '1'
         ) {
-            if ($debug > 0) error_log('New LP - Course has changed, discard lp object', 0);
+            if ($debug > 0) {
+                error_log('New LP - Course has changed, discard lp object', 0);
+            }
             if ($myrefresh == 1) {
                 $myrefresh_id = $oLP->get_id();
             }
@@ -250,13 +262,19 @@ 
 
 $course_id = api_get_course_int_id();
 
-if ($debug > 0) error_log('New LP - Passed data remains check', 0);
+if ($debug > 0) {
+    error_log('New LP - Passed data remains check', 0);
+}
 
 if (!$lp_found || (!empty($_REQUEST['lp_id']) && $_SESSION['oLP']->get_id() != $_REQUEST['lp_id'])) {
-    if ($debug > 0) error_log('New LP - oLP is not object, has changed or refresh been asked, getting new', 0);
+    if ($debug > 0) {
+        error_log('New LP - oLP is not object, has changed or refresh been asked, getting new', 0);
+    }
     // Regenerate a new lp object? Not always as some pages don't need the object (like upload?)
     if (!empty($_REQUEST['lp_id']) || !empty($myrefresh_id)) {
-        if ($debug > 0) error_log('New LP - lp_id is defined', 0);
+        if ($debug > 0) {
+            error_log('New LP - lp_id is defined', 0);
+        }
         // Select the lp in the database and check which type it is (scorm/dokeos/aicc) to generate the
         // right object.
         if (!empty($_REQUEST['lp_id'])) {
@@ -268,16 +286,22 @@ 
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         if (is_numeric($lp_id)) {
             $sel = "SELECT lp_type FROM $lp_table WHERE c_id = $course_id AND id = $lp_id";
-            if ($debug > 0) error_log('New LP - querying '.$sel, 0);
+            if ($debug > 0) {
+                error_log('New LP - querying '.$sel, 0);
+            }
             $res = Database::query($sel);
 
             if (Database::num_rows($res)) {
                 $row = Database::fetch_array($res);
                 $type = $row['lp_type'];
-                if ($debug > 0) error_log('New LP - found row - type '.$type.' - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                if ($debug > 0) {
+                    error_log('New LP - found row - type '.$type.' - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                }
                 switch ($type) {
                     case 1:
-                        if ($debug > 0) error_log('New LP - found row - type dokeos - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        if ($debug > 0) {
+                            error_log('New LP - found row - type dokeos - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        }
 
                         $oLP = new learnpath(api_get_course_id(), $lp_id, api_get_user_id());
                         if ($oLP !== false) {
@@ -287,7 +311,9 @@ 
                         }
                         break;
                     case 2:
-                        if ($debug > 0) error_log('New LP - found row - type scorm - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        if ($debug > 0) {
+                            error_log('New LP - found row - type scorm - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        }
                         $oLP = new scorm(api_get_course_id(), $lp_id, api_get_user_id());
                         if ($oLP !== false) {
                             $lp_found = true;
@@ -296,7 +322,9 @@ 
                         }
                         break;
                     case 3:
-                        if ($debug > 0) error_log('New LP - found row - type aicc - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        if ($debug > 0) {
+                            error_log('New LP - found row - type aicc - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        }
                         $oLP = new aicc(api_get_course_id(), $lp_id, api_get_user_id());
                         if ($oLP !== false) {
                             $lp_found = true;
@@ -305,7 +333,9 @@ 
                         }
                         break;
                     default:
-                        if ($debug > 0) error_log('New LP - found row - type other - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        if ($debug > 0) {
+                            error_log('New LP - found row - type other - Calling constructor with '.api_get_course_id().' - '.$lp_id.' - '.api_get_user_id(), 0);
+                        }
                         $oLP = new learnpath(api_get_course_id(), $lp_id, api_get_user_id());
                         if ($oLP !== false) {
                             $lp_found = true;
@@ -316,17 +346,23 @@ 
                 }
             }
         } else {
-            if ($debug > 0) error_log('New LP - Request[lp_id] is not numeric', 0);
+            if ($debug > 0) {
+                error_log('New LP - Request[lp_id] is not numeric', 0);
+            }
         }
     } else {
-        if ($debug > 0) error_log('New LP - Request[lp_id] and refresh_id were empty', 0);
+        if ($debug > 0) {
+            error_log('New LP - Request[lp_id] and refresh_id were empty', 0);
+        }
     }
     if ($lp_found) {
         Session::write('oLP', $oLP);
     }
 }
 
-if ($debug > 0) error_log('New LP - Passed oLP creation check', 0);
+if ($debug > 0) {
+    error_log('New LP - Passed oLP creation check', 0);
+}
 
 $is_allowed_to_edit = api_is_allowed_to_edit(false, true, false, false);
 
@@ -616,15 +652,15 @@ 
         }
         break;
     case 'auto_launch':
-        if (api_get_course_setting('enable_lp_auto_launch') == 1) { //Redirect to a specific LP
+        if (api_get_course_setting('enable_lp_auto_launch') == 1) {
+//Redirect to a specific LP
             if (!$is_allowed_to_edit) {
                 api_not_allowed(true);
             }
             if (!$lp_found) {
                 error_log('New LP - No learnpath given for set_autolaunch', 0);
                 require 'lp_list.php';
-            }
-            else {
+            } else {
                 $_SESSION['oLP']->set_autolaunch($_GET['lp_id'], $_GET['status']);
                 require 'lp_list.php';
                 exit;
@@ -1140,16 +1176,24 @@ 
         }
         break;
     case 'content':
-        if ($debug > 0) error_log('New LP - Item id is '.intval($_GET['item_id']), 0);
+        if ($debug > 0) {
+            error_log('New LP - Item id is '.intval($_GET['item_id']), 0);
+        }
         if (!$lp_found) {
             error_log('New LP - No learnpath given for content', 0);
             require 'lp_list.php';
         } else {
-            if ($debug > 0) error_log('New LP - save_last()', 0);
+            if ($debug > 0) {
+                error_log('New LP - save_last()', 0);
+            }
             $_SESSION['oLP']->save_last();
-            if ($debug > 0) error_log('New LP - set_current_item()', 0);
+            if ($debug > 0) {
+                error_log('New LP - set_current_item()', 0);
+            }
             $_SESSION['oLP']->set_current_item($_GET['item_id']);
-            if ($debug > 0) error_log('New LP - start_current_item()', 0);
+            if ($debug > 0) {
+                error_log('New LP - start_current_item()', 0);
+            }
             $_SESSION['oLP']->start_current_item();
             require 'lp_content.php';
         }
@@ -1264,8 +1308,7 @@ 
         require 'lp_message.php';
         break;
     case 'return_to_course_homepage':
-        if (!$lp_found) { error_log('New LP - No learnpath given for stats', 0); require 'lp_list.php'; }
-        else {
+        if (!$lp_found) { error_log('New LP - No learnpath given for stats', 0); require 'lp_list.php'; } else {
             $_SESSION['oLP']->save_current();
             $_SESSION['oLP']->save_last();
             $url = api_get_path(WEB_COURSE_PATH).api_get_course_path().'/index.php?id_session='.api_get_session_id();
@@ -1453,8 +1496,10 @@ 
 
 if (!empty($_SESSION['oLP'])) {
     $_SESSION['lpobject'] = serialize($_SESSION['oLP']);
-    if ($debug > 0) error_log('New LP - lpobject is serialized in session', 0);
-}
+    if ($debug > 0) {
+        error_log('New LP - lpobject is serialized in session', 0);
+    }
+    }
 
 if (!empty($redirectTo)) {
     header("Location: $redirectTo");

main/lp/aicc.class.php

Braces +7 added lines, -3 removed lines

@@ -532,7 +532,9 @@ 
                     if ($file != '.' && $file != '..') {
                         $filetype = 'file';
 
-                        if (is_dir($course_sys_dir.$new_dir.$file)) $filetype = 'folder';
+                        if (is_dir($course_sys_dir.$new_dir.$file)) {
+                            $filetype = 'folder';
+                        }
 
                         // TODO: RENAMING FILES CAN BE VERY DANGEROUS AICC-WISE, avoid that as much as possible!
                         //$safe_file = api_replace_dangerous_char($file, 'strict');
@@ -844,7 +846,8 @@ 
                         $v = @substr($v, 1, @strlen($v) - 2);
                     }
                     if ($sec) {
-                        if (api_strtolower($sec) == 'course_description') { // A special case.
+                        if (api_strtolower($sec) == 'course_description') {
+// A special case.
                             $r[api_strtolower($sec)] = $k;
                         } else {
                             $r[api_strtolower($sec)][api_strtolower($k)] = $v;
@@ -906,7 +909,8 @@ 
                         if ($pure_data) {
                             $r[api_strtolower($sec)] .= $f[$i];
                         } else {
-                            if (api_strtolower($sec) == 'course_description') { // A special case.
+                            if (api_strtolower($sec) == 'course_description') {
+// A special case.
                                 $r[api_strtolower($sec)] = $k;
                             } else {
                                 $r[api_strtolower($sec)][api_strtolower($k)] = $v;

main/inc/lib/myspace.lib.php

Braces +15 added lines, -7 removed lines

@@ -1059,11 +1059,13 @@ 
             );
             if ($last_login_date_tmp != false &&
                 $last_login_date == false
-            ) { // TODO: To be cleaned
+            ) {
+// TODO: To be cleaned
                 $last_login_date = $last_login_date_tmp;
             } elseif ($last_login_date_tmp != false &&
                 $last_login_date != false
-            ) { // TODO: Repeated previous condition. To be cleaned.
+            ) {
+// TODO: Repeated previous condition. To be cleaned.
                 // Find the max and assign it to first_login_date
                 if (strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
                     $last_login_date = $last_login_date_tmp;
@@ -1246,9 +1248,11 @@ 
                     null,
                     false
                 );
-                if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned.
+                if ($last_login_date_tmp != false && $last_login_date == false) {
+// TODO: To be cleaned.
                     $last_login_date = $last_login_date_tmp;
-                } elseif ($last_login_date_tmp != false && $last_login_date == false) { // TODO: Repeated previous condition. To be cleaned.
+                } elseif ($last_login_date_tmp != false && $last_login_date == false) {
+// TODO: Repeated previous condition. To be cleaned.
                     // Find the max and assign it to first_login_date
                     if (strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
                         $last_login_date = $last_login_date_tmp;
@@ -1634,9 +1638,11 @@ 
                         $session_id,
                         false
                     );
-                    if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned.
+                    if ($last_login_date_tmp != false && $last_login_date == false) {
+// TODO: To be cleaned.
                         $last_login_date = $last_login_date_tmp;
-                    } else if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: Repeated previous condition. To be cleaned.
+                    } else if ($last_login_date_tmp != false && $last_login_date == false) {
+// TODO: Repeated previous condition. To be cleaned.
                         // Find the max and assign it to first_login_date
                         if (strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
                             $last_login_date = $last_login_date_tmp;
@@ -1906,7 +1912,9 @@ 
                 FROM $tbl_course course
                 WHERE course.code IN (".implode(',', $courses_code).")";
 
-        if (!in_array($direction, array('ASC', 'DESC'))) $direction = 'ASC';
+        if (!in_array($direction, array('ASC', 'DESC'))) {
+            $direction = 'ASC';
+        }
 
         $column = intval($column);
         $from = intval($from);

main/inc/lib/geometry.lib.php

Braces +30 added lines, -14 removed lines

@@ -81,11 +81,15 @@ 
     for (; $i < sizeof($poly) && (!empty($poly[$i]['x']) && !empty($poly[$i]['y'])); $i++) {
         /* special cases */
         if ($poly[$i - 1]['y'] == $poly[$i]['y']) {
-            if ($poly[$i - 1]['x'] == $poly[$i]['x'])
-                continue; // twice the same point
-            else {    //  infinite elevation of the edge
-                if (is_array($bords[$poly[$i]['y']]))
-                    array_push($bords[$poly[$i]['y']], $poly[$i]['x']);
+            if ($poly[$i - 1]['x'] == $poly[$i]['x']) {
+                            continue;
+            }
+            // twice the same point
+            else {
+//  infinite elevation of the edge
+                if (is_array($bords[$poly[$i]['y']])) {
+                                    array_push($bords[$poly[$i]['y']], $poly[$i]['x']);
+                }
                 $old_pente = 0;
                 continue;
             }
@@ -93,9 +97,13 @@ 
 
         //echo 'point:'.$poly[$i]['y']; bug here
         // adding the point as a part of an edge
-        if (is_array($bords[$poly[$i]['y']])) //avoid warning
+        if (is_array($bords[$poly[$i]['y']])) {
+            //avoid warning
         array_push($bords[$poly[$i]['y']], $poly[$i]['x']);
-        if (DEBUG) echo '('.$poly[$i]['x'].';'.$poly[$i]['y'].')   ';
+        }
+        if (DEBUG) {
+            echo '('.$poly[$i]['x'].';'.$poly[$i]['y'].')   ';
+        }
 
         /* computing the elevation of the edge going */
         //        from $poly[$i-1] to $poly[$i]
@@ -108,10 +116,12 @@ 
         if ($i > 1) {
             if (($old_pente < 0 && $pente > 0)
                 || ($old_pente > 0 && $pente < 0)) {
-                if (is_array($bords[$poly[$i]['y']])) //avoid warning
+                if (is_array($bords[$poly[$i]['y']])) {
+                    //avoid warning
                 {
                     array_push($bords[$poly[$i]['y']], $poly[$i]['x']);
                 }
+                }
 
                 if (DEBUG) {
                     echo '*('.$poly[$i]['x'].
@@ -142,7 +152,8 @@ 
     }
 
     // closing the polygone (the edge between $poly[$i-1] and $poly[0])
-    if ($poly[$i - 1]['y'] != $poly[0]['y']) {// droite--> rien à faire
+    if ($poly[$i - 1]['y'] != $poly[0]['y']) {
+// droite--> rien à faire
 
         // elevation between $poly[0]['x'] and $poly[1]['x'])
         $rest = $poly[0]['y'] - $poly[1]['y'];
@@ -162,14 +173,17 @@ 
 
         // doubling the first point if needed (see above)
         if (($pente1 < 0 && $pente > 0) || ($pente1 > 0 && $pente < 0)) {
-            if (is_array($bords[$poly[$i - 1]['y']]))
-                array_push($bords[$poly[$i - 1]['y']], round($poly[$i - 1]['x']));
+            if (is_array($bords[$poly[$i - 1]['y']])) {
+                            array_push($bords[$poly[$i - 1]['y']], round($poly[$i - 1]['x']));
+            }
             //if (DEBUG) echo '('.$poly[$i-1]['x'].';'.$poly[$i-1]['y'].')   ';
         }
         //  doubling the last point if neededd
         if (($old_pente < 0 && $pente > 0) || ($old_pente > 0 && $pente < 0)) {
-            if (is_array($bords[$poly[$i - 1]['y']])) //avoid warning
+            if (is_array($bords[$poly[$i - 1]['y']])) {
+                //avoid warning
                 array_push($bords[$poly[$i - 1]['y']], round($poly[$i - 1]['x']));
+            }
             //if (DEBUG) echo '*('.$poly[$i-1]['x'].';'.$poly[$i-1]['y'].')   ';
         }
 
@@ -189,13 +203,15 @@ 
     /* basic idea: we sort a column of edges.
         For each pair of point, we color the points in between */
     $n = count($bords);
-    for ($i = 0; $i < $n; $i++) {  // Y
+    for ($i = 0; $i < $n; $i++) {
+// Y
         //error_log(__FILE__.' - Border Num '.$i,0);
         if (is_array($bords[$i])) {
             sort($bords[$i]);
         }
 
-        for ($j = 0; $j < sizeof($bords[$i]); $j += 2) { // bords
+        for ($j = 0; $j < sizeof($bords[$i]); $j += 2) {
+// bords
             if (!isset($bords[$i][$j + 1])) {
                 continue;
             }

main/webservices/registration.soap.php

Braces +192 added lines, -67 removed lines

@@ -576,7 +576,9 @@ 
 
     // First check wether the login already exists
     if (!UserManager::is_username_available($loginName)) {
-        if ($debug) error_log("Username $loginName is not available");
+        if ($debug) {
+            error_log("Username $loginName is not available");
+        }
         return 0;
     }
 
@@ -1084,9 +1086,15 @@ 
         );
     }
 
-    if ($debug) error_log('$userId found: '.$userId);
-    if ($debug) error_log('$courseId found: '.$courseId);
-    if ($debug) error_log('$sessionId found: '.$sessionId);
+    if ($debug) {
+        error_log('$userId found: '.$userId);
+    }
+    if ($debug) {
+        error_log('$courseId found: '.$courseId);
+    }
+    if ($debug) {
+        error_log('$sessionId found: '.$sessionId);
+    }
 
     return [
         'user_id' => $userId,
@@ -1115,13 +1123,17 @@ 
 function WSSubscribeTeacherToSessionCourse($params)
 {
     global $debug;
-    if ($debug) error_log('WSSubscribeTeacherToSessionCourse');
+    if ($debug) {
+        error_log('WSSubscribeTeacherToSessionCourse');
+    }
 
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
     }
 
-    if ($debug) error_log('Params '.print_r($params, 1));
+    if ($debug) {
+        error_log('Params '.print_r($params, 1));
+    }
 
     $params = parseCourseSessionUserParams($params);
 
@@ -1134,13 +1146,17 @@ 
     $result = 0;
 
     if (!empty($coaches)) {
-        if ($debug) error_log('Coaches:  '.print_r($coaches, 1));
+        if ($debug) {
+            error_log('Coaches:  '.print_r($coaches, 1));
+        }
         if (in_array($userId, $coaches)) {
             $result = 1;
         }
     }
 
-    if ($debug) error_log('Result:  '.$result);
+    if ($debug) {
+        error_log('Result:  '.$result);
+    }
 
     return $result;
 }
@@ -1166,13 +1182,17 @@ 
 {
     global $debug;
 
-    if ($debug) error_log('WSSubscribeTeacherToSessionCourse');
+    if ($debug) {
+        error_log('WSSubscribeTeacherToSessionCourse');
+    }
 
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
     }
 
-    if ($debug) error_log('Params '.print_r($params, 1));
+    if ($debug) {
+        error_log('Params '.print_r($params, 1));
+    }
 
     $params = parseCourseSessionUserParams($params);
 
@@ -1186,7 +1206,9 @@ 
     $result = 0;
 
     if (!empty($coaches)) {
-        if ($debug) error_log('Coaches:  '.print_r($coaches, 1));
+        if ($debug) {
+            error_log('Coaches:  '.print_r($coaches, 1));
+        }
         if (!in_array($userId, $coaches)) {
             $result = 1;
         }
@@ -1194,7 +1216,9 @@ 
         $result = 1;
     }
 
-    if ($debug) error_log('Final Result: '.$result);
+    if ($debug) {
+        error_log('Final Result: '.$result);
+    }
 
     return $result;
 }
@@ -1247,8 +1271,12 @@ 
 {
     global $_user, $_configuration, $debug;
     $debug = 1;
-    if ($debug) error_log('WSCreateUserPasswordCrypted');
-    if ($debug) error_log(print_r($params, 1));
+    if ($debug) {
+        error_log('WSCreateUserPasswordCrypted');
+    }
+    if ($debug) {
+        error_log(print_r($params, 1));
+    }
 
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
@@ -1280,22 +1308,30 @@ 
         if ($_configuration['password_encryption'] === $encrypt_method) {
             if ($encrypt_method == 'md5' && !preg_match('/^[A-Fa-f0-9]{32}$/', $password)) {
                 $msg = "Encryption $encrypt_method is invalid";
-                if ($debug) error_log($msg);
+                if ($debug) {
+                    error_log($msg);
+                }
                 return $msg;
 
             } elseif ($encrypt_method == 'sha1' && !preg_match('/^[A-Fa-f0-9]{40}$/', $password)) {
                 $msg = "Encryption $encrypt_method is invalid";
-                if ($debug) error_log($msg);
+                if ($debug) {
+                    error_log($msg);
+                }
                 return $msg;
             }
         } else {
             $msg = "This encryption $encrypt_method is not configured";
-            if ($debug) error_log($msg);
+            if ($debug) {
+                error_log($msg);
+            }
             return $msg;
         }
     } else {
         $msg = 'The chamilo setting $_configuration["password_encryption"] is not configured';
-        if ($debug) error_log($msg);
+        if ($debug) {
+            error_log($msg);
+        }
         return $msg;
     }
 
@@ -1315,10 +1351,14 @@ 
         $original_user_id_name
     );
 
-    if ($debug) error_log('Ready to create user');
+    if ($debug) {
+        error_log('Ready to create user');
+    }
 
     if ($user_id > 0) {
-        if ($debug) error_log('User found with id: '.$user_id);
+        if ($debug) {
+            error_log('User found with id: '.$user_id);
+        }
 
         // Check whether user is not active
         //@todo why this condition exists??
@@ -1328,7 +1368,9 @@ 
         $r_check_user = Database::fetch_row($resu);
         $count_check_user = Database::num_rows($resu);
         if ($count_check_user > 0) {
-            if ($debug) error_log('User id: '.$user_id.' exists and is NOT active. Updating user and setting setting active = 1');
+            if ($debug) {
+                error_log('User id: '.$user_id.' exists and is NOT active. Updating user and setting setting active = 1');
+            }
             $sql = "UPDATE $table_user SET
                     lastname='".Database::escape_string($lastName)."',
                     firstname='".Database::escape_string($firstName)."',
@@ -1364,11 +1406,15 @@ 
             }
             return $r_check_user[0];
         } else {
-            if ($debug) error_log('User exists but is active. Cant be updated');
+            if ($debug) {
+                error_log('User exists but is active. Cant be updated');
+            }
             return 0;
         }
     } else {
-        if ($debug) error_log("User not found with original_id = $original_user_id_value and original_name = $original_user_id_name");
+        if ($debug) {
+            error_log("User not found with original_id = $original_user_id_value and original_name = $original_user_id_name");
+        }
     }
 
     // Default language.
@@ -1384,12 +1430,16 @@ 
 
     // First check wether the login already exists
     if (!UserManager::is_username_available($loginName)) {
-        if ($debug) error_log("Username $loginName is not available");
+        if ($debug) {
+            error_log("Username $loginName is not available");
+        }
         return 0;
     }
 
     $queryExpirationDate = '';
-    if (!empty($params['expiration_date'])) $queryExpirationDate = "expiration_date     = '".Database::escape_string($expiration_date)."', ";
+    if (!empty($params['expiration_date'])) {
+        $queryExpirationDate = "expiration_date     = '".Database::escape_string($expiration_date)."', ";
+    }
 
     $sql = "INSERT INTO $table_user SET
             lastname            = '".Database::escape_string(trim($lastName))."',
@@ -1419,7 +1469,9 @@ 
 
         $url_id = api_get_current_access_url_id();
         UrlManager::add_user_to_url($return, $url_id);
-        if ($debug) error_log("Adding user_id = $return to URL id $url_id ");
+        if ($debug) {
+            error_log("Adding user_id = $return to URL id $url_id ");
+        }
 
         // Create extra field for the original_user_id_name
         UserManager::create_extra_field(
@@ -1456,7 +1508,9 @@ 
             }
         }
     } else {
-        if ($debug) error_log('Error while inserting a user');
+        if ($debug) {
+            error_log('Error while inserting a user');
+        }
 
         return 0;
     }
@@ -2947,10 +3001,12 @@ 
         $params['visibility'] = $visibility;
         $params['disk_quota'] = $diskQuota;
 
-        if (isset($subscribe) && $subscribe != '') { // Valid values: 0, 1
+        if (isset($subscribe) && $subscribe != '') {
+// Valid values: 0, 1
             $params['subscribe'] = $subscribe;
         }
-        if (isset($unsubscribe) && $subscribe != '') { // Valid values: 0, 1
+        if (isset($unsubscribe) && $subscribe != '') {
+// Valid values: 0, 1
             $params['unsubscribe'] = $unsubscribe;
         }
 
@@ -3327,7 +3383,8 @@ 
 );
 
 // Define the method WSEditCourse
-function WSEditCourse($params) {
+function WSEditCourse($params)
+{
 
     global $_configuration;
     if (!WSHelperVerifyKey($params)) {
@@ -4250,7 +4307,8 @@ 
         if (empty($name)) {
             $results[] = 0; //SessionNameIsRequired
             continue;
-        } elseif (empty($id_coach)) { // Session must have coach
+        } elseif (empty($id_coach)) {
+// Session must have coach
             $results[] = 0;
             continue;
         } elseif (empty($nolimit) && (!$month_start || !$day_start || !$year_start || !checkdate($month_start, $day_start, $year_start))) {
@@ -4545,12 +4603,15 @@ 
 );
 
 // define the method WSSubscribeUserToCourse
-function WSSubscribeUserToCourse($params) {
+function WSSubscribeUserToCourse($params)
+{
     global $debug;
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
     }
-    if ($debug) error_log('WSSubscribeUserToCourse params: '.print_r($params, 1));
+    if ($debug) {
+        error_log('WSSubscribeUserToCourse params: '.print_r($params, 1));
+    }
 
     $results = array();
     $userscourses = $params['userscourses'];
@@ -4569,7 +4630,9 @@ 
             $original_user_id['original_user_id_value'],
             $original_user_id['original_user_id_name']
         );
-        if ($debug) error_log('WSSubscribeUserToCourse user_id: '.$user_id);
+        if ($debug) {
+            error_log('WSSubscribeUserToCourse user_id: '.$user_id);
+        }
 
         if ($user_id == 0) {
             // If user was not found, there was a problem
@@ -4587,13 +4650,19 @@ 
                 // Course was not found
                 $resultValue = 0;
             } else {
-                if ($debug) error_log('WSSubscribeUserToCourse courseCode: '.$courseCode);
+                if ($debug) {
+                    error_log('WSSubscribeUserToCourse courseCode: '.$courseCode);
+                }
                 $result = CourseManager::add_user_to_course($user_id, $courseCode, $status, false);
                 if ($result) {
                     $resultValue = 1;
-                    if ($debug) error_log('WSSubscribeUserToCourse subscribed');
+                    if ($debug) {
+                        error_log('WSSubscribeUserToCourse subscribed');
+                    }
                 } else {
-                    if ($debug) error_log('WSSubscribeUserToCourse NOT subscribed: ');
+                    if ($debug) {
+                        error_log('WSSubscribeUserToCourse NOT subscribed: ');
+                    }
                 }
             }
         }
@@ -4649,11 +4718,16 @@ 
 );
 
 // define the method WSSubscribeUserToCourse
-function WSSubscribeUserToCourseSimple($params) {
+function WSSubscribeUserToCourseSimple($params)
+{
     global $debug;
 
-    if ($debug) error_log('WSSubscribeUserToCourseSimple');
-    if ($debug) error_log('Params '.print_r($params, 1));
+    if ($debug) {
+        error_log('WSSubscribeUserToCourseSimple');
+    }
+    if ($debug) {
+        error_log('Params '.print_r($params, 1));
+    }
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
     }
@@ -4671,7 +4745,9 @@ 
     if (empty($user_data)) {
         // If user was not found, there was a problem
         $result = "User $user_id does not exist";
-        if ($debug) error_log($result);
+        if ($debug) {
+            error_log($result);
+        }
         return $result;
     }
     if (!empty($course_code)) {
@@ -4679,14 +4755,22 @@ 
         if (empty($course_data)) {
             // Course was not found
             $result = "Course $course_code does not exist in the platform ";
-            if ($debug) error_log($result);
+            if ($debug) {
+                error_log($result);
+            }
         } else {
-            if ($debug) error_log('Try to register: user_id= '.$user_id.' to course: '.$course_data['code']);
+            if ($debug) {
+                error_log('Try to register: user_id= '.$user_id.' to course: '.$course_data['code']);
+            }
             if (!CourseManager::add_user_to_course($user_id, $course_data['code'], $status)) {
                 $result = 'User was not registered possible reasons: User already registered to the course, Course visibility doesnt allow user subscriptions ';
-                if ($debug) error_log($result);
+                if ($debug) {
+                    error_log($result);
+                }
             } else {
-                if ($debug) error_log('User registered to the course: '.$course_data['code']);
+                if ($debug) {
+                    error_log('User registered to the course: '.$course_data['code']);
+                }
                 $result = 1;
             }
         }
@@ -4737,8 +4821,12 @@ 
 function WSGetUser($params)
 {
     global $debug;
-    if ($debug) error_log('WSGetUser');
-    if ($debug) error_log('$params: '.print_r($params, 1));
+    if ($debug) {
+        error_log('WSGetUser');
+    }
+    if ($debug) {
+        error_log('$params: '.print_r($params, 1));
+    }
 
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
@@ -4793,8 +4881,12 @@ 
 function WSGetUserFromUsername($params)
 {
     global $debug;
-    if ($debug) error_log('WSGetUserFromUsername');
-    if ($debug) error_log('$params: '.print_r($params, 1));
+    if ($debug) {
+        error_log('WSGetUserFromUsername');
+    }
+    if ($debug) {
+        error_log('$params: '.print_r($params, 1));
+    }
 
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
@@ -5271,7 +5363,9 @@ 
                 );
                 $results[] = 1;
 
-                if ($debug) error_log("subscribe user:$user_id to session $sessionId");
+                if ($debug) {
+                    error_log("subscribe user:$user_id to session $sessionId");
+                }
             }
         }
     } // end principal foreach
@@ -5312,7 +5406,8 @@ 
     'encoded', // use
     'This service subscribes a user to a session in a simple way'                     // documentation
 );
-function WSSubscribeUserToSessionSimple($params) {
+function WSSubscribeUserToSessionSimple($params)
+{
     global $debug;
 
     if ($debug) {
@@ -5355,7 +5450,9 @@ 
                 SESSION_VISIBLE_READ_ONLY,
                 false
             );
-            if ($debug) error_log('User registered to the course: '.$session_id);
+            if ($debug) {
+                error_log('User registered to the course: '.$session_id);
+            }
             $result = 1;
         }
     }
@@ -5503,7 +5600,9 @@ 
 
                 $results[] = 1;
 
-                if ($debug) error_log("Unsubscribe user:$user_id to session:$id_session");
+                if ($debug) {
+                    error_log("Unsubscribe user:$user_id to session:$id_session");
+                }
             }
         }
     } // end principal foreach
@@ -5650,7 +5749,9 @@ 
         return returnError(WS_ERROR_SECRET_KEY);
     }
 
-    if ($debug) error_log('WSSuscribeCoursesToSession: '.print_r($params, 1));
+    if ($debug) {
+        error_log('WSSuscribeCoursesToSession: '.print_r($params, 1));
+    }
 
     $coursessessions_params = $params['coursessessions'];
     $results = array();
@@ -5696,7 +5797,9 @@ 
                     array($courseInfo['real_id']),
                     false
                 );
-                if ($debug) error_log("add_courses_to_session: course:$courseCode to session:$sessionId");
+                if ($debug) {
+                    error_log("add_courses_to_session: course:$courseCode to session:$sessionId");
+                }
 
                 $results[] = 1;
             }
@@ -6966,12 +7069,15 @@ 
 );
 
 // define the method WSRegisterUserVisibilityToCourseInCatalogue
-function WSAddUserVisibilityToCourseInCatalogue($params) {
+function WSAddUserVisibilityToCourseInCatalogue($params)
+{
     global $debug;
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
     }
-    if ($debug) error_log('WSAddUserVisibilityToCourseCatalogue params: '.print_r($params, 1));
+    if ($debug) {
+        error_log('WSAddUserVisibilityToCourseCatalogue params: '.print_r($params, 1));
+    }
 
     $results = array();
     $userscourses = $params['userscourses'];
@@ -6987,7 +7093,9 @@ 
             $original_user_id['original_user_id_value'],
             $original_user_id['original_user_id_name']
         );
-        if ($debug) error_log('WSAddUserVisibilityToCourseCatalogue userId: '.$userId);
+        if ($debug) {
+            error_log('WSAddUserVisibilityToCourseCatalogue userId: '.$userId);
+        }
 
         if ($userId == 0) {
             // If user was not found, there was a problem
@@ -7004,13 +7112,19 @@ 
                 // Course was not found
                 $resultValue = 0;
             } else {
-                if ($debug) error_log('WSAddUserVisibilityToCourseCatalogue courseCode: '.$courseCode);
+                if ($debug) {
+                    error_log('WSAddUserVisibilityToCourseCatalogue courseCode: '.$courseCode);
+                }
                 $result = CourseManager::addUserVisibilityToCourseInCatalogue($userId, $courseCode, $visible);
                 if ($result) {
                     $resultValue = 1;
-                    if ($debug) error_log('WSAddUserVisibilityToCourseCatalogue registered');
+                    if ($debug) {
+                        error_log('WSAddUserVisibilityToCourseCatalogue registered');
+                    }
                 } else {
-                    if ($debug) error_log('WSAddUserVisibilityToCourseCatalogue NOT registered: ');
+                    if ($debug) {
+                        error_log('WSAddUserVisibilityToCourseCatalogue NOT registered: ');
+                    }
                 }
             }
         }
@@ -7038,12 +7152,15 @@ 
 );
 
 // define the method WSRemoveUserVisibilityToCourseInCatalogue
-function WSRemoveUserVisibilityToCourseInCatalogue($params) {
+function WSRemoveUserVisibilityToCourseInCatalogue($params)
+{
     global $debug;
     if (!WSHelperVerifyKey($params)) {
         return returnError(WS_ERROR_SECRET_KEY);
     }
-    if ($debug) error_log('WSRemoveUserVisibilityToCourseInCatalogue params: '.print_r($params, 1));
+    if ($debug) {
+        error_log('WSRemoveUserVisibilityToCourseInCatalogue params: '.print_r($params, 1));
+    }
 
     $results = array();
     $userscourses = $params['userscourses'];
@@ -7059,7 +7176,9 @@ 
             $original_user_id['original_user_id_value'],
             $original_user_id['original_user_id_name']
         );
-        if ($debug) error_log('WSRemoveUserVisibilityToCourseInCatalogue user_id: '.$userId);
+        if ($debug) {
+            error_log('WSRemoveUserVisibilityToCourseInCatalogue user_id: '.$userId);
+        }
 
         if ($userId == 0) {
             // If user was not found, there was a problem
@@ -7076,13 +7195,19 @@ 
                 // Course was not found
                 $resultValue = 0;
             } else {
-                if ($debug) error_log('WSRemoveUserVisibilityToCourseInCatalogue courseCode: '.$courseCode);
+                if ($debug) {
+                    error_log('WSRemoveUserVisibilityToCourseInCatalogue courseCode: '.$courseCode);
+                }
                 $result = CourseManager::removeUserVisibilityToCourseInCatalogue($userId, $courseCode, $visible);
                 if ($result) {
                     $resultValue = 1;
-                    if ($debug) error_log('WSRemoveUserVisibilityToCourseInCatalogue removed');
+                    if ($debug) {
+                        error_log('WSRemoveUserVisibilityToCourseInCatalogue removed');
+                    }
                 } else {
-                    if ($debug) error_log('WSRemoveUserVisibilityToCourseInCatalogue NOT removed: ');
+                    if ($debug) {
+                        error_log('WSRemoveUserVisibilityToCourseInCatalogue NOT removed: ');
+                    }
                 }
             }
         }