[Mpuls-commits] r2261 - in wasko/branches/2.0: . mpulsweb/lib

[scm-commit@wald.intevation.org]

Author: bh
Date: 2010-03-30 16:20:29 +0200 (Tue, 30 Mar 2010)
New Revision: 2261

Modified:
   wasko/branches/2.0/ChangeLog
   wasko/branches/2.0/mpulsweb/lib/base.py
Log:
* mpulsweb/lib/base.py (BaseController._loadCase): Do not catch an
exception just to reraise it without doing anything else.


Modified: wasko/branches/2.0/ChangeLog
===================================================================
--- wasko/branches/2.0/ChangeLog	2010-03-29 18:57:00 UTC (rev 2260)
+++ wasko/branches/2.0/ChangeLog	2010-03-30 14:20:29 UTC (rev 2261)
@@ -1,3 +1,8 @@
+2010-03-30  Bernhard Herzog  <bh at intevation.de>
+
+	* mpulsweb/lib/base.py (BaseController._loadCase): Do not catch an
+	exception just to reraise it without doing anything else.
+
 2010-03-29  Bernhard Herzog  <bh at intevation.de>
 
 	Fix naming of CaseSearch methods.

Modified: wasko/branches/2.0/mpulsweb/lib/base.py
===================================================================
--- wasko/branches/2.0/mpulsweb/lib/base.py	2010-03-29 18:57:00 UTC (rev 2260)
+++ wasko/branches/2.0/mpulsweb/lib/base.py	2010-03-30 14:20:29 UTC (rev 2261)
@@ -47,7 +47,6 @@
 
 import mpulsweb.lib.helpers as h
 from mpulsweb.lib.security import getDbName, userIdentity
-from mpulsweb.model.case import LoadCaseNotExistsError
 
 # These names are exported from this module largely for historical
 # reasons.  Ideally the module should probably only export
@@ -72,11 +71,7 @@
         self.page_cache = None
 
     def _loadCase(self, id):
-        factory = g.case_factory
-        try:
-            return factory.loadById(id)
-        except LoadCaseNotExistsError, err:
-            raise
+        return g.case_factory.loadById(id)
 
     def __before__(self):
         try: