mirror of
https://github.com/moodle/moodle.git
synced 2025-01-19 06:18:28 +01:00
MDL-50699 Lesson: "Link to next activity" empty until Lesson saved
Due to a logic bug, the "Link to next activity" dropdown was empty (only contains the "None" option) until the Lesson had been saved. This minor change allows a "next activity" to be specified when initially creating the Lesson whilst still preventing the user from choosing the Lesson as its own next activity.
This commit is contained in:
parent
38cb73a47f
commit
a673650477
@ -146,15 +146,15 @@ class mod_lesson_mod_form extends moodleform_mod {
|
||||
if ($mods = get_course_mods($COURSE->id)) {
|
||||
$modinstances = array();
|
||||
foreach ($mods as $mod) {
|
||||
|
||||
// get the module name and then store it in a new array
|
||||
// Get the module name and then store it in a new array.
|
||||
if ($module = get_coursemodule_from_instance($mod->modname, $mod->instance, $COURSE->id)) {
|
||||
if (isset($this->_cm->id) and $this->_cm->id != $mod->id){
|
||||
// Exclude this lesson, if it's already been saved.
|
||||
if (!isset($this->_cm->id) || $this->_cm->id != $mod->id) {
|
||||
$modinstances[$mod->id] = $mod->modname.' - '.$module->name;
|
||||
}
|
||||
}
|
||||
}
|
||||
asort($modinstances); // sort by module name
|
||||
asort($modinstances); // Sort by module name.
|
||||
$modinstances=array(0=>get_string('none'))+$modinstances;
|
||||
|
||||
$mform->addElement('select', 'activitylink', get_string('activitylink', 'lesson'), $modinstances);
|
||||
|
Loading…
x
Reference in New Issue
Block a user