Add hooks before output for each action in wp-login.php

[jfarthing84]

7 years ago, in #9682, wp-login.php was made more pluggable. The thing is, if you just want to change the look (as my plugin "Theme My Login" does), you also have to replicate the logic.

Having an action that fires just before the login_header() call in each case of the action handler switch should be sufficient. One such hook is already present for one action: lost_password.

However, the hook register is already in use for the register link as is the format {$action}_form. So, I propose using pre_{$action}_form.

Patch incoming.

[voldemortensen]

38079.patch​ looks good to me, the @since docs will need to be updated though.

[dingo_bastard]

While I agree that this is cool thing to have, ​login_header function already has a lot of filters and action hooks that you can use to modify how the login form looks. So you can easily use login_header action for instance to place something before logo in the login form.
There is also login_messages filter that you can use to modify the login messages.

It would be better to place actions inside the login_header function, you could also filter them based on the error message.

[dingo_bastard]

Added action hooks inside login_header function

[jbpaul17]

Punting to 4.8.1 per today's bug scrub.

[jbpaul17]

Per today's bug scrub, we'll punt this as the focus for 4.8.1 is regressions only.

[voldemortensen]

38079.patch​ still applies cleanly, @since docs just need to be updated.

[SergeyBiryukov]

I agree with @dingo_bastard that existing hooks in login_header() can be used to customize the login page, so I don't see how 38079.patch​ would make it more customizable.

38079.2.patch​ makes more sense to me, but I'd like know a use case for these new actions that cannot be achieved with existing hooks.

[SergeyBiryukov]

On second thought, hooks before any output might indeed be helpful for redirects or more extensive customizations.

Perhaps they need better names though. pre_lostpassword_form sounds like it fires after the header and right before the form, not before any output on the page.

If the goal is to short-circuit login_header(), maybe pre_login_header should be added instead.

[SergeyBiryukov]

There's a ​login_form_{$action} dynamic hook, so maybe pre_login_form_{$action}_header would be a better name for these new hooks from 38079.patch​, which would expand to:

• pre_login_form_lostpassword_header
• pre_login_form_resetpass_header
• pre_login_form_register_header
• pre_login_form_login_header

[dingo_bastard]

The do_action hooks I added could be proven useful to add some additional messages (I think I was working for a client 6 months ago, and I could have benefitted from such hooks).