MDL-42869 calendar: HTML validation check fails on add event screen
authorDaniel Neis Araujo <danielneis@gmail.com>
Fri, 14 Mar 2014 22:14:20 +0000 (19:14 -0300)
committerDaniel Neis Araujo <danielneis@gmail.com>
Wed, 23 Apr 2014 17:50:35 +0000 (14:50 -0300)
calendar/event_form.php

index 5b12a4d..69a8924 100644 (file)
@@ -114,14 +114,23 @@ class event_form extends moodleform {
         $mform->addRule('timestart', get_string('required'), 'required');
 
         $mform->addElement('header', 'durationdetails', get_string('eventduration', 'calendar'));
-        $mform->addElement('radio', 'duration', null, get_string('durationnone', 'calendar'), 0);
 
-        $mform->addElement('radio', 'duration', null, get_string('durationuntil', 'calendar'), 1);
-        $mform->addElement('date_time_selector', 'timedurationuntil', '&nbsp;');
-        $mform->disabledIf('timedurationuntil','duration','noteq', 1);
+        $group = array();
+        $group[] =& $mform->createElement('radio', 'duration', null, get_string('durationnone', 'calendar'), 0);
+        $group[] =& $mform->createElement('radio', 'duration', null, get_string('durationuntil', 'calendar'), 1);
+        $group[] =& $mform->createElement('date_time_selector', 'timedurationuntil', '');
+        $group[] =& $mform->createElement('radio', 'duration', null, get_string('durationminutes', 'calendar'), 2);
+        $group[] =& $mform->createElement('text', 'timedurationminutes', null);
+
+        $mform->addGroup($group, 'durationgroup', '', '<br />', false);
+
+        $mform->disabledIf('timedurationuntil',         'duration', 'noteq', 1);
+        $mform->disabledIf('timedurationuntil[day]',    'duration', 'noteq', 1);
+        $mform->disabledIf('timedurationuntil[month]',  'duration', 'noteq', 1);
+        $mform->disabledIf('timedurationuntil[year]',   'duration', 'noteq', 1);
+        $mform->disabledIf('timedurationuntil[hour]',   'duration', 'noteq', 1);
+        $mform->disabledIf('timedurationuntil[minute]', 'duration', 'noteq', 1);
 
-        $mform->addElement('radio', 'duration', null, get_string('durationminutes', 'calendar'), 2);
-        $mform->addElement('text', 'timedurationminutes', null);
         $mform->setType('timedurationminutes', PARAM_INT);
         $mform->disabledIf('timedurationminutes','duration','noteq', 2);
 
@@ -130,7 +139,7 @@ class event_form extends moodleform {
         if ($newevent) {
 
             $mform->addElement('header', 'repeatevents', get_string('repeatedevents', 'calendar'));
-            $mform->addElement('checkbox', 'repeat', get_string('repeatevent', 'calendar'), null, 'repeat');
+            $mform->addElement('checkbox', 'repeat', get_string('repeatevent', 'calendar'), null);
             $mform->addElement('text', 'repeats', get_string('repeatweeksl', 'calendar'), 'maxlength="10" size="10"');
             $mform->setType('repeats', PARAM_INT);
             $mform->setDefault('repeats', 1);