1 files changed, 33 insertions, 29 deletions

diff --git a/lib/class/rating.class.php b/lib/class/rating.class.php
index 5d2f9bf2..d1b34822 100644
--- a/lib/class/rating.class.php
+++ b/lib/class/rating.class.php
@@ -65,41 +65,45 @@ class Rating extends database_object {
 	 */
 	public static function build_cache($type, $ids) {
 		
-		$user_id = Dba::escape($GLOBALS['user']->id); 
+		if ($ids) {
+			$user_id = Dba::escape($GLOBALS['user']->id); 
 
-		$idlist = '(' . implode(',', $ids) . ')';
-		$sql = "SELECT `rating`, `object_id`,`rating`.`rating` FROM `rating` WHERE `user`='$user_id' AND `object_id` IN $idlist " . 
-			"AND `object_type`='$type'";
-		$db_results = Dba::read($sql);
+			$idlist = '(' . implode(',', $ids) . ')';
+			$sql = "SELECT `rating`, `object_id`,`rating`.`rating` FROM `rating` WHERE `user`='$user_id' AND `object_id` IN $idlist " . 
+				"AND `object_type`='$type'";
+			$db_results = Dba::read($sql);
 
-		while ($row = Dba::fetch_assoc($db_results)) {
-			$user[$row['object_id']] = $row['rating']; 
-		}
+			while ($row = Dba::fetch_assoc($db_results)) {
+				$user[$row['object_id']] = $row['rating']; 
+			}
 		
-		$sql = "SELECT `rating`,`object_id` FROM `rating` WHERE `object_id` IN $idlist AND `object_type`='$type'"; 
-		$db_results = Dba::read($sql); 
+			$sql = "SELECT `rating`,`object_id` FROM `rating` WHERE `object_id` IN $idlist AND `object_type`='$type'"; 
+			$db_results = Dba::read($sql); 
 		
-		while ($row = Dba::fetch_assoc($db_results)) { 
-			$rating[$row['object_id']]['rating'] += $row['rating']; 
-			$rating[$row['object_id']]['total']++; 
-  		} 
-
-		foreach ($ids as $id) { 
-			parent::add_to_cache('rating_' . $type . '_user',$id,intval($user[$id])); 
-
-			// Do the bit of math required to store this
-			if (!isset($rating[$id])) { 
-				$entry = array('average'=>'0','percise'=>'0'); 
-			} 
-			else { 
-				$average = round($rating[$id]['rating']/$rating[$id]['total'],1); 
-				$entry = array('average'=>floor($average),'percise'=>$average); 
-			} 
+			while ($row = Dba::fetch_assoc($db_results)) { 
+				$rating[$row['object_id']]['rating'] += $row['rating']; 
+				$rating[$row['object_id']]['total']++; 
+	  		} 
+
+			foreach ($ids as $id) { 
+				parent::add_to_cache('rating_' . $type . '_user',$id,intval($user[$id])); 
+
+				// Do the bit of math required to store this
+				if (!isset($rating[$id])) { 
+					$entry = array('average'=>'0','percise'=>'0'); 
+				} 
+				else { 
+					$average = round($rating[$id]['rating']/$rating[$id]['total'],1); 
+					$entry = array('average'=>floor($average),'percise'=>$average); 
+				} 
 				
-			parent::add_to_cache('rating_' . $type . '_all',$id,$entry); 
-		} 
+				parent::add_to_cache('rating_' . $type . '_all',$id,$entry); 
+			} 
 
-		return true; 
+			return true; 
+		} else {
+			return false;
+		}
 
 	} // build_cache