Skip to content

Commit 7f85e8d

Browse files
Fix.. please?
Edited recreateItems() to use .entrySet().iterator() instead of for loop for my sanity
1 parent 4f27dfb commit 7f85e8d

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/main/java/me/flame/menus/menu/BaseMenu.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -608,11 +608,10 @@ public M updateTitle(String title) {
608608
}
609609

610610
protected void recreateItems() {
611-
int itemsLength = itemMap.size();
612-
if (itemsLength == 0) return;
613-
614-
for (int i = 1; i <= itemsLength; i++) {
615-
ItemStack item = itemMap.get(i).getItemStack();
611+
if (itemMap.isEmpty()) return;
612+
for (Map.Entry<Integer, MenuItem> entry : itemMap.entrySet()) {
613+
int i = entry.getKey();
614+
ItemStack item = entry.getValue().getItemStack();
616615
if (item == null || item.getType() == AIR) {
617616
inventory.setItem(i, null);
618617
continue;
@@ -621,6 +620,7 @@ protected void recreateItems() {
621620
}
622621
}
623622

623+
624624
/**
625625
* Open the inventory for the provided player.
626626
* @apiNote Will not work if the player is sleeping.

0 commit comments

Comments
 (0)