WordPress.org

Make WordPress Core

Ticket #44081: 44081.3.diff

File 44081.3.diff, 2.3 KB (added by javorszky, 14 months ago)

Simlifying code per feedback, fix return docblock of _wp_privacy_resend_request

  • src/wp-admin/includes/user.php

    diff --git src/wp-admin/includes/user.php src/wp-admin/includes/user.php
    index 4634c8e318..f3456ccf83 100644
    Please click the following link to activate your user account: 
    590590 * @access private
    591591 *
    592592 * @param int $request_id Request ID.
    593  * @return bool|WP_Error Returns true/false based on the success of sending the email, or a WP_Error object.
     593 * @return bool|WP_Error Returns true if sending the email was successful, or a WP_Error object.
    594594 */
    595595function _wp_privacy_resend_request( $request_id ) {
    596596        $request_id = absint( $request_id );
    abstract class WP_Privacy_Requests_Table extends WP_List_Table { 
    11891189                $action      = $this->current_action();
    11901190                $request_ids = isset( $_REQUEST['request_id'] ) ? wp_parse_id_list( wp_unslash( $_REQUEST['request_id'] ) ) : array();
    11911191
    1192                 $count = 0;
     1192                $count         = 0;
     1193                $failure_count = 0;
    11931194
    1194                 if ( $request_ids ) {
    1195                         check_admin_referer( 'bulk-privacy_requests' );
     1195                if ( empty( $request_ids ) ) {
     1196                                                add_settings_error(
     1197                                'bulk_action',
     1198                                'bulk_action',
     1199                                __( 'There were no requests to act on.' )
     1200                        );
     1201
     1202                        return;
    11961203                }
    11971204
     1205                check_admin_referer( 'bulk-privacy_requests' );
     1206
    11981207                switch ( $action ) {
    11991208                        case 'delete':
    12001209                                foreach ( $request_ids as $request_id ) {
    abstract class WP_Privacy_Requests_Table extends WP_List_Table { 
    12181227                                        if ( $resend && ! is_wp_error( $resend ) ) {
    12191228                                                $count++;
    12201229                                        }
     1230
     1231                                        if ( is_wp_error( $resend ) ) {
     1232                                                $failure_count++;
     1233                                        }
    12211234                                }
    12221235
    1223                                 add_settings_error(
    1224                                         'bulk_action',
    1225                                         'bulk_action',
    1226                                         /* translators: %d: number of requests */
    1227                                         sprintf( _n( 'Re-sent %d request', 'Re-sent %d requests', $count ), $count ),
    1228                                         'updated'
    1229                                 );
     1236                                if ( $count ) {
     1237                                        add_settings_error(
     1238                                                'bulk_action',
     1239                                                'bulk_action',
     1240                                                /* translators: %d: number of requests */
     1241                                                sprintf( _n( 'Re-sent %d request.', 'Re-sent %d requests.', $count ), $count ),
     1242                                                'updated'
     1243                                        );
     1244                                }
     1245
     1246                                if ( $failure_count ) {
     1247                                        add_settings_error(
     1248                                                'bulk_action',
     1249                                                'bulk_action',
     1250                                                /* translators: %d: number of requests */
     1251                                                sprintf( _n( 'Failed to re-send %d request.', 'Failed to re-send %d requests.', $failure_count ), $failure_count )
     1252                                        );
     1253                                }
    12301254                                break;
    12311255                }
    12321256        }