From 704ca25c574cfbd2b42cb4b7bd5c31b3856ae69a Mon Sep 17 00:00:00 2001
From: toyomoyo <toyomoyo>
Date: Wed, 7 Mar 2007 07:12:13 +0000
Subject: [PATCH] fixing a warning

---
 mod/forum/lib.php | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index 3b169850b6..5fd4e4fda7 100644
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -289,12 +289,13 @@ function forum_cron() {
             
             // caching subscribed users of each forum
             if (!isset($subscribedusers[$forum->id])) {
-                $subusers = forum_subscribed_users($course, $forum, 0, true);  
-                foreach ($subusers as $postuser) {
-                    // this user is subscribed to this forum
-                    $subscribedusers[$forum->id][] = $postuser->id;
-                    // this user is a user we have to process later
-                    $users[$postuser->id] = $postuser;  
+                if ($subusers = forum_subscribed_users($course, $forum, 0, true)) {
+                    foreach ($subusers as $postuser) {
+                        // this user is subscribed to this forum
+                        $subscribedusers[$forum->id][] = $postuser->id;
+                        // this user is a user we have to process later
+                        $users[$postuser->id] = $postuser;  
+                    }
                 }           
             }
             
@@ -353,7 +354,7 @@ function forum_cron() {
                     $forum = $postinfo[$pid]->forum;
                     $course = $postinfo[$pid]->course;    
                     
-                    if (!in_array($userto->id, $subscribedusers[$forum->id])) {
+                    if (empty($subscribedusers[$forum->id]) || !in_array($userto->id, $subscribedusers[$forum->id])) {
                         continue; // user does not subscribe to this forum  
                     }
                     
-- 
2.39.5