Changeset 30430

Timestamp:

11/20/2014 01:39:03 PM (12 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 );
+        }
 }