From cd52ee349751c226559a2c35e9bea2b13e8e4ecd Mon Sep 17 00:00:00 2001 From: Gustaf Lundh Date: Fri, 21 Apr 2017 12:38:24 +0200 Subject: [PATCH] Permissions may be lost when matching expanding params When we hit the first expanding parameter, no more matching parameters can be picked up. This change remove the faulty break. Change-Id: Ie46652bc4322812a4e1a55cd88e52a0040c6b00b --- .../com/google/gerrit/server/project/PermissionCollection.java | 1 - 1 file changed, 1 deletion(-) diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/project/PermissionCollection.java b/gerrit-server/src/main/java/com/google/gerrit/server/project/PermissionCollection.java index 4bf3e47b7e..9febb3f646 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/project/PermissionCollection.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/project/PermissionCollection.java @@ -98,7 +98,6 @@ public class PermissionCollection { perUser = true; if (sm.match(ref, user)) { sectionToProject.put(sm.section, sm.project); - break; } } else if (sm.match(ref, null)) { sectionToProject.put(sm.section, sm.project);