chamilo / chamilo-lms

main/session/session_export.php

Indentation +6 added lines, -6 removed lines

@@ -237,12 +237,12 @@
                 ];
             } else {
                 $add = "\t<Session>\n"
-                         ."\t\t<SessionName>$row[name]</SessionName>\n"
-                         ."\t\t<Coach>$row[username]</Coach>\n"
-                         ."\t\t<DateStart>$row[access_start_date]</DateStart>\n"
-                         ."\t\t<DateEnd>$row[access_end_date]</DateEnd>\n"
-                         ."\t\t<Visibility>$row[visibility]</Visibility>\n"
-                         ."\t\t<SessionCategory>$row[session_category]</SessionCategory>\n";
+                            ."\t\t<SessionName>$row[name]</SessionName>\n"
+                            ."\t\t<Coach>$row[username]</Coach>\n"
+                            ."\t\t<DateStart>$row[access_start_date]</DateStart>\n"
+                            ."\t\t<DateEnd>$row[access_end_date]</DateEnd>\n"
+                            ."\t\t<Visibility>$row[visibility]</Visibility>\n"
+                            ."\t\t<SessionCategory>$row[session_category]</SessionCategory>\n";
             }
 
             if (!$cvs) {

plugin/buycourses/src/service_success.php

Braces +9 added lines, -9 removed lines

@@ -10,7 +10,7 @@ 
 $plugin = BuyCoursesPlugin::create();

 $paypalEnabled = $plugin->get('paypal_enable') === 'true';

 

-if (!$paypalEnabled) {

+if (!$paypalEnabled) {
     api_not_allowed(true);

 }

 

@@ -18,7 +18,7 @@ 
 $serviceSale = $plugin->getServiceSale($serviceSaleId);

 $itemPrice = $serviceSale['price'];

 

-if (empty($serviceSale)) {

+if (empty($serviceSale)) {
     api_not_allowed(true);

 }

 

@@ -49,9 +49,9 @@ 
 );

 $form->addButtonCancel($plugin->get_lang('CancelOrder'), 'cancel');

 

-if ($form->validate()) {

+if ($form->validate()) {
     $formValues = $form->getSubmitValues();

-    if (isset($formValues['cancel'])) {

+    if (isset($formValues['cancel'])) {
         $plugin->cancelServiceSale($serviceSale['id']);

 

         unset($_SESSION['bc_service_sale_id']);

@@ -65,7 +65,7 @@ 
     }

 

     $confirmPayments = ConfirmPayment($itemPrice);

-    if ($confirmPayments['ACK'] !== 'Success') {

+    if ($confirmPayments['ACK'] !== 'Success') {
         $erroMessage = vsprintf(

             $plugin->get_lang('ErrorOccurred'),

             [$expressCheckout['L_ERRORCODE0'], $confirmPayments['L_LONGMESSAGE0']]

@@ -78,11 +78,11 @@ 
         exit;

     }

 

-    switch ($confirmPayments["PAYMENTINFO_0_PAYMENTSTATUS"]) {

+    switch ($confirmPayments["PAYMENTINFO_0_PAYMENTSTATUS"]) {
         case 'Completed':

             $serviceSaleIsCompleted = $plugin->completeServiceSale($serviceSale['id']);

 

-            if ($serviceSaleIsCompleted) {

+            if ($serviceSaleIsCompleted) {
                 Display::addFlash(

                     Display::return_message(

                         sprintf($plugin->get_lang('SubscriptionToServiceXSuccessful'), $serviceSale['service']['name']),

@@ -98,7 +98,7 @@ 
             );

             break;

         case 'Pending':

-            switch ($confirmPayments["PAYMENTINFO_0_PENDINGREASON"]) {

+            switch ($confirmPayments["PAYMENTINFO_0_PENDINGREASON"]) {
                 case 'address':

                     $purchaseStatus = $plugin->get_lang('PendingReasonByAddress');

                     break;

@@ -164,7 +164,7 @@ 
 }

 

 $token = isset($_GET['token']) ? Security::remove_XSS($_GET['token']) : null;

-if (empty($token)) {

+if (empty($token)) {
     api_not_allowed(true);

 }

 

main/survey/surveyUtil.class.php

Braces +2 added lines, -1 removed lines

@@ -2099,7 +2099,8 @@
 
         // Remind unanswered is a special version of remind all reminder
         $exclude_users = array();
-        if ($remindUnAnswered == 1) { // Remind only unanswered users
+        if ($remindUnAnswered == 1) {
+// Remind only unanswered users
             $reminder = 1;
             $exclude_users = SurveyManager::get_people_who_filled_survey($_GET['survey_id']);
         }

Doc Comments +5 added lines, -3 removed lines

@@ -137,7 +137,7 @@ 
     /**
      * This function checks the parameters that are used in this page
      *
-     * @return string $people_filled The header, an error and the footer if any parameter fails, else it returns true
+     * @return null|boolean $people_filled The header, an error and the footer if any parameter fails, else it returns true
      * @author Patrick Cool <patrick.cool@UGent.be>, Ghent University
      * @version February 2007
      */
@@ -729,6 +729,7 @@ 
      * Display score data about a survey question
      * @param    array    Question info
      * @param    integer    The offset of results shown
+     * @param integer $offset
      * @return   void    (direct output)
      */
     public static function display_question_report_score($survey_data, $question, $offset)
@@ -1431,7 +1432,7 @@ 
      * Quite similar to display_complete_report(), returns an HTML string
      * that can be used in a csv file
      * @todo consider merging this function with display_complete_report
-     * @return string The contents of a csv file
+     * @return false|null The contents of a csv file
      * @author Patrick Cool <patrick.cool@UGent.be>, Ghent University
      * @version February 2007
      */
@@ -2302,7 +2303,7 @@ 
 
     /**
      * @param $params
-     * @return bool|int
+     * @return false|string
      */
     public static function save_invitation($params)
     {
@@ -2358,6 +2359,7 @@ 
      *
      * @param int invitedUser - the userId (course user) or emailaddress of additional user
      * $param string $invitation_code - the unique invitation code for the URL
+     * @param string|null $invitation_text
      * @return void
      */
     public static function send_invitation_mail(

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

@@ -127,7 +127,7 @@ 
         Database::query($sql);
         $insertId = Database::insert_id();
 
-        $sql = "UPDATE $table_survey_answer SET answer_id = $insertId 
+        $sql = "update $table_survey_answer SET answer_id = $insertId 
                 WHERE iid = $insertId";
         Database::query($sql);
 
@@ -274,7 +274,7 @@ 
 			        WHERE c_id = $course_id AND survey_id = '".$survey_id."' AND user = '".$user_id."'";
             Database::query($sql);
             // update field answered from survey_invitation by user_id and survey_id
-            $sql = "UPDATE $table_survey_invitation SET answered = '0'
+            $sql = "update $table_survey_invitation SET answered = '0'
 			        WHERE
 			            c_id = $course_id AND
 			            survey_code = (
@@ -2153,7 +2153,7 @@ 
             $mail_field = 'reminder_mail';
         }
 
-        $sql = "UPDATE $table_survey SET
+        $sql = "update $table_survey SET
 		        mail_subject='".Database::escape_string($mail_subject)."',
 		        $mail_field = '".Database::escape_string($mailtext)."'
 		        WHERE c_id = $course_id AND survey_id = '".intval($_GET['survey_id'])."'";
@@ -2314,7 +2314,7 @@ 
         ) {
             $insertId = Database::insert($table, $params);
             if ($insertId) {
-                $sql = "UPDATE $table 
+                $sql = "update $table 
                         SET survey_invitation_id = $insertId
                         WHERE iid = $insertId";
                 Database::query($sql);
@@ -2456,7 +2456,7 @@ 
         $total_invited = $row['total'];
 
         // Updating the field in the survey table
-        $sql = "UPDATE $table_survey
+        $sql = "update $table_survey
 		        SET invited = '".Database::escape_string($total_invited)."'
 		        WHERE
 		            c_id = $course_id AND

main/exercise/exercise_submit.php

Indentation +1 added lines, -1 removed lines

@@ -880,7 +880,7 @@
 }
 
 if ($origin != 'learnpath') {
-   echo '<div id="highlight-plugin" class="glossary-content">';
+    echo '<div id="highlight-plugin" class="glossary-content">';
 }
 
 if ($reminder == 2) {

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

@@ -504,7 +504,7 @@
             // First we update the attempt to today
             /* How the expired time is changed into "track_e_exercises" table,
                then the last attempt for this student should be changed too */
-            $sql = "UPDATE $exercise_attempt_table SET
+            $sql = "update $exercise_attempt_table SET
                     tms = '".api_get_utc_datetime()."'
                     WHERE
                         exe_id = '".$exercise_stat_info['exe_id']."' AND

Braces +13 added lines, -5 removed lines

@@ -481,9 +481,15 @@ 
  * If the expired time is major that zero(0) then the expired time is compute on this time.
  */
 if ($time_control) {
-    if ($debug) error_log('7.1. Time control is enabled');
-    if ($debug) error_log('7.2. $current_expired_time_key  '.$current_expired_time_key);
-    if ($debug) error_log('7.3. $_SESSION[expired_time][$current_expired_time_key]  '.$_SESSION['expired_time'][$current_expired_time_key]);
+    if ($debug) {
+        error_log('7.1. Time control is enabled');
+    }
+    if ($debug) {
+        error_log('7.2. $current_expired_time_key  '.$current_expired_time_key);
+    }
+    if ($debug) {
+        error_log('7.3. $_SESSION[expired_time][$current_expired_time_key]  '.$_SESSION['expired_time'][$current_expired_time_key]);
+    }
 
     if (!isset($_SESSION['expired_time'][$current_expired_time_key])) {
         //Timer - Get expired_time for a student
@@ -548,7 +554,8 @@ 
  * The time control feature is enable here - this feature is enable for a jquery plugin called epiclock
  * for more details of how it works see this link : http://eric.garside.name/docs.html?p=epiclock
  */
-if ($time_control) { //Sends the exercise form when the expired time is finished
+if ($time_control) {
+//Sends the exercise form when the expired time is finished
     $htmlHeadXtra[] = $objExercise->showTimeControlJS($time_left);
 }
 
@@ -788,7 +795,8 @@ 
 );
 $interbreadcrumb[] = array("url" => "#", "name" => $objExercise->selectTitle(true));
 
-if ($origin != 'learnpath') { //so we are not in learnpath tool
+if ($origin != 'learnpath') {
+//so we are not in learnpath tool
     if (!api_is_allowed_to_session_edit()) {
         Display::addFlash(
             Display::return_message(get_lang('SessionIsReadOnly'), 'warning')

main/exercise/question_pool.php

Braces +6 added lines, -3 removed lines

@@ -675,7 +675,8 @@ 
         $actionIcon2 = "delete";
         // We are in the course, question title can be a link to the question edit page
         $questionTagA = 1;
-    } else { // NOT IN A TEST - NOT IN THE COURSE
+    } else {
+// NOT IN A TEST - NOT IN THE COURSE
         $actionLabel = get_lang('Reuse');
         $actionIcon1 = get_lang('MustBeInATest');
         $actionIcon2 = "";
@@ -828,7 +829,8 @@ 
     $in_questiontype,
     $in_questionname,
     $sessionId
-) {
+)
+{
     $res = $in_questionname;
     $sessionIcon = null;
     if ($in_addA) {
@@ -868,7 +870,8 @@ 
     $in_answerType,
     $in_session_id,
     $in_exercise_id
-) {
+)
+{
     $res = "";
     $getParams = "&selected_course=$in_selected_course&courseCategoryId=$in_courseCategoryId&exerciseId=$in_exercise_id&exerciseLevel=$in_exerciseLevel&answerType=$in_answerType&session_id=$in_session_id";
     switch ($in_action) {

main/inc/lib/fileUpload.lib.php

Doc Comments +6 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
  * Returns the name without extension, used for the title
  *
  * @param string $name
- * @return name without the extension
+ * @return string without the extension
  */
 function get_document_title($name)
 {
@@ -67,7 +67,7 @@ 
  * This function checks if the upload succeeded
  *
  * @param array $uploaded_file ($_FILES)
- * @return true if upload succeeded
+ * @return boolean if upload succeeded
  */
 function process_uploaded_file($uploaded_file, $show_output = true)
 {
@@ -1227,7 +1227,7 @@ 
  * @param int $session_id Session ID, if any
  * @param int $userId creator id
  *
- * @return int id if inserted document
+ * @return string|false id if inserted document
  */
 function add_document(
     $_course,
@@ -1368,7 +1368,7 @@ 
  *
  * @author	Olivier Cauberghe <olivier.cauberghe@ugent.be>
  * @param	path+filename eg: /main/document/document.php
- * @return	The directory depth
+ * @return	integer directory depth
  */
 function get_levels($filename)
 {
@@ -1386,6 +1386,8 @@ 
  * @author	Olivier Cauberghe <olivier.cauberghe@ugent.be>
  * @param	path,filename
  * action:	Adds an entry to the document table with the default settings.
+ * @param string $upload_path
+ * @param string $filename
  */
 function set_default_settings($upload_path, $filename, $filetype = 'file')
 {
@@ -1844,7 +1846,6 @@ 
  * @param int $groupId group.id
  * @param bool $output
  * @param array $parent
- * @param string $uploadPath
  *
  */
 function add_all_documents_in_folder_to_database(

Braces +19 added lines, -9 removed lines

@@ -223,7 +223,8 @@ 
     $comment = null,
     $sessionId = null,
     $treat_spaces_as_hyphens = true
-) {
+)
+{
     if (!$userId) {
         return false;
     }
@@ -985,7 +986,8 @@ 
             - add it to the database
             - parse & change relative html links
         */
-        if (PHP_OS == 'Linux' && !get_cfg_var('safe_mode') && false) { // *** UGent, changed by OC ***
+        if (PHP_OS == 'Linux' && !get_cfg_var('safe_mode') && false) {
+// *** UGent, changed by OC ***
             // Shell Method - if this is possible, it gains some speed
             exec("unzip -d \"".$base_work_dir.$upload_path."/\"".$uploaded_file['name']." ".$uploaded_file['tmp_name']);
         } else {
@@ -1004,7 +1006,9 @@ 
                     if ($file != '.' && $file != '..') {
 
                         $filetype = 'file';
-                        if (is_dir($base_work_dir.$upload_path.'/'.$file)) $filetype = 'folder';
+                        if (is_dir($base_work_dir.$upload_path.'/'.$file)) {
+                            $filetype = 'folder';
+                        }
 
                         $safe_file = api_replace_dangerous_char($file);
                         @rename($base_work_dir.$upload_path.'/'.$file, $base_work_dir.$upload_path.'/'.$safe_file);
@@ -1056,7 +1060,8 @@ 
     $groupId = 0,
     $output = true,
     $onlyUploadFile = false
-) {
+)
+{
     $zip = new PclZip($uploaded_file['tmp_name']);
 
     // Check the zip content (real size and file extension)
@@ -1170,7 +1175,8 @@ 
         return 1; // Authorize directories
     }
     $blacklist = api_get_setting('upload_extensions_list_type');
-    if ($blacklist != 'whitelist') { // if = blacklist
+    if ($blacklist != 'whitelist') {
+// if = blacklist
         $extensions = explode(';', strtolower(api_get_setting('upload_extensions_blacklist')));
 
         $skip = api_get_setting('upload_extensions_skip');
@@ -1241,7 +1247,8 @@ 
     $group_id = null,
     $session_id = 0,
     $userId = 0
-) {
+)
+{
     $session_id = empty($session_id) ? api_get_session_id() : $session_id;
     $userId = empty($userId) ? api_get_user_id() : $userId;
 
@@ -1509,7 +1516,8 @@ 
     $title = null,
     $visibility = null,
     $generateNewNameIfExists = false
-) {
+)
+{
     $course_id = $_course['real_id'];
     $session_id = intval($session_id);
 
@@ -1683,7 +1691,8 @@ 
     $to_group_id,
     $to_user_id,
     $max_filled_space
-) {
+)
+{
     $number_of_uploaded_images = count($uploaded_file_collection['name']);
     $new_file_list = array();
     for ($i = 0; $i < $number_of_uploaded_images; $i++) {
@@ -1856,7 +1865,8 @@ 
     $groupId = 0,
     $output = false,
     $parent = array()
-) {
+)
+{
     if (empty($userInfo) || empty($courseInfo)) {
 
         return false;

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

@@ -1261,7 +1261,7 @@ 
     ];
     $documentId = Database::insert($table_document, $params);
     if ($documentId) {
-        $sql = "UPDATE $table_document SET id = iid WHERE iid = $documentId";
+        $sql = "update $table_document SET id = iid WHERE iid = $documentId";
         Database::query($sql);
 
         if ($save_visibility) {
@@ -1299,7 +1299,7 @@ 
     $readonly = intval($readonly);
     $course_id = $_course['real_id'];
 
-    $sql = "UPDATE $document_table SET
+    $sql = "update $document_table SET
             size = '$filesize',
             readonly = '$readonly'
 			WHERE c_id = $course_id AND id = $documentId";
@@ -1351,7 +1351,7 @@ 
             $folder_id = DocumentManager::get_document_id($_course, $newpath);
 
             if ($folder_id) {
-                $sql = "UPDATE $table SET
+                $sql = "update $table SET
 				        lastedit_date = '$time',
 				        lastedit_type = 'DocumentInFolderUpdated', 
 				        lastedit_user_id='$user_id'
@@ -1419,7 +1419,7 @@ 
     $result = Database::query($query);
     $row = Database::fetch_array($result);
     if ($row['bestaat'] > 0) {
-        $query = "UPDATE $dbTable SET
+        $query = "update $dbTable SET
 		            path='$upload_path/$filename',
 		            visibility='$default_visibility',
 		            filetype='$filetype'

main/webservices/registration.soap.php

Doc Comments +3 added lines

@@ -2617,6 +2617,9 @@
     )
 );
 
+/**
+ * @param string $type
+ */
 function WSHelperActionOnUsers($params, $type)
 {
     if (!WSHelperVerifyKey($params)) {

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: ');
+                    }
                 }
             }
         }

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

@@ -576,7 +576,7 @@ 
 
     // 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;
     }
 
@@ -808,11 +808,11 @@ 
         if (!empty($_configuration['password_encryption'])) {
             if ($_configuration['password_encryption'] === $encrypt_method) {
                 if ($encrypt_method == 'md5' && !preg_match('/^[A-Fa-f0-9]{32}$/', $password)) {
-                    $msg = "Encryption $encrypt_method is invalid";
+                    $msg = "encryption $encrypt_method is invalid";
                     $results[] = $msg;
                     continue;
                 } else if ($encrypt_method == 'sha1' && !preg_match('/^[A-Fa-f0-9]{40}$/', $password)) {
-                    $msg = "Encryption $encrypt_method is invalid";
+                    $msg = "encryption $encrypt_method is invalid";
                     $results[] = $msg;
                     continue;
                 }
@@ -867,7 +867,7 @@ 
             $r_check_user = Database::fetch_row($resu);
             $count_check_user = Database::num_rows($resu);
             if ($count_check_user > 0) {
-                $sql = "UPDATE $table_user SET
+                $sql = "update $table_user SET
                         lastname='".Database::escape_string($lastName)."',
                         firstname='".Database::escape_string($firstName)."',
                         username='".Database::escape_string($loginName)."',";
@@ -947,7 +947,7 @@ 
             //echo "id returned";
             $return = Database::insert_id();
 
-            $sql = "UPDATE $table_user SET user_id = id WHERE id = $return";
+            $sql = "update $table_user SET user_id = id WHERE id = $return";
             Database::query($sql);
 
             if (api_is_multiple_url_enabled()) {
@@ -1279,12 +1279,12 @@ 
     if (!empty($_configuration['password_encryption'])) {
         if ($_configuration['password_encryption'] === $encrypt_method) {
             if ($encrypt_method == 'md5' && !preg_match('/^[A-Fa-f0-9]{32}$/', $password)) {
-                $msg = "Encryption $encrypt_method is invalid";
+                $msg = "encryption $encrypt_method is invalid";
                 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";
+                $msg = "encryption $encrypt_method is invalid";
                 if ($debug) error_log($msg);
                 return $msg;
             }
@@ -1329,7 +1329,7 @@ 
         $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');
-            $sql = "UPDATE $table_user SET
+            $sql = "update $table_user SET
                     lastname='".Database::escape_string($lastName)."',
                     firstname='".Database::escape_string($firstName)."',
                     username='".Database::escape_string($loginName)."',";
@@ -1384,7 +1384,7 @@ 
 
     // 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;
     }
 
@@ -1414,7 +1414,7 @@ 
     Database::query($sql);
     $return = Database::insert_id();
     if ($return) {
-        $sql = "UPDATE $table_user SET user_id = id WHERE id = $return";
+        $sql = "update $table_user SET user_id = id WHERE id = $return";
         Database::query($sql);
 
         $url_id = api_get_current_access_url_id();
@@ -2269,11 +2269,11 @@ 
             $encrypt_method = $user_param['encrypt_method'];
             if ($_configuration['password_encryption'] === $encrypt_method) {
                 if ($encrypt_method == 'md5' && !preg_match('/^[A-Fa-f0-9]{32}$/', $password)) {
-                    $msg = "Encryption $encrypt_method is invalid";
+                    $msg = "encryption $encrypt_method is invalid";
                     $results[] = $msg;
                     continue;
                 } elseif ($encrypt_method == 'sha1' && !preg_match('/^[A-Fa-f0-9]{40}$/', $password)) {
-                    $msg = "Encryption $encrypt_method is invalid";
+                    $msg = "encryption $encrypt_method is invalid";
                     $results[] = $msg;
                     continue;
                 }
@@ -2322,7 +2322,7 @@ 
             continue; // username already exits
         }
 
-        $sql = "UPDATE $table_user SET ";
+        $sql = "update $table_user SET ";
         if (!empty($lastname)) {
             $sql .= " lastname='".Database::escape_string($lastname)."', ";
         }
@@ -2473,10 +2473,10 @@ 
         $encrypt_method = $params['encrypt_method'];
         if ($_configuration['password_encryption'] === $encrypt_method) {
             if ($encrypt_method == 'md5' && !preg_match('/^[A-Fa-f0-9]{32}$/', $password)) {
-                $msg = "Encryption $encrypt_method is invalid";
+                $msg = "encryption $encrypt_method is invalid";
                 return $msg;
             } else if ($encrypt_method == 'sha1' && !preg_match('/^[A-Fa-f0-9]{40}$/', $password)) {
-                $msg = "Encryption $encrypt_method is invalid";
+                $msg = "encryption $encrypt_method is invalid";
                 return $msg;
             }
         } else {
@@ -2524,7 +2524,7 @@ 
         return 0;
     }
     // Edit lastname and firstname only if not empty
-    $sql = "UPDATE $table_user SET ";
+    $sql = "update $table_user SET ";
     if (!empty($lastname)) {
         $sql .= " lastname='".Database::escape_string($lastname)."', ";
     }
@@ -2901,7 +2901,7 @@ 
 
         if (!empty($courseInfo)) {
             if ($courseInfo['visibility'] != 0) {
-                $sql = "UPDATE $table_course SET
+                $sql = "update $table_course SET
                             course_language='".Database::escape_string($course_language)."',
                             title='".Database::escape_string($title)."',
                             category_code='".Database::escape_string($category_code)."',
@@ -3136,7 +3136,7 @@ 
 
         if (!empty($courseInfo)) {
             if ($courseInfo['visibility'] != 0) {
-                $sql = "UPDATE $table_course SET
+                $sql = "update $table_course SET
                             course_language='".Database::escape_string($course_language)."',
                             title='".Database::escape_string($title)."',
                             category_code='".Database::escape_string($category_code)."',
@@ -3388,7 +3388,7 @@ 
             $visual_code = CourseManager::generate_course_code(substr($title, 0, $maxlength));
         }
         $tutor_name = $tutor_name[0];
-        $sql = "UPDATE $course_table SET
+        $sql = "update $course_table SET
                     course_language='".Database::escape_string($course_language)."',
                     title='".Database::escape_string($title)."',
                     category_code='".Database::escape_string($category_code)."',
@@ -3820,7 +3820,7 @@ 
         }
 
         $courseId = $courseInfo['real_id'];
-        $sql = "UPDATE $table_course SET visibility = '0' WHERE id = '$courseId'";
+        $sql = "update $table_course SET visibility = '0' WHERE id = '$courseId'";
         $return = Database::query($sql);
         $results[] = $return;
     }
@@ -4670,7 +4670,7 @@ 
 
     if (empty($user_data)) {
         // If user was not found, there was a problem
-        $result = "User $user_id does not exist";
+        $result = "user $user_id does not exist";
         if ($debug) error_log($result);
         return $result;
     }
@@ -4678,7 +4678,7 @@ 
         $course_data = api_get_course_info($course_code);
         if (empty($course_data)) {
             // Course was not found
-            $result = "Course $course_code does not exist in the platform ";
+            $result = "course $course_code does not exist in the platform ";
             if ($debug) error_log($result);
         } else {
             if ($debug) error_log('Try to register: user_id= '.$user_id.' to course: '.$course_data['code']);
@@ -5047,8 +5047,8 @@ 
 
     if ($user_id) {
         if ($debug) {
-            error_log("User $original_user_id_value, $original_user_id_name found");
-            error_log("Course $original_course_id_value, $original_course_id_name found");
+            error_log("user $original_user_id_value, $original_user_id_name found");
+            error_log("course $original_course_id_value, $original_course_id_name found");
         }
 
         $courseInfo = CourseManager::getCourseInfoFromOriginalId(
@@ -5065,7 +5065,7 @@ 
             }
         } else {
             if ($debug) {
-                error_log("Course $courseCode found");
+                error_log("course $courseCode found");
             }
             CourseManager::unsubscribe_user($user_id, $courseCode, 0);
             $result['result'] = 1;
@@ -5335,7 +5335,7 @@ 
     $result = 0;
 
     if (empty($user_data)) {
-        $result = "User {$user_id} does not exist";
+        $result = "user {$user_id} does not exist";
         if ($debug) {
             error_log($result);
         }
@@ -5344,7 +5344,7 @@ 
     if (!empty($session_id) && is_numeric($session_id)) {
         $session_data = api_get_session_info($session_id);
         if (empty($session_data)) {
-            $result = "Session {$session_id} does not exist.";
+            $result = "session {$session_id} does not exist.";
             if ($debug) {
                 error_log($result);
             }
@@ -5885,7 +5885,7 @@ 
         }
 
         // Update number of users in the session.
-        $update_sql = "UPDATE $tbl_session SET nbr_courses= $nbr_courses WHERE id='$id_session' ";
+        $update_sql = "update $tbl_session SET nbr_courses= $nbr_courses WHERE id='$id_session' ";
         Database::query($update_sql);
 
         $results[] = 1;

main/inc/lib/auth.lib.php

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

@@ -182,7 +182,7 @@ 
 
         $TABLECOURSUSER = Database::get_main_table(TABLE_MAIN_COURSE_USER);
         $max_sort_value = api_max_sort_value($newcategory, $current_user);
-        $sql = "UPDATE $TABLECOURSUSER SET
+        $sql = "update $TABLECOURSUSER SET
                     user_course_cat='".$newcategory."',
                     sort='" . ($max_sort_value + 1)."'
                 WHERE
@@ -245,7 +245,7 @@ 
             $targetCourseInfo = api_get_course_info($target_course['code']);
             $targetCourseId = $targetCourseInfo['real_id'];
 
-            $sql = "UPDATE $table
+            $sql = "update $table
                     SET sort='".$target_course['sort']."'
                     WHERE
                         c_id = '" . $courseId."' AND
@@ -254,7 +254,7 @@ 
 
             $result1 = Database::query($sql);
 
-            $sql = "UPDATE $table SET sort='".$source_course['sort']."'
+            $sql = "update $table SET sort='".$source_course['sort']."'
                     WHERE
                         c_id ='" . $targetCourseId."' AND
                         user_id='" . $current_user_id."' AND
@@ -305,11 +305,11 @@ 
         $result = false;
         if (count($target_category) > 0 && count($source_category) > 0) {
             $table = Database::get_main_table(TABLE_USER_COURSE_CATEGORY);
-            $sql = "UPDATE $table SET 
+            $sql = "update $table SET 
                     sort = '".Database::escape_string($target_category['sort'])."'
                     WHERE id='" . intval($source_category['id'])."' AND user_id='".$userId."'";
             $resultFirst = Database::query($sql);
-            $sql = "UPDATE $table SET 
+            $sql = "update $table SET 
                     sort = '".Database::escape_string($source_category['sort'])."'
                     WHERE id='" . intval($target_category['id'])."' AND user_id='".$userId."'";
             $resultSecond = Database::query($sql);
@@ -334,7 +334,7 @@ 
         $category_id = intval($category_id);
         $result = false;
         $tucc = Database::get_main_table(TABLE_USER_COURSE_CATEGORY);
-        $sql = "UPDATE $tucc
+        $sql = "update $tucc
                 SET title='".api_htmlentities($title, ENT_QUOTES, api_get_system_encoding())."'
                 WHERE id='" . $category_id."'";
         $resultQuery = Database::query($sql);
@@ -364,7 +364,7 @@ 
         if (Database::affected_rows($resultQuery)) {
             $result = true;
         }
-        $sql = "UPDATE $TABLECOURSUSER
+        $sql = "update $TABLECOURSUSER
                 SET user_course_cat='0'
                 WHERE
                     user_course_cat='".$category_id."' AND
@@ -723,7 +723,7 @@ 
                 OR (s.access_end_date IS NULL)
                 OR (s.access_start_date IS NULL AND
                 s.access_end_date IS NOT NULL AND s.access_end_date > '$date')
-SQL;
+sql;
         }
         $sql = "SELECT COUNT(*) 
                 FROM $sessionTable s

Doc Comments +7 added lines

@@ -172,6 +172,7 @@ 
      * (moving a course to a different course category)
      * @param  int    $courseId
      * @param  int       Category id
+     * @param integer $newcategory
      * @return bool      True if it success
      */
     public function updateCourseCategory($courseId, $newcategory)
@@ -204,6 +205,9 @@ 
      * @param   string    Direction (up/down)
      * @param   string    Course code
      * @param   int       Category id
+     * @param string $direction
+     * @param string $course2move
+     * @param integer $category
      * @return  bool      True if it success
      */
     public function move_course($direction, $course2move, $category)
@@ -325,6 +329,8 @@ 
      * Updates the user course category in the chamilo_user database
      * @param   string  Category title
      * @param   int     Category id
+     * @param string $title
+     * @param integer $category_id
      * @return  bool    True if it success
      */
     public function store_edit_course_category($title, $category_id)
@@ -517,6 +523,7 @@ 
     /**
      * stores the user course category in the chamilo_user database
      * @param   string  Category title
+     * @param string $category_title
      * @return  bool    True if it success
      */
     public function store_course_category($category_title)

plugin/buycourses/src/process_confirm.php

Braces +14 added lines, -14 removed lines

@@ -11,7 +11,7 @@ 
 

 $saleId = $_SESSION['bc_sale_id'];

 

-if (empty($saleId)) {

+if (empty($saleId)) {
     api_not_allowed(true);

 }

 

@@ -19,14 +19,14 @@ 
 

 $userInfo = api_get_user_info($sale['user_id']);

 

-if (empty($sale)) {

+if (empty($sale)) {
     api_not_allowed(true);

 }

 

 $currency = $plugin->getCurrency($sale['currency_id']);

 $terms = $plugin->getGlobalParameters();

 

-switch ($sale['payment_type']) {

+switch ($sale['payment_type']) {
     case BuyCoursesPlugin::PAYMENT_TYPE_PAYPAL:

         $paypalParams = $plugin->getPaypalParams();

 

@@ -51,7 +51,7 @@ 
             $extra

         );

 

-        if ($expressCheckout["ACK"] !== 'Success') {

+        if ($expressCheckout["ACK"] !== 'Success') {
             $erroMessage = vsprintf(

                 $plugin->get_lang('ErrorOccurred'),

                 [$expressCheckout['L_ERRORCODE0'], $expressCheckout['L_LONGMESSAGE0']]

@@ -69,7 +69,7 @@ 
         $buyingCourse = false;

         $buyingSession = false;

 

-        switch ($sale['product_type']) {

+        switch ($sale['product_type']) {
             case BuyCoursesPlugin::PRODUCT_TYPE_COURSE:

                 $buyingCourse = true;

                 $course = $plugin->getCourseInfo($sale['product_id']);

@@ -91,10 +91,10 @@ 
             FormValidator::LAYOUT_INLINE

         );

 

-        if ($form->validate()) {

+        if ($form->validate()) {
             $formValues = $form->getSubmitValues();

 

-            if (isset($formValues['cancel'])) {

+            if (isset($formValues['cancel'])) {
                 $plugin->cancelSale($sale['id']);

 

                 unset($_SESSION['bc_sale_id']);

@@ -153,9 +153,9 @@ 
 

         $template = new Template();

 

-        if ($buyingCourse) {

+        if ($buyingCourse) {
             $template->assign('course', $course);

-        } elseif ($buyingSession) {

+        } elseif ($buyingSession) {
             $template->assign('session', $session);

         }

 

@@ -183,7 +183,7 @@ 
         $buyingCourse = false;

         $buyingSession = false;

 

-        switch ($sale['product_type']) {

+        switch ($sale['product_type']) {
             case BuyCoursesPlugin::PRODUCT_TYPE_COURSE:

                 $buyingCourse = true;

                 $course = $plugin->getCourseInfo($sale['product_id']);

@@ -203,10 +203,10 @@ 
             FormValidator::LAYOUT_INLINE

         );

 

-        if ($form->validate()) {

+        if ($form->validate()) {
             $formValues = $form->getSubmitValues();

 

-            if (isset($formValues['cancel'])) {

+            if (isset($formValues['cancel'])) {
                 $plugin->cancelSale($sale['id']);

 

                 unset($_SESSION['bc_sale_id']);

@@ -244,9 +244,9 @@ 
 

         $template = new Template();

 

-        if ($buyingCourse) {

+        if ($buyingCourse) {
             $template->assign('course', $course);

-        } elseif ($buyingSession) {

+        } elseif ($buyingSession) {
             $template->assign('session', $session);

         }