From b688a2793ba95eb2baede25faa61096409b4837e Mon Sep 17 00:00:00 2001 From: mathiascode Date: Sat, 21 Dec 2019 16:12:26 +0200 Subject: [PATCH] More style fixes --- pom.xml | 8 -- src/main/java/pw/kaboom/extras/Main.java | 6 +- .../extras/commands/CommandServerInfo.java | 8 +- .../kaboom/extras/helpers/SkinDownloader.java | 12 ++- .../extras/modules/block/BlockCheck.java | 10 ++- .../extras/modules/block/BlockPhysics.java | 39 +++++----- .../modules/entity/EntityExplosion.java | 20 +++-- .../modules/entity/EntityKnockback.java | 4 +- .../extras/modules/entity/EntityTeleport.java | 4 +- .../extras/modules/player/PlayerChat.java | 14 ++-- .../extras/modules/player/PlayerCommand.java | 4 +- .../modules/player/PlayerConnection.java | 27 +++---- .../extras/modules/player/PlayerDamage.java | 20 ++--- .../extras/modules/player/PlayerInteract.java | 4 +- .../extras/modules/player/PlayerTeleport.java | 4 +- .../extras/modules/server/ServerCommand.java | 77 ++++++++++--------- .../extras/modules/server/ServerPing.java | 7 +- 17 files changed, 136 insertions(+), 132 deletions(-) diff --git a/pom.xml b/pom.xml index 7f863aa..5d03dbb 100644 --- a/pom.xml +++ b/pom.xml @@ -49,14 +49,6 @@ - - org.apache.maven.plugins - maven-compiler-plugin - - 8 - 8 - - diff --git a/src/main/java/pw/kaboom/extras/Main.java b/src/main/java/pw/kaboom/extras/Main.java index 0bc7a04..155a366 100644 --- a/src/main/java/pw/kaboom/extras/Main.java +++ b/src/main/java/pw/kaboom/extras/Main.java @@ -1,8 +1,6 @@ package pw.kaboom.extras; import java.util.Collections; -import java.util.HashSet; -import java.util.UUID; import org.bukkit.block.BlockFace; import org.bukkit.plugin.java.JavaPlugin; @@ -36,9 +34,7 @@ import pw.kaboom.extras.modules.player.PlayerTeleport; import pw.kaboom.extras.modules.server.ServerCommand; import pw.kaboom.extras.modules.server.ServerPing; -public class Main extends JavaPlugin { - public static HashSet skinInProgress = new HashSet(); - +public final class Main extends JavaPlugin { @Override public void onLoad() { /* Fill lists */ diff --git a/src/main/java/pw/kaboom/extras/commands/CommandServerInfo.java b/src/main/java/pw/kaboom/extras/commands/CommandServerInfo.java index 5ca2e32..6527f75 100644 --- a/src/main/java/pw/kaboom/extras/commands/CommandServerInfo.java +++ b/src/main/java/pw/kaboom/extras/commands/CommandServerInfo.java @@ -45,8 +45,8 @@ public final class CommandServerInfo implements CommandExecutor { ); sendInfoMessage(sender, "Java version", ManagementFactory.getRuntimeMXBean().getSpecVersion() - + " " + - ManagementFactory.getRuntimeMXBean().getVmVersion() + + " " + + ManagementFactory.getRuntimeMXBean().getVmVersion() ); try { @@ -103,8 +103,8 @@ public final class CommandServerInfo implements CommandExecutor { final long seconds = (ManagementFactory.getRuntimeMXBean().getUptime() / 1000) % 60; sendInfoMessage(sender, "Server uptime", - minutes + " minute(s) " + - seconds + " second(s)" + minutes + " minute(s) " + + seconds + " second(s)" ); return true; } diff --git a/src/main/java/pw/kaboom/extras/helpers/SkinDownloader.java b/src/main/java/pw/kaboom/extras/helpers/SkinDownloader.java index 5541a7d..b2240b8 100644 --- a/src/main/java/pw/kaboom/extras/helpers/SkinDownloader.java +++ b/src/main/java/pw/kaboom/extras/helpers/SkinDownloader.java @@ -3,6 +3,8 @@ package pw.kaboom.extras.helpers; import java.io.IOException; import java.io.InputStreamReader; import java.net.URL; +import java.util.HashSet; +import java.util.UUID; import javax.net.ssl.HttpsURLConnection; @@ -18,6 +20,8 @@ import com.google.gson.JsonParser; import pw.kaboom.extras.Main; public final class SkinDownloader { + public static HashSet skinInProgress = new HashSet(); + private HttpsURLConnection skinConnection; private InputStreamReader skinStream; @@ -28,7 +32,7 @@ public final class SkinDownloader { new BukkitRunnable() { @Override public void run() { - Main.skinInProgress.add(player.getUniqueId()); + skinInProgress.add(player.getUniqueId()); final PlayerProfile profile = player.getPlayerProfile(); @@ -48,14 +52,14 @@ public final class SkinDownloader { try { skinStream.close(); skinConnection.disconnect(); - } catch (Exception e) { + } catch (Exception ignored) { } if (!shouldChangeName && shouldSendMessage) { player.sendMessage("A player with that username doesn't exist"); } - Main.skinInProgress.remove(player.getUniqueId()); + skinInProgress.remove(player.getUniqueId()); if (!shouldChangeName) { return; @@ -75,7 +79,7 @@ public final class SkinDownloader { // Do nothing } - Main.skinInProgress.remove(player.getUniqueId()); + skinInProgress.remove(player.getUniqueId()); } }.runTask(JavaPlugin.getPlugin(Main.class)); } diff --git a/src/main/java/pw/kaboom/extras/modules/block/BlockCheck.java b/src/main/java/pw/kaboom/extras/modules/block/BlockCheck.java index afdcc89..329456e 100644 --- a/src/main/java/pw/kaboom/extras/modules/block/BlockCheck.java +++ b/src/main/java/pw/kaboom/extras/modules/block/BlockCheck.java @@ -6,10 +6,12 @@ import org.bukkit.event.Listener; import org.bukkit.event.block.BlockPlaceEvent; import org.bukkit.event.block.SignChangeEvent; -public class BlockCheck implements Listener { +public final class BlockCheck implements Listener { @EventHandler - void onBlockPlace(BlockPlaceEvent event) { - if (event.getItemInHand().toString().length() > 3019) { + void onBlockPlace(final BlockPlaceEvent event) { + final int maxItemStringLength = 3019; + + if (event.getItemInHand().toString().length() > maxItemStringLength) { event.setCancelled(true); } @@ -21,7 +23,7 @@ public class BlockCheck implements Listener { } @EventHandler - void onSignChange(SignChangeEvent event) { + void onSignChange(final SignChangeEvent event) { try { event.getLines(); } catch (Exception exception) { diff --git a/src/main/java/pw/kaboom/extras/modules/block/BlockPhysics.java b/src/main/java/pw/kaboom/extras/modules/block/BlockPhysics.java index d785c7f..ff4d3ba 100644 --- a/src/main/java/pw/kaboom/extras/modules/block/BlockPhysics.java +++ b/src/main/java/pw/kaboom/extras/modules/block/BlockPhysics.java @@ -16,16 +16,16 @@ import org.bukkit.event.entity.EntityChangeBlockEvent; import com.destroystokyo.paper.event.block.BlockDestroyEvent; -public class BlockPhysics implements Listener { +public final class BlockPhysics implements Listener { public static HashSet blockFaces = new HashSet(); @EventHandler - void onBlockForm(BlockFormEvent event) { - if (event.getBlock().getType() == Material.LAVA || - event.getBlock().getType() == Material.WATER) { + void onBlockForm(final BlockFormEvent event) { + if (event.getBlock().getType() == Material.LAVA + || event.getBlock().getType() == Material.WATER) { for (BlockFace face : blockFaces) { - if (event.getBlock().getRelative(face).getType() != Material.LAVA && - event.getBlock().getRelative(face).getType() != Material.WATER) { + if (event.getBlock().getRelative(face).getType() != Material.LAVA + && event.getBlock().getRelative(face).getType() != Material.WATER) { return; } event.setCancelled(true); @@ -34,9 +34,9 @@ public class BlockPhysics implements Listener { } @EventHandler - void onBlockFromTo(BlockFromToEvent event) { - if (event.getBlock().getType() == Material.LAVA || - event.getBlock().getType() == Material.WATER) { + void onBlockFromTo(final BlockFromToEvent event) { + if (event.getBlock().getType() == Material.LAVA + || event.getBlock().getType() == Material.WATER) { boolean lavaFound = false; boolean waterFound = false; @@ -55,7 +55,7 @@ public class BlockPhysics implements Listener { } @EventHandler - void onBlockDestroy(BlockDestroyEvent event) { + void onBlockDestroy(final BlockDestroyEvent event) { if (!event.getBlock().getType().isSolid()) { for (BlockFace face : blockFaces) { if (event.getBlock().getRelative(face).getType() != event.getBlock().getType()) { @@ -68,7 +68,7 @@ public class BlockPhysics implements Listener { } @EventHandler - void onBlockPhysics(BlockPhysicsEvent event) { + void onBlockPhysics(final BlockPhysicsEvent event) { switch (event.getChangedType()) { case COMPARATOR: case REDSTONE_TORCH: @@ -99,23 +99,26 @@ public class BlockPhysics implements Listener { } @EventHandler - void onBlockRedstone(BlockRedstoneEvent event) { + void onBlockRedstone(final BlockRedstoneEvent event) { final double tps = Bukkit.getServer().getTPS()[0]; + final int maxTps = 10; - if (tps < 10) { + if (tps < maxTps) { event.setNewCurrent(0); } } - int fallingBlockCount; + private int fallingBlockCount; @EventHandler - void onEntityChangeBlock(EntityChangeBlockEvent event) { - if (event.getEntityType() == EntityType.FALLING_BLOCK && - event.getTo() == Material.AIR) { + void onEntityChangeBlock(final EntityChangeBlockEvent event) { + if (event.getEntityType() == EntityType.FALLING_BLOCK + && event.getTo() == Material.AIR) { fallingBlockCount++; - if (fallingBlockCount == 10) { + final int maxFallingBlockCount = 10; + + if (fallingBlockCount == maxFallingBlockCount) { event.setCancelled(true); fallingBlockCount = 0; } diff --git a/src/main/java/pw/kaboom/extras/modules/entity/EntityExplosion.java b/src/main/java/pw/kaboom/extras/modules/entity/EntityExplosion.java index 2da5483..41d0ab3 100644 --- a/src/main/java/pw/kaboom/extras/modules/entity/EntityExplosion.java +++ b/src/main/java/pw/kaboom/extras/modules/entity/EntityExplosion.java @@ -7,16 +7,20 @@ import org.bukkit.event.Listener; import org.bukkit.event.entity.ExplosionPrimeEvent; -public class EntityExplosion implements Listener { +public final class EntityExplosion implements Listener { @EventHandler - void onExplosionPrime(ExplosionPrimeEvent event) { - if (event.getRadius() > 20) { - event.setRadius(20); + void onExplosionPrime(final ExplosionPrimeEvent event) { + final int maxRadius = 20; + + if (event.getRadius() > maxRadius) { + event.setRadius(maxRadius); } - - if (event.getEntity().getWorld().getEntitiesByClass(Fireball.class).size() > 30 && - event.getRadius() > 1) { - event.setRadius(1); + + final int maxFireballCount = 30; + + if (event.getEntity().getWorld().getEntitiesByClass(Fireball.class).size() > maxFireballCount + && event.getRadius() > 1) { + event.setRadius(1); } } } diff --git a/src/main/java/pw/kaboom/extras/modules/entity/EntityKnockback.java b/src/main/java/pw/kaboom/extras/modules/entity/EntityKnockback.java index 2881512..b562fbe 100644 --- a/src/main/java/pw/kaboom/extras/modules/entity/EntityKnockback.java +++ b/src/main/java/pw/kaboom/extras/modules/entity/EntityKnockback.java @@ -5,9 +5,9 @@ import org.bukkit.event.Listener; import com.destroystokyo.paper.event.entity.EntityKnockbackByEntityEvent; -public class EntityKnockback implements Listener { +public final class EntityKnockback implements Listener { @EventHandler - void onEntityKnockbackByEntity(EntityKnockbackByEntityEvent event) { + void onEntityKnockbackByEntity(final EntityKnockbackByEntityEvent event) { final int knockbackLimit = 60; if (event.getKnockbackStrength() > knockbackLimit) { diff --git a/src/main/java/pw/kaboom/extras/modules/entity/EntityTeleport.java b/src/main/java/pw/kaboom/extras/modules/entity/EntityTeleport.java index 0bfc7bd..001d0fc 100644 --- a/src/main/java/pw/kaboom/extras/modules/entity/EntityTeleport.java +++ b/src/main/java/pw/kaboom/extras/modules/entity/EntityTeleport.java @@ -5,9 +5,9 @@ import org.bukkit.event.Listener; import org.bukkit.event.entity.EntityTeleportEvent; -public class EntityTeleport implements Listener { +public final class EntityTeleport implements Listener { @EventHandler - void onEntityTeleport(EntityTeleportEvent event) { + void onEntityTeleport(final EntityTeleportEvent event) { event.setTo(EntitySpawn.limitLocation(event.getTo())); } } diff --git a/src/main/java/pw/kaboom/extras/modules/player/PlayerChat.java b/src/main/java/pw/kaboom/extras/modules/player/PlayerChat.java index 451ccdb..93c62ed 100644 --- a/src/main/java/pw/kaboom/extras/modules/player/PlayerChat.java +++ b/src/main/java/pw/kaboom/extras/modules/player/PlayerChat.java @@ -19,26 +19,26 @@ import org.bukkit.plugin.java.JavaPlugin; import pw.kaboom.extras.Main; -public class PlayerChat implements Listener { +public final class PlayerChat implements Listener { @EventHandler - void onAsyncPlayerChat(AsyncPlayerChatEvent event) { + void onAsyncPlayerChat(final AsyncPlayerChatEvent event) { final Player player = event.getPlayer(); final UUID playerUuid = event.getPlayer().getUniqueId(); - + if (PlayerCommand.commandMillisList.get(playerUuid) != null) { final long millisDifference = System.currentTimeMillis() - PlayerCommand.commandMillisList.get(playerUuid); - + if (millisDifference < 5) { event.setCancelled(true); } } - + PlayerCommand.commandMillisList.put(playerUuid, System.currentTimeMillis()); - + if (event.isCancelled()) { return; } - + final File configFile = new File(JavaPlugin.getPlugin(Main.class).getDataFolder(), "prefixes.yml"); final FileConfiguration prefixConfig = YamlConfiguration.loadConfiguration(configFile); final String prefix; diff --git a/src/main/java/pw/kaboom/extras/modules/player/PlayerCommand.java b/src/main/java/pw/kaboom/extras/modules/player/PlayerCommand.java index b048456..16feb14 100644 --- a/src/main/java/pw/kaboom/extras/modules/player/PlayerCommand.java +++ b/src/main/java/pw/kaboom/extras/modules/player/PlayerCommand.java @@ -10,11 +10,11 @@ import org.bukkit.event.player.PlayerCommandPreprocessEvent; import pw.kaboom.extras.modules.server.ServerCommand; -public class PlayerCommand implements Listener { +public final class PlayerCommand implements Listener { static HashMap commandMillisList = new HashMap(); @EventHandler - void onPlayerCommandPreprocess(PlayerCommandPreprocessEvent event) { + void onPlayerCommandPreprocess(final PlayerCommandPreprocessEvent event) { final UUID playerUuid = event.getPlayer().getUniqueId(); if (commandMillisList.get(playerUuid) != null) { diff --git a/src/main/java/pw/kaboom/extras/modules/player/PlayerConnection.java b/src/main/java/pw/kaboom/extras/modules/player/PlayerConnection.java index 8881e3a..1cd31bd 100644 --- a/src/main/java/pw/kaboom/extras/modules/player/PlayerConnection.java +++ b/src/main/java/pw/kaboom/extras/modules/player/PlayerConnection.java @@ -1,7 +1,5 @@ package pw.kaboom.extras.modules.player; -import java.io.IOException; - import org.bukkit.Chunk; import org.bukkit.World; import org.bukkit.block.BlockState; @@ -19,17 +17,16 @@ import org.bukkit.plugin.java.JavaPlugin; import pw.kaboom.extras.Main; import pw.kaboom.extras.helpers.SkinDownloader; -public class PlayerConnection implements Listener { +public final class PlayerConnection implements Listener { @EventHandler - void onAsyncPlayerPreLogin(AsyncPlayerPreLoginEvent event) throws IOException { + void onAsyncPlayerPreLogin(final AsyncPlayerPreLoginEvent event) { if (event.getName().length() > 16) { event.disallow(AsyncPlayerPreLoginEvent.Result.KICK_OTHER, "Your username can't be longer than 16 characters"); - return; } } @EventHandler - void onPlayerJoin(PlayerJoinEvent event) { + void onPlayerJoin(final PlayerJoinEvent event) { final Player player = event.getPlayer(); final String title = JavaPlugin.getPlugin(Main.class).getConfig().getString("playerJoinTitle"); final String subtitle = JavaPlugin.getPlugin(Main.class).getConfig().getString("playerJoinSubtitle"); @@ -37,8 +34,8 @@ public class PlayerConnection implements Listener { final int stay = 160; final int fadeOut = 5; - if (title != null || - subtitle != null) { + if (title != null + || subtitle != null) { player.sendTitle( title, subtitle, @@ -50,16 +47,16 @@ public class PlayerConnection implements Listener { } @EventHandler - void onPlayerKick(PlayerKickEvent event) { + void onPlayerKick(final PlayerKickEvent event) { if (!JavaPlugin.getPlugin(Main.class).getConfig().getBoolean("enableKick")) { event.setCancelled(true); } } @EventHandler - void onPlayerLogin(PlayerLoginEvent event) { - if (event.getHostname().startsWith("play.flame.ga") && - event.getHostname().endsWith(":25565")) { + void onPlayerLogin(final PlayerLoginEvent event) { + if (event.getHostname().startsWith("play.flame.ga") + && event.getHostname().endsWith(":25565")) { event.disallow(Result.KICK_OTHER, "You connected to the server using an outdated server address/IP.\nPlease use the following address/IP:\n\nkaboom.pw"); return; } @@ -70,8 +67,8 @@ public class PlayerConnection implements Listener { event.allow(); } - if (event.getResult() == Result.KICK_FULL && - JavaPlugin.getPlugin(Main.class).getConfig().getBoolean("allowJoinOnFullServer")) { + if (event.getResult() == Result.KICK_FULL + && JavaPlugin.getPlugin(Main.class).getConfig().getBoolean("allowJoinOnFullServer")) { event.allow(); } @@ -92,7 +89,7 @@ public class PlayerConnection implements Listener { void onPlayerQuit(PlayerQuitEvent event) { PlayerCommand.commandMillisList.remove(event.getPlayer().getUniqueId()); PlayerInteract.interactMillisList.remove(event.getPlayer().getUniqueId()); - Main.skinInProgress.remove(event.getPlayer().getUniqueId()); + SkinDownloader.skinInProgress.remove(event.getPlayer().getUniqueId()); final World world = event.getPlayer().getWorld(); diff --git a/src/main/java/pw/kaboom/extras/modules/player/PlayerDamage.java b/src/main/java/pw/kaboom/extras/modules/player/PlayerDamage.java index f01e509..2fcf5ab 100644 --- a/src/main/java/pw/kaboom/extras/modules/player/PlayerDamage.java +++ b/src/main/java/pw/kaboom/extras/modules/player/PlayerDamage.java @@ -23,19 +23,19 @@ import org.bukkit.event.entity.PlayerDeathEvent; import org.bukkit.inventory.ItemStack; -public class PlayerDamage implements Listener { +public final class PlayerDamage implements Listener { @EventHandler - void onEntityDamage(EntityDamageEvent event) { + void onEntityDamage(final EntityDamageEvent event) { if (event.getEntityType() == EntityType.PLAYER) { - if (event.getCause() == DamageCause.VOID && - event.getDamage() == Float.MAX_VALUE) { + if (event.getCause() == DamageCause.VOID + && event.getDamage() == Float.MAX_VALUE) { event.setCancelled(true); } } } @EventHandler - void onPlayerDeath(PlayerDeathEvent event) { + void onPlayerDeath(final PlayerDeathEvent event) { final Player player = event.getEntity(); final AttributeInstance maxHealth = player.getAttribute(Attribute.GENERIC_MAX_HEALTH); @@ -45,7 +45,7 @@ public class PlayerDamage implements Listener { if (!event.getKeepInventory()) { player.getInventory().clear(); - + for (ItemStack item : event.getDrops()) { player.getWorld().dropItemNaturally(player.getLocation(), item); } @@ -65,13 +65,13 @@ public class PlayerDamage implements Listener { } else { final World world = Bukkit.getWorld("world"); final Location spawnLocation = world.getSpawnLocation(); - + for (double y = spawnLocation.getY(); y <= 256; y++) { final Location yLocation = new Location(world, spawnLocation.getX(), y, spawnLocation.getZ()); final Block coordBlock = world.getBlockAt(yLocation); - - if (!coordBlock.getType().isSolid() && - !coordBlock.getRelative(BlockFace.UP).getType().isSolid()) { + + if (!coordBlock.getType().isSolid() + && !coordBlock.getRelative(BlockFace.UP).getType().isSolid()) { player.teleportAsync(yLocation); break; } diff --git a/src/main/java/pw/kaboom/extras/modules/player/PlayerInteract.java b/src/main/java/pw/kaboom/extras/modules/player/PlayerInteract.java index 06ad988..2052357 100644 --- a/src/main/java/pw/kaboom/extras/modules/player/PlayerInteract.java +++ b/src/main/java/pw/kaboom/extras/modules/player/PlayerInteract.java @@ -7,11 +7,11 @@ import org.bukkit.event.EventHandler; import org.bukkit.event.Listener; import org.bukkit.event.player.PlayerInteractEvent; -public class PlayerInteract implements Listener { +public final class PlayerInteract implements Listener { static HashMap interactMillisList = new HashMap(); @EventHandler - void onPlayerInteract(PlayerInteractEvent event) { + void onPlayerInteract(final PlayerInteractEvent event) { final UUID playerUuid = event.getPlayer().getUniqueId(); if (interactMillisList.get(playerUuid) != null) { diff --git a/src/main/java/pw/kaboom/extras/modules/player/PlayerTeleport.java b/src/main/java/pw/kaboom/extras/modules/player/PlayerTeleport.java index 6b18f3f..cb5ae2e 100644 --- a/src/main/java/pw/kaboom/extras/modules/player/PlayerTeleport.java +++ b/src/main/java/pw/kaboom/extras/modules/player/PlayerTeleport.java @@ -7,9 +7,9 @@ import org.bukkit.event.player.PlayerTeleportEvent; import pw.kaboom.extras.modules.entity.EntitySpawn; -public class PlayerTeleport implements Listener { +public final class PlayerTeleport implements Listener { @EventHandler - void onPlayerTeleport(PlayerTeleportEvent event) { + void onPlayerTeleport(final PlayerTeleportEvent event) { event.setTo(EntitySpawn.limitLocation(event.getTo())); } } diff --git a/src/main/java/pw/kaboom/extras/modules/server/ServerCommand.java b/src/main/java/pw/kaboom/extras/modules/server/ServerCommand.java index 48179bd..4fd6170 100644 --- a/src/main/java/pw/kaboom/extras/modules/server/ServerCommand.java +++ b/src/main/java/pw/kaboom/extras/modules/server/ServerCommand.java @@ -10,10 +10,10 @@ import org.bukkit.event.EventHandler; import org.bukkit.event.Listener; import org.bukkit.event.server.ServerCommandEvent; -public class ServerCommand implements Listener { +public final class ServerCommand implements Listener { public static HashSet consoleCommandBlacklist = new HashSet(); - public static String checkCommand(CommandSender sender, String command, boolean isConsoleCommand) { + public static String checkCommand(final CommandSender sender, final String command, final boolean isConsoleCommand) { final String[] arr = command.split(" "); String commandName = arr[0].toLowerCase(); @@ -30,26 +30,26 @@ public class ServerCommand implements Listener { for (int i = 1; i < arr.length; i++) { if ("run".equalsIgnoreCase(arr[i])) { - if (i+1 < arr.length) { - if ("execute".equalsIgnoreCase(arr[i+1]) || - "particle".equalsIgnoreCase(arr[i+1]) || - "save-off".equalsIgnoreCase(arr[i+1]) || - "spreadplayers".equalsIgnoreCase(arr[i+1]) || - "stop".equalsIgnoreCase(arr[i+1])) { + if (i + 1 < arr.length) { + if ("execute".equalsIgnoreCase(arr[i + 1]) + || "particle".equalsIgnoreCase(arr[i + 1]) + || "save-off".equalsIgnoreCase(arr[i + 1]) + || "spreadplayers".equalsIgnoreCase(arr[i + 1]) + || "stop".equalsIgnoreCase(arr[i + 1])) { Command.broadcastCommandMessage(sender, "Forbidden execute command detected"); return "cancel"; - } else if (i+3 < arr.length && - "gamerule".equalsIgnoreCase(arr[i+1])) { - if ("randomTickSpeed".equalsIgnoreCase(arr[i+2]) && - Double.parseDouble(arr[i+3]) > 6) { - return command.replaceFirst("(?i)" + "randomTickSpeed " + arr[i+3], "randomTickSpeed 6"); - } else if ("spawnRadius".equalsIgnoreCase(arr[i+2]) && - Double.parseDouble(arr[i+3]) > 100) { - return command.replaceFirst("(?i)" + "spawnRadius " + arr[i+3], "spawnRadius 100"); + } else if (i + 3 < arr.length + && "gamerule".equalsIgnoreCase(arr[i + 1])) { + if ("randomTickSpeed".equalsIgnoreCase(arr[i + 2]) + && Double.parseDouble(arr[i + 3]) > 6) { + return command.replaceFirst("(?i)" + "randomTickSpeed " + arr[i + 3], "randomTickSpeed 6"); + } else if ("spawnRadius".equalsIgnoreCase(arr[i + 2]) + && Double.parseDouble(arr[i + 3]) > 100) { + return command.replaceFirst("(?i)" + "spawnRadius " + arr[i + 3], "spawnRadius 100"); } - } else if ("give".equalsIgnoreCase(arr[i+1])) { - if (Double.parseDouble(arr[arr.length-1]) > 64) { - arr[arr.length-1] = "64"; + } else if ("give".equalsIgnoreCase(arr[i + 1])) { + if (Double.parseDouble(arr[arr.length - 1]) > 64) { + arr[arr.length - 1] = "64"; return String.join(" ", arr); } } @@ -57,8 +57,8 @@ public class ServerCommand implements Listener { break; } - if ("as".equalsIgnoreCase(arr[i]) || - "at".equalsIgnoreCase(arr[i])) { + if ("as".equalsIgnoreCase(arr[i]) + || "at".equalsIgnoreCase(arr[i])) { asAtCount++; } } @@ -72,26 +72,26 @@ public class ServerCommand implements Listener { case "/minecraft:gamerule": case "/gamerule": if (arr.length >= 3) { - if ("randomTickSpeed".equalsIgnoreCase(arr[1]) && - Double.parseDouble(arr[2]) > 6) { + if ("randomTickSpeed".equalsIgnoreCase(arr[1]) + && Double.parseDouble(arr[2]) > 6) { return command.replaceFirst(arr[2], "6"); - } else if ("spawnRadius".equalsIgnoreCase(arr[1]) && - Double.parseDouble(arr[2]) > 100) { + } else if ("spawnRadius".equalsIgnoreCase(arr[1]) + && Double.parseDouble(arr[2]) > 100) { return command.replaceFirst(arr[2], "100"); } } break; case "/minecraft:give": case "/give": - if (Double.parseDouble(arr[arr.length-1]) > 64) { - arr[arr.length-1] = "64"; + if (Double.parseDouble(arr[arr.length - 1]) > 64) { + arr[arr.length - 1] = "64"; return String.join(" ", arr); } break; case "/minecraft:particle": case "/particle": - if (arr.length >= 10 && - Double.parseDouble(arr[9]) > 10) { + if (arr.length >= 10 + && Double.parseDouble(arr[9]) > 10) { arr[9] = "10"; return String.join(" ", arr); } @@ -100,9 +100,9 @@ public class ServerCommand implements Listener { case "/bukkit:rl": case "/reload": case "/rl": - if (sender.hasPermission("bukkit.command.reload") && - arr.length >= 2 && - "confirm".equalsIgnoreCase(arr[1])) { + if (sender.hasPermission("bukkit.command.reload") + && arr.length >= 2 + && "confirm".equalsIgnoreCase(arr[1])) { Command.broadcastCommandMessage(sender, ChatColor.RED + "Please note that this command is not supported and may cause issues when using some plugins."); Command.broadcastCommandMessage(sender, ChatColor.RED + "If you encounter any issues please use the /stop command to restart your server."); Command.broadcastCommandMessage(sender, ChatColor.GREEN + "Reload complete."); @@ -125,12 +125,15 @@ public class ServerCommand implements Listener { case "/minecraft:spreadplayers": case "/spreadplayers": if (arr.length >= 5) { - if (Double.parseDouble(arr[3]) > 0) + if (Double.parseDouble(arr[3]) > 0) { arr[3] = "0"; - if (Double.parseDouble(arr[4]) < 8) + } + if (Double.parseDouble(arr[4]) < 8) { arr[4] = "8"; - if (Double.parseDouble(arr[4]) > 50) + } + if (Double.parseDouble(arr[4]) > 50) { arr[4] = "50"; + } return String.join(" ", arr); } @@ -141,6 +144,8 @@ public class ServerCommand implements Listener { Command.broadcastCommandMessage(sender, "Stopping the server"); return "cancel"; } + default: + break; } } catch (NumberFormatException exception) { // Do nothing @@ -149,7 +154,7 @@ public class ServerCommand implements Listener { } @EventHandler - void onServerCommand(ServerCommandEvent event) { + void onServerCommand(final ServerCommandEvent event) { final CommandSender sender = event.getSender(); final String[] arr = event.getCommand().split(" "); diff --git a/src/main/java/pw/kaboom/extras/modules/server/ServerPing.java b/src/main/java/pw/kaboom/extras/modules/server/ServerPing.java index c31534d..dd129e5 100644 --- a/src/main/java/pw/kaboom/extras/modules/server/ServerPing.java +++ b/src/main/java/pw/kaboom/extras/modules/server/ServerPing.java @@ -5,11 +5,12 @@ import org.bukkit.event.Listener; import com.destroystokyo.paper.event.server.PaperServerListPingEvent; -public class ServerPing implements Listener { +public final class ServerPing implements Listener { @EventHandler - void onServerListPing(PaperServerListPingEvent event) { + void onServerListPing(final PaperServerListPingEvent event) { if (event.getClient().getProtocolVersion() == -1) { - event.setProtocolVersion(573); + final int protocol = 573; + event.setProtocolVersion(protocol); } else { event.setProtocolVersion(event.getClient().getProtocolVersion()); }