From dfeee2f7503da7f7b1a810fa5a928d9e0fe47ec3 Mon Sep 17 00:00:00 2001 From: Edwin Kempin Date: Fri, 24 Jun 2016 18:16:56 +0200 Subject: [PATCH] Fix JavaDoc of AccountControl#canSee Change-Id: I174c85b6da06b65bb3515e797ba24689ef2f72c3 Signed-off-by: Edwin Kempin --- .../src/main/java/com/google/gerrit/server/ReviewersUtil.java | 2 +- .../java/com/google/gerrit/server/account/AccountControl.java | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/ReviewersUtil.java b/gerrit-server/src/main/java/com/google/gerrit/server/ReviewersUtil.java index c73c32b78f..2f98a0c8cd 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/ReviewersUtil.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/ReviewersUtil.java @@ -231,7 +231,7 @@ public class ReviewersUtil { if (!map.containsKey(account) // Can the suggestion see the change? && visibilityControl.isVisibleTo(account) - // Can the account see the current user? + // Can the current user see the account? && accountControl.canSee(account)) { map.put(account, accountLoader.get(account)); return true; diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/account/AccountControl.java b/gerrit-server/src/main/java/com/google/gerrit/server/account/AccountControl.java index aad427b46d..05ae51fc0a 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/account/AccountControl.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/account/AccountControl.java @@ -79,7 +79,7 @@ public class AccountControl { } /** - * Returns true if the otherUser is allowed to see the current user, based + * Returns true if the current user is allowed to see the otherUser, based * on the account visibility policy. Depending on the group membership * realms supported, this may not be able to determine SAME_GROUP or * VISIBLE_GROUP correctly (defaulting to not being visible). This is because @@ -91,7 +91,7 @@ public class AccountControl { } /** - * Returns true if the otherUser is allowed to see the current user, based + * Returns true if the current user is allowed to see the otherUser, based * on the account visibility policy. Depending on the group membership * realms supported, this may not be able to determine SAME_GROUP or * VISIBLE_GROUP correctly (defaulting to not being visible). This is because