Minor - format code.

main/inc/lib/course.lib.php

@@ -3133,6 +3133,9 @@ class CourseManager
     /**
      * Builds the course block in user_portal.php
      * @todo use Twig
+     *
+     * @param array $params
+     * @return string
      */
     public static function course_item_html_no_icon($params)
     {
@@ -3167,6 +3170,11 @@ class CourseManager
         return $html;
     }
 
+    /**
+     * @param $params
+     * @param bool|false $is_sub_content
+     * @return string
+     */
     public static function session_items_html($params, $is_sub_content = false)
     {
         $html = '';
@@ -3192,6 +3200,9 @@ class CourseManager
     /**
      * Builds the course block in user_portal.php
      * @todo use Twig
+     * @param array $params
+     * @param bool|false $is_sub_content
+     * @return string
      */
     public static function course_item_html($params, $is_sub_content = false)
     {
@@ -3216,9 +3227,9 @@ class CourseManager
         }
 
         $html .= '</div>';
-        $notifications = isset($params['notifications']) ? $params['notifications'] : null;
+        $notifications = isset($params['notifications']) ? $params['notifications'] : '';
-        $param_class = isset($params['class']) ? $params['class'] : null;
+        $param_class = isset($params['class']) ? $params['class'] : '';
-        $params['right_actions'] = isset($params['right_actions']) ? $params['right_actions'] : null;
+        $params['right_actions'] = isset($params['right_actions']) ? $params['right_actions'] : '';
 
         $html .= '<div class="col-md-10 ' . $param_class . '">';
         $html .= '<div class="pull-right">' . $params['right_actions'] . '</div>';
@@ -3231,12 +3242,14 @@ class CourseManager
             $html .= '<div class="subtitle-session">' . $params['subtitle'] . '</div>';
         }
         if (!empty($params['teachers'])) {
-            $html .= '<h5>' . Display::return_icon('teacher.png', get_lang('Teacher'), array(),
+            $html .= '<h5>' .
-                    ICON_SIZE_TINY) . $params['teachers'] . '</h5>';
+                    Display::return_icon('teacher.png', get_lang('Teacher'), array(), ICON_SIZE_TINY) .
+                $params['teachers'] . '</h5>';
         }
         if (!empty($params['coaches'])) {
-            $html .= '<h5>' . Display::return_icon('teacher.png', get_lang('Coach'), array(),
+            $html .= '<h5>' .
-                    ICON_SIZE_TINY) . $params['coaches'] . '</h5>';
+                Display::return_icon('teacher.png', get_lang('Coach'), array(), ICON_SIZE_TINY) .
+                $params['coaches'] . '</h5>';
         }
 
         $html .= '</div>';
@@ -3520,7 +3533,6 @@ class CourseManager
         $sql .= " ORDER BY course_rel_user.user_course_cat, course_rel_user.sort ASC";
 
         $result = Database::query($sql);
-        $status_icon = '';
         $html = '';
 
         $course_list = array();
@@ -3605,8 +3617,6 @@ class CourseManager
                 }
             }
 
-            $course_title = $course_info['title'];
-
             $course_title_url = '';
             if ($course_info['visibility'] != COURSE_VISIBILITY_CLOSED || $course['status'] == COURSEMANAGER) {
                 $course_title_url = api_get_path(WEB_COURSE_PATH) . $course_info['path'] . '/index.php?id_session=0';
@@ -3638,11 +3648,11 @@ class CourseManager
                 $params['notifications'] = $show_notification;
             }
 
-            $isSubcontent = true;
+            $isSubContent = true;
             if (empty($user_category_id)) {
-                $isSubcontent = false;
+                $isSubContent = false;
             }
-            $html .= self::course_item_html($params, $isSubcontent);
+            $html .= self::course_item_html($params, $isSubContent);
         }
 
         return [

main/inc/lib/userportal.lib.php

@@ -941,7 +941,11 @@ class IndexManager
         $my_account_content .= '</ul>';
 
         if (!empty($my_account_content)) {
-            $html =  self::show_right_block(get_lang('Courses'), $my_account_content, 'course_block');
+            $html = self::show_right_block(
+                get_lang('Courses'),
+                $my_account_content,
+                'course_block'
+            );
         }
         return $html;
     }
@@ -967,9 +971,7 @@ class IndexManager
         }
 
         $html = '';
-
         // Showing history title
-
         if ($load_history) {
             $html .= Display::page_subheader(get_lang('HistoryTrainingSession'));
             if (empty($session_categories)) {
@@ -1294,7 +1296,7 @@ class IndexManager
         }
 
         return [
-            'html' => $sessions_with_category . $sessions_with_no_category . $courses_html . $special_courses,
+            'html' => $sessions_with_category.$sessions_with_no_category.$courses_html.$special_courses,
             'session_count' => $sessionCount,
             'course_count' => $courseCount
         ];

user_portal.php

@@ -94,9 +94,9 @@ $nameTools = get_lang('MyCourses');
     Include the HTTP, HTML headers plus the top banner.
 */
 if ($load_dirs) {
-	$url 			= api_get_path(WEB_AJAX_PATH).'document.ajax.php?a=document_preview';
+	$url = api_get_path(WEB_AJAX_PATH).'document.ajax.php?a=document_preview';
-	$folder_icon 	= api_get_path(WEB_IMG_PATH).'icons/22/folder.png';
+	$folder_icon = api_get_path(WEB_IMG_PATH).'icons/22/folder.png';
-	$close_icon 	= api_get_path(WEB_IMG_PATH).'loading1.gif';
+	$close_icon = api_get_path(WEB_IMG_PATH).'loading1.gif';
 
 	$htmlHeadXtra[] =  '<script>
 	$(document).ready(function() {
@@ -121,7 +121,6 @@ if ($load_dirs) {
 	            success: function(return_value) {
 	            	image.attr("src", "'.$folder_icon.'");
 	            	$("#document_result_" +course_id+"_" + session_id).html(return_value);
-
 	            }
 	        });
 
@@ -130,8 +129,6 @@ if ($load_dirs) {
 	</script>';
 }
 
-
-
 //Show the chamilo mascot
 if (empty($courseAndSessions['html']) && !isset($_GET['history'])) {
 	$controller->tpl->assign('welcome_to_course_block', $controller->return_welcome_to_course_block());