Make WordPress Core

Ticket #43601: for43601.diff

File for43601.diff, 4.1 KB (added by bamadesigner, 7 years ago)

Patch for ticket 43601

  • src/wp-admin/network/site-new.php

     
    120120                }
    121121                $password = wp_generate_password( 12, false );
    122122                $user_id  = wpmu_create_user( $domain, $password, $email );
    123                 if ( false === $user_id ) {
    124                         wp_die( __( 'There was an error creating the user.' ) );
     123
     124                $is_user_error = is_wp_error( $user_id );
     125                if ( $is_user_error || ! $user_id ) {
     126                        $error_message = __( 'There was an error creating the user.' );
     127                        if ( $is_user_error ) {
     128                                $error_message .= ' ' . $user_id->get_error_message();
     129                        }
     130                        wp_die( $error_message );
    125131                }
    126132
    127133                /**
  • src/wp-admin/network/site-users.php

     
    6868                                $password = wp_generate_password( 12, false );
    6969                                $user_id  = wpmu_create_user( esc_html( strtolower( $user['username'] ) ), $password, esc_html( $user['email'] ) );
    7070
    71                                 if ( false === $user_id ) {
     71                                if ( is_wp_error( $user_id ) || ! $user_id ) {
    7272                                        $update = 'err_new_dup';
    7373                                } else {
    7474                                        $result = add_user_to_blog( $id, $user_id, $_POST['new_role'] );
  • src/wp-admin/network/user-new.php

     
    5050                $password = wp_generate_password( 12, false );
    5151                $user_id  = wpmu_create_user( esc_html( strtolower( $user['username'] ) ), $password, sanitize_email( $user['email'] ) );
    5252
    53                 if ( ! $user_id ) {
    54                         $add_user_errors = new WP_Error( 'add_user_fail', __( 'Cannot add user.' ) );
     53                $is_user_error = is_wp_error( $user_id );
     54                if ( $is_user_error || ! $user_id ) {
     55                        $error_message = __( 'Cannot add user.' );
     56                        if ( $is_user_error ) {
     57                                $error_message .= ' ' . $user_id->get_error_message();
     58                        }
     59                        $add_user_errors = new WP_Error( 'add_user_fail', $error_message );
    5560                } else {
    5661                        /**
    5762                         * Fires after a new user has been created via the network user-new.php page.
  • src/wp-includes/ms-functions.php

     
    11201120                $user_already_exists = true;
    11211121        }
    11221122
    1123         if ( ! $user_id ) {
    1124                 return new WP_Error( 'create_user', __( 'Could not create user' ), $signup );
     1123        $is_user_error = is_wp_error( $user_id );
     1124        if ( $is_user_error || ! $user_id ) {
     1125                $error_message = __( 'Could not create user.' );
     1126                if ( $is_user_error ) {
     1127                        $error_message .= ' ' . $user_id->get_error_message();
     1128                }
     1129                return new WP_Error( 'create_user', $error_message, $signup );
    11251130        }
    11261131
    11271132        $now = current_time( 'mysql', true );
     
    12211226
    12221227        $user_id = wp_create_user( $user_name, $password, $email );
    12231228        if ( is_wp_error( $user_id ) ) {
    1224                 return false;
     1229                return $user_id->get_error_message();
    12251230        }
    12261231
    12271232        // Newly created users have no roles or caps until they are added to a blog.
  • src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php

     
    487487                if ( is_multisite() ) {
    488488                        $user_id = wpmu_create_user( $user->user_login, $user->user_pass, $user->user_email );
    489489
    490                         if ( ! $user_id ) {
    491                                 return new WP_Error( 'rest_user_create', __( 'Error creating new user.' ), array( 'status' => 500 ) );
     490                        $is_user_error = is_wp_error( $user_id );
     491                        if ( $is_user_error || ! $user_id ) {
     492                                $error_message = __( 'Error creating new user.' );
     493                                if ( $is_user_error ) {
     494                                        $error_message .= ' ' . $user_id->get_error_message();
     495                                }
     496                                return new WP_Error( 'rest_user_create', $error_message, array( 'status' => 500 ) );
    492497                        }
    493498
    494499                        $user->ID = $user_id;