From feed81934b78402ea71b199b214ea34575db8c9b Mon Sep 17 00:00:00 2001
From: tjhunt <tjhunt>
Date: Mon, 16 Mar 2009 06:09:43 +0000
Subject: [PATCH] course settings: use the optional option on date selectors,
 rather than a manua disabled checkbox. Also use new duration element.

---
 course/edit_form.php | 24 +++++-------------------
 1 file changed, 5 insertions(+), 19 deletions(-)

diff --git a/course/edit_form.php b/course/edit_form.php
index 358164f9f4..6b43c4b3a1 100644
--- a/course/edit_form.php
+++ b/course/edit_form.php
@@ -232,29 +232,15 @@ class course_edit_form extends moodleform {
         $mform->setHelpButton('enrollable', array('courseenrollable2', get_string('enrollable')), true);
         $mform->setDefault('enrollable', $courseconfig->enrollable);
 
-        $enroldatestartgrp = array();
-        $enroldatestartgrp[] = &MoodleQuickForm::createElement('date_selector', 'enrolstartdate');
-        $enroldatestartgrp[] = &MoodleQuickForm::createElement('checkbox', 'enrolstartdisabled', null, get_string('disable'));
-        $mform->addGroup($enroldatestartgrp, 'enrolstartdategrp', get_string('enrolstartdate'), ' ', false);
+        $mform->addElement('date_selector', 'enrolstartdate', get_string('enrolstartdate'), array('optional' => true));
         $mform->setDefault('enrolstartdate', 0);
-        $mform->setDefault('enrolstartdisabled', 1);
-        $mform->disabledIf('enrolstartdategrp', 'enrolstartdisabled', 'checked');
+        $mform->disabledIf('enrolstartdate', 'enrollable', 'neq', 2);
 
-        $enroldateendgrp = array();
-        $enroldateendgrp[] = &MoodleQuickForm::createElement('date_selector', 'enrolenddate');
-        $enroldateendgrp[] = &MoodleQuickForm::createElement('checkbox', 'enrolenddisabled', null, get_string('disable'));
-        $mform->addGroup($enroldateendgrp, 'enroldateendgrp', get_string('enrolenddate'), ' ', false);
+        $mform->addElement('date_selector', 'enrolenddate', get_string('enrolenddate'), array('optional' => true));
         $mform->setDefault('enrolenddate', 0);
-        $mform->setDefault('enrolenddisabled', 1);
-        $mform->disabledIf('enroldateendgrp', 'enrolenddisabled', 'checked');
+        $mform->disabledIf('enrolenddate', 'enrollable', 'neq', 2);
 
-        $periodmenu=array();
-        $periodmenu[0] = get_string('unlimited');
-        for ($i=1; $i<=365; $i++) {
-            $seconds = $i * 86400;
-            $periodmenu[$seconds] = get_string('numdays', '', $i);
-        }
-        $mform->addElement('select', 'enrolperiod', get_string('enrolperiod'), $periodmenu);
+        $mform->addElement('duration', 'enrolperiod', get_string('enrolperiod'), array('optional' => true, 'defaultunit' => 86400));
         $mform->setDefault('enrolperiod', $courseconfig->enrolperiod);
 
 
-- 
2.39.5