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: | 
  
     | 
      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
Mailcomponent.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 :-)