diff --git a/src/main/java/codechicken/enderstorage/api/Frequency.java b/src/main/java/codechicken/enderstorage/api/Frequency.java index 007a6f1..14c91f4 100644 --- a/src/main/java/codechicken/enderstorage/api/Frequency.java +++ b/src/main/java/codechicken/enderstorage/api/Frequency.java @@ -6,6 +6,7 @@ import codechicken.lib.util.Copyable; import net.minecraft.nbt.CompoundTag; import net.minecraft.network.chat.Component; +import net.minecraft.world.entity.player.Player; import net.minecraft.world.item.ItemStack; import java.util.UUID; @@ -82,13 +83,15 @@ public Frequency setRight(EnumColour right) { return this; } - public Frequency setOwner(UUID owner) { - this.owner = owner; + public Frequency setOwner(Player player) { + owner = player.getUUID(); + ownerName = player.getName(); return this; } - public Frequency setOwnerName(Component ownerName) { - this.ownerName = ownerName; + public Frequency clearOwner() { + owner = null; + ownerName = null; return this; } @@ -104,11 +107,11 @@ public Frequency set(EnumColour[] colours) { } public Frequency set(Frequency frequency) { - setLeft(frequency.left); - setMiddle(frequency.middle); - setRight(frequency.right); - setOwner(frequency.owner); - setOwnerName(frequency.ownerName); + left = frequency.left; + middle = frequency.middle; + right = frequency.right; + owner = frequency.owner; + ownerName = frequency.ownerName; return this; } @@ -136,7 +139,7 @@ public EnumColour[] toArray() { return new EnumColour[] { left, middle, right }; } - protected Frequency read_internal(CompoundTag tagCompound) { + private Frequency read_internal(CompoundTag tagCompound) { left = EnumColour.fromWoolMeta(tagCompound.getInt("left")); middle = EnumColour.fromWoolMeta(tagCompound.getInt("middle")); right = EnumColour.fromWoolMeta(tagCompound.getInt("right")); @@ -149,7 +152,7 @@ protected Frequency read_internal(CompoundTag tagCompound) { return this; } - protected CompoundTag write_internal(CompoundTag tagCompound) { + private CompoundTag write_internal(CompoundTag tagCompound) { tagCompound.putInt("left", left.getWoolMeta()); tagCompound.putInt("middle", middle.getWoolMeta()); tagCompound.putInt("right", right.getWoolMeta()); diff --git a/src/main/java/codechicken/enderstorage/block/BlockEnderStorage.java b/src/main/java/codechicken/enderstorage/block/BlockEnderStorage.java index cd5caad..bd05cff 100644 --- a/src/main/java/codechicken/enderstorage/block/BlockEnderStorage.java +++ b/src/main/java/codechicken/enderstorage/block/BlockEnderStorage.java @@ -107,14 +107,11 @@ public InteractionResult use(BlockState state, Level world, BlockPos pos, Player return InteractionResult.FAIL; } - owner.setFreq(owner.getFrequency().copy().setOwner(null)); + owner.setFreq(owner.getFrequency().copy().clearOwner()); return InteractionResult.SUCCESS; } else if (!item.isEmpty() && ItemUtils.areStacksSameType(item, EnderStorageConfig.getPersonalItem())) { if (!owner.getFrequency().hasOwner()) { - owner.setFreq(owner.getFrequency().copy()// - .setOwner(player.getUUID())// - .setOwnerName(player.getName())// - ); + owner.setFreq(owner.getFrequency().copy().setOwner(player)); if (!player.getAbilities().instabuild) { item.shrink(1); } diff --git a/src/main/java/codechicken/enderstorage/container/ContainerEnderItemStorage.java b/src/main/java/codechicken/enderstorage/container/ContainerEnderItemStorage.java index 7eabc23..ae09d62 100644 --- a/src/main/java/codechicken/enderstorage/container/ContainerEnderItemStorage.java +++ b/src/main/java/codechicken/enderstorage/container/ContainerEnderItemStorage.java @@ -12,7 +12,6 @@ import net.minecraft.world.inventory.Slot; import net.minecraft.world.item.ItemStack; -//@ChestContainer public class ContainerEnderItemStorage extends AbstractContainerMenu { public EnderItemStorage chestInv; diff --git a/src/main/java/codechicken/enderstorage/item/ItemEnderPouch.java b/src/main/java/codechicken/enderstorage/item/ItemEnderPouch.java index 3efb331..d9322db 100644 --- a/src/main/java/codechicken/enderstorage/item/ItemEnderPouch.java +++ b/src/main/java/codechicken/enderstorage/item/ItemEnderPouch.java @@ -53,11 +53,10 @@ public InteractionResult onItemUseFirst(ItemStack stack, UseOnContext context) { } BlockEntity tile = world.getBlockEntity(context.getClickedPos()); - if (tile instanceof TileEnderChest && context.getPlayer().isCrouching()) { - TileEnderChest chest = (TileEnderChest) tile; + if (tile instanceof TileEnderChest chest && context.getPlayer().isCrouching()) { Frequency frequency = chest.getFrequency().copy(); if (EnderStorageConfig.anarchyMode && !(frequency.owner != null && frequency.owner.equals(context.getPlayer().getUUID()))) { - frequency.setOwner(null); + frequency.clearOwner(); } frequency.writeToStack(stack);