From 814277eb364018c421d43a93add4c0d3c0844fad Mon Sep 17 00:00:00 2001 From: Tomas Slusny Date: Thu, 17 Jan 2019 18:31:08 +0100 Subject: [PATCH] Inverse collapse menu entry loop This makes sure that menu collapses to first entry because menu is built in inverse. Before setting the entries, inverse them back to original order. Signed-off-by: Tomas Slusny --- .../client/plugins/grounditems/GroundItemsPlugin.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/runelite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsPlugin.java b/runelite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsPlugin.java index d4714453a2..6e6f7c53b8 100644 --- a/runelite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsPlugin.java +++ b/runelite-client/src/main/java/net/runelite/client/plugins/grounditems/GroundItemsPlugin.java @@ -35,6 +35,7 @@ import java.awt.Rectangle; import static java.lang.Boolean.TRUE; import java.util.ArrayList; import java.util.Collection; +import java.util.Collections; import java.util.HashSet; import java.util.LinkedHashMap; import java.util.List; @@ -308,8 +309,10 @@ public class GroundItemsPlugin extends Plugin final List newEntries = new ArrayList<>(menuEntries.length); outer: - for (MenuEntry menuEntry : menuEntries) + for (int i = menuEntries.length - 1; i >= 0; i--) { + MenuEntry menuEntry = menuEntries[i]; + int menuType = menuEntry.getType(); if (menuType == FIRST_OPTION || menuType == SECOND_OPTION || menuType == THIRD_OPTION || menuType == FOURTH_OPTION || menuType == FIFTH_OPTION || menuType == EXAMINE_ITEM) @@ -327,6 +330,8 @@ public class GroundItemsPlugin extends Plugin newEntries.add(new MenuEntryWithCount(menuEntry)); } + Collections.reverse(newEntries); + client.setMenuEntries(newEntries.stream().map(e -> { final MenuEntry entry = e.getEntry();