Changeset 31828

Timestamp:

03/19/2015 02:25:05 AM (10 years ago)

Author:

wonderboymusic

Message:

Add WP_Posts_List_Table::is_base_request() to determine if the current "view" is the "All" (default) view.

Fixes #24869.

File:

• trunk/src/wp-admin/includes/class-wp-posts-list-table.php (modified) (2 diffs)

trunk/src/wp-admin/includes/class-wp-posts-list-table.php

@@ -149 +149 @@
     }
 
+    /**
+     * Determine if the current view is the "All" view
+     *
+     * @since 4.2.0
+     *
+     * @return boolean
+     */
+    protected function is_base_request() {
+        if ( empty( $_GET ) ) {
+            return true;
+        } elseif ( 1 === count( $_GET ) && ! empty( $_GET['post_type'] ) ) {
+            return $this->screen->post_type === $_GET['post_type'];
+        }
+    }
+
     protected function get_views() {
         global $locked_post_status, $avail_post_stati;
@@ -177 +192 @@
             $total_posts -= $num_posts->$state;
 
-        $class = empty( $class ) && empty( $_REQUEST['post_status'] ) && empty( $_REQUEST['show_sticky'] ) ? ' class="current"' : '';
-        $status_links['all'] = "<a href='edit.php?post_type=$post_type{$allposts}'$class>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_posts, 'posts' ), number_format_i18n( $total_posts ) ) . '</a>';
+        if ( empty( $class ) && $this->is_base_request() ) {
+            $class =  ' class="current"';
+        }
+
+        $all_inner_html = sprintf(
+            _nx(
+                'All <span class="count">(%s)</span>',
+                'All <span class="count">(%s)</span>',
+                $total_posts,
+                'posts'
+            ),
+            number_format_i18n( $total_posts )
+        );
+
+        $status_links['all'] = "<a href='edit.php?post_type=$post_type{$allposts}'$class>" . $all_inner_html . '</a>';
 
         foreach ( get_post_stati(array('show_in_admin_status_list' => true), 'objects') as $status ) {