Modify

Opened 4 years ago

Last modified 20 months ago

#7834 new defect

Emails are attempted to be sent even when there are no recipients

Reported by: anonymous Owned by: hasienda
Priority: normal Component: AnnouncerPlugin
Severity: normal Keywords: email notification
Cc: rjollos, doki_pen Trac Release: 0.12

Description (last modified by hasienda)

Before starting delivery, EmailDistributor._do_send should check the length of recip_adds and only send email, if there are actually recipients.

Attachments (0)

Change History (3)

comment:1 Changed 20 months ago by hasienda

  • Cc rjollos doki_pen added
  • Description modified (diff)
  • Keywords email notification added
  • Owner changed from doki_pen to hasienda

comment:2 Changed 20 months ago by hasienda

(In [12326]) TracAnnouncer: Don't attempt email distribution without recipients, refs #7834.

Felt like a re-arranged and slightly simplified code would make it easier to
implement the requested check in EmailDistributor._do_send. IMHO it did.

comment:3 Changed 20 months ago by hasienda

But events without subscription won't even fire EmailDistributor, so I'm currently at a loss, how to test this. Ideas?

Add Comment

Modify Ticket

Action
as new .
Author


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

 
Note: See TracTickets for help on using tickets.