Controller sorts by energy usage in GUI

CHANGELOG.md

@@ -10,6 +10,7 @@
 - Added Silk Touch Upgrade for Destructor (InusualZ)
 - Added Fortune Upgrade for Destructor (InusualZ)
 - Added ore dictionary compare toggle to storage I/O blocks (raoulvdberge)
+- Controller sorts by energy usage in GUI (raoulvdberge)
 - The Constructor can now drop items in the world (raoulvdberge)
 - The Constructor can now place skulls (modmuss50)
 - The Destructor can now pick up items in the world (InusualZ)

src/main/java/com/raoulvdberge/refinedstorage/tile/TileController.java

@@ -114,22 +114,22 @@ public class TileController extends TileBase implements INetworkMaster, IEnergyR
                         node.getEnergyUsage()
                     );
 
-                    if (clientNode.getStack().getItem() != null) {
+                    if (clientNode.getStack().getItem() == null) {
-                        if (nodes.contains(clientNode)) {
+                        continue;
-                            for (ClientNode other : nodes) {
+                    }
-                                if (other.equals(clientNode)) {
-                                    other.setAmount(other.getAmount() + 1);
 
-                                    break;
+                    if (nodes.contains(clientNode)) {
-                                }
+                        ClientNode other = nodes.get(nodes.indexOf(clientNode));
-                            }
+
-                        } else {
+                        other.setAmount(other.getAmount() + 1);
-                            nodes.add(clientNode);
+                    } else {
-                        }
+                        nodes.add(clientNode);
                     }
                 }
             }
 
+            Collections.sort(nodes, CLIENT_NODE_COMPARATOR);
+
             return nodes;
         }
     });
@@ -139,6 +139,14 @@ public class TileController extends TileBase implements INetworkMaster, IEnergyR
 
     private static final String NBT_CRAFTING_TASKS = "CraftingTasks";
 
+    private static final Comparator<ClientNode> CLIENT_NODE_COMPARATOR = (left, right) -> {
+        if (left.getEnergyUsage() == right.getEnergyUsage()) {
+            return 0;
+        }
+
+        return (left.getEnergyUsage() > right.getEnergyUsage()) ? -1 : 1;
+    };
+
     private static final Comparator<IItemStorage> ITEM_SIZE_COMPARATOR = (left, right) -> {
         if (left.getStored() == right.getStored()) {
             return 0;