diff --git a/gerrit-common/src/main/java/com/google/gerrit/common/data/AccessSection.java b/gerrit-common/src/main/java/com/google/gerrit/common/data/AccessSection.java index a9b5e85503..5ddf1ae249 100644 --- a/gerrit-common/src/main/java/com/google/gerrit/common/data/AccessSection.java +++ b/gerrit-common/src/main/java/com/google/gerrit/common/data/AccessSection.java @@ -124,7 +124,7 @@ public class AccessSection extends RefConfigSection implements if (!super.equals(obj) || !(obj instanceof AccessSection)) { return false; } - return new HashSet(permissions).equals(new HashSet( - ((AccessSection) obj).permissions)); + return new HashSet(getPermissions()).equals(new HashSet( + ((AccessSection) obj).getPermissions())); } } diff --git a/gerrit-common/src/main/java/com/google/gerrit/common/data/Permission.java b/gerrit-common/src/main/java/com/google/gerrit/common/data/Permission.java index 0585651098..43fefba481 100644 --- a/gerrit-common/src/main/java/com/google/gerrit/common/data/Permission.java +++ b/gerrit-common/src/main/java/com/google/gerrit/common/data/Permission.java @@ -241,8 +241,8 @@ public class Permission implements Comparable { if (!name.equals(other.name) || exclusiveGroup != other.exclusiveGroup) { return false; } - return new HashSet(rules) - .equals(new HashSet(other.rules)); + return new HashSet(getRules()) + .equals(new HashSet(other.getRules())); } @Override