MDLSITE-2455 Do not throw coding exception when the activity is not found
Under certain conditions, coding exception is thrown when fetching items from the navigation bar. The patch prevents them. See more details in the tracker.
This commit is contained in:
committed by
Aparup Banerjee
parent
1d93c9ae07
commit
069c29b987
@@ -1222,12 +1222,14 @@ class global_navigation extends navigation_node {
|
||||
// Load the course sections into the page
|
||||
$this->load_course_sections($course, $coursenode, null, $cm);
|
||||
$activity = $coursenode->find($cm->id, navigation_node::TYPE_ACTIVITY);
|
||||
// Finally load the cm specific navigaton information
|
||||
$this->load_activity($cm, $course, $activity);
|
||||
// Check if we have an active ndoe
|
||||
if (!$activity->contains_active_node() && !$activity->search_for_active_node()) {
|
||||
// And make the activity node active.
|
||||
$activity->make_active();
|
||||
if (!empty($activity)) {
|
||||
// Finally load the cm specific navigaton information
|
||||
$this->load_activity($cm, $course, $activity);
|
||||
// Check if we have an active ndoe
|
||||
if (!$activity->contains_active_node() && !$activity->search_for_active_node()) {
|
||||
// And make the activity node active.
|
||||
$activity->make_active();
|
||||
}
|
||||
}
|
||||
break;
|
||||
case CONTEXT_USER :
|
||||
|
||||
Reference in New Issue
Block a user