Ticket #16166: garyc40.16166.diff

wp-admin/edit-comments.php

@@ -13 +13 @@
 
 $wp_list_table = get_list_table('WP_Comments_List_Table');
 $pagenum = $wp_list_table->get_pagenum();
-
+list( $current_orderby, $current_order ) = $wp_list_table->get_order_info();
 $doaction = $wp_list_table->current_action();
 
 if ( $doaction ) {
@@ -38 +38 @@
 
         $redirect_to = remove_query_arg( array( 'trashed', 'untrashed', 'deleted', 'spammed', 'unspammed', 'approved', 'unapproved', 'ids' ), wp_get_referer() );
         $redirect_to = add_query_arg( 'paged', $pagenum, $redirect_to );
+        if ( $current_orderby )
+                $redirect_to = add_query_arg( array( 'order' => $current_order, 'orderby' => $current_orderby ), $redirect_to );
 
         foreach ( $comment_ids as $comment_id ) { // Check the permissions on each
                 if ( !current_user_can( 'edit_comment', $comment_id ) )

wp-admin/edit-tags.php

@@ -94 +94 @@
         foreach ( $tags as $tag_ID ) {
                 wp_delete_term( $tag_ID, $taxonomy );
         }
-
+        $pagenum = $wp_list_table->get_pagenum();
+        list( $current_orderby, $current_order ) = $wp_list_table->get_order_info();
         $location = 'edit-tags.php?taxonomy=' . $taxonomy;
         if ( 'post' != $post_type )
                 $location .= '&post_type=' . $post_type;
@@ -103 +104 @@
                         $location = $referer;
         }
 
-        $location = add_query_arg( 'message', 6, $location );
+        $location = add_query_arg( array( 'message' => 6, 'paged' => $pagenum ), $location );
+        if ( $current_orderby )
+                $location = add_query_arg( array( 'order' => $current_order, 'orderby' => $current_orderby ), $location );
+
         wp_redirect( $location );
         exit;
 

wp-admin/edit.php

@@ -25 +25 @@
 
 $wp_list_table = get_list_table('WP_Posts_List_Table');
 $pagenum = $wp_list_table->get_pagenum();
+list( $current_orderby, $current_order ) = $wp_list_table->get_order_info();
 
 // Back-compat for viewing comments of an entry
 foreach ( array( 'p', 'attachment_id', 'page_id' ) as $_redirect ) {
@@ -42 +43 @@
 
         $sendback = remove_query_arg( array('trashed', 'untrashed', 'deleted', 'ids'), wp_get_referer() );
         $sendback = add_query_arg( 'paged', $pagenum, $sendback );
+        if ( $current_orderby )
+                $sendback = add_query_arg( array( 'order' => $current_order, 'orderby' => $current_orderby ), $sendback );
         if ( strpos($sendback, 'post.php') !== false )
                 $sendback = admin_url($post_new_file);
 

wp-admin/includes/class-wp-list-table.php

@@ -611 +611 @@
                 $hidden = array_intersect( array_keys( $columns ), array_filter( $hidden ) );
                 return count( $columns ) - count( $hidden );
         }
+        
+        function get_order_info() {
+                if ( isset( $_GET['orderby'] ) )
+                        $current_orderby = $_GET['orderby'];
+                else
+                        $current_orderby = '';
+
+                if ( ! $current_orderby )
+                        $current_order = '';
+                elseif ( isset( $_GET['order'] ) && 'desc' == $_GET['order'] )
+                        $current_order = 'desc';
+                else
+                        $current_order = 'asc';
+                        
+                return array($current_orderby, $current_order);
+        }
 
         /**
          * Print column headers, accounting for hidden and sortable columns.
@@ -627 +643 @@
 
                 $current_url = ( is_ssl() ? 'https://' : 'http://' ) . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];
 
-                if ( isset( $_GET['orderby'] ) )
-                        $current_orderby = $_GET['orderby'];
-                else
-                        $current_orderby = '';
-
-                if ( isset( $_GET['order'] ) && 'desc' == $_GET['order'] )
-                        $current_order = 'desc';
-                else
-                        $current_order = 'asc';
+                list( $current_orderby, $current_order ) = $this->get_order_info();
 
                 foreach ( $columns as $column_key => $column_display_name ) {
                         $class = array( 'manage-column', "column-$column_key" );
@@ -728 +736 @@
          * @access protected
          */
         function display_tablenav( $which ) {
-                if ( 'top' == $which )
+                if ( 'top' == $which ) {
                         wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+                        list( $current_orderby, $current_order ) = $this->get_order_info();
+                        echo '<input type="hidden" name="orderby" value="' . $current_orderby . '" /><input type="hidden" name="order" value="' . $current_order . '" />';
+                }
 ?>
         <div class="tablenav <?php echo esc_attr( $which ); ?>">
 

wp-admin/js/list-table.dev.js

@@ -103 +103 @@
                 if ( 'object' != typeof response ) {
                         this.handle_error();
                 } else {
-                        var tablenav = $('.tablenav-pages');
+                        var tablenav = $('.tablenav-pages'),
+                                order = $.query.GET('order'),
+                                orderby = (order) ? $.query.GET('orderby') : '';
 
                         this.stop_loading();
 
@@ -124 +126 @@
                         // Disable buttons that should noop.
                         tablenav.find('.first-page, .prev-page').toggleClass('disabled', 1 == $.query.GET('paged'));
                         tablenav.find('.next-page, .last-page').toggleClass('disabled', response.total_pages == $.query.GET('paged'));
-
+                        
+                        $('input[name=order]').val(order);
+                        $('input[name=orderby]').val(orderby);
+                        
                         $('th.column-cb :input').attr('checked', false);
 
                         if ( history.replaceState ) {