Changeset 59759

Timestamp:

02/04/2025 03:24:19 AM (18 months ago)

Author:

jorbin

Message:

Administration: Fix pagination in categories, tags, and plugins tables.

Fix an issue introduced in [59134] that prevented manual entry of a page number in the pagination input field from navigating pages. Requiring validation of the bulk actions input also impacted other inputs nested in the same form.

Also fixes a pre-existing bug where it was not possible to navigate to page 1 using the input field.

Reviewed by jorbin.
Merges [59727] and [59746] to the 6.7 branch.

Props ffffelix, im3dabasia1, apermo, rishavdutta, joedolson, swissspidy, jorbin, joedolson, tobiasbg, mamaduka.
Fixes #62534.

Location:

branches/6.7

Files:

• . (modified) (1 prop)
• src/js/_enqueues/admin/common.js (modified) (1 diff)
• src/wp-admin/edit-tags.php (modified) (1 diff)

branches/6.7/src/js/_enqueues/admin/common.js

@@ -1313 +1313 @@
         $( '.bulkactions' ).parents( 'form' ).on( 'submit', function( event ) {
                 var form = this,
-                        submitterName = event.originalEvent && event.originalEvent.submitter ? event.originalEvent.submitter.name : false;
+                        submitterName = event.originalEvent && event.originalEvent.submitter ? event.originalEvent.submitter.name : false,
+                        currentPageSelector = form.querySelector( '#current-page-selector' );
+
+                if ( currentPageSelector && currentPageSelector.defaultValue !== currentPageSelector.value ) {
+                        return; // Pagination form submission.
+                }
 
                 // Observe submissions from posts lists for 'bulk_action' or users lists for 'new_role'.

branches/6.7/src/wp-admin/edit-tags.php

@@ -216 +216 @@
         if ( $pagenum > 1 ) {
                 $location = add_query_arg( 'paged', $pagenum, $location ); // $pagenum takes care of $total_pages.
+        }
+        if ( 1 === $pagenum ) {
+                $location = remove_query_arg( 'paged', $location );
         }