WordPress.org

Make WordPress Core

Opened 13 years ago

Closed 12 years ago

Last modified 12 months ago

#1352 closed enhancement (wontfix)

wp_email() From: header doesn't include name

Reported by: doey Owned by:
Milestone: Priority: normal
Severity: normal Version: 1.5.1
Component: General Keywords: has-patch
Focuses: 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 13 years ago.
1352.diff (888 bytes) - added by Nazgul 12 years ago.

Download all attachments as: .zip

Change History (15)

#2 @doey
13 years ago

  • Patch set to No

#3 @MC_incubus
13 years ago

  • Patch changed from No to Yes

#4 @filosofo
12 years ago

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

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

#5 @westi
12 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

#6 @westi
12 years ago

  • Keywords bg|needs-patch added

#7 @filosofo
12 years ago

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

#8 @westi
12 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.

#9 @Nazgul
12 years ago

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

Brought the patch up to speed.

@Nazgul
12 years ago

#10 @matt
12 years ago

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

#11 @Nazgul
11 years ago

  • Milestone 2.1 deleted

This ticket was mentioned in Slack in #polyglots by ocean90. View the logs.


2 years ago

This ticket was mentioned in Slack in #core-editor by brentjett. View the logs.


12 months ago

Note: See TracTickets for help on using tickets.