MDL-68235 core_course: Sections work after drag and drop reordering

diff --git a/course/amd/build/activitychooser.min.js b/course/amd/build/activitychooser.min.js
index ea7a72e..d9441d4 100644 (file)
Binary files a/course/amd/build/activitychooser.min.js and b/course/amd/build/activitychooser.min.js differ

diff --git a/course/amd/build/activitychooser.min.js.map b/course/amd/build/activitychooser.min.js.map
index f410d70..3348e19 100644 (file)
Binary files a/course/amd/build/activitychooser.min.js.map and b/course/amd/build/activitychooser.min.js.map differ

diff --git a/course/amd/src/activitychooser.js b/course/amd/src/activitychooser.js
index 0207281..d3b83d0 100644 (file)
--- a/course/amd/src/activitychooser.js
+++ b/course/amd/src/activitychooser.js
@@ -79,7 +79,13 @@ const registerListenerEvents = (courseId) => {
         document.addEventListener(event, async(e) => {
             if (e.target.closest(selectors.elements.sectionmodchooser)) {
                 const data = await fetchModuleData();
-                const caller = e.target.closest(selectors.elements.sectionmodchooser);
+                // We need to know who called this.
+                // Standard courses use the ID in the main section info.
+                const sectionDiv = e.target.closest(selectors.elements.section);
+                // Front page courses need some special handling.
+                const button = e.target.closest(selectors.elements.sectionmodchooser);
+                // If we don't have a section ID use the fallback ID.
+                const caller = sectionDiv || button;
                 const favouriteFunction = partiallyAppliedFavouriteManager(data, caller.dataset.sectionid);
                 const builtModuleData = sectionIdMapper(data, caller.dataset.sectionid);
                 const sectionModal = await modalBuilder(builtModuleData);

diff --git a/course/renderer.php b/course/renderer.php
index 784d6ee..cc24507 100644 (file)
--- a/course/renderer.php
+++ b/course/renderer.php
@@ -368,11 +368,11 @@ class core_course_renderer extends plugin_renderer_base {
             $ajaxcontrol .= html_writer::start_tag('div', array('class' => 'section-modchooser'));
             $icon = $this->output->pix_icon('t/add', '');
             $span = html_writer::tag('span', $straddeither, array('class' => 'section-modchooser-text'));
-            $ajaxcontrol .= html_writer::tag('button', $icon . $span, array(
+            $ajaxcontrol .= html_writer::tag('button', $icon . $span, [
                     'class' => 'section-modchooser-link btn btn-link',
                     'data-action' => 'open-chooser',
                     'data-sectionid' => $section,
-                )
+                ]
             );
             $ajaxcontrol .= html_writer::end_tag('div');
             $ajaxcontrol .= html_writer::end_tag('div');