opendev/gerrit
Code Issues Proposed changes

Change AvatarProvider to accept IdentifiedUser

Browse Source 
This concept is more likely to survive the upcoming account UUID
refactoring being done to support external account management
systems. It also gives the provider more information about the
account, such as the complete list of email addresses or groups
the user is a member of. This may make it easier for the provider
implementation to select the correct image.

Change-Id: I2056914425e52e60cc8591c83d6746faca3d491c
This commit is contained in:
Shawn Pearce
2013-01-17 11:34:02 -08:00
parent 24b10578df
commit b0d0a64755
2 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/account/GetAvatar.java
View File

@@ -44,7 +44,7 @@ class GetAvatar implements RestReadView<AccountResource> {
throw new ResourceNotFoundException();
}
String url = impl.getUrl(rsrc.getUser().getAccount(), size);
String url = impl.getUrl(rsrc.getUser(), size);
if (Strings.isNullOrEmpty(url)) {
throw new ResourceNotFoundException();
} else {