Make WordPress Core

Opened 13 years ago

Closed 10 years ago

#21729 closed defect (bug) (wontfix)

Better logic for comments email notification

Reported by: pavelevap's profile pavelevap Owned by:
Milestone: 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 13 years ago.

Download all attachments as: .zip

Change History (6)

#2 @knutsp
12 years ago

  • Cc knut@… added

#4 @jeremyfelt
11 years 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

#5 @rachelbaker
10 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.