Changeset 48134

Timestamp:

06/23/2020 05:54:15 AM (2 years ago)

Author:

whyisjake

Message:

Quick/Bulk Edit: Ensure the proper actions is triggered when using the bulk updater.

If a user selects the top option, then chooses a different option, the top selection takes precedence. This update gives a new name to the bottom action, ensuring the proper update is carried out.

Fixes 46872.

Props clayray, garrett-eclipse, subrataemfluence.

File:

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

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

@@ -474 +474 @@
         echo "</select>\n";
 
-        submit_button( __( 'Apply' ), 'action', '', false, array( 'id' => "doaction$two" ) );
+        submit_button( __( 'Apply' ), 'action', "doaction$two", false, array( 'id' => "doaction$two" ) );
         echo "\n";
     }
@@ -490 +490 @@
         }
 
+        if ( isset( $_REQUEST['doaction2'] ) && isset( $_REQUEST['action2'] ) && -1 != $_REQUEST['action2'] ) {
+            return $_REQUEST['action2'];
+        }
+
         if ( isset( $_REQUEST['action'] ) && -1 != $_REQUEST['action'] ) {
             return $_REQUEST['action'];
-        }
-
-        if ( isset( $_REQUEST['action2'] ) && -1 != $_REQUEST['action2'] ) {
-            return $_REQUEST['action2'];
         }