Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#51276 closed defect (bug) (fixed)

Allow attachments to be added to a recovery mode email

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

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 4 years ago.

Download all attachments as: .zip

Change History (7)

@desrosj
4 years ago

#1 @TimothyBlynJacobs
4 years ago

  • Keywords commit added

LGTM!

#2 @desrosj
4 years ago

  • Owner set to desrosj
  • Status changed from new to assigned

#3 @desrosj
4 years ago

  • Resolution set to fixed
  • Status changed from assigned to 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
4 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
4 years ago

  • Keywords needs-dev-note added

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

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

#6 @daisyo
4 years ago

  • Keywords has-dev-note added; needs-dev-note removed
Note: See TracTickets for help on using tickets.