Add $error argument to lost_password action

[wonderyan]

I'm trying to hook login process to display my own forms and in wp-login.php for lostpassword case the action lost_password is fired just before displaying the corresponding form.

It would be very usefull that this action fires with $errors as arguments as it could have errors of type invalidkey & expiredkey ?

[sebakurzyn]

Hey guys! I added $error parameter and docs.

[sebakurzyn]

Added $error parameter in lost_password action.

[peterwilsoncc]

@sebakurzyn

Thanks for the patch!

It can be a little confusing but we use the resolution and status to indicate whether a ticket a patch has been committed to the code base. Marking it worksforme closes the ticket and indicates the decision has been made not to commit it.

I've reopened this ticket to make sure it appears in searches for tickets that are going to be committed to the project.

I'll also refer you to the ​WordPress PHP Coding standards. There are a few things in the patch that need to be reformatted but they can be fixed by the person who commits the code.

[sebakurzyn]

Thanks a lot for answer and help. I will check the coding standards. So next time I should simply add the patch and don't change a ticket status? Am I right? :)

[peterwilsoncc]

@sebakurzyn that's correct, adding the has-patch keyword (as you did) is helpful too.

[wonderyan]

Thanks for your considerations

[johnbillion]

In 43542:

Login and Registration: Pass the $errors parameter to the lost_password action.

Props sebakurzyn

Fixes #44512

[desrosj]

This @since tag needs to be changed to 5.0.1.

[audrasjb]

@since refresh for 5.0.3

[audrasjb]

Hi,

44512.2.diff adds @since refresh for 5.0.3.

[desrosj]

Let's leave this in the 5.1 release as it is outside of the 5.0.3 scope. The @since just needs to be updated to 5.1.0.

[desrosj]

In 44396:

Docs: Update since annotation for new lost_password action parameter.

Introduced in [43542].

Fixes #44512.