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 =