From cfbb2e735f633c47898841f4887078205c109cd4 Mon Sep 17 00:00:00 2001 From: Gary Pendergast Date: Tue, 8 Jan 2019 06:06:04 +0000 Subject: [PATCH] Users: Use `email_exists()` in `send_confirmation_on_profile_email()`, instead of a manual query. This gives us caching, and removes the use of the `$wpdb` global. Props spacedmonkey, pento. Fixes #45748. git-svn-id: https://develop.svn.wordpress.org/trunk@44465 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/user.php | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/wp-includes/user.php b/src/wp-includes/user.php index 09be161e15..d99885142d 100644 --- a/src/wp-includes/user.php +++ b/src/wp-includes/user.php @@ -2692,10 +2692,9 @@ function _wp_get_current_user() { * @since 4.9.0 This function was moved from wp-admin/includes/ms.php so it's no longer Multisite specific. * * @global WP_Error $errors WP_Error object. - * @global wpdb $wpdb WordPress database object. */ function send_confirmation_on_profile_email() { - global $errors, $wpdb; + global $errors; $current_user = wp_get_current_user(); if ( ! is_object( $errors ) ) { @@ -2719,7 +2718,7 @@ function send_confirmation_on_profile_email() { return; } - if ( $wpdb->get_var( $wpdb->prepare( "SELECT user_email FROM {$wpdb->users} WHERE user_email=%s", $_POST['email'] ) ) ) { + if ( email_exists( $_POST['email'] ) ) { $errors->add( 'user_email', __( 'ERROR: The email address is already used.' ),