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 .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from ixokai. Next status will be '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.