[wp-trac] [WordPress Trac] #38071: Add status links to network/sites.php

WordPress Trac noreply at wordpress.org
Thu Oct 6 20:10:13 UTC 2016


#38071: Add status links to network/sites.php
-------------------------------------------------+-------------------------
 Reporter:  johnjamesjacoby                      |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  4.7
Component:  Networks and Sites                   |     Version:  3.0
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-screenshots needs-     |     Focuses:  multisite
  unit-tests                                     |
-------------------------------------------------+-------------------------

Comment (by rheinardkorf):

 Definitely needs unit tests.

 The following is not a reliable method.

 {{{
 protected function is_base_request() {
                 $vars = $_GET;
                 unset( $vars['paged'] );

                 if ( empty( $vars ) ) {
                         return true;
                 }

                 return 1 === count( $vars ) && ! empty( $vars['mode'] );
         }
 }}}

 Plugins may add additional functionality via URI, so simply unsetting
 'paged' or checking for only 'mode' is does not give the intended result.
 It is better to check for the presence/absence of 'site_status'.

 The rest is looking pretty good.

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


More information about the wp-trac mailing list