diff options
author | Shawn O. Pearce <sop@google.com> | 2010-11-08 17:10:20 -0800 |
---|---|---|
committer | Shawn O. Pearce <sop@google.com> | 2010-11-08 17:10:20 -0800 |
commit | fc26d13c4555a87ad35d5b41b65b9c71133e187f (patch) | |
tree | 41ba7ff605a67e4e9fda2f86738bc807d746fa71 | |
parent | cd67a8403f4040b55cf100c4f4474f54f7cfd8a2 (diff) | |
parent | ee5320abd7c58bc2c563dad4bb612e9a70d8a364 (diff) |
Merge branch 'sec-access-panel'v2.1.6-rc0
-rw-r--r-- | gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/project/ProjectDetailFactory.java | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/project/ProjectDetailFactory.java b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/project/ProjectDetailFactory.java index 3ff3892f9a..ef632c44bd 100644 --- a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/project/ProjectDetailFactory.java +++ b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/project/ProjectDetailFactory.java @@ -26,6 +26,7 @@ import com.google.gerrit.server.account.GroupCache; import com.google.gerrit.server.project.NoSuchProjectException; import com.google.gerrit.server.project.ProjectControl; import com.google.gerrit.server.project.ProjectState; +import com.google.gerrit.server.project.RefControl; import com.google.inject.Inject; import com.google.inject.assistedinject.Assisted; @@ -75,8 +76,14 @@ class ProjectDetailFactory extends Handler<ProjectDetail> { final List<InheritedRefRight> refRights = new ArrayList<InheritedRefRight>(); for (final RefRight r : projectState.getInheritedRights()) { - InheritedRefRight refRight = new InheritedRefRight( - r, true, pc.controlForRef(r.getRefPattern()).isOwner()); + RefControl rc = pc.controlForRef(r.getRefPattern()); + boolean isOwner = rc.isOwner(); + + if (!isOwner && !rc.isVisible()) { + continue; + } + + InheritedRefRight refRight = new InheritedRefRight(r, true, isOwner); if (!refRights.contains(refRight)) { refRights.add(refRight); wantGroup(r.getAccountGroupId()); @@ -84,8 +91,14 @@ class ProjectDetailFactory extends Handler<ProjectDetail> { } for (final RefRight r : projectState.getLocalRights()) { - refRights.add(new InheritedRefRight( - r, false, pc.controlForRef(r.getRefPattern()).isOwner())); + RefControl rc = pc.controlForRef(r.getRefPattern()); + boolean isOwner = rc.isOwner(); + + if (!isOwner && !rc.isVisible()) { + continue; + } + + refRights.add(new InheritedRefRight(r, false, isOwner)); wantGroup(r.getAccountGroupId()); } |