Format files with google-java-format
Change-Id: Idd93cde7b8d5a30f19f583706a7c9f9432147121
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/Action.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/Action.java
index 315ed7d..04a15fc 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/Action.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/Action.java
@@ -15,5 +15,6 @@
package com.googlesource.gerrit.plugins.reviewers;
public enum Action {
- ADD, REMOVE
+ ADD,
+ REMOVE
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java
index 441c605..1716dbd 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java
@@ -49,20 +49,17 @@
import com.google.inject.Provider;
import com.google.inject.ProvisionException;
import com.google.inject.Singleton;
-
+import java.io.IOException;
+import java.util.List;
+import java.util.Set;
import org.eclipse.jgit.lib.Repository;
import org.eclipse.jgit.revwalk.RevWalk;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.io.IOException;
-import java.util.List;
-import java.util.Set;
-
@Singleton
class ChangeEventListener implements RevisionCreatedListener {
- private static final Logger log = LoggerFactory
- .getLogger(ChangeEventListener.class);
+ private static final Logger log = LoggerFactory.getLogger(ChangeEventListener.class);
private final AccountResolver accountResolver;
private final AccountByEmailCache byEmailCache;
@@ -126,68 +123,72 @@
try (Repository git = repoManager.openRepository(projectName);
RevWalk rw = new RevWalk(git);
ReviewDb reviewDb = schemaFactory.open()) {
- ChangeData changeData = changeDataFactory.create(
- reviewDb, projectName, new Change.Id(c._number));
+ ChangeData changeData =
+ changeDataFactory.create(reviewDb, projectName, new Change.Id(c._number));
Set<String> reviewers = findReviewers(sections, changeData);
if (reviewers.isEmpty()) {
return;
}
final Change change = changeData.change();
- final Runnable task = reviewersFactory.create(change,
- toAccounts(reviewDb, reviewers, projectName,
- event.getWho().email));
+ final Runnable task =
+ reviewersFactory.create(
+ change, toAccounts(reviewDb, reviewers, projectName, event.getWho().email));
- workQueue.getDefaultQueue().submit(new Runnable() {
- ReviewDb db = null;
+ workQueue
+ .getDefaultQueue()
+ .submit(
+ new Runnable() {
+ ReviewDb db = null;
- @Override
- public void run() {
- RequestContext old = tl.setContext(new RequestContext() {
-
- @Override
- public CurrentUser getUser() {
- return identifiedUserFactory.create(change.getOwner());
- }
-
- @Override
- public Provider<ReviewDb> getReviewDbProvider() {
- return new Provider<ReviewDb>() {
@Override
- public ReviewDb get() {
- if (db == null) {
- try {
- db = schemaFactory.open();
- } catch (OrmException e) {
- throw new ProvisionException("Cannot open ReviewDb", e);
+ public void run() {
+ RequestContext old =
+ tl.setContext(
+ new RequestContext() {
+
+ @Override
+ public CurrentUser getUser() {
+ return identifiedUserFactory.create(change.getOwner());
+ }
+
+ @Override
+ public Provider<ReviewDb> getReviewDbProvider() {
+ return new Provider<ReviewDb>() {
+ @Override
+ public ReviewDb get() {
+ if (db == null) {
+ try {
+ db = schemaFactory.open();
+ } catch (OrmException e) {
+ throw new ProvisionException("Cannot open ReviewDb", e);
+ }
+ }
+ return db;
+ }
+ };
+ }
+ });
+ try {
+ task.run();
+ } finally {
+ tl.setContext(old);
+ if (db != null) {
+ db.close();
+ db = null;
}
}
- return db;
}
- };
- }
- });
- try {
- task.run();
- } finally {
- tl.setContext(old);
- if (db != null) {
- db.close();
- db = null;
- }
- }
- }
- });
+ });
} catch (OrmException | IOException | QueryParseException x) {
log.error(x.getMessage(), x);
}
}
- private Set<String> findReviewers(List<ReviewerFilterSection> sections,
- ChangeData changeData) throws OrmException, QueryParseException {
+ private Set<String> findReviewers(List<ReviewerFilterSection> sections, ChangeData changeData)
+ throws OrmException, QueryParseException {
ImmutableSet.Builder<String> reviewers = ImmutableSet.builder();
- List<ReviewerFilterSection> found = findReviewerSections(sections,
- changeData);
+ List<ReviewerFilterSection> found = findReviewerSections(sections, changeData);
for (ReviewerFilterSection s : found) {
reviewers.addAll(s.getReviewers());
}
@@ -196,11 +197,10 @@
private List<ReviewerFilterSection> findReviewerSections(
List<ReviewerFilterSection> sections, ChangeData changeData)
- throws OrmException, QueryParseException {
+ throws OrmException, QueryParseException {
ImmutableList.Builder<ReviewerFilterSection> found = ImmutableList.builder();
for (ReviewerFilterSection s : sections) {
- if (Strings.isNullOrEmpty(s.getFilter())
- || s.getFilter().equals("*")) {
+ if (Strings.isNullOrEmpty(s.getFilter()) || s.getFilter().equals("*")) {
found.add(s);
} else if (filterMatch(s.getFilter(), changeData)) {
found.add(s);
@@ -220,8 +220,8 @@
return filterPredicate.asMatchable().match(changeData);
}
- private Set<Account> toAccounts(ReviewDb reviewDb, Set<String> in,
- Project.NameKey p, String uploaderEMail) {
+ private Set<Account> toAccounts(
+ ReviewDb reviewDb, Set<String> in, Project.NameKey p, String uploaderEMail) {
Set<Account> reviewers = Sets.newHashSetWithExpectedSize(in.size());
GroupMembers groupMembers = null;
for (String r : in) {
@@ -239,18 +239,17 @@
}
if (groupMembers == null) {
groupMembers =
- groupMembersFactory.create(identifiedUserFactory.create(Iterables
- .getOnlyElement(byEmailCache.get(uploaderEMail))));
+ groupMembersFactory.create(
+ identifiedUserFactory.create(
+ Iterables.getOnlyElement(byEmailCache.get(uploaderEMail))));
}
try {
- reviewers.addAll(groupMembers.listAccounts(
- groupsCollection.get().parse(r).getGroupUUID(), p));
+ reviewers.addAll(
+ groupMembers.listAccounts(groupsCollection.get().parse(r).getGroupUUID(), p));
} catch (UnprocessableEntityException | NoSuchGroupException e) {
- log.warn(String.format(
- "Reviewer %s is neither an account nor a group", r));
+ log.warn(String.format("Reviewer %s is neither an account nor a group", r));
} catch (NoSuchProjectException e) {
- log.warn(String.format(
- "Failed to list accounts for group %s and project %s", r, p));
+ log.warn(String.format("Failed to list accounts for group %s and project %s", r, p));
} catch (IOException | OrmException e) {
log.warn(String.format("Failed to list accounts for group %s", r), e);
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/DefaultReviewers.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/DefaultReviewers.java
index 5459f11..a26e6c1 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/DefaultReviewers.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/DefaultReviewers.java
@@ -22,16 +22,13 @@
import com.google.gerrit.reviewdb.client.Change;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
-
+import java.util.ArrayList;
+import java.util.Set;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.util.ArrayList;
-import java.util.Set;
-
class DefaultReviewers implements Runnable {
- private static final Logger log = LoggerFactory
- .getLogger(DefaultReviewers.class);
+ private static final Logger log = LoggerFactory.getLogger(DefaultReviewers.class);
private final GerritApi gApi;
private final Change change;
@@ -42,10 +39,7 @@
}
@Inject
- DefaultReviewers(
- GerritApi gApi,
- @Assisted Change change,
- @Assisted Set<Account> reviewers) {
+ DefaultReviewers(GerritApi gApi, @Assisted Change change, @Assisted Set<Account> reviewers) {
this.gApi = gApi;
this.change = change;
this.reviewers = reviewers;
@@ -73,10 +67,7 @@
addReviewerInput.reviewer = account.getId().toString();
in.reviewers.add(addReviewerInput);
}
- gApi.changes()
- .id(change.getId().get())
- .current()
- .review(in);
+ gApi.changes().id(change.getId().get()).current().review(in);
} catch (RestApiException e) {
log.error("Couldn't add reviewers to the change", e);
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/GetReviewers.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/GetReviewers.java
index 77cd4b8..b0e735f 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/GetReviewers.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/GetReviewers.java
@@ -21,7 +21,6 @@
import com.google.gerrit.server.project.ProjectResource;
import com.google.inject.Inject;
import com.google.inject.Singleton;
-
import java.util.List;
@Singleton
@@ -34,8 +33,8 @@
}
@Override
- public List<ReviewerFilterSection> apply(ProjectResource resource) throws AuthException,
- BadRequestException, ResourceConflictException, Exception {
+ public List<ReviewerFilterSection> apply(ProjectResource resource)
+ throws AuthException, BadRequestException, ResourceConflictException, Exception {
return configFactory.create(resource.getNameKey()).getReviewerFilterSections();
}
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/Module.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/Module.java
index 2857105..98dad1b 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/Module.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/Module.java
@@ -26,7 +26,6 @@
import com.google.gerrit.extensions.webui.WebUiPlugin;
import com.google.gerrit.server.config.PluginConfigFactory;
import com.google.inject.Inject;
-
import org.eclipse.jgit.lib.Config;
public class Module extends FactoryModule {
@@ -34,13 +33,10 @@
private final boolean enableREST;
@Inject
- public Module(@PluginName String pluginName,
- PluginConfigFactory pluginCfgFactory) {
+ public Module(@PluginName String pluginName, PluginConfigFactory pluginCfgFactory) {
Config c = pluginCfgFactory.getGlobalPluginConfig(pluginName);
this.enableREST = c.getBoolean("reviewers", null, "enableREST", true);
- this.enableUI = enableREST
- ? c.getBoolean("reviewers", null, "enableUI", true)
- : false;
+ this.enableUI = enableREST ? c.getBoolean("reviewers", null, "enableUI", true) : false;
}
public Module(boolean enableUI, boolean enableREST) {
@@ -51,26 +47,24 @@
@Override
protected void configure() {
if (enableUI) {
- DynamicSet.bind(binder(), TopMenu.class)
- .to(ReviewersTopMenu.class);
- DynamicSet.bind(binder(), WebUiPlugin.class)
- .toInstance(new GwtPlugin("reviewers"));
+ DynamicSet.bind(binder(), TopMenu.class).to(ReviewersTopMenu.class);
+ DynamicSet.bind(binder(), WebUiPlugin.class).toInstance(new GwtPlugin("reviewers"));
}
- DynamicSet.bind(binder(), RevisionCreatedListener.class)
- .to(ChangeEventListener.class);
+ DynamicSet.bind(binder(), RevisionCreatedListener.class).to(ChangeEventListener.class);
factory(DefaultReviewers.Factory.class);
factory(ReviewersConfig.Factory.class);
if (enableREST) {
- install(new RestApiModule() {
- @Override
- protected void configure() {
- get(PROJECT_KIND, "reviewers").to(GetReviewers.class);
- put(PROJECT_KIND, "reviewers").to(PutReviewers.class);
- get(PROJECT_KIND, "suggest_reviewers").to(SuggestProjectReviewers.class);
- }
- });
+ install(
+ new RestApiModule() {
+ @Override
+ protected void configure() {
+ get(PROJECT_KIND, "reviewers").to(GetReviewers.class);
+ put(PROJECT_KIND, "reviewers").to(PutReviewers.class);
+ get(PROJECT_KIND, "suggest_reviewers").to(SuggestProjectReviewers.class);
+ }
+ });
}
}
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/PutReviewers.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/PutReviewers.java
index b83ea10..48c35fb 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/PutReviewers.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/PutReviewers.java
@@ -32,21 +32,17 @@
import com.google.inject.Inject;
import com.google.inject.Provider;
import com.google.inject.Singleton;
-
import com.googlesource.gerrit.plugins.reviewers.PutReviewers.Input;
-
+import java.io.IOException;
+import java.util.List;
import org.eclipse.jgit.errors.ConfigInvalidException;
import org.eclipse.jgit.errors.RepositoryNotFoundException;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.io.IOException;
-import java.util.List;
-
@Singleton
class PutReviewers implements RestModifyView<ProjectResource, Input> {
- private static final Logger log = LoggerFactory
- .getLogger(PutReviewers.class);
+ private static final Logger log = LoggerFactory.getLogger(PutReviewers.class);
public static class Input {
public Action action;
@@ -63,7 +59,8 @@
private final Provider<ReviewDb> reviewDbProvider;
@Inject
- PutReviewers(@PluginName String pluginName,
+ PutReviewers(
+ @PluginName String pluginName,
ReviewersConfig.Factory configFactory,
Provider<MetaDataUpdate.User> metaDataUpdateFactory,
ProjectCache projectCache,
@@ -85,8 +82,7 @@
Project.NameKey projectName = rsrc.getNameKey();
ReviewersConfig cfg = configFactory.create(projectName);
if (!rsrc.getControl().isOwner() || cfg == null) {
- throw new ResourceNotFoundException(
- "Project" + projectName.get() + " not found");
+ throw new ResourceNotFoundException("Project" + projectName.get() + " not found");
}
try (MetaDataUpdate md = metaDataUpdateFactory.get().create(projectName)) {
@@ -94,20 +90,21 @@
validateReviewer(input.reviewer);
}
try {
- StringBuilder message = new StringBuilder(pluginName)
- .append(" plugin: ");
+ StringBuilder message = new StringBuilder(pluginName).append(" plugin: ");
cfg.load(md);
if (input.action == Action.ADD) {
- message.append("Add reviewer ")
- .append(input.reviewer)
- .append(" to filter ")
- .append(input.filter);
+ message
+ .append("Add reviewer ")
+ .append(input.reviewer)
+ .append(" to filter ")
+ .append(input.filter);
cfg.addReviewer(input.filter, input.reviewer);
} else {
- message.append("Remove reviewer ")
- .append(input.reviewer)
- .append(" from filter ")
- .append(input.filter);
+ message
+ .append("Remove reviewer ")
+ .append(input.reviewer)
+ .append(" from filter ")
+ .append(input.filter);
cfg.removeReviewer(input.filter, input.reviewer);
}
message.append("\n");
@@ -117,17 +114,17 @@
projectCache.evict(projectName);
} catch (IOException e) {
if (e.getCause() instanceof ConfigInvalidException) {
- throw new ResourceConflictException("Cannot update " + projectName
- + ": " + e.getCause().getMessage());
+ throw new ResourceConflictException(
+ "Cannot update " + projectName + ": " + e.getCause().getMessage());
}
throw new ResourceConflictException("Cannot update " + projectName);
}
} catch (ConfigInvalidException err) {
- throw new ResourceConflictException("Cannot read " + pluginName
- + " configurations for project " + projectName, err);
+ throw new ResourceConflictException(
+ "Cannot read " + pluginName + " configurations for project " + projectName, err);
} catch (IOException err) {
- throw new ResourceConflictException("Cannot update " + pluginName
- + " configurations for project " + projectName, err);
+ throw new ResourceConflictException(
+ "Cannot update " + pluginName + " configurations for project " + projectName, err);
}
} catch (RepositoryNotFoundException err) {
throw new ResourceNotFoundException(projectName.get());
@@ -144,8 +141,7 @@
try {
groupsCollection.get().parse(reviewer);
} catch (UnprocessableEntityException e) {
- throw new ResourceNotFoundException(
- "Account or group " + reviewer + " not found");
+ throw new ResourceNotFoundException("Account or group " + reviewer + " not found");
}
}
} catch (OrmException e) {
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewerFilterSection.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewerFilterSection.java
index 58e7c89..1112bdd 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewerFilterSection.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewerFilterSection.java
@@ -20,8 +20,7 @@
private final String filter;
private final Set<String> reviewers;
- ReviewerFilterSection(String filter,
- Set<String> reviewers) {
+ ReviewerFilterSection(String filter, Set<String> reviewers) {
this.filter = filter;
this.reviewers = reviewers;
}
@@ -33,4 +32,4 @@
Set<String> getReviewers() {
return reviewers;
}
-}
\ No newline at end of file
+}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersConfig.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersConfig.java
index 4b8d1ff..6ee15ce 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersConfig.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersConfig.java
@@ -25,15 +25,13 @@
import com.google.gerrit.server.project.NoSuchProjectException;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
-
-import org.eclipse.jgit.errors.ConfigInvalidException;
-import org.eclipse.jgit.lib.CommitBuilder;
-import org.eclipse.jgit.lib.Config;
-
import java.io.IOException;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
+import org.eclipse.jgit.errors.ConfigInvalidException;
+import org.eclipse.jgit.lib.CommitBuilder;
+import org.eclipse.jgit.lib.Config;
class ReviewersConfig extends VersionedMetaData {
private static final String FILENAME = "reviewers.config";
@@ -46,12 +44,12 @@
}
@Inject
- ReviewersConfig(PluginConfigFactory cfgFactory,
+ ReviewersConfig(
+ PluginConfigFactory cfgFactory,
@PluginName String pluginName,
@Assisted Project.NameKey projectName)
throws NoSuchProjectException {
- cfg = cfgFactory.getProjectPluginConfigWithInheritance(projectName,
- pluginName);
+ cfg = cfgFactory.getProjectPluginConfigWithInheritance(projectName, pluginName);
}
List<ReviewerFilterSection> getReviewerFilterSections() {
@@ -64,8 +62,8 @@
void addReviewer(String filter, String reviewer) {
if (!newReviewerFilterSection(filter).getReviewers().contains(reviewer)) {
- List<String> values = new ArrayList<>(Arrays.asList(cfg.getStringList(
- FILTER, filter, REVIEWER)));
+ List<String> values =
+ new ArrayList<>(Arrays.asList(cfg.getStringList(FILTER, filter, REVIEWER)));
values.add(reviewer);
cfg.setStringList(FILTER, filter, REVIEWER, values);
}
@@ -73,8 +71,8 @@
void removeReviewer(String filter, String reviewer) {
if (newReviewerFilterSection(filter).getReviewers().contains(reviewer)) {
- List<String> values = new ArrayList<>(Arrays.asList(cfg.getStringList(
- FILTER, filter, REVIEWER)));
+ List<String> values =
+ new ArrayList<>(Arrays.asList(cfg.getStringList(FILTER, filter, REVIEWER)));
values.remove(reviewer);
if (values.isEmpty()) {
cfg.unsetSection(FILTER, filter);
@@ -103,8 +101,7 @@
}
@Override
- protected boolean onSave(CommitBuilder commit) throws IOException,
- ConfigInvalidException {
+ protected boolean onSave(CommitBuilder commit) throws IOException, ConfigInvalidException {
if (Strings.isNullOrEmpty(commit.getMessage())) {
commit.setMessage("Update reviewers configuration\n");
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersTopMenu.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersTopMenu.java
index b905858..2c839a0 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersTopMenu.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersTopMenu.java
@@ -19,7 +19,6 @@
import com.google.gerrit.extensions.client.MenuItem;
import com.google.gerrit.extensions.webui.TopMenu;
import com.google.inject.Inject;
-
import java.util.Collections;
import java.util.List;
@@ -29,9 +28,11 @@
@Inject
ReviewersTopMenu(@PluginName String pluginName) {
menuEntries = Lists.newArrayList();
- menuEntries.add(new MenuEntry("Projects", Collections.singletonList(
- new MenuItem(
- "Reviewers", "#/x/" + pluginName + "/p/${projectName}", "_self"))));
+ menuEntries.add(
+ new MenuEntry(
+ "Projects",
+ Collections.singletonList(
+ new MenuItem("Reviewers", "#/x/" + pluginName + "/p/${projectName}", "_self"))));
}
@Override
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/SuggestProjectReviewers.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/SuggestProjectReviewers.java
index 3e0768e..f71c591 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/SuggestProjectReviewers.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/SuggestProjectReviewers.java
@@ -29,16 +29,15 @@
import com.google.gwtorm.server.OrmException;
import com.google.inject.Inject;
import com.google.inject.Provider;
-
-import org.eclipse.jgit.lib.Config;
-
import java.io.IOException;
import java.util.List;
+import org.eclipse.jgit.lib.Config;
public class SuggestProjectReviewers extends SuggestReviewers
- implements RestReadView<ProjectResource> {
+ implements RestReadView<ProjectResource> {
@Inject
- SuggestProjectReviewers(AccountVisibility av,
+ SuggestProjectReviewers(
+ AccountVisibility av,
IdentifiedUser.GenericFactory identifiedUserFactory,
Provider<ReviewDb> dbProvider,
@GerritServerConfig Config cfg,
@@ -49,18 +48,16 @@
@Override
public List<SuggestedReviewerInfo> apply(ProjectResource rsrc)
throws BadRequestException, OrmException, IOException {
- return reviewersUtil.suggestReviewers(null, this, rsrc.getControl(),
- getVisibility(rsrc), true);
+ return reviewersUtil.suggestReviewers(null, this, rsrc.getControl(), getVisibility(rsrc), true);
}
private VisibilityControl getVisibility(final ProjectResource rsrc) {
return new VisibilityControl() {
@Override
public boolean isVisibleTo(Account.Id account) throws OrmException {
- IdentifiedUser who =
- identifiedUserFactory.create(account);
+ IdentifiedUser who = identifiedUserFactory.create(account);
return rsrc.getControl().forUser(who).isVisible();
}
};
}
-}
\ No newline at end of file
+}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/Action.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/Action.java
index 52b7091..96ebd14 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/Action.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/Action.java
@@ -15,5 +15,6 @@
package com.googlesource.gerrit.plugins.reviewers.client;
public enum Action {
- ADD, REMOVE
+ ADD,
+ REMOVE
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ChangeReviewersInput.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ChangeReviewersInput.java
index d5f955e..9560feb 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ChangeReviewersInput.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ChangeReviewersInput.java
@@ -21,19 +21,15 @@
return (ChangeReviewersInput) createObject();
}
- protected ChangeReviewersInput() {
- }
+ protected ChangeReviewersInput() {}
final void setAction(Action a) {
setActionRaw(a.name());
}
- final native void setActionRaw(String a)
- /*-{ if(a)this.action=a; }-*/;
+ final native void setActionRaw(String a) /*-{ if(a)this.action=a; }-*/;
- final native void setFilter(String f)
- /*-{ if(f)this.filter=f; }-*/;
+ final native void setFilter(String f) /*-{ if(f)this.filter=f; }-*/;
- final native void setReviewer(String r)
- /*-{ if(r)this.reviewer=r; }-*/;
+ final native void setReviewer(String r) /*-{ if(r)this.reviewer=r; }-*/;
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ProjectAccessInfo.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ProjectAccessInfo.java
index 20db766..48010ed 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ProjectAccessInfo.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ProjectAccessInfo.java
@@ -19,6 +19,5 @@
public class ProjectAccessInfo extends JavaScriptObject {
public final native boolean isOwner() /*-{ return this.is_owner ? true : false; }-*/;
- protected ProjectAccessInfo() {
- }
+ protected ProjectAccessInfo() {}
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerFilterSection.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerFilterSection.java
index f7088a4..eed5448 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerFilterSection.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerFilterSection.java
@@ -19,8 +19,8 @@
public class ReviewerFilterSection extends JavaScriptObject {
public final native String filter() /*-{ return this.filter; }-*/;
+
public final native JsArrayString reviewers() /*-{ return this.reviewers; }-*/;
- protected ReviewerFilterSection() {
- }
+ protected ReviewerFilterSection() {}
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerSuggestOracle.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerSuggestOracle.java
index b7b86e9..2f601dc 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerSuggestOracle.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewerSuggestOracle.java
@@ -21,7 +21,6 @@
import com.google.gwt.core.client.JavaScriptObject;
import com.google.gwt.user.client.rpc.AsyncCallback;
import com.google.gwt.user.client.ui.SuggestOracle;
-
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
@@ -60,10 +59,8 @@
this.name = groupName;
}
- ReviewerSuggestion(String query, String fullname, String email,
- String accountId) {
- super(query, fullname
- + ((!email.isEmpty()) ? " <" + email + ">" : " (" + accountId + ")"));
+ ReviewerSuggestion(String query, String fullname, String email, String accountId) {
+ super(query, fullname + ((!email.isEmpty()) ? " <" + email + ">" : " (" + accountId + ")"));
this.name = fullname;
}
@@ -84,36 +81,36 @@
if (req.getLimit() > 0) {
rest.addParameter("n", req.getLimit());
}
- rest.get(new AsyncCallback<NativeMap<JavaScriptObject>>() {
- @Override
- public void onSuccess(NativeMap<JavaScriptObject> result) {
- List<String> keys0 = result.sortedKeys();
- List<Suggestion> suggestions = new ArrayList<>(keys0.size());
- for (String key0 : keys0) {
- Set<String> keys1 = Natives.keys(result.get(key0));
- NativeMap<JavaScriptObject> map1 = result.get(key0).cast();
- for (String key1 : keys1) {
- NativeMap<JavaScriptObject> map2 = map1.get(key1).cast();
- String name = map2.get(NAME_KEY).toString();
- if (ACCOUNT_KEY.equals(key1)) {
- String email = (map2.containsKey(EMAIL_KEY))
- ? map2.get(EMAIL_KEY).toString() : "";
- String accountId = map2.get(ACCOUNT_ID_KEY).toString();
- suggestions
- .add(new ReviewerSuggestion(req.getQuery(), name, email, accountId));
- } else if (GROUP_KEY.equals(key1)) {
- suggestions.add(new ReviewerSuggestion(req.getQuery(), name));
+ rest.get(
+ new AsyncCallback<NativeMap<JavaScriptObject>>() {
+ @Override
+ public void onSuccess(NativeMap<JavaScriptObject> result) {
+ List<String> keys0 = result.sortedKeys();
+ List<Suggestion> suggestions = new ArrayList<>(keys0.size());
+ for (String key0 : keys0) {
+ Set<String> keys1 = Natives.keys(result.get(key0));
+ NativeMap<JavaScriptObject> map1 = result.get(key0).cast();
+ for (String key1 : keys1) {
+ NativeMap<JavaScriptObject> map2 = map1.get(key1).cast();
+ String name = map2.get(NAME_KEY).toString();
+ if (ACCOUNT_KEY.equals(key1)) {
+ String email =
+ (map2.containsKey(EMAIL_KEY)) ? map2.get(EMAIL_KEY).toString() : "";
+ String accountId = map2.get(ACCOUNT_ID_KEY).toString();
+ suggestions.add(new ReviewerSuggestion(req.getQuery(), name, email, accountId));
+ } else if (GROUP_KEY.equals(key1)) {
+ suggestions.add(new ReviewerSuggestion(req.getQuery(), name));
+ }
+ }
}
+ done.onSuggestionsReady(req, new Response(suggestions));
}
- }
- done.onSuggestionsReady(req, new Response(suggestions));
- }
- @Override
- public void onFailure(Throwable caught) {
- responseEmptySuggestion(req, done);
- }
- });
+ @Override
+ public void onFailure(Throwable caught) {
+ responseEmptySuggestion(req, done);
+ }
+ });
}
private static void responseEmptySuggestion(Request req, Callback done) {
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewersScreen.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewersScreen.java
index ebd1f48..ba601ae 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewersScreen.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/client/ReviewersScreen.java
@@ -32,20 +32,18 @@
import com.google.gwt.user.client.ui.SuggestBox;
import com.google.gwt.user.client.ui.VerticalPanel;
import com.google.gwtexpui.globalkey.client.NpTextBox;
-
import java.util.HashSet;
import java.util.Objects;
import java.util.Set;
public class ReviewersScreen extends HorizontalPanel {
- private static final String REMOVE_BUTTON_IMG =
- "plugins/reviewers/static/remove_reviewer.png";
+ private static final String REMOVE_BUTTON_IMG = "plugins/reviewers/static/remove_reviewer.png";
+
static class Factory implements Screen.EntryPoint {
@Override
public void onLoad(Screen screen) {
screen.setPageTitle("Reviewers");
- screen.show(
- new ReviewersScreen(URL.decodeQueryString(screen.getToken(1))));
+ screen.show(new ReviewersScreen(URL.decodeQueryString(screen.getToken(1))));
}
}
@@ -58,34 +56,37 @@
this.projectName = projectName;
this.rEntries = new HashSet<>();
- new RestApi("access/").addParameter("project", projectName).get(
- new AsyncCallback<NativeMap<ProjectAccessInfo>>() {
+ new RestApi("access/")
+ .addParameter("project", projectName)
+ .get(
+ new AsyncCallback<NativeMap<ProjectAccessInfo>>() {
- @Override
- public void onSuccess(NativeMap<ProjectAccessInfo> result) {
- isOwner = result.get(projectName).isOwner();
- display();
- }
+ @Override
+ public void onSuccess(NativeMap<ProjectAccessInfo> result) {
+ isOwner = result.get(projectName).isOwner();
+ display();
+ }
- @Override
- public void onFailure(Throwable caught) {
- }
- });
+ @Override
+ public void onFailure(Throwable caught) {}
+ });
}
void display() {
- new RestApi("projects").id(projectName).view("reviewers").get(
- new AsyncCallback<JsArray<ReviewerFilterSection>>() {
+ new RestApi("projects")
+ .id(projectName)
+ .view("reviewers")
+ .get(
+ new AsyncCallback<JsArray<ReviewerFilterSection>>() {
- @Override
- public void onSuccess(JsArray<ReviewerFilterSection> result) {
- display(result);
- }
+ @Override
+ public void onSuccess(JsArray<ReviewerFilterSection> result) {
+ display(result);
+ }
- @Override
- public void onFailure(Throwable caught) {
- }
- });
+ @Override
+ public void onFailure(Throwable caught) {}
+ });
}
void display(JsArray<ReviewerFilterSection> sections) {
@@ -116,12 +117,13 @@
Button removeButton = Button.wrap(img.getElement());
removeButton.setStyleName("reviewers-removeButton");
removeButton.setTitle("remove reviewer");
- removeButton.addClickHandler(new ClickHandler() {
- @Override
- public void onClick(final ClickEvent event) {
- doSave(Action.REMOVE, e);
- }
- });
+ removeButton.addClickHandler(
+ new ClickHandler() {
+ @Override
+ public void onClick(final ClickEvent event) {
+ doSave(Action.REMOVE, e);
+ }
+ });
removeButton.setVisible(isOwner);
HorizontalPanel p = new HorizontalPanel();
@@ -130,7 +132,7 @@
return p;
}
- Panel createInputPanel(){
+ Panel createInputPanel() {
Grid inputGrid = new Grid(2, 2);
final NpTextBox filterBox = new NpTextBox();
@@ -147,19 +149,18 @@
Button addButton = new Button("Add");
addButton.setStyleName("reviewers-addButton");
- addButton.addClickHandler(new ClickHandler() {
- @Override
- public void onClick(final ClickEvent event) {
- ReviewerEntry e = new ReviewerEntry(filterBox.getValue(),
- reviewerBox.getValue());
- if (!rEntries.contains(e) && !e.filter.isEmpty() &&
- !e.reviewer.isEmpty()) {
- doSave(Action.ADD, e);
- }
- filterBox.setText("");
- reviewerBox.setText("");
- }
- });
+ addButton.addClickHandler(
+ new ClickHandler() {
+ @Override
+ public void onClick(final ClickEvent event) {
+ ReviewerEntry e = new ReviewerEntry(filterBox.getValue(), reviewerBox.getValue());
+ if (!rEntries.contains(e) && !e.filter.isEmpty() && !e.reviewer.isEmpty()) {
+ doSave(Action.ADD, e);
+ }
+ filterBox.setText("");
+ reviewerBox.setText("");
+ }
+ });
filterBox.setEnabled(isOwner);
reviewerBox.setEnabled(isOwner);
addButton.setEnabled(isOwner);
@@ -178,18 +179,21 @@
in.setReviewer(entry.reviewer);
reset();
- new RestApi("projects").id(projectName).view("reviewers").put(
- in, new AsyncCallback<JsArray<ReviewerFilterSection>>() {
+ new RestApi("projects")
+ .id(projectName)
+ .view("reviewers")
+ .put(
+ in,
+ new AsyncCallback<JsArray<ReviewerFilterSection>>() {
- @Override
- public void onSuccess(JsArray<ReviewerFilterSection> result) {
- display(result);
- }
+ @Override
+ public void onSuccess(JsArray<ReviewerFilterSection> result) {
+ display(result);
+ }
- @Override
- public void onFailure(Throwable caught) {
- }
- });
+ @Override
+ public void onFailure(Throwable caught) {}
+ });
}
void reset() {
@@ -217,8 +221,7 @@
return false;
}
ReviewerEntry other = (ReviewerEntry) o;
- if (!this.filter.equals(other.filter)
- || !this.reviewer.equals(other.reviewer)) {
+ if (!this.filter.equals(other.filter) || !this.reviewer.equals(other.reviewer)) {
return false;
}
return true;