WordPress.org

Make WordPress Core

Opened 20 months ago

Last modified 5 months ago

#21729 new defect (bug)

Better logic for comments email notification

Reported by: pavelevap Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Comments Keywords: close
Focuses: Cc:

Description

Please see attached patch.

  • If comment_author_email is known, it can be used in "From" email header.
  • Also in "Reply-To" header we can use comment_author.

I am not sure if there is any reason for current logic, but my friend noticed it and he thought that this way it would be better...

Attachments (1)

comments_email.patch (883 bytes) - added by pavelevap 20 months ago.

Download all attachments as: .zip

Change History (5)

pavelevap20 months ago

comment:2 knutsp17 months ago

  • Cc knut@… added

comment:4 jeremyfelt5 months ago

  • Component changed from General to Comments
  • Keywords close added

Suggesting we close due to the reasoning laid out in #1593. Setting From to anything other than settings from the originating server can quickly cause spam issues.

Filters are available to modify the default settings if different behavior is desired:

  • comment_notification_recipients
  • comment_notification_headers
Note: See TracTickets for help on using tickets.