Modify

Opened 6 years ago

Closed 6 years ago

#2975 closed defect (fixed)

CarbonCopySubscriber is not properly invoked on a ticket change event

Reported by: cedrichurst Owned by: ixokai
Priority: normal Component: AnnouncerPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

With the CarbonCopySubscriber module activated, 'cedric' listed as owner, and 'steven' listed as cc:

2008-04-30 16:07:38,672 Trac[api] DEBUG: AnnouncementSystem found the following subscribers capable of handling 'ticket, changed': LegacyTicketSubscrib$
2008-04-30 16:07:38,675 Trac[ticket_compat] DEBUG: LegacyTicketSubscriber added 'cedric (authenticated)' because of rule: always_notify_owner
2008-04-30 16:07:38,676 Trac[ticket_compat] DEBUG: LegacyTicketSubscriber added 'cedric (authenticated)' because of rule: always_notify_updater
2008-04-30 16:07:38,678 Trac[api] DEBUG: AnnouncementSystem has found the following subscriptions: [cedric(authenticated) via email]
2008-04-30 16:07:38,681 Trac[email_distributor] DEBUG: EmailDistributor has found the following formats capable of handling 'email' of 'ticket': text/h$
2008-04-30 16:07:38,697 Trac[email_distributor] DEBUG: EmailDistributor found the address 'cedric@company.com' for 'cedric (authenticated)' via: Session$
2008-04-30 16:07:38,698 Trac[email_distributor] DEBUG: EmailDistributor is sending event as 'text/html' to: cedric@company.com
2008-04-30 16:07:39,725 Trac[email_distributor] DEBUG: EmailDistributor took 0.65 seconds to send.
2008-04-30 16:07:39,725 Trac[api] DEBUG: AnnouncementSystem sent event in 1.06 seconds.

i figured this had something to do with the get_subscription_categories method of
the the CarbonCopySubscriber. So I simply mirrored the body of the same method in LegacyTicketSubscriber and the problem was resolved:

2008-04-30 16:45:11,119 Trac[api] DEBUG: AnnouncementSystem found the following subscribers capable of handling 'ticket, changed': LegacyTicketSubscrib$
2008-04-30 16:45:11,122 Trac[ticket_compat] DEBUG: LegacyTicketSubscriber added 'cedric (authenticated)' because of rule: always_notify_owner
2008-04-30 16:45:11,123 Trac[ticket_compat] DEBUG: LegacyTicketSubscriber added 'cedric (authenticated)' because of rule: always_notify_updater
2008-04-30 16:45:11,125 Trac[ticket_compat] DEBUG: CarbonCopySubscriber added 'steven <None>' because of rule: carbon copied
2008-04-30 16:45:11,126 Trac[api] DEBUG: AnnouncementSystem has found the following subscriptions: [cedric(authenticated) via email], [steven(authentic$
2008-04-30 16:45:11,130 Trac[email_distributor] DEBUG: EmailDistributor has found the following formats capable of handling 'email' of 'ticket': text/h$
2008-04-30 16:45:11,136 Trac[email_distributor] DEBUG: EmailDistributor found the address 'cedric@company.com' for 'cedric (authenticated)' via: Session$
2008-04-30 16:45:11,140 Trac[email_distributor] DEBUG: EmailDistributor found the address 'steven@company.com' for 'steven (authenticated)' via: Sessi$
2008-04-30 16:45:11,140 Trac[email_distributor] DEBUG: EmailDistributor is sending event as 'text/html' to: cedric@company.com, steven@company.com
2008-04-30 16:45:12,278 Trac[email_distributor] DEBUG: EmailDistributor took 0.78 seconds to send.
2008-04-30 16:45:12,278 Trac[api] DEBUG: AnnouncementSystem sent event in 1.17 seconds.
2008-04-30 16:07:39,725 Trac[email_distributor] DEBUG: EmailDistributor took 0.65 seconds to send.
2008-04-30 16:07:39,725 Trac[api] DEBUG: AnnouncementSystem sent event in 1.06 seconds.

Attachments (1)

announcerplugin-carboncopysubscriberpatch.patch (665 bytes) - added by cedrichurst 6 years ago.

Download all attachments as: .zip

Change History (2)

Changed 6 years ago by cedrichurst

comment:1 Changed 6 years ago by doki_pen

  • Resolution set to fixed
  • Status changed from new to closed

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.