Browse Source

minor: avoid trailing separator in course list on homepage

Laurent Opprecht 12 years ago
parent
commit
ab21c1b338
1 changed files with 9 additions and 7 deletions
  1. 9 7
      main/inc/lib/userportal.lib.php

+ 9 - 7
main/inc/lib/userportal.lib.php

@@ -543,21 +543,23 @@ class IndexManager {
 						|| api_is_platform_admin()) {
 	                    $courses_list_string .= '</a><br />';
 				}
+                        $course_details = array();
 						if (api_get_setting('display_coursecode_in_courselist') == 'true') {
-						$courses_list_string .= ' '.$course['visual_code'];
-				}
-						if (api_get_setting('display_coursecode_in_courselist') == 'true' && api_get_setting('display_teacher_in_courselist') == 'true') {
-	                    $courses_list_string .= ' - ';
+						$course_details[] = $course['visual_code'];
 				}
+//						if (api_get_setting('display_coursecode_in_courselist') == 'true' && api_get_setting('display_teacher_in_courselist') == 'true') {
+//	                    $courses_list_string .= ' - ';
+//				}
 						if (api_get_setting('display_teacher_in_courselist') == 'true') {
-						$courses_list_string .= $course['tutor_name'];
+						$course_details[] = $course['tutor_name'];
 	                }
 	                if (api_get_setting('show_different_course_language') == 'true' && $course['course_language'] != api_get_setting('platformLanguage')) {
-						$courses_list_string .= ' - '.$course['course_language'];
+						$course_details[] = $course['course_language'];
 	                }
 						if (api_get_setting('show_different_course_language') == 'true' && $course['course_language'] != api_get_setting('platformLanguage')) {
-	                    $courses_list_string .= ' - '.$course['course_language'];
+	                    $course_details[] = $course['course_language'];
 	                }
+                    $courses_list_string .= implode(' - ', $course_details);
 						// We display a subscription link if:
 	                // 1. it is allowed to register for the course and if the course is not already in the courselist of the user and if the user is identiefied
 	                // 2.