[wp-trac] [WordPress Trac] #32508: Action Hooks for empty username and empty password
WordPress Trac
noreply at wordpress.org
Wed May 27 20:24:51 UTC 2015
#32508: Action Hooks for empty username and empty password
-------------------------+------------------------------
Reporter: jwarren | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Plugins | Version: trunk
Severity: normal | Resolution:
Keywords: | Focuses:
-------------------------+------------------------------
Comment (by jwarren):
Replying to [comment:1 MikeHansenMe]:
> Instead of adding new hooks is there a reason we exclude those from
triggering 'wp_login_failed'? If not we could just pass the additional
param into that hook. [attachment: 32508.2.diff] is an example and still
needs some docs.
That was actually my initial idea and would definitely be a cleaner
option. But I'm concerned about breaking current implementations. If
people don't expect their action to fire on `wp_login_failed` then it
could go awry. We also don't have a `$username` to pass if we're failing
due to `empty_username`. We could end up with white screen of death PHP
errors in code that expects it.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/32508#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list