MDL-13542 More return value checking
This commit is contained in:
+11
-3
@@ -163,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;
|
||||
@@ -299,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;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user