commit eba3c41b8d4b3d32f266822bd1c694b3c0161bc4
author Sven Selberg <svense@axis.com> Wed Nov 17 12:07:33 2021 +0100
committer Sven Selberg <svense@axis.com> Wed Nov 17 12:07:33 2021 +0100
tree c4fd7995e44d1ba18e4c68e5d04bb693127d5f03
parent 3846bd50e858e75dac308710cf8441387a654590
parent 5dcf72bf260f56e5094ab2d3bdd54e7da8bcc4e8

Merge 'stable-3.4' into master

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

Change-Id: I7c37f5016185823958478227b7cfce1316a604ef

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 5375481..099db37 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());