Changeset 4427

Show
Ignore:
Timestamp:
10/08/08 15:13:18 (1 month ago)
Author:
doki_pen
Message:

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

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • announcerplugin/0.11/announcerplugin/subscribers/ticket_compat.py

    r4274 r4427  
    55from trac.config import BoolOption 
    66import re 
     7from trac.resource import ResourceNotFound 
    78 
    89class StaticTicketSubscriber(Component): 
    …… 
    99100             
    100101            if event.category in ('created', 'changed', 'attachment added'): 
    101                 component = model.Component(self.env, ticket['component']) 
    102                 if component.owner: 
    103                     ## TODO: Is this an option? 
    104                     self.log.debug("LegacyTicketSubscriber added '%s' because of rule: component owner" % (component.owner,)) 
    105                     yield ('email', component.owner, True, None) 
    106                      
     102                try: 
     103                    # this throws an exception if the component does not exist 
     104                    component = model.Component(self.env, ticket['component']) 
     105                    if component.owner: 
     106                        ## TODO: Is this an option? 
     107                        self.log.debug("LegacyTicketSubscriber added '%s' because of rule: component owner" % (component.owner,)) 
     108                        yield ('email', component.owner, True, None) 
     109                except ResourceNotFound, message: 
     110                    self.log.warn("LegacyTicketSubscriber couldn't add component owner because component was not found, message: '%s'" % (message,))     
     111 
    107112                if self.always_notify_owner and ticket['owner'] and not self._check_opt_out('notify_owner', ticket['owner']):                    
    108113                    owner = ticket['owner']