moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-06 01:16:44 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

MDL-68821 course: Add a attr check for the chooser

Browse source
This commit is contained in:
Mathew May 2020-05-25 14:22:33 +08:00
parent 71965a8572
commit e74bcf19f3
4 changed files with 16 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
course/amd/build/activitychooser.min.js vendored
View file

File diff suppressed because one or more lines are too long

2
course/amd/build/activitychooser.min.js.map
View file

File diff suppressed because one or more lines are too long

12
course/amd/src/activitychooser.js
View file

@ -91,13 +91,23 @@ const registerListenerEvents = (courseId, chooserConfig) => {
events.forEach((event) => {
document.addEventListener(event, async(e) => {
if (e.target.closest(selectors.elements.sectionmodchooser)) {
let caller;
// 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;
// We always want the sectionDiv caller first as it keeps track of section ID's after DnD changes.
// The button attribute is always just a fallback for us as the section div is not always available.
// A YUI change could be done maybe to only update the button attribute but we are going for minimal change here.
if (sectionDiv !== null && sectionDiv.hasAttribute('data-sectionid')) {
// We check for attributes just in case of outdated contrib course formats.
caller = sectionDiv;
} else {
caller = button;
}
// We want to show the modal instantly but loading whilst waiting for our data.
let bodyPromiseResolver;