Upstream/Essentials
1
0
Fork 0
mirror of https://github.com/EssentialsX/Essentials.git synced 2024-12-26 02:58:03 +01:00
Code Issues Projects Releases Wiki Activity

Fix sticky piston retract breaking essentials sign

Browse Source 
Sticky pistons could break signs that were mounted on blocks the pistons were retracting.

This patch extends the protection from just the piston base to include the extended piston arm and the block at the end of the piston arm as well.

This method will very likely need to be changed again when 1.8 comes out, but this should bring us a bit closer, by wrapping the block tests in a loop.
This commit is contained in:
SidShakal 2014-07-19 09:37:37 -05:00 committed by KHobbits
parent 465041b98b
commit d1c014fe16
1 changed files with 22 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
Essentials/src/com/earth2me/essentials/signs/SignBlockListener.java
View File

@ -271,23 +271,33 @@ public class SignBlockListener implements Listener
if (event.isSticky())
{
final Block block = event.getBlock();
if (((block.getType() == WALL_SIGN
|| block.getType() == SIGN_POST)
&& EssentialsSign.isValidSign(new EssentialsSign.BlockSign(block)))
|| EssentialsSign.checkIfBlockBreaksSigns(block))
final Block pistonBaseBlock = event.getBlock();
final Block[] affectedBlocks = new Block[]
{
pistonBaseBlock,
pistonBaseBlock.getRelative(event.getDirection()),
event.getRetractLocation().getBlock()
};
for (Block block : affectedBlocks)
{
event.setCancelled(true);
return;
}
for (EssentialsSign sign : ess.getSettings().enabledSigns())
{
if (sign.areHeavyEventRequired() && sign.getBlocks().contains(block.getType())
&& !sign.onBlockPush(block, ess))
if (((block.getType() == WALL_SIGN
|| block.getType() == SIGN_POST)
&& EssentialsSign.isValidSign(new EssentialsSign.BlockSign(block)))
|| EssentialsSign.checkIfBlockBreaksSigns(block))
{
event.setCancelled(true);
return;
}
for (EssentialsSign sign : ess.getSettings().enabledSigns())
{
if (sign.areHeavyEventRequired() && sign.getBlocks().contains(block.getType())
&& !sign.onBlockPush(block, ess))
{
event.setCancelled(true);
return;
}
}
}
}
}