Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Add check for author->mail in function _comment_notify_mailalert($comment)
Empty author->mail causes Error sending e-mail
Comment | File | Size | Author |
---|---|---|---|
#3 | 1576878_no_anonymous_notification.patch | 1.37 KB | greggles |
Comments
Comment #1
gregglesMakes sense, maybe we should look for an empty mail. That said...how did anonymous get subscribed to a notification?
Comment #2
dusov CreditAttribution: dusov commentedadmin/config/people/comment_notify
Check-box "Subscribe users to their node follow-up notification emails by default"
Thus, anonymous (who create node) get subscribed to a notification
Comment #3
gregglesHow about this, which adds some comments.
Comment #5
gregglesComment #6
greggles#3: 1576878_no_anonymous_notification.patch queued for re-testing.
Comment #7
gregglesNow committed http://drupalcode.org/project/comment_notify.git/commit/b1a8cf3
Thanks for pointing this out, dusov.