Make WordPress Core

Ticket #33654: 33654.2.diff

File 33654.2.diff, 3.5 KB (added by kraftbj, 9 years ago)

Missed the mark on inline doc. null is of the null type, not a bool.

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

     
    176176                $user_id = wp_update_user( $user );
    177177        } else {
    178178                $user_id = wp_insert_user( $user );
    179                 wp_new_user_notification( $user_id, 'both' );
     179                wp_new_user_notification( $user_id, null, 'both' );
    180180        }
    181181        return $user_id;
    182182}
  • src/wp-admin/network/site-new.php

     
    7979                if ( false === $user_id )
    8080                        wp_die( __( 'There was an error creating the user.' ) );
    8181                else
    82                         wp_new_user_notification( $user_id, 'both' );
     82                        wp_new_user_notification( $user_id, null, 'both' );
    8383        }
    8484
    8585        $wpdb->hide_errors();
  • src/wp-admin/network/site-users.php

     
    7777                                if ( false === $user_id ) {
    7878                                        $update = 'err_new_dup';
    7979                                } else {
    80                                         wp_new_user_notification( $user_id, 'both' );
     80                                        wp_new_user_notification( $user_id, null, 'both' );
    8181                                        add_user_to_blog( $id, $user_id, $_POST['new_role'] );
    8282                                        $update = 'newuser';
    8383                                }
  • src/wp-admin/network/user-new.php

     
    5151                if ( ! $user_id ) {
    5252                        $add_user_errors = new WP_Error( 'add_user_fail', __( 'Cannot add user.' ) );
    5353                } else {
    54                         wp_new_user_notification( $user_id, 'both' );
     54                        wp_new_user_notification( $user_id, null, 'both' );
    5555                        wp_redirect( add_query_arg( array('update' => 'added'), 'user-new.php' ) );
    5656                        exit;
    5757                }
  • src/wp-includes/pluggable.php

     
    16901690 *
    16911691 * @since 2.0.0
    16921692 * @since 4.3.0 The `$plaintext_pass` parameter was changed to `$notify`.
     1693 * @since 4.3.1 The `$plaintext_pass` deprecated. `$notify` added as a third argument.
    16931694 *
    16941695 * @param int    $user_id User ID.
     1696 * @param null   $deprecated Not used (argument deprecated).
    16951697 * @param string $notify  Optional. Type of notification that should happen. Accepts 'admin' or an empty
    16961698 *                        string (admin only), or 'both' (admin and user). The empty string value was kept
    16971699 *                        for backward-compatibility purposes with the renamed parameter. Default empty.
    16981700 */
    1699 function wp_new_user_notification( $user_id, $notify = '' ) {
     1701function wp_new_user_notification( $user_id, , $deprecated = null, $notify = '' ) {
     1702        if ( $deprecated !== null ) {
     1703                _deprecated_argument( __FUNCTION__, '4.3.1' );
     1704        }
     1705
    17001706        global $wpdb;
    17011707        $user = get_userdata( $user_id );
    17021708
  • src/wp-includes/user-functions.php

     
    20032003
    20042004        update_user_option( $user_id, 'default_password_nag', true, true ); //Set up the Password change nag.
    20052005
    2006         wp_new_user_notification( $user_id, 'both' );
     2006        wp_new_user_notification( $user_id, null, 'both' );
    20072007
    20082008        return $user_id;
    20092009}