WordPress.org

Make WordPress Core

Opened 8 years ago

Closed 7 years ago

Last modified 6 years ago

#1352 closed enhancement (wontfix)

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 (2)

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

Download all attachments as: .zip

Change History (13)

comment:2 doey8 years ago

  • Patch set to No

comment:3 MC_incubus8 years ago

  • Patch changed from No to Yes

comment:4 filosofo7 years ago

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

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

comment:5 westi7 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

comment:6 westi7 years ago

  • Keywords bg|needs-patch added

comment:7 filosofo7 years ago

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

comment:8 westi7 years ago

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.

comment:9 Nazgul7 years ago

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

Brought the patch up to speed.

Nazgul7 years ago

comment:10 matt7 years ago

  • Resolution set to wontfix
  • Status changed from reopened to closed

comment:11 Nazgul6 years ago

  • Milestone 2.1 deleted
Note: See TracTickets for help on using tickets.