Minor - format code

main/course_home/course_home.php

@@ -171,13 +171,13 @@ $htmlHeadXtra[] = '<script type="text/javascript">
                     if (info.message == "is_active") {
                         message = "'.get_lang('ToolIsNowVisible', '').'";
                         $("#" + tool_id)
-                        .attr("alt", "' . get_lang('Activate', '') . '")
+                        .attr("alt", "'.get_lang('Activate', '').'")
-                        .attr("title", "' . get_lang('Activate', '') . '");
+                        .attr("title", "'.get_lang('Activate', '').'");
                     } else {
                         message = "'.get_lang('ToolIsNowHidden', '').'";
                         $("#" + tool_id)
-                        .attr("alt", "' . get_lang('Deactivate', '') . '")
+                        .attr("alt", "'.get_lang('Deactivate', '').'")
-                        .attr("title", "' . get_lang('Deactivate', '') . '");
+                        .attr("title", "'.get_lang('Deactivate', '').'");
                     }
                     $(".normal-message").hide();
                     $("#id_confirmation_message").html(message);

main/course_info/delete_course.php

@@ -32,30 +32,30 @@ if (isset($_GET['delete']) && $_GET['delete'] === 'yes' && $_GET['course_code']
         // DELETE CONFIRMATION MESSAGE
         Session::erase('_cid');
         Session::erase('_real_cid');
-        $message = '<h2>' . get_lang('Course') . ' : ' . $current_course_name . ' (' . $current_course_code . ') </h2>';
+        $message = '<h2>'.get_lang('Course').' : '.$current_course_name.' ('.$current_course_code.') </h2>';
         $message .= get_lang('HasDel');
-        $message .= '<br /><br /><a href="../../index.php">' . get_lang('BackHome') . '</a>';
+        $message .= '<br /><br /><a href="../../index.php">'.get_lang('BackHome').'</a>';
     } else {
         /* message if code course is incorrect */
-        $message = '<h2>' . get_lang('Course') . ' : ' . $current_course_name . ' (' . $current_course_code . ') </h2>';
+        $message = '<h2>'.get_lang('Course').' : '.$current_course_name.' ('.$current_course_code.') </h2>';
         $message .= '<p>'.get_lang('CourseRegistrationCodeIncorrect').'</p>';
         $message .= '<p><a class="btn btn-primary" href="'
             .api_get_path(WEB_CODE_PATH)
             .'course_info/delete_course.php?'
             .api_get_cidreq()
             .'">'.get_lang('BackToPreviousPage').'</a>';
-        $message .= '<br /><br /><a href="../../index.php">' . get_lang('BackHome') . '</a>';
+        $message .= '<br /><br /><a href="../../index.php">'.get_lang('BackHome').'</a>';
         $type_info_message = 'error';
     }
 } else {
-    $message = '<h3>' . get_lang('Course') . ' : ' . $current_course_name . ' (' . $current_course_code . ') </h3>';
+    $message = '<h3>'.get_lang('Course').' : '.$current_course_name.' ('.$current_course_code.') </h3>';
-    $message .= '<p>' . get_lang('ByDel') . '</p>';
+    $message .= '<p>'.get_lang('ByDel').'</p>';
     $message .= '<p><span class="form_required">*</span>'
         .get_lang('CourseCodeConfirmation')
         .'&nbsp;<input type="text" name="course_code" id="course_code"></p>';
 
     $message .= '<p>';
-    $message .= '<button class="btn btn-danger delete-course">' . get_lang('ValidateChanges') . '</button>';
+    $message .= '<button class="btn btn-danger delete-course">'.get_lang('ValidateChanges').'</button>';
     $message .= '&nbsp;';
     $message .= '<a class="btn btn-primary"href="'
         .api_get_path(WEB_CODE_PATH)
@@ -69,7 +69,7 @@ if (isset($_GET['delete']) && $_GET['delete'] === 'yes' && $_GET['course_code']
         'name' => get_lang('Maintenance'),
     ];
 }
-    $htmlHeadXtra[]  ='<script>
+    $htmlHeadXtra[] = '<script>
 $(function(){
 	/* Asking by course code to confirm recycling*/
 	$(".delete-course").on("click",function(){

main/coursecopy/recycle_course.php

@@ -16,7 +16,6 @@ require_once __DIR__.'/../inc/global.inc.php';
 $current_course_tool = TOOL_COURSE_MAINTENANCE;
 api_protect_course_script(true);
 
-
 $_course = api_get_course_info();
 $current_course_code = $_course['official_code'];
 
@@ -60,7 +59,7 @@ if (Security::check_token('post') && (
     }
     $recycle_type = '';
     $fullDelete = 0;
-    $courseCodeConfirmation ='';
+    $courseCodeConfirmation = '';
     if (isset($_POST['course_code_confirmation'])) {
         $courseCodeConfirmation = $_POST['course_code_confirmation'];
     }
@@ -78,13 +77,12 @@ if (Security::check_token('post') && (
             $cr->recycle($recycle_type);
             echo Display::return_message(get_lang('RecycleFinished'), 'confirm');
         } else {
-            $messageFailCourseCode = '<p>' . get_lang('CourseRegistrationCodeIncorrect') . '</p>';
+            $messageFailCourseCode = '<p>'.get_lang('CourseRegistrationCodeIncorrect').'</p>';
             $messageFailCourseCode .= '<p><a class="btn btn-primary" href="'.api_get_self().'?'.api_get_cidreq().'">'.
                 get_lang('BackToPreviousPage').
                 '</a></p>';
             echo Display::return_message($messageFailCourseCode, 'error', false);
         }
-
     } elseif ($recycle_type == 'select_items') {
         $cr->recycle($recycle_type);
         echo Display::return_message(get_lang('RecycleFinished'), 'confirm');

main/exercise/exercise_result.php

@@ -106,7 +106,7 @@ if (api_is_course_admin() && !in_array($origin, ['learnpath', 'embeddable'])) {
             .Display::url(
                 Display::return_icon('edit.png', get_lang('ModifyExercise'), [], 32),
                 'exercise_admin.php?'.api_get_cidreq().'&modifyExercise=yes&exerciseId='.$objExercise->id
-            )
+            ),
         ]
     );
 }

main/inc/lib/document.lib.php

@@ -591,7 +591,7 @@ class DocumentManager
         }
         $where = '';
         if ($search != true) {
-            $where .= "docs.path NOT LIKE '" . Database::escape_string($path . $addedSlash . '%/%') . "' AND";
+            $where .= "docs.path NOT LIKE '".Database::escape_string($path.$addedSlash.'%/%')."' AND";
         }
         $sql = "SELECT
                     docs.id,

main/inc/lib/events.lib.php

@@ -1,7 +1,7 @@
 <?php
+
 /* See license terms in /license.txt */
 
-//use Chamilo\UserBundle\Entity\User;
 use Chamilo\CoreBundle\Component\Utils\ChamiloApi;
 use ChamiloSession as Session;
 
@@ -2237,11 +2237,11 @@ class Event
      * updating the current one to be just the considered_working_time and
      * end at the same second as the user connected to the course.
      *
-     * @param int    $courseId    The course in which to add the time
+     * @param int    $courseId     The course in which to add the time
-     * @param int    $userId      The user for whom to add the time
+     * @param int    $userId       The user for whom to add the time
-     * @param int    $sessionId   The session in which to add the time (if any)
+     * @param int    $sessionId    The session in which to add the time (if any)
-     * @param string $virtualTime The amount of time to be added,
+     * @param string $virtualTime  The amount of time to be added,
-     *                            in a hh:mm:ss format. If int, we consider it is expressed in hours.
+     *                             in a hh:mm:ss format. If int, we consider it is expressed in hours.
      * @param int    $resultWorkId Student publication id result
      *
      * @return true on successful insertion, false otherwise

main/inc/lib/formvalidator/Element/SelectAjax.php

@@ -76,7 +76,7 @@ class SelectAjax extends HTML_QuickForm_select
         $max = !empty($max) ? "maximumSelectionLength: $max, " : '';
 
         // wait XX milliseconds before triggering the request
-        $delay = (int)$this->getAttribute('delay');
+        $delay = (int) $this->getAttribute('delay');
         $delay = (0 !== $delay) ? "delay: $delay, " : '';
 
         $html = <<<JS

main/inc/lib/message.lib.php

@@ -1537,10 +1537,10 @@ class MessageManager
                 $message_content .= '<a href="inbox.php?action=deleteone&id='.$messageId.'&'.$social_link.'" >'.
                     Display::return_icon('delete.png', get_lang('DeleteMessage')).'</a>&nbsp;';
                 if ($idPrevMessage != 0) {
-                    $message_content .= '<a title="' . get_lang('PrevMessage') . '" href="view_message.php?type=' . $type . '&id=' . $idPrevMessage . '" ">' . Display::return_icon('icons/22/back.png', get_lang('ScormPrevious')) . '</a> &nbsp';
+                    $message_content .= '<a title="'.get_lang('PrevMessage').'" href="view_message.php?type='.$type.'&id='.$idPrevMessage.'" ">'.Display::return_icon('icons/22/back.png', get_lang('ScormPrevious')).'</a> &nbsp';
                 }
                 if ($idNextMessage != 0) {
-                    $message_content .= '<a title="' . get_lang('NextMessage') . '" href="view_message.php?type=' . $type . '&id=' . $idNextMessage . '">' . Display::return_icon('icons/22/move.png', get_lang('ScormNext')) . '</a> &nbsp';
+                    $message_content .= '<a title="'.get_lang('NextMessage').'" href="view_message.php?type='.$type.'&id='.$idNextMessage.'">'.Display::return_icon('icons/22/move.png', get_lang('ScormNext')).'</a> &nbsp';
                 }
                 break;
         }

main/inc/lib/social.lib.php

@@ -54,9 +54,9 @@ class SocialManager extends UserManager
     /**
      * Get the kind of relation between contacts.
      *
-     * @param int $user_id user id
+     * @param int  $user_id     user id
-     * @param int $user_friend user friend id
+     * @param int  $user_friend user friend id
-     * @param boolean $includeRH include the RH relationship
+     * @param bool $includeRH   include the RH relationship
      *
      * @return int
      *
@@ -112,15 +112,14 @@ class SocialManager extends UserManager
                 foreach ($targetUserCoursesList as $course) {
                     $teachersList = CourseManager::get_teacher_list_from_course_code($course['code']);
                     foreach ($teachersList as $teacher) {
-                         if ($currentUserId == $teacher['user_id']) {
+                        if ($currentUserId == $teacher['user_id']) {
-                             return USER_RELATION_TYPE_GOODFRIEND;
+                            return USER_RELATION_TYPE_GOODFRIEND;
-                         }
+                        }
                     }
                 }
             } else {
                 return USER_UNKNOWN;
             }
-
         }
     }
 

main/social/vcard_export.php

@@ -25,7 +25,7 @@ $userInfo = api_get_user_info($userId, true, false, true);
 
 /* Get the relationship between current user and vCard user */
 $currentUserId = api_get_user_id();
-$hasRelation =  SocialManager::get_relation_between_contacts(
+$hasRelation = SocialManager::get_relation_between_contacts(
     $currentUserId,
     $userId,
     true
@@ -38,7 +38,7 @@ if ($hasRelation == 0) {
 if (empty($userInfo)) {
     api_not_allowed(true);
 }
-if (api_get_user_id() != $userId && !SocialManager::get_relation_between_contacts(api_get_user_id(),$userId)) {
+if (api_get_user_id() != $userId && !SocialManager::get_relation_between_contacts(api_get_user_id(), $userId)) {
     api_not_allowed(true);
 }
 // Pre-Loaded User Info