Changeset 20022 for trunk/wp-admin/includes/class-wp-ms-themes-list-table.php

Timestamp:

02/28/2012 08:13:21 PM (15 years ago)

Author:

nacin

Message:

Do not save the last visited tab on the multisite Network Themes page and Site Themes tab. These are poor UX, see #18810 for plugins. Entering these screens will always default to 'all' themes view. see #20103.

File:

• trunk/wp-admin/includes/class-wp-ms-themes-list-table.php (modified) (1 diff)

trunk/wp-admin/includes/class-wp-ms-themes-list-table.php

@@ -16 +16 @@
                 global $status, $page;
 
-                $default_status = get_user_option( 'themes_last_view' );
-                if ( empty( $default_status ) )
-                        $default_status = 'all';
-                $status = isset( $_REQUEST['theme_status'] ) ? $_REQUEST['theme_status'] : $default_status;
+                $status = isset( $_REQUEST['theme_status'] ) ? $_REQUEST['theme_status'] : 'all';
                 if ( !in_array( $status, array( 'all', 'enabled', 'disabled', 'upgrade', 'search' ) ) )
                         $status = 'all';
-                if ( $status != $default_status && 'search' != $status )
-                        update_user_meta( get_current_user_id(), 'themes_last_view', $status );
 
                 $page = $this->get_pagenum();