Opened 8 months ago
Last modified 2 months ago
#47605 new feature request
Email sender name change with site title in wp-includes/pluggable.php
Reported by: | viitorcloudvc | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | 2.2 |
Component: | Keywords: | has-patch needs-testing | |
Focuses: | Cc: |
Description
A variable $from_name has default WordPress name as sender on line 316 in file name pluggable.php exist in wp-include folder.
Where there is a line replacement with site name of WordPress website. The result will show in email with Site title of website in email of sender name.
Attachments (3)
Change History (12)
#2
@
8 months ago
Hello @audrasjb,
Thank you.
Thank you again for suggestion. I am aware about hook but it would be great to have default inbuilt function for sender name. For this purpose I am requesting to add in core functionality.
#3
in reply to:
↑ 1
@
8 months ago
- Keywords needs-refresh added
- Version set to 2.2
As @audrasjb said there is already filter available to change default $from_name
. please remove an extra space from 45562.patch.patch patch.
Replying to audrasjb:
Also, looks like there is an extra space that should be removed in the patch provided :-)
#5
in reply to:
↑ 1
@
8 months ago
Hello @audrasjb,
I have added new patch file named as:45562.patch.2.patch . Please review it.
Replying to audrasjb:
Hello @viitorcloudvc and welcome to WordPress Trac,
Thanks for the ticket, I'm moving it to the
By the way, there is already a hook to filter the From Name parameter for all WP mails: https://developer.wordpress.org/reference/hooks/wp_mail_from_name/
Also, looks like there is an extra space that should be removed in the patch provided :-)
Hello @viitorcloudvc and welcome to WordPress Trac,
Thanks for the ticket, I'm moving it to the
Mail
component.By the way, there is already a hook to filter the From Name parameter for all WP mails: https://developer.wordpress.org/reference/hooks/wp_mail_from_name/
Also, looks like there is an extra space that should be removed in the patch provided :-)