Changeset 35280 for trunk/src/wp-includes/user-functions.php

Timestamp:

10/20/2015 05:27:32 AM (11 years ago)

Author:

wonderboymusic

Message:

Users: when calling wp_insert_user() with an valid user ID, return WP_Error instead of arbitrarily updating user meta.

Adds unit test.

Props swissspidy, bilalcoder.
Fixes #28004.

File:

• trunk/src/wp-includes/user-functions.php (modified) (1 diff)

trunk/src/wp-includes/user-functions.php

@@ -1276 +1276 @@
         $userdata = $userdata->to_array();
     }
+
     // Are we updating or creating?
     if ( ! empty( $userdata['ID'] ) ) {
         $ID = (int) $userdata['ID'];
         $update = true;
-        $old_user_data = WP_User::get_data_by( 'id', $ID );
+        $old_user_data = get_userdata( $ID );
+
+        if ( ! $old_user_data ) {
+            return new WP_Error( 'invalid_user_id', __( 'Invalid user ID.' ) );
+        }
+
         // hashed in wp_update_user(), plaintext if called directly
         $user_pass = $userdata['user_pass'];