From 1d693c8fd0144ad12d94dfa0ae747279c147e8ad Mon Sep 17 00:00:00 2001
From: tjhunt <tjhunt>
Date: Thu, 19 Mar 2009 03:14:11 +0000
Subject: [PATCH] course settings: MDL-18595: Fix regression caused by
 switching the course settings form to use the new date selector.

The bad commit was http://cvs.moodle.org/moodle/course/edit_form.php?r1=1.65&r2=1.66

Sorry everyone who was inconvenienced by this.
---
 course/edit.php      | 17 -----------------
 course/edit_form.php |  7 +++----
 2 files changed, 3 insertions(+), 21 deletions(-)

diff --git a/course/edit.php b/course/edit.php
index 85e16d532b..6961566f64 100644
--- a/course/edit.php
+++ b/course/edit.php
@@ -52,18 +52,9 @@
                 } // it'll be greyed out but we want these by default anyway.
             }
             $course->allowedmods = $allowedmods;
-
-            if ($course->enrolstartdate){
-                $course->enrolstartdisabled = 0;
-            }
-
-            if ($course->enrolenddate) {
-                $course->enrolenddisabled = 0;
-            }
         }
     }
 
-
 /// first create the form
     $editform = new course_edit_form('edit.php', compact('course', 'category'));
     // now override defaults if course already exists
@@ -84,14 +75,6 @@
 /// process data if submitted
 
         //preprocess data
-        if ($data->enrolstartdisabled){
-            $data->enrolstartdate = 0;
-        }
-
-        if ($data->enrolenddisabled) {
-            $data->enrolenddate = 0;
-        }
-
         $data->timemodified = time();
 
         if (empty($course)) {
diff --git a/course/edit_form.php b/course/edit_form.php
index 6b43c4b3a1..1cb7a83bbb 100644
--- a/course/edit_form.php
+++ b/course/edit_form.php
@@ -464,10 +464,9 @@ class course_edit_form extends moodleform {
             }
         }
 
-        if (empty($data['enrolenddisabled'])){
-            if ($data['enrolenddate'] <= $data['enrolstartdate']){
-                $errors['enroldateendgrp'] = get_string('enrolenddaterror');
-            }
+        if (!empty($data['enrolstartdate']) && !empty($data['enrolenddate']) &&
+                $data['enrolenddate'] <= $data['enrolstartdate']){
+            $errors['enrolenddate'] = get_string('enrolenddaterror');
         }
 
         if (!empty($CFG->enrol_manual_usepasswordpolicy) and isset($data['enrolpassword']) and $data['enrolpassword'] != '') {
-- 
2.39.5