clean up

Author: SaloEater

src/main/java/com/robertx22/mine_and_slash/capability/player/container/JewelsMenu.java

@@ -35,6 +35,10 @@ public JewelsMenu(MenuType<?> pType, int pContainerId, Inventory pPlayerInventor
         }
     }
 
+    private boolean isJewelSlot(int pIndex) {
+        return pIndex >= maxJewels && pIndex < firstPlayerInventoryIndex;
+    }
+
     @Override
     public ItemStack quickMoveStack(Player pPlayer, int pIndex) {
         if (isJewelSlot(pIndex)) {
@@ -43,10 +47,6 @@ public ItemStack quickMoveStack(Player pPlayer, int pIndex) {
         return super.quickMoveStack(pPlayer, pIndex);
     }
 
-    private boolean isJewelSlot(int pIndex) {
-        return pIndex >= maxJewels && pIndex < firstPlayerInventoryIndex;
-    }
-
     @Override
     public void clicked(int pSlotId, int pButton, ClickType pClickType, Player pPlayer) {
         if (isJewelSlot(pSlotId) || pClickType == ClickType.PICKUP_ALL) {

src/main/java/com/robertx22/mine_and_slash/vanilla_mc/packets/OpenJewelsPacket.java

@@ -40,15 +40,11 @@ public void saveToData(FriendlyByteBuf buf) {
 
     @Override
     public void onReceived(ExilePacketContext ctx) {
-
         JewelInvHelper jewels = Load.player(ctx.getPlayer()).getJewels();
-        var inv = jewels.inv;
-
         Player p = ctx.getPlayer();
         int maxJewels = jewels.getJewelSocketsMaxStat(p);
-
         p.openMenu(new SimpleMenuProvider((i, playerInventory, playerEntity) -> {
-            return oneRow(i, playerInventory, inv, maxJewels); // todo why doesnt vanilla have this
+            return oneRow(i, playerInventory, jewels.inv, maxJewels); // todo why doesnt vanilla have this
         }, Component.literal("")));
 
     }