[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