Make WordPress Core

Opened 14 years ago

Closed 10 years ago

#13857 closed enhancement (invalid)

Email notifications lack a link to the comment, from address could be improved

Reported by: hexley's profile hexley Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Comments Keywords:
Focuses: Cc:

Description

Email Notification
For the 6+ years I believe I have been using WordPress, the emails that notify me someone has made a comment have several links in them.

Fields within an email notification
Email, Url, Whois, Comment, All Comments, Delete, Spam

The comment, at least in plain text emails, contains html markup, which often is encoded making it hard to read, depending on the comment complexity. I suggest a simple strip_html style function be ran on the comment copy.

The link to the comments only takes you to the comments, but does not add in the identifier in the url to take you to that specific comment. The blog I am managing has 30+ comments on each page, making it harder than it should be to get to the comment and reply.

An example of a better method might be:
http://example.com/foo-bar-baz/#comment-7348
Compared to:
http://example.com/foo-bar-baz/#comments


Email Headers
The From header in the email is is correctly set to the "Name" value the user filled into the comment form, the email address is set to wordpress@… where example.com is the domain of the site.

However, the reply-to header is correctly set to the actual sender. Bounces (NDR's) and the cosmetic view of an email could all be improved with a simple condition:

if is_valid(emaIl) then
  from = name & ' <' & email & '>'
  reply-to = from
else
  // looks like they did not set the data
  // fall back to some sane defaults
end if

Change History (4)

#1 @nacin
13 years ago

  • Milestone changed from Awaiting Review to Future Release
  • Type changed from defect (bug) to enhancement

The link now takes you directly to the comment (in 3.1-alpha).

Not entirely sure what you mean by the reply-to headers.

Stripping HTML tags makes sense there.

#2 @nacin
13 years ago

  • Component changed from General to Comments

#3 @jeremyfelt
10 years ago

  • Keywords changed from email notifications,reply-to header, from header to email notifications reply-to header, from header

Comment links issue is resolved in #12797

Related reply-to headers ticket: #21729 (possibly duplicate)

#4 @jeremyfelt
10 years ago

  • Keywords email notifications reply-to header from header removed
  • Milestone Future Release deleted
  • Resolution set to invalid
  • Status changed from new to closed

Closing due to (a) almost identical issue in #21729, (b) concerns raised in #1593, and (c) filters available to modify default behavior.

  • comment_notification_recipients
  • comment_notification_headers

Please feel free to reopen with additional feedback.

Note: See TracTickets for help on using tickets.