WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#51276 closed defect (bug) (fixed)

Allow attachments to be added to a recovery mode email

Reported by: desrosj Owned by: desrosj
Priority: low Milestone: 5.6
Component: Site Health Version: 5.2
Severity: normal Keywords: has-patch commit has-dev-note
Cc: Focuses: administration

Description

wp_mail() allows attachments to be passed but the recovery mode email (more specifically the recovery_mode_email filter) does not support this.

Attachments (1)

51276.diff​ (1.8 KB ) - added by desrosj 6 years ago.

Download all attachments as: .zip

Change History (7)

@desrosj
6 years ago

#1 @TimothyBlynJacobs
6 years ago

  • Keywords commit added

LGTM!

#2 @desrosj
6 years ago

  • Owner set to desrosj
  • Status new → assigned

#3 @desrosj
6 years ago

  • Resolution → fixed
  • Status assigned → closed

In 48964:

Site Health: Allow attachments to be added to recovery mode emails.

The wp_mail() function has the ability to add attachments to emails. There is currently no way to add attachments to the recovery mode email sent out to site admins when a PHP error is encountered on their site.

This change adds that ability through the use of the recovery_mode_email filter, allowing developers to use the full capabilities of wp_mail().

Props desrosj, timothyblynjacobs.
Fixes #51276.

#4 @SergeyBiryukov
6 years ago

In 48994:

Docs: Add a @since note for the recovery_mode_email filter about the $email argument now including the attachments key.

Follow-up to [48964].

See #51276.

#5 @desrosj
6 years ago

  • Keywords needs-dev-note added

Let's note this bug fix and the use case in the miscellaneous dev note.

Last edited 6 years ago by desrosj (previous) (diff)

#6 @daisyo
6 years ago

  • Keywords has-dev-note added; needs-dev-note removed

Dev note published: ​https://make.wordpress.org/core/2020/11/20/miscellaneous-developer-focused-changes-in-wordpress-5-6/

Note: See TracTickets for help on using tickets.