From 6bd58b0c451a2eceb23b8c7c42baf55435d50c62 Mon Sep 17 00:00:00 2001 From: Naoki Takezoe Date: Fri, 8 Dec 2017 15:10:41 +0900 Subject: [PATCH] Don't filter pull request target repositories because users who can access the forked repository should see the original repositories and other forked repositories basically. --- .../gitbucket/core/controller/PullRequestsController.scala | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/main/scala/gitbucket/core/controller/PullRequestsController.scala b/src/main/scala/gitbucket/core/controller/PullRequestsController.scala index dcaeda08b..9520c7501 100644 --- a/src/main/scala/gitbucket/core/controller/PullRequestsController.scala +++ b/src/main/scala/gitbucket/core/controller/PullRequestsController.scala @@ -382,12 +382,11 @@ trait PullRequestsControllerBase extends ControllerBase { diffs, ((forkedRepository.repository.originUserName, forkedRepository.repository.originRepositoryName) match { case (Some(userName), Some(repositoryName)) => getRepository(userName, repositoryName) match { - case Some(originRepository) => originRepository.repository :: getForkedRepositories(userName, repositoryName) - case None => getForkedRepositories(userName, repositoryName) + case Some(x) => x.repository :: getForkedRepositories(userName, repositoryName) + case None => getForkedRepositories(userName, repositoryName) } case _ => forkedRepository.repository :: getForkedRepositories(forkedRepository.owner, forkedRepository.name) - }).filter { repository => isReadable(repository, context.loginAccount) } - .map { repository => (repository.userName, repository.repositoryName) }, + }).map { repository => (repository.userName, repository.repositoryName) }, commits.flatten.map(commit => getCommitComments(forkedRepository.owner, forkedRepository.name, commit.id, false)).flatten.toList, originId, forkedId,