[Mpuls-commits] r5004 - in base/trunk: . mpulsweb/lib

[scm-commit@wald.intevation.org]

Author: ludwig
Date: 2011-05-25 11:50:21 +0200 (Wed, 25 May 2011)
New Revision: 5004

Modified:
   base/trunk/ChangeLog
   base/trunk/mpulsweb/lib/validators.py
Log:
Issue1972: Modification to the CreateAppointmentForm validator.
Start/end date/time should be filled and no default should be used.
The old defaults were confusing.


Modified: base/trunk/ChangeLog
===================================================================
--- base/trunk/ChangeLog	2011-05-24 18:59:45 UTC (rev 5003)
+++ base/trunk/ChangeLog	2011-05-25 09:50:21 UTC (rev 5004)
@@ -1,3 +1,10 @@
+2011-05-25  Ludwig Reiter <ludwig.reiter at inteation.de>
+
+	* mpulsweb/lib/validators.py:
+	Issue1972: Modification to the CreateAppointmentForm validator.
+	Start/end/date/time should not be empty and no defaults are choosen,
+	because the defaults were confusing.
+
 2011-05-24  Ludwig Reiter <ludwig.reiter at inteation.de>
 
 	* mpulsweb/controllers/case_bundle.py,

Modified: base/trunk/mpulsweb/lib/validators.py
===================================================================
--- base/trunk/mpulsweb/lib/validators.py	2011-05-24 18:59:45 UTC (rev 5003)
+++ base/trunk/mpulsweb/lib/validators.py	2011-05-25 09:50:21 UTC (rev 5004)
@@ -610,10 +610,10 @@
 class CreateAppointmentForm(BaseFormValidator):
 
     start_date = MyDateConverter(not_empty=True, month_style='dd.mm.jjjj')
-    start_time = TimeConverter(if_empty="00:00", use_datetime=True)
+    start_time = TimeConverter(not_empty=True, use_datetime=True)
 
-    end_date = MyDateConverter(month_style='dd.mm.jjjj')
-    end_time = TimeConverter(use_datetime=True)
+    end_date = MyDateConverter(not_empty=True, month_style='dd.mm.jjjj')
+    end_time = TimeConverter(not_empty=True, use_datetime=True)
 
     chained_validators = [DateOrder('start_date', 'end_date')]
     title = All(String(not_empty=True), MaxLength(128))