Changeset 51161

Timestamp:

06/15/2021 04:50:49 PM (5 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;
                         }