commit 5dcf72bf260f56e5094ab2d3bdd54e7da8bcc4e8
author Sven Selberg <svense@axis.com> Wed Nov 17 11:53:04 2021 +0100
committer Sven Selberg <sven.selberg@axis.com> Wed Nov 17 10:59:29 2021 +0000
tree 8ecda21f6dca1472749056f43c5b58856d10e82d
parent 2a9102ef08dff8e6a08788e6507cc006140bde6b
parent 857bfbce4ca466b1cb2c7a78d18b690271989545

Merge branch 'stable-3.3' into stable-3.4

* stable-3.3:
  Don't call 'review' with empty ReviewInput

Change-Id: I9b1a0a7444f504f1b2e24c17ab18642b59a7e2e0

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

diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/AddReviewers.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/AddReviewers.java
index 43db627..80fab28 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/AddReviewers.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/AddReviewers.java
@@ -88,6 +88,9 @@
               .filter(c -> !existingReviewers.contains(c))
               .filter(r -> !reviewersToAdd.contains(r))
               .collect(Collectors.toSet());
+      if (reviewersToAdd.isEmpty() && ccsToAdd.isEmpty()) {
+        return;
+      }
 
       ReviewInput in = new ReviewInput();
       in.reviewers = new ArrayList<>(reviewers.size() + ccs.size());