[wp-trac] [WordPress Trac] #38079: Add hooks before output for each action in wp-login.php
WordPress Trac
noreply at wordpress.org
Thu May 2 17:33:21 UTC 2024
#38079: Add hooks before output for each action in wp-login.php
-------------------------------------------------+-------------------------
Reporter: jfarthing84 | Owner:
| voldemortensen
Type: enhancement | Status: assigned
Priority: normal | Milestone: Future
| Release
Component: Login and Registration | Version: 4.7
Severity: normal | Resolution:
Keywords: has-patch reporter-feedback | Focuses:
changes-requested close |
-------------------------------------------------+-------------------------
Changes (by hellofromTonya):
* keywords: has-patch reporter-feedback changes-requested => has-patch
reporter-feedback changes-requested close
Comment:
== Is it still needed today?
My first impressions lean towards "no" for these reasons:
* The ticket being stale this long can indicate not enough interest or use
cases.
* As previously noted, there are plenty of filter and action hooks
available for customization including style, messaging, redirects, etc.
In considering adding additional hooks, benefits vs impacts need to also
be considered.
* Impacts:
* Performance: each new hook adds processing time and memory usage.
* Maintenance: initial changes, long term backwards-compatibility,
documentation, etc.
* Benefits:
* Are there enough use cases?
* is there enough interests and value to users and extenders?
If benefits outweigh the impacts, then adding the hooks makes sense.
For the reasonings I shared, I'll mark the ticket as a `close` candidate.
If there's no follow-up or continued discussion within 2 months, this
ticket can be closed. Please note, if it gets closed, it could be reopened
when there's enough interest and benefits to move it forward.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/38079#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list