Changeset 4427

Timestamp:

Oct 8, 2008, 8:13:18 PM (16 years ago)

Author:

Robert Corsaro

Message:

fixes deleted component defect that causes notification to fail. ticket #3715. Thanks jdio!

File:

• announcerplugin/0.11/announcerplugin/subscribers/ticket_compat.py (modified) (2 diffs)

announcerplugin/0.11/announcerplugin/subscribers/ticket_compat.py

@@ -5 +5 @@
 from trac.config import BoolOption
 import re
+from trac.resource import ResourceNotFound
 
 class StaticTicketSubscriber(Component):
@@ -99 +100 @@
             
             if event.category in ('created', 'changed', 'attachment added'):
-                component = model.Component(self.env, ticket['component'])
-                if component.owner:
-                    ## TODO: Is this an option?
-                    self.log.debug("LegacyTicketSubscriber added '%s' because of rule: component owner" % (component.owner,))
-                    yield ('email', component.owner, True, None)
-                    
+                try:
+                    # this throws an exception if the component does not exist
+                    component = model.Component(self.env, ticket['component'])
+                    if component.owner:
+                        ## TODO: Is this an option?
+                        self.log.debug("LegacyTicketSubscriber added '%s' because of rule: component owner" % (component.owner,))
+                        yield ('email', component.owner, True, None)
+                except ResourceNotFound, message:
+                    self.log.warn("LegacyTicketSubscriber couldn't add component owner because component was not found, message: '%s'" % (message,))    
+
                 if self.always_notify_owner and ticket['owner'] and not self._check_opt_out('notify_owner', ticket['owner']):                   
                     owner = ticket['owner']