Merge branch 'stable-3.7' into stable-3.8 * stable-3.7: Fix misleading log message when adding reviewers Change-Id: Ie94419ca4e2b3a49a5b88a55b4be020bd0841a61
diff --git a/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java b/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java index 83ba531..bdfa7f6 100644 --- a/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java +++ b/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java
@@ -106,7 +106,10 @@ in.reviewers = new ArrayList<>(accountsIds.size()); Collection<Account.Id> validOwnersForAttentionSet = new ArrayList<>(accountsIds.size()); for (Account.Id account : accountsIds) { - if (!currentReviewers.contains(account.get()) && isVisibleTo(changeInfo, account)) { + if (currentReviewers.contains(account.get())) { + continue; + } + if (isVisibleTo(changeInfo, account)) { ReviewerInput addReviewerInput = new ReviewerInput(); addReviewerInput.reviewer = account.toString(); addReviewerInput.state = reviewerState;