From 778a74e8037ea4eb8082e373010d4988d2131ac8 Mon Sep 17 00:00:00 2001
From: Yannick Warnier <yannick.warnier@beeznest.com>
Date: Thu, 3 May 2007 00:04:33 +0200
Subject: [PATCH] [svn r12246] Ensure user_id and group_id are defined before
 issuing DB query

---
 main/inc/lib/groupmanager.lib.php | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/main/inc/lib/groupmanager.lib.php b/main/inc/lib/groupmanager.lib.php
index 490c1a04af..530a468c80 100644
--- a/main/inc/lib/groupmanager.lib.php
+++ b/main/inc/lib/groupmanager.lib.php
@@ -806,9 +806,11 @@ class GroupManager
 	 */
 	function is_subscribed($user_id, $group_id)
 	{
+		if(empty($user_id) or empty($group_id)){return false;}
 		$table_group_user = Database :: get_course_table(TABLE_GROUP_USER);
-		$db_result = api_sql_query('SELECT 1 FROM '.$table_group_user.' WHERE group_id = '.$group_id.' AND user_id = '.$user_id);
-		return mysql_num_rows($db_result) > 0;
+		$sql = 'SELECT 1 FROM '.$table_group_user.' WHERE group_id = '.$group_id.' AND user_id = '.$user_id;
+		$db_result = api_sql_query($sql);
+		return Database::num_rows($db_result) > 0;
 	}
 	/**
 	 * Can a user subscribe to a specified group in a course