Adapt to new AccountResolver interface Change-Id: Ia7d8254aed0e42c8c96df077f1ee4bbb26ec5751
diff --git a/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowAccountCommand.java b/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowAccountCommand.java index 99c140e..ffeaed4 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowAccountCommand.java +++ b/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowAccountCommand.java
@@ -21,7 +21,6 @@ import com.google.gerrit.extensions.common.SshKeyInfo; import com.google.gerrit.extensions.restapi.AuthException; import com.google.gerrit.reviewdb.client.Account; -import com.google.gerrit.reviewdb.client.Account.Id; import com.google.gerrit.server.IdentifiedUser; import com.google.gerrit.server.account.AccountCache; import com.google.gerrit.server.account.AccountResolver; @@ -89,7 +88,7 @@ @Override public void run() throws UnloggedFailure, Exception { - Account account; + AccountState account; if (name.isEmpty()) { throw new UnloggedFailure( @@ -97,7 +96,7 @@ "You need to tell me who to find: LastName,\\\\ Firstname, email@address.com, account id or an user name. " + "Be sure to double-escape spaces, for example: \"show-account Last,\\\\ First\""); } - Set<Id> idList = accountResolver.findAll(name); + Set<Account.Id> idList = accountResolver.resolve(name).asIdSet(); if (idList.isEmpty()) { throw new UnloggedFailure( 1, @@ -116,26 +115,26 @@ + "\""); stdout.println(); - for (Id id : idList) { - account = accountResolver.find(id.toString()); + for (Account.Id id : idList) { + account = accountResolver.resolve(id.toString()).asUnique(); if (account == null) { throw new UnloggedFailure("Account " + id.toString() + " does not exist."); } - stdout.println("Full name: " + account.getFullName()); + stdout.println("Full name: " + account.getAccount().getFullName()); stdout.println("Account Id: " + id.toString()); - stdout.println("Preferred Email: " + account.getPreferredEmail()); + stdout.println("Preferred Email: " + account.getAccount().getPreferredEmail()); Optional<AccountState> accountState = accountCache.get(id); if (accountState.isPresent()) { stdout.println("User Name: " + accountState.get().getUserName().get()); } - stdout.println("Active: " + account.isActive()); - stdout.println("Registered on: " + account.getRegisteredOn()); + stdout.println("Active: " + account.getAccount().isActive()); + stdout.println("Registered on: " + account.getAccount().getRegisteredOn()); stdout.println(""); stdout.println("External Ids:"); stdout.println(String.format("%-50s %s", "Email Address:", "External Id:")); try { - for (ExternalId externalId : externalIds.byAccount(account.getId())) { + for (ExternalId externalId : externalIds.byAccount(account.getAccount().getId())) { stdout.println( String.format( "%-50s %s",
diff --git a/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowRepoAccountAccessCommand.java b/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowRepoAccountAccessCommand.java index 80ccea1..4ed3b8c 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowRepoAccountAccessCommand.java +++ b/src/main/java/com/googlesource/gerrit/plugins/adminconsole/ShowRepoAccountAccessCommand.java
@@ -22,11 +22,11 @@ import com.google.gerrit.extensions.annotations.RequiresCapability; import com.google.gerrit.extensions.common.GroupInfo; import com.google.gerrit.reviewdb.client.Account; -import com.google.gerrit.reviewdb.client.Account.Id; import com.google.gerrit.reviewdb.client.Project; import com.google.gerrit.server.IdentifiedUser; import com.google.gerrit.server.account.AccountResolver; import com.google.gerrit.server.account.AccountResource; +import com.google.gerrit.server.account.AccountState; import com.google.gerrit.server.git.meta.MetaDataUpdate; import com.google.gerrit.server.project.ProjectConfig; import com.google.gerrit.server.restapi.account.GetGroups; @@ -84,7 +84,7 @@ @Override public void run() throws UnloggedFailure, Failure, Exception { - Account account; + AccountState account; String sectionNameFormatter = " %-25s\n"; String ruleNameFormatter = " %-15s\n "; String permissionNameFormatter = " %5s %9s %s\n"; @@ -102,7 +102,7 @@ "You need to tell me who to find: LastName,\\\\ Firstname, email@address.com, account id or an user name. " + "Be sure to double-escape spaces, for example: \"show-repo-account-access All-Projects --user Last,\\\\ First\""); } - Set<Id> idList = accountResolver.findAll(name); + Set<Account.Id> idList = accountResolver.resolve(name).asIdSet(); if (idList.isEmpty()) { throw new UnloggedFailure( 1, @@ -121,10 +121,10 @@ permissionGroupWidth = wide ? Integer.MAX_VALUE : columns - 9 - 5 - 9; - for (Id id : idList) { + for (Account.Id id : idList) { userHasPermissionsInProject = false; - account = accountResolver.find(id.toString()); - stdout.println("Full name: " + account.getFullName()); + account = accountResolver.resolve(id.toString()).asUnique(); + stdout.println("Full name: " + account.getAccount().getFullName()); // Need to know what groups the user is in. This is not a great // solution, but it does work. List<GroupInfo> groupInfos =