Ticket #1352 (closed enhancement: wontfix)

Opened 7 years ago

Last modified 5 years ago

wp_email() From: header doesn't include name

Reported by: doey Owned by: anonymous
Priority: normal Milestone:
Component: General Version: 1.5.1
Severity: normal Keywords: has-patch
Cc:

Description

The wp_email() From: header only includes the sender email address -- no name. In a shared hosting environment this is undesirable b/c while the admin email address is used the name string is whatever the php default is, usually the hostname. Adding the blogname setting makes this more aesthetically pleasing. A patch is attached.

Attachments

pluggable-functions.php.diff Download (666 bytes) - added by doey 7 years ago.
1352.diff Download (888 bytes) - added by Nazgul 5 years ago.

Change History

comment:1   doey7 years ago

comment:2   doey7 years ago

  • Patch set to No
  • Patch changed from No to Yes

doey7 years ago

  • Status changed from new to closed
  • Resolution set to fixed

As of 2.0 WP uses "wordpress@ . . . " as the name.

  • Status changed from closed to reopened
  • Resolution fixed deleted

I don't think this is really "fixed" having the name as well would be usefull.

It will need encoding though as the headers are 7bit

  • Keywords bg|needs-patch added

westi, isn't "being useful" more plugin territory? This is a pluggable function, after all.

For a simple change like this I wouldn't call it plugin territory.

Having to copy and paste the whole function into a plugin to change one line seems a bit extreme.

  • Keywords has-patch added; bg|needs-patch removed
  • Milestone set to 2.1

Brought the patch up to speed.

Nazgul5 years ago

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