[buddypress-trac] [BuddyPress Trac] #5979: `bp_core_userdata_` cache is unnecessary

[buddypress-trac]

#5979: `bp_core_userdata_` cache is unnecessary
-----------------------------+------------------
 Reporter:  johnjamesjacoby  |       Owner:
     Type:  defect (bug)     |      Status:  new
 Priority:  high             |   Milestone:  2.2
Component:  Members          |     Version:
 Severity:  normal           |  Resolution:
 Keywords:  has-patch        |
-----------------------------+------------------
Changes (by boonebgorges):

 * keywords:  2nd-opinion has-patch => has-patch


Comment:

 `cache_users()` is only called if we call `fields=all_with_meta`, which
 `BP_User_Query` does not do. But yes, you are correct about `WP_User`.

 Would be nice to have a unit test that shows it's cached by WP, and I
 shudder at your ternary syntax, but I'm on board with tearing out our
 redundant caching.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5979#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac