diff options
Diffstat (limited to 'gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AccessSectionEditor.java')
-rw-r--r-- | gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AccessSectionEditor.java | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AccessSectionEditor.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AccessSectionEditor.java index e518d26295..7bd8b825d4 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AccessSectionEditor.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AccessSectionEditor.java @@ -14,6 +14,8 @@ package com.google.gerrit.client.admin; +import static java.util.stream.Collectors.toCollection; + import com.google.gerrit.client.Gerrit; import com.google.gerrit.common.data.AccessSection; import com.google.gerrit.common.data.LabelType; @@ -45,7 +47,6 @@ import com.google.gwt.user.client.ui.FlowPanel; import com.google.gwt.user.client.ui.HTMLPanel; import com.google.gwt.user.client.ui.ValueListBox; import java.util.ArrayList; -import java.util.Collections; import java.util.List; public class AccessSectionEditor extends Composite @@ -169,7 +170,7 @@ public class AccessSectionEditor extends Composite @Override public void setValue(AccessSection value) { - Collections.sort(value.getPermissions()); + sortPermissions(value); this.value = value; this.readOnly = !editing || !(projectAccess.isOwnerOf(value) || projectAccess.canUpload()); @@ -204,6 +205,11 @@ public class AccessSectionEditor extends Composite } } + private void sortPermissions(AccessSection accessSection) { + accessSection.setPermissions( + accessSection.getPermissions().stream().sorted().collect(toCollection(ArrayList::new))); + } + void setEditing(boolean editing) { this.editing = editing; } |