Homec4science

Consolidate User Profile loading

Authored by Anh Nhan Nguyen <anhnhan@outlook.com> on Mar 24 2013, 14:42.

Description

Consolidate User Profile loading

Summary:
Introduce PhabricatorUserQuery::attachProfilesForUsers() for batch attachment of user profiles (for a future diff)
Introduce PhabricatorUser::loadUserProfile() to load attached user profiles (or load them on their own, if no one is attached).

USed them in code

Test Plan:
verified that use sites did not break

  • Uploaded user profile image for bots
  • changed my own profile image and blurb
  • looked a lot at my own profile

Reviewers: epriestley, btrahan, chad

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D5395

Details

Committed
epriestley <git@epriestley.com>Mar 24 2013, 14:42
Pushed
aubortJan 31 2017, 17:16
Parents
rPH1a48c7ac40c8: Lighten up gradient on cards
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHb0e9371a9c92: Consolidate User Profile loading (authored by Anh Nhan Nguyen <anhnhan@outlook.com>).Mar 24 2013, 14:42