From 960d97de4903de279e02bd2def60850db8a30b8c Mon Sep 17 00:00:00 2001 From: Electroblob77 <35599699+Electroblob77@users.noreply.github.com> Date: Mon, 2 Sep 2019 16:03:40 +0100 Subject: [PATCH] Prevent IndexOutOfBoundsException when clicking an already-remembered stone circle, fixes second crash in #217 --- .../electroblob/wizardry/block/BlockTransportationStone.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/electroblob/wizardry/block/BlockTransportationStone.java b/src/main/java/electroblob/wizardry/block/BlockTransportationStone.java index ea0595a1..979fa0f4 100644 --- a/src/main/java/electroblob/wizardry/block/BlockTransportationStone.java +++ b/src/main/java/electroblob/wizardry/block/BlockTransportationStone.java @@ -73,7 +73,7 @@ public class BlockTransportationStone extends Block { public boolean isOpaqueCube(IBlockState state){ return false; } - + @Override public boolean isSideSolid(IBlockState base_state, IBlockAccess world, BlockPos pos, EnumFacing side){ return side == EnumFacing.DOWN; @@ -147,7 +147,7 @@ public class BlockTransportationStone extends Block { if(locations.isEmpty()) locations.add(here); else{ locations.remove(here); // Prevents duplicates - locations.set(locations.size() - 1, here); + locations.set(Math.max(locations.size() - 1, 0), here); } if(!world.isRemote) player.sendStatusMessage(new TextComponentTranslation("tile." + Wizardry.MODID + ":transportation_stone.confirm", Spells.transportation.getNameForTranslationFormatted()), true); }