opendev/gerrit
Code Issues Proposed changes

AccountIndex: Fix username field

Browse Source 
Change-Id: Ie05d36898b426cd52cb5b1e12c245b3651b44623
This commit is contained in:
David Ostrovsky
2016-03-20 20:34:58 +01:00
committed by David Pursehouse
parent a3e684c61b
commit 0a5d463287
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-server/src/main/java/com/google/gerrit/server/index/account/AccountField.java
View File

@@ -16,6 +16,7 @@ package com.google.gerrit.server.index.account;
import com.google.common.base.Function; import com.google.common.base.Function;
import com.google.common.base.Predicates; import com.google.common.base.Predicates;
import com.google.common.base.Strings;
import com.google.common.collect.FluentIterable; import com.google.common.collect.FluentIterable;
import com.google.common.collect.Iterables; import com.google.common.collect.Iterables;
import com.google.gerrit.reviewdb.client.AccountExternalId; import com.google.gerrit.reviewdb.client.AccountExternalId;
@@ -128,10 +129,10 @@ public class AccountField {
public static final FieldDef<AccountState, String> USERNAME = public static final FieldDef<AccountState, String> USERNAME =
new FieldDef.Single<AccountState, String>( new FieldDef.Single<AccountState, String>(
"username", null, false) { "username", FieldType.EXACT, false) {
@Override @Override
public String get(AccountState input, FillArgs args) { public String get(AccountState input, FillArgs args) {
return input.getUserName().toLowerCase(); return Strings.nullToEmpty(input.getUserName()).toLowerCase();
} }
}; };