Apply sorting to configuration items (#94)

Author: Crim

kafka-webview-ui/src/main/java/org/sourcelab/kafka/webview/ui/controller/cluster/ClusterController.java

@@ -65,7 +65,7 @@ public String clusterIndex(final Model model, final RedirectAttributes redirectA
         manager.addCrumb("Cluster Explorer", null);
 
         // Retrieve all clusters
-        final Iterable<Cluster> clusterList = clusterRepository.findAll();
+        final Iterable<Cluster> clusterList = clusterRepository.findAllByOrderByNameAsc();
         model.addAttribute("clusterList", clusterList);
 
         // Retrieve how many views for each cluster

kafka-webview-ui/src/main/java/org/sourcelab/kafka/webview/ui/manager/kafka/dto/BrokerConfig.java

@@ -24,7 +24,9 @@
 
 package org.sourcelab.kafka.webview.ui.manager.kafka.dto;
 
+import java.util.ArrayList;
 import java.util.Collections;
+import java.util.Comparator;
 import java.util.List;
 
 /**
@@ -34,7 +36,14 @@ public class BrokerConfig {
     private final List<ConfigItem> configEntries;
 
     public BrokerConfig(final List<ConfigItem> configEntries) {
-        this.configEntries = Collections.unmodifiableList(configEntries);
+        // Create new list from source.
+        final List<ConfigItem> sorted = new ArrayList<>(configEntries);
+
+        // Sort the list by name
+        sorted.sort(Comparator.comparing(ConfigItem::getName));
+
+        // Make immutable.
+        this.configEntries = Collections.unmodifiableList(sorted);
     }
 
     public List<ConfigItem> getConfigEntries() {

kafka-webview-ui/src/main/java/org/sourcelab/kafka/webview/ui/manager/kafka/dto/TopicConfig.java

@@ -24,7 +24,9 @@
 
 package org.sourcelab.kafka.webview.ui.manager.kafka.dto;
 
+import java.util.ArrayList;
 import java.util.Collections;
+import java.util.Comparator;
 import java.util.List;
 
 /**
@@ -37,7 +39,14 @@ public class TopicConfig {
      * Constructor.
      */
     public TopicConfig(final List<ConfigItem> configEntries) {
-        this.configEntries = Collections.unmodifiableList(configEntries);
+        // Create new list from source.
+        final List<ConfigItem> sorted = new ArrayList<>(configEntries);
+
+        // Sort the list by name
+        sorted.sort(Comparator.comparing(ConfigItem::getName));
+
+        // Make immutable.
+        this.configEntries = Collections.unmodifiableList(sorted);
     }
 
     public List<ConfigItem> getConfigEntries() {