Changeset 30430 for trunk

Timestamp:

11/20/2014 01:39:03 PM (10 years ago)

Author:

nacin

Message:

Invalidate password keys when a user's email changes.

Location:

trunk

Files:

• src/wp-includes/user.php (modified) (1 diff)
• tests/phpunit/tests/user.php (modified) (1 diff)

trunk/src/wp-includes/user.php

@@ -1886 +1886 @@
 
     if ( $update ) {
+        if ( $user_email !== $old_user_data->user_email ) {
+            $data['user_activation_key'] = '';
+        }
         $wpdb->update( $wpdb->users, $data, compact( 'ID' ) );
         $user_id = (int) $ID;

trunk/tests/phpunit/tests/user.php

@@ -673 +673 @@
         $this->assertSame( $user->user_nicename, $updated_user->user_nicename );
     }
+
+    function test_changing_email_invalidates_password_reset_key() {
+        global $wpdb;
+
+        $user = $this->factory->user->create_and_get();
+        $wpdb->update( $wpdb->users, array( 'user_activation_key' => 'key' ), array( 'ID' => $user->ID ) );
+        clean_user_cache( $user );
+
+        $user = get_userdata( $user->ID );
+        $this->assertEquals( 'key', $user->user_activation_key );
+
+        // Check that changing something other than the email doesn't remove the key.
+        $userdata = array(
+            'ID'            => $user->ID,
+            'user_nicename' => 'wat',
+        );
+        wp_update_user( $userdata );
+
+        $user = get_userdata( $user->ID );
+        $this->assertEquals( 'key', $user->user_activation_key );
+
+        // Now check that changing the email does remove it.
+        $userdata = array(
+            'ID'            => $user->ID,
+            'user_nicename' => 'cat',
+            'user_email'    => 'foo@bar.dev',
+        );
+        wp_update_user( $userdata );
+
+        $user = get_userdata( $user->ID );
+        $this->assertEmpty( $user->user_activation_key );
+    }
 }