Opened 5 years ago
Last modified 5 years ago
#47004 assigned enhancement
WordPress email subject update - phase 2
Reported by: | ramiy | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Keywords: | has-patch | |
Focuses: | Cc: |
Description
WordPress send emails, using wp_mail()
function, to inform the site admin when some important tasks are done and when it needs user confirmation.
In #37940 we standardized email subjects used by WordPress. We updated subject texts, added unified translation comments and added site title as a prefix to all the email subjects.
Now we should standardized the filters used to filter WordPress emails. Why? Because not all emails can be filtered.
I prefer to do this in two steps:
- Separate the email subject from the
wp_mail()
function. - Add new filters to filter the email subject.
See the attached patches.
Attachments (1)
Change History (6)
#2
@
5 years ago
- Focuses administration removed
- Type changed from defect (bug) to enhancement
Related:
Unofficial wp_mail()
usage docs for reference: https://github.com/johnbillion/wp_mail
Note: See
TracTickets for help on using
tickets.
This patch removed the email subject translation string from the
wp_mail()
functions into a separate$subject
var.I should mention that this conventions is used in other places in the code.
As a separate variable, we will be able to filter those email subjects.