Changeset 718

Timestamp:

11/11/08 16:23:22 (5 years ago)

Author:

rcorsaro

Message:

better error handling and handles null values for properties

File:

• trunk/plugins/oforgeplugin/oforge/properties.py (modified) (3 diffs)

trunk/plugins/oforgeplugin/oforge/properties.py

@@ -1 +1 @@
 import traceback
 import re
+import sys
 
 from trac.config import Option
@@ -53 +54 @@
     def resource_property_changed(self, resource, property, old_v, new_v):
         oforge = OForge(self.config.filename)
+        self.log.debug("status change listener invoked")
         try:
             project = oforge.get_project(self.env)
@@ -73 +75 @@
                       """%(c_status, c_comment, p_status))
     
-                    project.set_attribute('current_status', c_status)
-                    project.set_attribute('current_status_comment', c_comment)
-                    project.set_attribute('prev_status', p_status)
-        except Exception:
+                    project.set_attribute('current_status', c_status or '')
+                    project.set_attribute('current_status_comment', c_comment or '')
+                    project.set_attribute('prev_status', p_status or '')
+        except Exception, e:
+            self.log.debug("an error occured in project status listener")
+            self.log.error(traceback.format_exc(e))
             # This should handle the Django ObjectDoesNotExist exception
             # but we don't want the import dependency on Django