WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 11 years ago

#21729 closed defect (bug) (wontfix)

Better logic for comments email notification

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

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 14 years ago.

Download all attachments as: .zip

Change History (6)

@pavelevap
14 years ago

#2 @knutsp
14 years ago

  • Cc knut@… added

#4 @jeremyfelt
13 years ago

  • Component General → 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
11 years ago

  • Milestone Awaiting Review
  • Resolution → wontfix
  • Status new → closed
Note: See TracTickets for help on using tickets.