From d18cd1513e2e0a2bf10feb5fabd75349cddb6cc7 Mon Sep 17 00:00:00 2001 From: Jake Potrebic Date: Tue, 15 Mar 2022 01:38:15 -0700 Subject: [PATCH] Make some itemstacks nonnull --- .../craftbukkit/inventory/CraftInventoryPlayer.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventoryPlayer.java b/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventoryPlayer.java index 107fa1d4b..e62baea16 100644 --- a/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventoryPlayer.java +++ b/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventoryPlayer.java @@ -155,13 +155,13 @@ public class CraftInventoryPlayer extends CraftInventory implements org.bukkit.i case OFF_HAND: return this.getItemInOffHand(); case FEET: - return this.getBoots(); + return java.util.Objects.requireNonNullElseGet(this.getBoots(), () -> new ItemStack(org.bukkit.Material.AIR)); // Paper - make nonnull case LEGS: - return this.getLeggings(); + return java.util.Objects.requireNonNullElseGet(this.getLeggings(), () -> new ItemStack(org.bukkit.Material.AIR)); // Paper - make nonnull case CHEST: - return this.getChestplate(); + return java.util.Objects.requireNonNullElseGet(this.getChestplate(), () -> new ItemStack(org.bukkit.Material.AIR)); // Paper - make nonnull case HEAD: - return this.getHelmet(); + return java.util.Objects.requireNonNullElseGet(this.getHelmet(), () -> new ItemStack(org.bukkit.Material.AIR)); // Paper - make nonnull default: throw new IllegalArgumentException("Could not get slot " + slot + " - not a valid slot for PlayerInventory"); }