[Mpuls-commits] r3418 - in base/trunk: . mpulsweb/model

[scm-commit@wald.intevation.org]

Author: bh
Date: 2010-08-17 14:45:11 +0200 (Tue, 17 Aug 2010)
New Revision: 3418

Modified:
   base/trunk/ChangeLog
   base/trunk/mpulsweb/model/phase.py
Log:
* mpulsweb/model/phase.py (PhasePart.__init__): Log a warning if a
phase does not have an associated date field.  The missing date
field name will likely interfere with date-range searches.


Modified: base/trunk/ChangeLog
===================================================================
--- base/trunk/ChangeLog	2010-08-17 10:46:56 UTC (rev 3417)
+++ base/trunk/ChangeLog	2010-08-17 12:45:11 UTC (rev 3418)
@@ -1,5 +1,11 @@
 2010-08-17  Bernhard Herzog  <bh at intevation.de>
 
+	* mpulsweb/model/phase.py (PhasePart.__init__): Log a warning if a
+	phase does not have an associated date field.  The missing date
+	field name will likely interfere with date-range searches.
+
+2010-08-17  Bernhard Herzog  <bh at intevation.de>
+
 	* mpulsweb/lib/search.py (CaseSearch.get_where): Simply join the
 	sub-clauses with " AND ".  The SEARCH_WHERE template didn't do
 	anything else now.

Modified: base/trunk/mpulsweb/model/phase.py
===================================================================
--- base/trunk/mpulsweb/model/phase.py	2010-08-17 10:46:56 UTC (rev 3417)
+++ base/trunk/mpulsweb/model/phase.py	2010-08-17 12:45:11 UTC (rev 3418)
@@ -279,6 +279,9 @@
             if self.id in flags['time']:
                 self.datefield = p.getName()
 
+        if self.datefield is None:
+            log.warning("Phase %s has no associated datefield", id)
+
     def getDescription(self):
         desc = []
         desc.append(g.mpuls_config.get('phases',