[wp-trac] [WordPress Trac] #43536: Network registration page

WordPress Trac noreply at wordpress.org
Tue Mar 20 22:21:30 UTC 2018


#43536: Network registration page
------------------------------------+------------------------------
 Reporter:  SGr33n                  |       Owner:
     Type:  defect (bug)            |      Status:  new
 Priority:  normal                  |   Milestone:  Awaiting Review
Component:  Login and Registration  |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:                          |     Focuses:  multisite
------------------------------------+------------------------------

Comment (by jeremyfelt):

 Hi @SGr33n, thanks for the ticket.

 I think this can be traced to an assumption where if a
 [https://github.com/WordPress/wordpress-
 develop/blob/8df8cf2df14fe26174f97af5bb17d63f2e867231/src/wp-includes
 /class-wp-query.php#L949 query isn't detected to be anything else], then
 we assume `is_home()` is `true`.

 Lacking some kind of `is_wp_signup()` or a refactor of that area, there
 might not be a great way of avoiding the page ID being output.

 A couple possibilities:

 * Add a `body_class` filter in `wp-signup.php` to always add a `wp-signup`
 or similar class.
 * Leave it to themes to check `$_SERVER['REQUEST_URI']` or `$wp->request`
 and add their own body class filter.

 My gut says that if we add a filter in `wp-signup.php`, we should do so
 after things have been refactored via a ticket like #31928 or #33500. It's
 entirely possible to add a filter at the theme level now and if we made
 changes to `wp-signup.php`, we add to the list of things to keep track of.

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


More information about the wp-trac mailing list