MDL-84140 course: more resilient check for activity footer component.

paulholden · Jan 9, 2025 · 4e96885 · 4e96885
1 parent a843eab
commit 4e96885
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/course/externallib.php b/course/externallib.php
@@ -4596,9 +4596,17 @@ public static function get_activity_chooser_footer(int $courseid, int $sectionid
         $coursecontext = context_course::instance($courseid);
         self::validate_context($coursecontext);
 
-        $activeplugin = get_config('core', 'activitychooseractivefooter');
+        // The active plugin must be set, and be present on the site.
+        $activeplugin = clean_param(
+            get_config('core', 'activitychooseractivefooter'),
+            PARAM_COMPONENT,
+        );
 
-        if ($activeplugin !== COURSE_CHOOSER_FOOTER_NONE) {
+        if (
+            $activeplugin !== COURSE_CHOOSER_FOOTER_NONE &&
+            !empty($activeplugin) &&
+            core_component::get_component_directory($activeplugin) !== null
+        ) {
             $footerdata = component_callback($activeplugin, 'custom_chooser_footer', [$courseid, $sectionid]);
             return [
                 'footer' => true,