From e5953e0daee45d4be54611bf5e70de943f8da4cb Mon Sep 17 00:00:00 2001
From: Yannick Warnier <yannick.warnier@beeznest.com>
Date: Thu, 19 Jul 2007 07:27:21 +0200
Subject: [PATCH] [svn r12758] Fixed bug whereby if only one category exists
 (can be white spaces) and most courses belong to this category, they are
 shown on the anonymous homepage although they are not "visible to the world".

---
 index.php | 32 +++++++++++++++++++++-----------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/index.php b/index.php
index a1fa758d57..6e12ce10c6 100644
--- a/index.php
+++ b/index.php
@@ -20,7 +20,7 @@
 /**
 *	@package dokeos.main
 * 	@author Patrick Cool <patrick.cool@UGent.be>, Ghent University, Refactoring
-* 	@version $Id: index.php 12708 2007-07-05 20:46:47Z yannoo $
+* 	@version $Id: index.php 12758 2007-07-19 05:27:21Z yannoo $
 *   @todo check the different @todos in this page and really do them
 * 	@todo check if the news management works as expected
 */
@@ -603,30 +603,40 @@ function display_anonymous_course_list()
 		echo "<h3>", $categoryName['name'], "</h3>\n";
 	}
 	$numrows = mysql_num_rows($sql_result_courses);
+	$courses_list_string = '';
+	$courses_shown = 0;
 	if ($numrows > 0)
 	{
 		if ($thereIsSubCat)
 		{
-			echo "<hr size=\"1\" noshade=\"noshade\">\n";
+			$courses_list_string .= "<hr size=\"1\" noshade=\"noshade\">\n";
 		}
-		echo "<h4 style=\"margin-top: 0px;\">", get_lang("CourseList"), "</h4>\n", "<ul>\n";
+		$courses_list_string .= "<h4 style=\"margin-top: 0px;\">".get_lang("CourseList")."</h4>\n"."<ul>\n";
 		foreach ($course_list AS $course)
 		{
-			echo "<li>\n";
-			echo "<a href=\"".$web_course_path.$course['directory'], "/\">", $course['title'], "</a>";
-			echo "<br/>", $course['visual_code'], " - ", $course['tutor_name'];
-			if (api_get_setting('show_different_course_language') == 'true' && $course['course_language'] <> api_get_setting('platformLanguage'))
-			{
-				echo ' - '.$course['course_language'];
+			if($course['visibility'] == COURSE_VISIBILITY_OPEN_WORLD){
+				$courses_shown++;
+				$courses_list_string .= "<li>\n";
+				$courses_list_string .= "<a href=\"".$web_course_path.$course['directory']."/\">".$course['title']."</a>";
+				$courses_list_string .= "<br/>".$course['visual_code']." - ".$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'];
+				}
+				$courses_list_string .= "</li>\n";
 			}
-			echo "</li>\n";
 		}
-		echo "</ul>\n";
+		$courses_list_string .= "</ul>\n";
 	}
 	else
 	{
 		// echo "<blockquote>",get_lang('_No_course_publicly_available'),"</blockquote>\n";
 	}
+	if($courses_shown > 0)
+	{ //only display the list of courses and categories if there was more than
+	  // 0 courses visible to the world (we're in the anonymous list here)
+		echo $courses_list_string;
+	}
 	if ($category != "")
 	{
 		echo "<p>", "<a href=\"".api_get_self()."\"><b>&lt;&lt;</b> ", get_lang("BackToHomePage"), "</a>", "</p>\n";