From 150be0dfd0cd588ba3ca93f07b3d8f9bb254f417 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Wed, 20 Feb 2008 06:16:07 +0000
Subject: [PATCH] MDL-13542  Merged return checking from stable 1.9

---
 mod/forum/index.php | 23 ++++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/mod/forum/index.php b/mod/forum/index.php
index 59ecbb2506..5e0b941c50 100644
--- a/mod/forum/index.php
+++ b/mod/forum/index.php
@@ -92,8 +92,13 @@
 
         foreach ($forums as $forum) {
 
-            $cm = get_coursemodule_from_instance("forum", $forum->id, $course->id);
-            $context = get_context_instance(CONTEXT_MODULE, $cm->id);
+            if (!$cm = get_coursemodule_from_instance('forum', $forum->id, $course->id)) {
+                continue;   // Shouldn't happen
+            }
+
+            if (!$context = get_context_instance(CONTEXT_MODULE, $cm->id)) {
+                continue;   // Shouldn't happen
+            }
             
             if (!has_capability('mod/forum:viewdiscussion', $context)) {
                 if (isset($forum->keyreference)) {
@@ -158,8 +163,12 @@
     if ($generalforums) {
         foreach ($generalforums as $forum) {
 
-            $cm = get_coursemodule_from_instance("forum", $forum->id, $course->id);
-            $context = get_context_instance(CONTEXT_MODULE, $cm->id);
+            if (!$cm = get_coursemodule_from_instance("forum", $forum->id, $course->id)) {
+                continue;  // Shouldn't happen
+            }
+            if (!$context = get_context_instance(CONTEXT_MODULE, $cm->id)) {
+                continue;  // Shouldn't happen
+            }
 
             if (!groups_course_module_visible($cm)) {
                 continue;
@@ -294,7 +303,11 @@
         if ($learningforums) {
             $currentsection = "";
             foreach ($learningforums as $key => $forum) {
-                $cm = get_coursemodule_from_instance("forum", $forum->id, $course->id);
+
+                if (!$cm = get_coursemodule_from_instance('forum', $forum->id, $course->id)) {
+                    continue;   // Shouldn't happen
+                }
+
                 if (!coursemodule_visible_for_user($cm)) {
                     continue;
                 }
-- 
2.39.5