commit a1cf02d22c31563f551fc1322b36b9623600c937
author David Pursehouse <dpursehouse@collab.net> Tue Jan 30 11:57:24 2018 +0900
committer David Pursehouse <dpursehouse@collab.net> Tue Jan 30 11:57:24 2018 +0900
tree 9b7f27da36facbd03caa66583733556d3822daad
parent c6de78b04391aafb3e9518bef81c6cb24f9d55cd
parent 9524d6a074d43a174e3a4ec2619813f266631584

Merge branch 'stable-2.14' into stable-2.15

* stable-2.14:
  Update to latest bazlets on stable-2.14 branch
  Ignore inactive reviewers in groups
  Use latest bazlets

Change-Id: I609c1057898f4ecd88b175ad0c4e14b1ee417d68

src/main/java/com/googlesource/gerrit/plugins/reviewers/DefaultReviewers.java

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 a26e6c1..1d5dac4 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/DefaultReviewers.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/DefaultReviewers.java
@@ -64,8 +64,10 @@
       in.reviewers = new ArrayList<>(reviewers.size());
       for (Account account : reviewers) {
         AddReviewerInput addReviewerInput = new AddReviewerInput();
-        addReviewerInput.reviewer = account.getId().toString();
-        in.reviewers.add(addReviewerInput);
+        if (account.isActive()) {
+          addReviewerInput.reviewer = account.getId().toString();
+          in.reviewers.add(addReviewerInput);
+        }
       }
       gApi.changes().id(change.getId().get()).current().review(in);
     } catch (RestApiException e) {