WordPress.org

Make WordPress Core

Ticket #44081: 44081.8.diff

File 44081.8.diff, 3.8 KB (added by hellofromTonya, 9 months ago)

Refreshed to latest wordpress-develop master branch. Added guard clause to unwrap code for improved readability.

  • src/wp-admin/includes/class-wp-privacy-requests-table.php

    diff --git a/src/wp-admin/includes/class-wp-privacy-requests-table.php b/src/wp-admin/includes/class-wp-privacy-requests-table.php
    index 7f3476202f..dbc9309a10 100644
    a b abstract class WP_Privacy_Requests_Table extends WP_List_Table { 
    226226                $action      = $this->current_action();
    227227                $request_ids = isset( $_REQUEST['request_id'] ) ? wp_parse_id_list( wp_unslash( $_REQUEST['request_id'] ) ) : array();
    228228
    229                 $count = 0;
    230 
    231                 if ( $request_ids ) {
    232                         check_admin_referer( 'bulk-privacy_requests' );
     229                if ( empty( $request_ids ) ) {
     230                        return;
    233231                }
    234232
     233                $count    = 0;
     234                $failures = 0;
     235
     236                check_admin_referer( 'bulk-privacy_requests' );
     237
    235238                switch ( $action ) {
    236239                        case 'resend':
    237240                                foreach ( $request_ids as $request_id ) {
    abstract class WP_Privacy_Requests_Table extends WP_List_Table { 
    239242
    240243                                        if ( $resend && ! is_wp_error( $resend ) ) {
    241244                                                $count++;
     245                                        } else {
     246                                                $failures++;
    242247                                        }
    243248                                }
    244249
    245                                 add_settings_error(
    246                                         'bulk_action',
    247                                         'bulk_action',
    248                                         /* translators: %d: Number of requests. */
    249                                         sprintf( _n( 'Re-sent %d request.', 'Re-sent %d requests.', $count ), $count ),
    250                                         'success'
    251                                 );
     250                                if ( $count ) {
     251                                        add_settings_error(
     252                                                'bulk_action',
     253                                                'bulk_action',
     254                                                /* translators: %d: Number of requests. */
     255                                                sprintf( _n( '%d confirmation request re-sent successfully.', '%d confirmation requests re-sent successfully.', $count ), $count ),
     256                                                'success'
     257                                        );
     258                                }
     259
     260                                if ( $failures ) {
     261                                        add_settings_error(
     262                                                'bulk_action',
     263                                                'bulk_action',
     264                                                /* translators: %d: Number of requests. */
     265                                                sprintf( _n( '%d confirmation request failed to resend.', '%d confirmation requests failed to resend.', $failures ), $failures ),
     266                                                'error'
     267                                        );
     268                                }
     269
    252270                                break;
     271
    253272                        case 'complete':
    254273                                foreach ( $request_ids as $request_id ) {
    255274                                        $result = _wp_privacy_completed_request( $request_id );
    abstract class WP_Privacy_Requests_Table extends WP_List_Table { 
    267286                                        'success'
    268287                                );
    269288                                break;
     289
    270290                        case 'delete':
    271291                                foreach ( $request_ids as $request_id ) {
    272292                                        if ( wp_delete_post( $request_id, true ) ) {
    273                                                 $count ++;
     293                                                $count++;
     294                                        } else {
     295                                                $failures++;
    274296                                        }
    275297                                }
    276298
    277                                 add_settings_error(
    278                                         'bulk_action',
    279                                         'bulk_action',
    280                                         /* translators: %d: Number of requests. */
    281                                         sprintf( _n( 'Deleted %d request.', 'Deleted %d requests.', $count ), $count ),
    282                                         'success'
    283                                 );
     299                                if ( $count ) {
     300                                        add_settings_error(
     301                                                'bulk_action',
     302                                                'bulk_action',
     303                                                /* translators: %d: Number of requests. */
     304                                                sprintf( _n( '%d request deleted successfully.', '%d requests deleted successfully.', $count ), $count ),
     305                                                'success'
     306                                        );
     307                                }
     308
     309                                if ( $failures ) {
     310                                        add_settings_error(
     311                                                'bulk_action',
     312                                                'bulk_action',
     313                                                /* translators: %d: Number of requests. */
     314                                                sprintf( _n( '%d request failed to delete.', '%d requests failed to delete.', $failures ), $failures ),
     315                                                'error'
     316                                        );
     317                                }
     318
    284319                                break;
    285320                }
    286321        }
  • src/wp-admin/includes/privacy-tools.php

    diff --git a/src/wp-admin/includes/privacy-tools.php b/src/wp-admin/includes/privacy-tools.php
    index 2e4589d6d2..e13c402d69 100644
    a b  
    1313 * @access private
    1414 *
    1515 * @param int $request_id Request ID.
    16  * @return bool|WP_Error Returns true/false based on the success of sending the email, or a WP_Error object.
     16 * @return bool|WP_Error Returns true if sending the email was successful, or a WP_Error object.
    1717 */
    1818function _wp_privacy_resend_request( $request_id ) {
    1919        $request_id = absint( $request_id );