warning #128: Update mod_form.php to add completion rules with suffix…

… based on branch version
udima-university · Aug 29, 2024 · 84df295 · 84df295
1 parent 21d020e
commit 84df295
Showing 1 changed file with 21 additions and 8 deletions.
diff --git a/mod_form.php b/mod_form.php
@@ -150,19 +150,25 @@ public function definition() {
      * @return array List of added element names, or names of wrapping group elements.
      */
     public function add_completion_rules() {
+        global $CFG;
+        if ($CFG->branch < 403) {
+            $suffix = '';
+        } else {
+            $suffix = $this->get_suffix();
+        }
 
         $mform =&  $this->_form;
 
         $group = [
-            $mform->createElement('checkbox', 'completionminutesenabled', ' ', get_string('completionminutesex', 'jitsi')),
-            $mform->createElement('text', 'completionminutes', ' ', ['size' => 3]),
+            $mform->createElement('checkbox', 'completionminutesenabled'.$suffix, ' ', get_string('completionminutesex', 'jitsi')),
+            $mform->createElement('text', 'completionminutes'.$suffix, ' ', ['size' => 3]),
         ];
-        $mform->setType('completionminutes', PARAM_INT);
-        $mform->addGroup($group, 'completionminutesgroup', get_string('completionminutes', 'jitsi'), [' '], false);
-        $mform->addHelpButton('completionminutesgroup', 'completionminutes', 'jitsi');
-        $mform->disabledIf('completionminutes', 'completionminutesenabled', 'notchecked');
+        $mform->setType('completionminutes'.$suffix, PARAM_INT);
+        $mform->addGroup($group, 'completionminutesgroup'.$suffix, get_string('completionminutes', 'jitsi'), [' '], false);
+        $mform->addHelpButton('completionminutesgroup'.$suffix, 'completionminutes', 'jitsi');
+        $mform->disabledIf('completionminutes'.$suffix, 'completionminutesenabled', 'notchecked');
 
-        return ['completionminutesgroup'];
+        return ['completionminutesgroup'.$suffix];
     }
 
     /**
@@ -172,7 +178,14 @@ public function add_completion_rules() {
      * @return bool True if one or more rules is enabled, false if none are.
      */
     public function completion_rule_enabled($data) {
-        return (!empty($data['completionminutesenabled']) && $data['completionminutes'] != 0);
+        global $CFG;
+        if ($CFG->branch < 403) {
+            $suffix = '';
+        } else {
+            $suffix = $this->get_suffix();
+        }
+
+        return (!empty($data['completionminutesenabled'.$suffix]) && $data['completionminutes'.$suffix] != 0);
     }
 
     /**