Skip to content

Commit 7835aa3

Browse files
author
Matthias Opitz
committed
Added suffix for completion rules for Moodle 4.3+
1 parent 5657d4d commit 7835aa3

File tree

1 file changed

+32
-17
lines changed

1 file changed

+32
-17
lines changed

mod_form.php

Lines changed: 32 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -742,30 +742,45 @@ function data_preprocessing(&$default_values) {
742742
* @return array Array of string IDs of added items, empty array if none
743743
*/
744744
public function add_completion_rules() {
745+
global $CFG;
746+
745747
$mform =& $this->_form;
746748

747-
$group=array();
748-
$group[] =& $mform->createElement('checkbox', 'completionpostsenabled', '', get_string('completionposts','hsuforum'));
749-
$group[] =& $mform->createElement('text', 'completionposts', '', array('size'=>3));
750-
$mform->setType('completionposts',PARAM_INT);
751-
$mform->addGroup($group, 'completionpostsgroup', get_string('completionpostsgroup','hsuforum'), array(' '), false);
752-
$mform->disabledIf('completionposts','completionpostsenabled','notchecked');
749+
// Changes for Moodle 4.3 - MDL-78516.
750+
if ($CFG->branch < 403) {
751+
$suffix = '';
752+
} else {
753+
$suffix = $this->get_suffix();
754+
}
753755

754756
$group=array();
755-
$group[] =& $mform->createElement('checkbox', 'completiondiscussionsenabled', '', get_string('completiondiscussions','hsuforum'));
756-
$group[] =& $mform->createElement('text', 'completiondiscussions', '', array('size'=>3));
757-
$mform->setType('completiondiscussions',PARAM_INT);
758-
$mform->addGroup($group, 'completiondiscussionsgroup', get_string('completiondiscussionsgroup','hsuforum'), array(' '), false);
759-
$mform->disabledIf('completiondiscussions','completiondiscussionsenabled','notchecked');
757+
$group[] =& $mform->createElement('checkbox', 'completionpostsenabled' . $suffix, '',
758+
get_string('completionposts','hsuforum'));
759+
$group[] =& $mform->createElement('text', 'completionposts' . $suffix, '', array('size'=>3));
760+
$mform->setType('completionposts' . $suffix,PARAM_INT);
761+
$mform->addGroup($group, 'completionpostsgroup' . $suffix,
762+
get_string('completionpostsgroup','hsuforum'), array(' '), false);
763+
$mform->disabledIf('completionposts' . $suffix,'completionpostsenabled' . $suffix,'notchecked');
760764

761765
$group=array();
762-
$group[] =& $mform->createElement('checkbox', 'completionrepliesenabled', '', get_string('completionreplies','hsuforum'));
763-
$group[] =& $mform->createElement('text', 'completionreplies', '', array('size'=>3));
764-
$mform->setType('completionreplies',PARAM_INT);
765-
$mform->addGroup($group, 'completionrepliesgroup', get_string('completionrepliesgroup','hsuforum'), array(' '), false);
766-
$mform->disabledIf('completionreplies','completionrepliesenabled','notchecked');
766+
$group[] =& $mform->createElement('checkbox', 'completiondiscussionsenabled' . $suffix, '',
767+
get_string('completiondiscussions','hsuforum'));
768+
$group[] =& $mform->createElement('text', 'completiondiscussions' . $suffix, '', array('size'=>3));
769+
$mform->setType('completiondiscussions' . $suffix,PARAM_INT);
770+
$mform->addGroup($group, 'completiondiscussionsgroup' . $suffix,
771+
get_string('completiondiscussionsgroup','hsuforum'), array(' '), false);
772+
$mform->disabledIf('completiondiscussions' . $suffix,'completiondiscussionsenabled' . $suffix,'notchecked');
767773

768-
return array('completiondiscussionsgroup','completionrepliesgroup','completionpostsgroup');
774+
$group=array();
775+
$group[] =& $mform->createElement('checkbox', 'completionrepliesenabled' . $suffix, '',
776+
get_string('completionreplies','hsuforum'));
777+
$group[] =& $mform->createElement('text', 'completionreplies' . $suffix, '', array('size'=>3));
778+
$mform->setType('completionreplies' . $suffix,PARAM_INT);
779+
$mform->addGroup($group, 'completionrepliesgroup' . $suffix,
780+
get_string('completionrepliesgroup','hsuforum'), array(' '), false);
781+
$mform->disabledIf('completionreplies' . $suffix,'completionrepliesenabled' . $suffix,'notchecked');
782+
783+
return ['completiondiscussionsgroup' . $suffix, 'completionrepliesgroup' . $suffix, 'completionpostsgroup' . $suffix];
769784
}
770785

771786
function completion_rule_enabled($data) {

0 commit comments

Comments
 (0)