WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#12441 closed enhancement (duplicate)

_fill_user() should use get_user_meta()

Reported by: scribu Owned by:
Priority: normal Milestone:
Component: Optimization Version:
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

Currently, _fill_user() does a direct query, when it could be using get_user_meta(), which is cached.

Attachments (1)

_fill_user.diff​ (1.3 KB ) - added by scribu 17 years ago.

Download all attachments as: .zip

Change History (5)

@scribu
17 years ago

#1 @scribu
17 years ago

  • Keywords commit added

#2 @nacin
17 years ago

  • Milestone 3.0
  • Resolution → duplicate
  • Status new → closed

Duplicate of #11914 it seems

#3 @miqrogroove
17 years ago

Agreed. _fill_user() should never be called when the data are already cached, because the user object has already been filled. In any case, my patch already blows _fill_user() out of the water and hooks it up to goodies that are superior to get_user_meta(). :) Feedback would be appreciated on the other ticket.

#4 @voyagerfan5761
17 years ago

  • Cc WordPress@… added
Note: See TracTickets for help on using tickets.