1h - Fix INdexOutOfBoundsException and use guidline formating for code.
This commit is contained in:
parent
ef9f7d6dca
commit
99bb301e0d
@ -5,6 +5,7 @@ import java.util.HashMap;
|
|||||||
import java.util.List;
|
import java.util.List;
|
||||||
import java.util.UUID;
|
import java.util.UUID;
|
||||||
|
|
||||||
|
import org.bukkit.World;
|
||||||
import org.bukkit.block.Block;
|
import org.bukkit.block.Block;
|
||||||
import org.bukkit.entity.Player;
|
import org.bukkit.entity.Player;
|
||||||
import org.bukkit.entity.TNTPrimed;
|
import org.bukkit.entity.TNTPrimed;
|
||||||
@ -20,10 +21,10 @@ import org.bukkit.Material;
|
|||||||
|
|
||||||
import com.massivecraft.factions.entity.MConf;
|
import com.massivecraft.factions.entity.MConf;
|
||||||
import com.massivecraft.massivecore.Engine;
|
import com.massivecraft.massivecore.Engine;
|
||||||
|
import com.massivecraft.massivecore.collections.MassiveList;
|
||||||
import com.massivecraft.massivecore.ps.PS;
|
import com.massivecraft.massivecore.ps.PS;
|
||||||
import com.massivecraft.massivecore.util.MUtil;
|
import com.massivecraft.massivecore.util.MUtil;
|
||||||
|
|
||||||
|
|
||||||
public class EngineExploit extends Engine
|
public class EngineExploit extends Engine
|
||||||
{
|
{
|
||||||
// -------------------------------------------- //
|
// -------------------------------------------- //
|
||||||
@ -132,121 +133,121 @@ public class EngineExploit extends Engine
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
// -------------------------------------------- //
|
// -------------------------------------------- //
|
||||||
// NETHER PORTAL TRAP
|
// NETHER PORTAL TRAP
|
||||||
// -------------------------------------------- //
|
// -------------------------------------------- //
|
||||||
// A nether portal trap can be created by the destination portal being enclosed (trapped) - resulting in the player not being able to run commands.
|
// A nether portal trap can be created by the destination portal being enclosed (trapped) - resulting in the player not being able to run commands.
|
||||||
// This fix removes the portal blocks (client side) from the destination until they are away from the portal.
|
// This fix removes the portal blocks (client side) from the destination until they are away from the portal.
|
||||||
|
|
||||||
private int NETHER_TRAP_RADIUS_CHECK = 5;
|
private static final int NETHER_TRAP_RADIUS_CHECK = 5;
|
||||||
private int NETHER_TRAP_RESET_RADIUS = 3;
|
private static final int NETHER_TRAP_RESET_RADIUS_SQUARED = 9;
|
||||||
|
|
||||||
private HashMap<UUID, List<Block>> netherTrapBlockSet = new HashMap<UUID, List<Block>>();
|
private HashMap<UUID, List<Block>> portalTraps = new HashMap<UUID, List<Block>>();
|
||||||
|
|
||||||
// Detect teleport from a nether portal and remove animation if required
|
|
||||||
@EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
|
@EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
|
||||||
public void netherportalTrapRemoveAnimation(PlayerTeleportEvent event)
|
public void portalTrapRemoveAnimation(PlayerTeleportEvent event)
|
||||||
{
|
{
|
||||||
if ( ! MConf.get().handleNetherPortalTrap) return;
|
// If there is a teleport caused by a nether portal ...
|
||||||
|
if ( ! MConf.get().handleNetherPortalTrap || event.getCause() != TeleportCause.NETHER_PORTAL) return;
|
||||||
if (event.getCause() != TeleportCause.NETHER_PORTAL) return;
|
|
||||||
|
|
||||||
// If they can build at the target destination then we will not do any further checks
|
|
||||||
if (EngineMain.canPlayerBuildAt(event.getPlayer(), PS.valueOf(event.getTo()), false)) return;
|
|
||||||
|
|
||||||
final Player player = event.getPlayer();
|
|
||||||
final UUID uuid = player.getUniqueId();
|
|
||||||
|
|
||||||
|
Player player = event.getPlayer();
|
||||||
Block from = event.getTo().getBlock();
|
Block from = event.getTo().getBlock();
|
||||||
|
|
||||||
// If a list exists, then we're dealing with a new portal - so revert the old portal blocks
|
// ... and the player can't build at the destination ...
|
||||||
this.netherportalReset(player);
|
if (EngineMain.canPlayerBuildAt(player, PS.valueOf(from), false)) return;
|
||||||
|
|
||||||
// Store some temporary data
|
// ... reset the old portal blocks stored ...
|
||||||
this.netherTrapBlockSet.put(uuid, this.getPortalBlocks(from));
|
this.portalReset(player);
|
||||||
|
|
||||||
// Send the air update
|
// ... get all the portal blocks belonging to the new portal at the destination ...
|
||||||
this.netherportalSendAir(player);
|
List<Block> portalTrap = getPortal(from);
|
||||||
|
if (portalTrap.isEmpty()) return;
|
||||||
|
|
||||||
|
// ... and then store those blocks and send an update as if they were air.
|
||||||
|
this.portalTraps.put(player.getUniqueId(), portalTrap);
|
||||||
|
portalUpdateAir(player, portalTrap);
|
||||||
}
|
}
|
||||||
|
|
||||||
// When they leave the portal we will update it, otherwise send the updates again
|
|
||||||
@EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
|
@EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
|
||||||
public void netherportalTrapUpdate(PlayerMoveEvent event)
|
public void portalUpdate(PlayerMoveEvent event)
|
||||||
{
|
{
|
||||||
if ( ! MConf.get().handleNetherPortalTrap) return;
|
// If a player moves ...
|
||||||
|
if ( ! MConf.get().handleNetherPortalTrap || MUtil.isSameBlock(event)) return;
|
||||||
// Only check if we're changing blocks
|
|
||||||
if (MUtil.isSameBlock(event)) return;
|
|
||||||
|
|
||||||
Player player = event.getPlayer();
|
Player player = event.getPlayer();
|
||||||
UUID uuid = player.getUniqueId();
|
UUID uuid = player.getUniqueId();
|
||||||
|
|
||||||
// Check if we're sending updates to this player
|
// ... and he recently used a portal ...
|
||||||
if ( ! this.netherTrapBlockSet.containsKey(uuid)) return;
|
List<Block> portalTrap = this.portalTraps.get(uuid);
|
||||||
|
if (portalTrap == null) return;
|
||||||
|
|
||||||
// If they've changed worlds we don't bother checking this
|
Location locationTo = event.getTo();
|
||||||
if (event.getTo().getWorld() != this.netherTrapBlockSet.get(uuid).get(0).getWorld())
|
Location locationFrom = portalTrap.get(0).getLocation();
|
||||||
|
|
||||||
|
World worldTo = locationTo.getWorld();
|
||||||
|
World worldFrom = locationFrom.getWorld();
|
||||||
|
|
||||||
|
// ... update reset the portal near them, if they have moved away too far ...
|
||||||
|
if ( ! worldTo.equals(worldFrom) || locationTo.distanceSquared(locationFrom) > NETHER_TRAP_RESET_RADIUS_SQUARED)
|
||||||
{
|
{
|
||||||
this.netherportalReset(player);
|
portalUpdateReset(player, portalTrap);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
// When the player moves away from the portal, we put it back to normal
|
// ... or send an update as if the portal blocks were air.
|
||||||
if (event.getTo().distance(this.netherTrapBlockSet.get(uuid).get(0).getLocation()) > NETHER_TRAP_RESET_RADIUS)
|
portalUpdateAir(player, portalTrap);
|
||||||
{
|
|
||||||
this.netherportalReset(player);
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
// Send updates as air
|
|
||||||
this.netherportalSendAir(player);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Sends block update to player with original blocks (if required)
|
public void portalReset(Player player)
|
||||||
@SuppressWarnings("deprecation")
|
|
||||||
public void netherportalReset(Player player)
|
|
||||||
{
|
{
|
||||||
UUID uuid = player.getUniqueId();
|
UUID uuid = player.getUniqueId();
|
||||||
|
|
||||||
if ( ! this.netherTrapBlockSet.containsKey(uuid)) return;
|
// If a player has already a portal registered to him ...
|
||||||
|
List<Block> portalTrap = this.portalTraps.get(uuid);
|
||||||
|
if (portalTrap == null) return;
|
||||||
|
|
||||||
// Only send updates if they're in the same world
|
// ... remove them from the registry ...
|
||||||
if (this.netherTrapBlockSet.get(uuid).get(0).getWorld() == player.getWorld()) for (Block block : this.netherTrapBlockSet.get(uuid)) player.sendBlockChange(block.getLocation(), block.getType(), block.getData());
|
this.portalTraps.remove(uuid);
|
||||||
|
|
||||||
// Remove the block set
|
// ... and send updates if the player and portal are in the same world.
|
||||||
this.netherTrapBlockSet.remove(uuid);
|
if ( ! player.getWorld().equals(portalTrap.get(0).getWorld())) return;
|
||||||
|
portalUpdateReset(player, portalTrap);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Send air block updates to player
|
public static void portalUpdateReset(Player player, List<Block> portal) { portalUpdate(player, portal, null, null); }
|
||||||
|
public static void portalUpdateAir(Player player, List<Block> portal) { portalUpdate(player, portal, Material.AIR, (byte) 0); }
|
||||||
|
|
||||||
@SuppressWarnings("deprecation")
|
@SuppressWarnings("deprecation")
|
||||||
public void netherportalSendAir(Player player)
|
private static void portalUpdate(Player player, List<Block> portal, Material material, Byte data)
|
||||||
{
|
{
|
||||||
for (Block block : this.netherTrapBlockSet.get(player.getUniqueId())) player.sendBlockChange(block.getLocation(), Material.AIR, (byte) 0);
|
boolean usingDefault = material == null && data == null;
|
||||||
|
for (Block block : portal)
|
||||||
|
{
|
||||||
|
Material updateMaterial = usingDefault ? block.getType() : material;
|
||||||
|
byte updateData = usingDefault ? block.getData() : data;
|
||||||
|
player.sendBlockChange(block.getLocation(), updateMaterial, updateData);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Get portal blocks near a block
|
public static List<Block> getPortal(Block from)
|
||||||
public List<Block> getPortalBlocks(Block from) {
|
{
|
||||||
List<Block> blocks = new ArrayList<Block>();
|
// Create
|
||||||
|
List<Block> ret = new MassiveList<>();
|
||||||
|
|
||||||
// Check in a radius of the block to find the portal blocks
|
// Fill - Check in a radius of the block to find the portal blocks
|
||||||
for (int x = -(NETHER_TRAP_RADIUS_CHECK); x <= NETHER_TRAP_RADIUS_CHECK; x ++)
|
for (int x = -(NETHER_TRAP_RADIUS_CHECK); x <= NETHER_TRAP_RADIUS_CHECK; x ++)
|
||||||
{
|
{
|
||||||
for (int y = -(NETHER_TRAP_RADIUS_CHECK); y <= NETHER_TRAP_RADIUS_CHECK; y ++)
|
for (int y = -(NETHER_TRAP_RADIUS_CHECK); y <= NETHER_TRAP_RADIUS_CHECK; y ++)
|
||||||
{
|
{
|
||||||
for (int z = -(NETHER_TRAP_RADIUS_CHECK); z <= NETHER_TRAP_RADIUS_CHECK; z ++)
|
for (int z = -(NETHER_TRAP_RADIUS_CHECK); z <= NETHER_TRAP_RADIUS_CHECK; z ++)
|
||||||
{
|
{
|
||||||
if (from.getRelative(x, y, z).getType() == Material.PORTAL)
|
if (from.getRelative(x, y, z).getType() == Material.PORTAL) ret.add(from.getRelative(x, y, z));
|
||||||
{
|
|
||||||
// Store block
|
|
||||||
blocks.add(from.getRelative(x, y, z));
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return blocks;
|
// Return
|
||||||
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user