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

Timestamp:

02/28/2012 08:13:21 PM (12 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();