commit 273e0a45a8b7d8d6031b2df8367ef6b4e0466f52
author Sven Selberg <svense@axis.com> Wed Mar 23 15:36:43 2022 +0100
committer Sven Selberg <svense@axis.com> Wed Mar 23 15:37:36 2022 +0100
tree e47e440a1e645f32a2085a6fe24cd991a3610b94
parent 857bfbce4ca466b1cb2c7a78d18b690271989545

Count uploader as a resolved reviewer

If we don't consider the uploader a resolved reviewer we will try to
resolve it as a group later on which will fail and spam the logs with
unnecessary warnings.

Change-Id: I11d96d9976242226ac34af4fd586ad098e4f4aa1

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

diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersResolver.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersResolver.java
index fc5d9e0..ea7d8e1 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersResolver.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ReviewersResolver.java
@@ -98,8 +98,8 @@
         Account.Id id = result.asList().get(0).account().id();
         if (uploader == null || id.get() != uploader._accountId) {
           reviewers.add(id);
-          return true;
         }
+        return true;
       }
       return false;
     } catch (StorageException | IOException | ConfigInvalidException e) {