If a user is "blocked" (instead of active) then they probably shouldn't get e-mails. This can be checked when the e-mail is being sent but should also be checked probably in cron and we should delete subscriptions for inactive users in cron.

Anything else?

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

greggles’s picture

Status: Active » Needs review
FileSize
1.68 KB

And here's a patch to do this...

greggles’s picture

Title: no mailing to blocked, periodic cleanup of subscriptions » Don't mail to blocked users
Assigned: Unassigned » greggles
Status: Needs review » Fixed

After thinking about this some more, I realized that maybe it's OK to keep around the subscriptions for blocked users. If that status = 1 becomes an issue on a site then we can reconsider this.

Other than that...this is fixed.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.