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

[scm-commit@wald.intevation.org]

Author: bh
Date: 2010-07-02 21:23:44 +0200 (Fri, 02 Jul 2010)
New Revision: 3132

Modified:
   base/trunk/ChangeLog
   base/trunk/mpulsweb/model/case.py
Log:
* mpulsweb/model/case.py (MpulsCaseState.setPhase): Use
log.exception instead of printing a traceback directly to stderr.


Modified: base/trunk/ChangeLog
===================================================================
--- base/trunk/ChangeLog	2010-07-02 19:18:00 UTC (rev 3131)
+++ base/trunk/ChangeLog	2010-07-02 19:23:44 UTC (rev 3132)
@@ -1,5 +1,10 @@
 2010-07-02  Bernhard Herzog  <bh at intevation.de>
 
+	* mpulsweb/model/case.py (MpulsCaseState.setPhase): Use
+	log.exception instead of printing a traceback directly to stderr.
+
+2010-07-02  Bernhard Herzog  <bh at intevation.de>
+
 	* mpulsweb/model/appointment.py (Appointment.store)
 	(Appointment.delete): Use DB.execute to execute the database
 	transactions

Modified: base/trunk/mpulsweb/model/case.py
===================================================================
--- base/trunk/mpulsweb/model/case.py	2010-07-02 19:18:00 UTC (rev 3131)
+++ base/trunk/mpulsweb/model/case.py	2010-07-02 19:23:44 UTC (rev 3132)
@@ -1199,7 +1199,8 @@
                 db.recycleConnection(conn, cur)
             return True
         except:
-            traceback.print_exc(file=sys.stderr)
+            log.exception("Error while setting phase of case %r to %r"
+                          % (self.id, phase))
         return False
 
     def getPhaseDescription(self):