Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#18333 closed enhancement (fixed)

Deprecate get_userdatabylogin() and get_user_by_email()

Reported by: scribu's profile scribu Owned by: ryan's profile ryan
Milestone: 3.3 Priority: normal
Severity: normal Version:
Component: Users Keywords: has-patch
Focuses: Cc:

Description

They have ugly names and are just wrappers for get_user_by().

We should at least not use them in Core anymore.

Attachments (3)

18333.diff (6.1 KB) - added by scribu 13 years ago.
18333.2.diff (6.8 KB) - added by scribu 13 years ago.
18333.3.diff (638 bytes) - added by scribu 13 years ago.

Download all attachments as: .zip

Change History (11)

@scribu
13 years ago

#1 @scribu
13 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
13 years ago

While adding them to pluggable-deprecated.php, the patch doesn't remove them from pluggable.php. Is that intended?

@scribu
13 years ago

#3 @scribu
13 years ago

Nope; good catch.

#4 @ryan
13 years ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In [18513]:

Deprecate get_userdatabylogin() and get_user_by_email(). Props scribu. fixes #18333

#5 @ryan
13 years ago

  • Milestone changed from Awaiting Review to 3.3

#6 @ocean90
13 years ago

Typo in inline docs?

@deprecated 2.5 should be @deprecated 3.3.0.

@scribu
13 years ago

#7 @scribu
13 years ago

Indeed. See 18333.3.diff.

By the way, in [18513] the svn:mergeinfo property was changed for some reason.

#8 @ryan
13 years ago

In [18514]:

Update @deprecated. Props scribu. see #18333

Note: See TracTickets for help on using tickets.