Browse Source

fix getCategory in catalog courses BT#11017

Alex Aragon 9 years ago
parent
commit
c88c8490fa
2 changed files with 13 additions and 19 deletions
  1. 5 10
      main/inc/lib/course.lib.php
  2. 8 9
      main/template/default/auth/courses_categories.php

+ 5 - 10
main/inc/lib/course.lib.php

@@ -2948,18 +2948,13 @@ class CourseManager
     public static function getListCategory()
     {
         $table_categories = Database::get_main_table(TABLE_MAIN_CATEGORY);
-        $sql = "SELECT * FROM $table_categories ";
-        $result =  Database::query($sql);
-        $count = 0;
-        $list = array();
+        $sql = "SELECT * FROM $table_categories";
+        $result = Database::query($sql);
         $category = array();
-        while($row = Database::fetch_array($result, 'ASSOC')){
-            $category['code'] = $row['code'];
-            $category['name'] = $row['name'];
-            $count++;
-            $list[$count] = $category;
+        while ($row = Database::fetch_array($result, 'ASSOC')) {
+            $category[$row['code']] = $row['name'];
         }
-        return $list;
+        return $category;
     }
 
     /**

+ 8 - 9
main/template/default/auth/courses_categories.php

@@ -222,7 +222,7 @@ if ($showCourses && $action != 'display_sessions') {
             $html .= '<div class="col-xs-6 col-sm-6 col-md-3"><div class="items">';
 
             // display thumbnail
-            $html .= return_thumbnail($course, $icon_title, $listCategory);
+            $html .= returnThumbnail($course, $listCategory[$course['category']]);
 
             // display course title and button bloc
             $html .= '<div class="description">';
@@ -288,10 +288,11 @@ echo $cataloguePagination;
 
 /**
  * Display the course catalog image of a course
- * @param $course
- * @param $icon_title
+ * @param array $course
+ * @param string $categoryTitle
+ * @return string HTML string
  */
-function return_thumbnail($course, $icon_title, $list = array())
+function returnThumbnail($course, $categoryTitle=null)
 {
     $html = '';
     $title = cut($course['title'], 70);
@@ -307,11 +308,9 @@ function return_thumbnail($course, $icon_title, $list = array())
     
         $html .= '<div class="image">';
         $html .= '<img class="img-responsive" src="'.$course_medium_image.'" alt="'.api_htmlentities($title).'"/>';
-        foreach ($list as $cat) {
-                if($course['category'] == $cat['code']){
-                    $html .= '<span class="category">'. $cat['name'].'</span>';
-                    $html .= '<div class="cribbon"></div>';
-                }
+        if (!empty($categoryTitle)) {
+            $html .= '<span class="category">'. $categoryTitle.'</span>';
+            $html .= '<div class="cribbon"></div>';
         }
         $teachers = CourseManager::getTeacheCourseCode($course['code']);
         $html .= '<div class="black_shadow">';