Procházet zdrojové kódy

Use current template, don't create new object.

jmontoyaa před 8 roky
rodič
revize
47b7a94f49
2 změnil soubory, kde provedl 11 přidání a 40 odebrání
  1. 10 34
      main/inc/lib/userportal.lib.php
  2. 1 6
      user_portal.php

+ 10 - 34
main/inc/lib/userportal.lib.php

@@ -1123,19 +1123,10 @@ class IndexManager
             $session_categories = UserManager::get_sessions_by_category($user_id, false);
         }
 
-        $html = '';
-        // Showing history title
-        if ($load_history) {
-            $html .= Display::page_subheader(get_lang('HistoryTrainingSession'));
-            if (empty($session_categories)) {
-                $html .= get_lang('YouDoNotHaveAnySessionInItsHistory');
-            }
-        }
-
         $sessionCount = 0;
         $courseCount = 0;
 
-        $template = new Template(null, false, false, false, false, false, false);
+        //$template = new Template(null, false, false, false, false, false, false);
 
         // If we're not in the history view...
         if (!isset($_GET['history'])) {
@@ -1157,21 +1148,21 @@ class IndexManager
             }
 
             if ($specialCourses) {
-                $template->assign('courses', $specialCourses);
+                $this->tpl->assign('courses', $specialCourses);
 
-                $specialCourseList = $template->fetch(
+                $specialCourseList = $this->tpl->fetch(
                     $this->tpl->get_template($coursesWithoutCategoryTemplate)
                 );
             }
 
             if ($courses['in_category'] || $courses['not_category']) {
-                $template->assign('courses', $courses['not_category']);
-                $template->assign('categories', $courses['in_category']);
+                $this->tpl->assign('courses', $courses['not_category']);
+                $this->tpl->assign('categories', $courses['in_category']);
 
-                $listCourse = $template->fetch(
+                $listCourse = $this->tpl->fetch(
                     $this->tpl->get_template($coursesWithCategoryTemplate)
                 );
-                $listCourse .= $template->fetch(
+                $listCourse .= $this->tpl->fetch(
                     $this->tpl->get_template($coursesWithoutCategoryTemplate)
                 );
             }
@@ -1191,7 +1182,6 @@ class IndexManager
         if (is_array($session_categories)) {
             foreach ($session_categories as $session_category) {
                 $session_category_id = $session_category['session_category']['id'];
-
                 // Sessions and courses that are not in a session category
                 if (
                     empty($session_category_id) &&
@@ -1312,7 +1302,6 @@ class IndexManager
                                 $params['points'] = GamificationUtils::getSessionPoints($params['id'], $this->user_id);
                             }
                             $listSession[] = $params;
-
                             $sessionCount++;
                         }
                     }
@@ -1519,7 +1508,7 @@ class IndexManager
     }
 
     /**
-     * UserPortal view for session, return the HTLK of the course list
+     * UserPortal view for session, return the HTML of the course list
      * @param $user_id
      * @return string
      */
@@ -1527,7 +1516,6 @@ class IndexManager
     {
         $sessionCount = 0;
         $courseCount = 0;
-
         $load_history = (isset($_GET['history']) && intval($_GET['history']) == 1) ? true : false;
 
         if ($load_history) {
@@ -1538,16 +1526,6 @@ class IndexManager
             $session_categories = UserManager::get_sessions_by_category($user_id, false);
         }
 
-        $html = '';
-
-        //Showing history title
-        if ($load_history) {
-            $html .= Display::page_subheader(get_lang('HistoryTrainingSession'));
-            if (empty($session_categories)) {
-                $html .= get_lang('YouDoNotHaveAnySessionInItsHistory');
-            }
-        }
-
         $html = '';
         $loadDirs = $this->load_directories_preview;
 
@@ -1562,10 +1540,8 @@ class IndexManager
             $specialCourses = $specialCoursesResult;
 
             if ($specialCourses) {
-                $specialCoursesTemplate = new Template(null, false, false, false, false, false, false);
-                $specialCoursesTemplate->assign('courses', $specialCourses);
-
-                $html = $specialCoursesTemplate->fetch(
+                $this->tpl->assign('courses', $specialCourses);
+                $html = $this->tpl->fetch(
                     $this->tpl->get_template('/user_portal/classic_courses_without_category.tpl')
                 );
             }

+ 1 - 6
user_portal.php

@@ -104,9 +104,6 @@ if ($displayMyCourseViewBySessionLink) {
 
 $controller = new IndexManager(get_lang('MyCourses'));
 
-// Main courses and session list
-//$courseAndSessions = $controller->returnCoursesAndSessions($userId);
-
 // Main courses and session list
 if (isset($_COOKIE['defaultMyCourseView'.$userId]) &&
     $_COOKIE['defaultMyCourseView'.$userId] == IndexManager::VIEW_BY_SESSION && $displayMyCourseViewBySessionLink
@@ -225,9 +222,7 @@ $controller->tpl->assign('course_block', $controller->return_course_block());
 $controller->tpl->assign('navigation_course_links', $controller->return_navigation_links());
 $controller->tpl->assign('search_block', $controller->return_search_block());
 $controller->tpl->assign('classes_block', $controller->return_classes_block());
-//if (api_is_platform_admin() || api_is_drh()) {
 $controller->tpl->assign('skills_block', $controller->return_skills_links());
-//}
 $historyClass = '';
 if (!empty($_GET['history'])) {
     $historyClass = 'courses-history';
@@ -238,4 +233,4 @@ $controller->tpl->display_two_col_template();
 // Deleting the session_id.
 Session::erase('session_id');
 Session::erase('studentview');
-api_remove_in_gradebook('in_gradebook');
+api_remove_in_gradebook();