commit 9ae15a31d0092e68d675c8e417e89f68c96f0fbc
author David Pursehouse <dpursehouse@collab.net> Fri Feb 08 08:54:59 2019 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Feb 08 08:54:59 2019 +0000
tree d76f1878688b09f46ddddebf95f076263d0717c7
parent 7f442e5d950af271849bf540ddab92e0af6e2166
parent a55bfeedba8178b197f73db38dab8586152dd141

Merge "Don't return single change ref if skipMetadata is set"

java/com/google/gerrit/server/permissions/DefaultRefFilter.java

diff --git a/java/com/google/gerrit/server/permissions/DefaultRefFilter.java b/java/com/google/gerrit/server/permissions/DefaultRefFilter.java
index c588bdf..c189f33 100644
--- a/java/com/google/gerrit/server/permissions/DefaultRefFilter.java
+++ b/java/com/google/gerrit/server/permissions/DefaultRefFilter.java
@@ -133,6 +133,9 @@
     // See if we can get away with a single, cheap ref evaluation.
     if (refs.size() == 1) {
       String refName = Iterables.getOnlyElement(refs.values()).getName();
+      if (opts.filterMeta() && isMetadata(refName)) {
+        return ImmutableMap.of();
+      }
       if (RefNames.isRefsChanges(refName)) {
         return canSeeSingleChangeRef(refName) ? refs : ImmutableMap.of();
       }