// Copyright (C) 2016 The Android Open Source Project
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

package com.google.gerrit.server.change;

import com.google.gerrit.extensions.common.SuggestedReviewerInfo;
import com.google.gerrit.extensions.restapi.AuthException;
import com.google.gerrit.extensions.restapi.BadRequestException;
import com.google.gerrit.extensions.restapi.RestReadView;
import com.google.gerrit.reviewdb.client.Account;
import com.google.gerrit.reviewdb.server.ReviewDb;
import com.google.gerrit.server.CurrentUser;
import com.google.gerrit.server.IdentifiedUser;
import com.google.gerrit.server.IdentifiedUser.GenericFactory;
import com.google.gerrit.server.ReviewersUtil;
import com.google.gerrit.server.ReviewersUtil.VisibilityControl;
import com.google.gerrit.server.account.AccountVisibility;
import com.google.gerrit.server.config.GerritServerConfig;
import com.google.gwtorm.server.OrmException;
import com.google.inject.Inject;
import com.google.inject.Provider;
import java.io.IOException;
import java.util.List;
import org.eclipse.jgit.lib.Config;
import org.kohsuke.args4j.Option;

public class SuggestChangeReviewers extends SuggestReviewers
    implements RestReadView<ChangeResource> {

  @Option(
    name = "--exclude-groups",
    aliases = {"-e"},
    usage = "exclude groups from query"
  )
  boolean excludeGroups;

  private final Provider<CurrentUser> self;

  @Inject
  SuggestChangeReviewers(
      AccountVisibility av,
      GenericFactory identifiedUserFactory,
      Provider<ReviewDb> dbProvider,
      Provider<CurrentUser> self,
      @GerritServerConfig Config cfg,
      ReviewersUtil reviewersUtil) {
    super(av, identifiedUserFactory, dbProvider, cfg, reviewersUtil);
    this.self = self;
  }

  @Override
  public List<SuggestedReviewerInfo> apply(ChangeResource rsrc)
      throws AuthException, BadRequestException, OrmException, IOException {
    if (!self.get().isIdentifiedUser()) {
      throw new AuthException("Authentication required");
    }
    return reviewersUtil.suggestReviewers(
        rsrc.getNotes(),
        this,
        rsrc.getControl().getProjectControl(),
        getVisibility(rsrc),
        excludeGroups);
  }

  private VisibilityControl getVisibility(final ChangeResource rsrc) {
    if (rsrc.getControl().getRefControl().isVisibleByRegisteredUsers()) {
      return new VisibilityControl() {
        @Override
        public boolean isVisibleTo(Account.Id account) throws OrmException {
          return true;
        }
      };
    }
    return new VisibilityControl() {
      @Override
      public boolean isVisibleTo(Account.Id account) throws OrmException {
        IdentifiedUser who = identifiedUserFactory.create(account);
        // we can't use changeControl directly as it won't suggest reviewers
        // to drafts
        return rsrc.getControl().forUser(who).isRefVisible();
      }
    };
  }
}

@@ -1375,24 +1375,146 @@
 rverConfig;%0A
+import com.google.gerrit.server.permissions.PermissionBackend;%0Aimport com.google.gerrit.server.permissions.RefPermission;%0A
 import com.g
@@ -1539,32 +1539,32 @@
 r.OrmException;%0A
-
 import com.googl
@@ -1979,16 +1979,69 @@
 roups;%0A%0A
+  private final PermissionBackend permissionBackend;%0A
   privat
@@ -2222,16 +2222,59 @@
 ovider,%0A
+      PermissionBackend permissionBackend,%0A
       Pr
@@ -2444,16 +2444,64 @@
 sUtil);%0A
+    this.permissionBackend = permissionBackend;%0A
     this
@@ -3007,22 +3007,16 @@
 ibility(
-final 
 ChangeRe
@@ -3281,24 +3281,241 @@
     %7D;%0A    %7D
+%0A%0A    // Use the destination reference, not the change, as drafts may deny%0A    // anyone who is not already a reviewer.%0A    PermissionBackend.ForRef perm = permissionBackend.user(self).ref(rsrc.getChange().getDest());
 %0A    return 
@@ -3711,157 +3711,60 @@
     
-// we can't use changeControl directly as it won't suggest reviewers%0A        // to drafts%0A        return rsrc.getControl().forUser(who).isRefVisible(
+return perm.user(who).testOrFalse(RefPermission.READ
 );%0A 

Convert SuggestChangeReviewers to use PermissionBackend