WordPress.org

Make WordPress Core


Ignore:
Timestamp:
02/05/2021 03:48:59 PM (4 months ago)
Author:
SergeyBiryukov
Message:

Privacy: Rename the $send_confirmation_email parameter of wp_create_user_request() to $status, for clarity.

Follow-up to [50159], [50165].

Props xkon, TimothyBlynJacobs.
Fixes #52430. See #43890.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/privacy-tools.php

    r50165 r50230  
    112112                $username_or_email_address = sanitize_text_field( wp_unslash( $_POST['username_or_email_for_privacy_request'] ) );
    113113                $email_address             = '';
    114                 $send_confirmation_email   = true;
     114                $status                    = 'pending';
    115115
    116116                if ( ! isset( $_POST['send_confirmation_email'] ) ) {
    117                     $send_confirmation_email = false;
     117                    $status = 'confirmed';
    118118                }
    119119
     
    147147                }
    148148
    149                 $request_id = wp_create_user_request( $email_address, $action_type, array(), $send_confirmation_email );
     149                $request_id = wp_create_user_request( $email_address, $action_type, array(), $status );
     150                $message    = '';
    150151
    151152                if ( is_wp_error( $request_id ) ) {
     153                    $message = $request_id->get_error_message();
     154                } elseif ( ! $request_id ) {
     155                    $message = __( 'Unable to initiate confirmation request.' );
     156                }
     157
     158                if ( $message ) {
    152159                    add_settings_error(
    153160                        'username_or_email_for_privacy_request',
    154161                        'username_or_email_for_privacy_request',
    155                         $request_id->get_error_message(),
    156                         'error'
    157                     );
    158                     break;
    159                 } elseif ( ! $request_id ) {
    160                     add_settings_error(
    161                         'username_or_email_for_privacy_request',
    162                         'username_or_email_for_privacy_request',
    163                         __( 'Unable to initiate confirmation request.' ),
     162                        $message,
    164163                        'error'
    165164                    );
     
    167166                }
    168167
    169                 if ( $send_confirmation_email ) {
     168                if ( 'pending' === $status ) {
    170169                    wp_send_user_request( $request_id );
     170
     171                    $message = __( 'Confirmation request initiated successfully.' );
     172                } elseif ( 'confirmed' === $status ) {
     173                    $message = __( 'Request added successfully.' );
    171174                }
    172175
    173                 add_settings_error(
    174                     'username_or_email_for_privacy_request',
    175                     'username_or_email_for_privacy_request',
    176                     __( 'Confirmation request initiated successfully.' ),
    177                     'success'
    178                 );
    179                 break;
     176                if ( $message ) {
     177                    add_settings_error(
     178                        'username_or_email_for_privacy_request',
     179                        'username_or_email_for_privacy_request',
     180                        $message,
     181                        'success'
     182                    );
     183                    break;
     184                }
    180185        }
    181186    }
Note: See TracChangeset for help on using the changeset viewer.