WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#49521 closed enhancement (fixed)

Append lostpassword_errors filter to `retrieve_password()`

Reported by: wpdo5ea Owned by: SergeyBiryukov
Priority: normal Milestone: 5.5
Component: Login and Registration Version: 5.4
Severity: normal Keywords: has-patch has-dev-note
Cc: Focuses:

Description

Currently comparing to login and register process, lost password submission didn't have any error filter, which makes it hard to append more errors.

Similar to the part in register_new_user() function, added the filter lostpassword_errors to errors encountered.

Attachments (2)

lostpassword_errors_filter.diff​ (667 bytes ) - added by wpdo5ea 7 years ago.
49521.2.patch​ (681 bytes ) - added by dilipbheda 7 years ago.
I did change in filter comment area. I am not sure it is better to understand or not for our user.

Download all attachments as: .zip

Change History (8)

@wpdo5ea
7 years ago

#1 @SergeyBiryukov
7 years ago

  • Milestone Awaiting Review → 5.5
  • Owner set to SergeyBiryukov
  • Status new → reviewing

@dilipbheda
7 years ago

I did change in filter comment area. I am not sure it is better to understand or not for our user.

This ticket was mentioned in ​Slack in #core by david.baumwald. ​View the logs.
6 years ago

#3 @davidbaumwald
6 years ago

  • Keywords needs-dev-note added

Since this adds a new filter, marking for a dev-note call-out on the Misc Dev Note.

#4 @SergeyBiryukov
6 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 48084:

Login and Registration: Introduce lostpassword_errors filter in retrieve_password() for errors encountered on a password reset request.

This complements the registration_errors filter in register_new_user().

Props wpdo5ea, dilipbheda.
Fixes #49521.

#5 @SergeyBiryukov
6 years ago

In 48085:

Docs: Add missing $user_data parameter name to lostpassword_post and lostpassword_errors filter DocBlocks.

Follow-up to [48084].

See #49521, #49572.

#6 @desrosj
6 years ago

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

This received a call out in the Miscellaneous Developer Changes in 5.5 dev note: ​https://make.wordpress.org/core/2020/07/29/miscellaneous-developer-focused-changes-in-wordpress-5-5/.

Note: See TracTickets for help on using tickets.