Changeset 51161

Timestamp:

06/15/2021 04:50:49 PM (3 years ago)

Author:

davidbaumwald

Message:

Media: Ensure $post_ids is evaluated properly when processing bulk actions.

After [51111], the $post_ids variable is now initialized as an empty array when processing a bulk action. As such, the original check using isset on $post_ids will always evaluate to true. This change swaps the isset checks for empty to check array length instead.

Props david.binda, hellofromTonya.
Fixes #53411.

File:

• trunk/src/wp-admin/upload.php (modified) (3 diffs)

trunk/src/wp-admin/upload.php

@@ -149 +149 @@
 
         case 'trash':
-            if ( ! isset( $post_ids ) ) {
+            if ( empty( $post_ids ) ) {
                 break;
             }
@@ -170 +170 @@
             break;
         case 'untrash':
-            if ( ! isset( $post_ids ) ) {
+            if ( empty( $post_ids ) ) {
                 break;
             }
@@ -185 +185 @@
             break;
         case 'delete':
-            if ( ! isset( $post_ids ) ) {
+            if ( empty( $post_ids ) ) {
                 break;
             }