WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#45748 closed enhancement (fixed)

Use email_exists in send_confirmation_on_profile_email

Reported by: spacedmonkey's profile spacedmonkey Owned by: pento's profile pento
Milestone: 5.1 Priority: normal
Severity: normal Version: 3.0
Component: Users Keywords: good-first-bug has-patch
Focuses: Cc:

Description

Use email_exists in send_confirmation_on_profile_email function, as email_exists is a cached call.

Attachments (1)

45748.diff​ (499 bytes) - added by spacedmonkey 6 years ago.

Download all attachments as: .zip

Change History (5)

@spacedmonkey
6 years ago

#1 @SergeyBiryukov
6 years ago

  • Milestone changed from Awaiting Review to 5.1
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#2 @pento
6 years ago

  • Owner changed from SergeyBiryukov to pento
  • Status changed from reviewing to accepted

#3 @pento
6 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 44465:

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.

#4 @pento
6 years ago

In 44470:

Coding Standards: Fix an error introduced in [44465].

See #45748.

Note: See TracTickets for help on using tickets.