From be3fc64dba088319ccf03e328ed04bd37a80d367 Mon Sep 17 00:00:00 2001 From: Scott Burns Date: Thu, 16 May 2019 00:58:51 +0200 Subject: [PATCH] Refactor LootingBagViewer --- .../LootingBagViewerOverlay.java | 20 ++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/runelite-client/src/main/java/net/runelite/client/plugins/lootingbagviewer/LootingBagViewerOverlay.java b/runelite-client/src/main/java/net/runelite/client/plugins/lootingbagviewer/LootingBagViewerOverlay.java index f583d5b768..4b66517489 100644 --- a/runelite-client/src/main/java/net/runelite/client/plugins/lootingbagviewer/LootingBagViewerOverlay.java +++ b/runelite-client/src/main/java/net/runelite/client/plugins/lootingbagviewer/LootingBagViewerOverlay.java @@ -74,20 +74,26 @@ class LootingBagViewerOverlay extends Overlay { if(client.getItemContainer(InventoryID.LOOTING_BAG) != null) { itemContainer = client.getItemContainer(InventoryID.LOOTING_BAG); - items = itemContainer.getItems(); + if (itemContainer != null) + { + items = itemContainer.getItems(); + } } return null; } - else if(itemContainer != null && client.getItemContainer(InventoryID.LOOTING_BAG) != null) + else if(items != null && client.getItemContainer(InventoryID.LOOTING_BAG) != null) { itemContainer = client.getItemContainer(InventoryID.LOOTING_BAG); - Item[] tempItems = itemContainer.getItems(); - - for(int i = 0; i < items.length; i++) + if (itemContainer != null) { - if(!items[i].equals(tempItems[i])) + Item[] tempItems = itemContainer.getItems(); + + for (int i = 0; i < items.length; i++) { - items = tempItems; + if (!items[i].equals(tempItems[i])) + { + items = tempItems; + } } } }