[wp-trac] [WordPress Trac] #15384: wp-login.php refactor

WordPress Trac noreply at wordpress.org
Wed Oct 7 02:50:24 UTC 2015


#15384: wp-login.php refactor
-------------------------------------------------+-------------------------
 Reporter:  nacin                                |       Owner:
     Type:  enhancement                          |  valendesigns
 Priority:  normal                               |      Status:  assigned
Component:  Login and Registration               |   Milestone:  Awaiting
 Severity:  normal                               |  Review
 Keywords:  dev-feedback needs-patch needs-      |     Version:
  unit-tests                                     |  Resolution:
                                                 |     Focuses:
-------------------------------------------------+-------------------------
Changes (by valendesigns):

 * keywords:  has-patch dev-feedback => dev-feedback needs-patch needs-unit-
     tests
 * owner:   => valendesigns
 * status:  reopened => assigned


Comment:

 This ticket has the potential to greatly impact the Two Factor feature
 plugin. I can do a thorough review of  [attachment:15384.patch] in the
 next day or two and discuss my findings here and at the weekly meeting on
 Thursday in `core-passwords`.

 In order to merge 2fa into Core for the 4.5 release, we would have to do
 something along these lines anyhow. So I'm for changing how we access
 these functions and build the `wp-login.php` page. Though making sure we
 are thinking about backwards compatibility, potential plugin breakage, and
 all possible use cases is extremely important for such a critical system.

 @jfarthing84 Great effort so far, though because this patch is unfinished
 and doesn't work with interim login in the Customizer, I feel it should
 still be set to `needs-patch`.

 Does anyone have objections to setting the milestone to 4.4?

--
Ticket URL: <https://core.trac.wordpress.org/ticket/15384#comment:32>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list