From 75fbd44af7fec95bd0344cf23a5410f298a8084c Mon Sep 17 00:00:00 2001 From: EpicKnarvik97 Date: Thu, 2 Sep 2021 00:31:03 +0200 Subject: [PATCH 1/2] Removes an s in class name of PlayerEventsListener --- src/main/java/net/knarcraft/stargate/Stargate.java | 4 ++-- ...{PlayerEventsListener.java => PlayerEventListener.java} | 7 ++----- 2 files changed, 4 insertions(+), 7 deletions(-) rename src/main/java/net/knarcraft/stargate/listener/{PlayerEventsListener.java => PlayerEventListener.java} (98%) diff --git a/src/main/java/net/knarcraft/stargate/Stargate.java b/src/main/java/net/knarcraft/stargate/Stargate.java index 5037c61..ded5f53 100644 --- a/src/main/java/net/knarcraft/stargate/Stargate.java +++ b/src/main/java/net/knarcraft/stargate/Stargate.java @@ -6,7 +6,7 @@ import net.knarcraft.stargate.event.StargateAccessEvent; import net.knarcraft.stargate.listener.BlockEventListener; import net.knarcraft.stargate.listener.BungeeCordListener; import net.knarcraft.stargate.listener.EntityEventListener; -import net.knarcraft.stargate.listener.PlayerEventsListener; +import net.knarcraft.stargate.listener.PlayerEventListener; import net.knarcraft.stargate.listener.PluginEventListener; import net.knarcraft.stargate.listener.VehicleEventListener; import net.knarcraft.stargate.listener.WorldEventListener; @@ -535,7 +535,7 @@ public class Stargate extends JavaPlugin { log.info(pluginDescriptionFile.getName() + " v." + pluginDescriptionFile.getVersion() + " is enabled."); // Register events before loading gates to stop weird things happening. - pm.registerEvents(new PlayerEventsListener(), this); + pm.registerEvents(new PlayerEventListener(), this); pm.registerEvents(new BlockEventListener(), this); pm.registerEvents(new VehicleEventListener(), this); diff --git a/src/main/java/net/knarcraft/stargate/listener/PlayerEventsListener.java b/src/main/java/net/knarcraft/stargate/listener/PlayerEventListener.java similarity index 98% rename from src/main/java/net/knarcraft/stargate/listener/PlayerEventsListener.java rename to src/main/java/net/knarcraft/stargate/listener/PlayerEventListener.java index 108470a..298464a 100644 --- a/src/main/java/net/knarcraft/stargate/listener/PlayerEventsListener.java +++ b/src/main/java/net/knarcraft/stargate/listener/PlayerEventListener.java @@ -31,7 +31,7 @@ import java.util.Objects; * This listener listens to any player-related events related to stargates */ @SuppressWarnings("unused") -public class PlayerEventsListener implements Listener { +public class PlayerEventListener implements Listener { private static long eventTime; private static PlayerInteractEvent previousEvent; @@ -208,10 +208,7 @@ public class PlayerEventsListener implements Listener { * @return

True if the player should be denied

*/ private boolean cannotAccessPortal(Player player, Portal portal) { - boolean deny = false; - if (!Stargate.canAccessNetwork(player, portal.getNetwork())) { - deny = true; - } + boolean deny = !Stargate.canAccessNetwork(player, portal.getNetwork()); if (!Stargate.canAccessPortal(player, portal, deny)) { Stargate.sendMessage(player, Stargate.getString("denyMsg")); From daa3c6f8680fc9f6dad39b1dbe797c25a7e215cb Mon Sep 17 00:00:00 2001 From: EpicKnarvik97 Date: Thu, 9 Sep 2021 15:25:08 +0200 Subject: [PATCH 2/2] Cleans up a bit and changes to compile for java 16 and spigot 1.17 --- pom.xml | 16 +++++++++++----- .../knarcraft/stargate/RelativeBlockVector.java | 6 +++--- .../java/net/knarcraft/stargate/Stargate.java | 10 +++++----- .../java/net/knarcraft/stargate/TwoTuple.java | 4 ++-- .../stargate/command/CommandReload.java | 2 +- .../stargate/command/CommandStarGate.java | 2 +- .../stargate/listener/BungeeCordListener.java | 2 +- .../java/net/knarcraft/stargate/portal/Gate.java | 10 +++++----- .../knarcraft/stargate/portal/GateHandler.java | 8 ++++---- .../knarcraft/stargate/portal/GateLayout.java | 2 +- .../net/knarcraft/stargate/portal/Portal.java | 4 ++-- .../knarcraft/stargate/portal/PortalHandler.java | 2 +- .../stargate/portal/GateLayoutTest.java | 3 +-- 13 files changed, 38 insertions(+), 33 deletions(-) diff --git a/pom.xml b/pom.xml index f7d86c3..c4063b2 100644 --- a/pom.xml +++ b/pom.xml @@ -14,8 +14,8 @@ UTF-8 - 1.8 - 1.8 + 16 + 16 @@ -33,7 +33,7 @@ org.spigotmc spigot-api - 1.16.5-R0.1-SNAPSHOT + 1.17.1-R0.1-SNAPSHOT net.milkbowl.vault @@ -48,8 +48,8 @@ com.github.seeseemelk - MockBukkit-v1.16 - 0.24.0 + MockBukkit-v1.17 + 1.7.0 test @@ -58,6 +58,12 @@ 19.0.0 compile + + junit + junit + 4.13.1 + test + diff --git a/src/main/java/net/knarcraft/stargate/RelativeBlockVector.java b/src/main/java/net/knarcraft/stargate/RelativeBlockVector.java index 8bf2624..15c17ab 100644 --- a/src/main/java/net/knarcraft/stargate/RelativeBlockVector.java +++ b/src/main/java/net/knarcraft/stargate/RelativeBlockVector.java @@ -10,9 +10,9 @@ package net.knarcraft.stargate; */ public class RelativeBlockVector { - private int right; - private int depth; - private int distance; + private final int right; + private final int depth; + private final int distance; /** * Instantiates a new relative block vector diff --git a/src/main/java/net/knarcraft/stargate/Stargate.java b/src/main/java/net/knarcraft/stargate/Stargate.java index ded5f53..23cfe82 100644 --- a/src/main/java/net/knarcraft/stargate/Stargate.java +++ b/src/main/java/net/knarcraft/stargate/Stargate.java @@ -67,13 +67,13 @@ public class Stargate extends JavaPlugin { // Used for debug public static boolean debug = false; public static boolean permDebug = false; - public static ConcurrentLinkedQueue activeList = new ConcurrentLinkedQueue<>(); + public static final ConcurrentLinkedQueue activeList = new ConcurrentLinkedQueue<>(); // Used for populating gate open/closed material. - public static Queue blockPopulatorQueue = new LinkedList<>(); + public static final Queue blockPopulatorQueue = new LinkedList<>(); // HashMap of player names for Bungee support - public static Map bungeeQueue = new HashMap<>(); + public static final Map bungeeQueue = new HashMap<>(); // World names that contain stargates - public static HashSet managedWorlds = new HashSet<>(); + public static final HashSet managedWorlds = new HashSet<>(); private static String pluginVersion; private static String portalFolder; private static String gateFolder; @@ -656,7 +656,7 @@ public class Stargate extends JavaPlugin { File newFile = new File(portalFolder, getServer().getWorlds().get(0).getName() + ".db"); if (!newFile.exists()) { if (!newFile.getParentFile().mkdirs()) { - log.severe("Unable to create portal directory"); + log.severe("Unable to create portal database folder: " + newFile.getParentFile().getPath()); } } } diff --git a/src/main/java/net/knarcraft/stargate/TwoTuple.java b/src/main/java/net/knarcraft/stargate/TwoTuple.java index 6167fac..438590d 100644 --- a/src/main/java/net/knarcraft/stargate/TwoTuple.java +++ b/src/main/java/net/knarcraft/stargate/TwoTuple.java @@ -8,8 +8,8 @@ package net.knarcraft.stargate; */ public class TwoTuple { - private K firstValue; - private L secondValue; + private final K firstValue; + private final L secondValue; /** * Instantiate a new TwoTuple diff --git a/src/main/java/net/knarcraft/stargate/command/CommandReload.java b/src/main/java/net/knarcraft/stargate/command/CommandReload.java index 3147062..2d4ea4d 100644 --- a/src/main/java/net/knarcraft/stargate/command/CommandReload.java +++ b/src/main/java/net/knarcraft/stargate/command/CommandReload.java @@ -12,7 +12,7 @@ import org.jetbrains.annotations.NotNull; */ public class CommandReload implements CommandExecutor { - private Stargate plugin; + private final Stargate plugin; /** * Instantiates the reload command diff --git a/src/main/java/net/knarcraft/stargate/command/CommandStarGate.java b/src/main/java/net/knarcraft/stargate/command/CommandStarGate.java index bcc86ab..182e4c8 100644 --- a/src/main/java/net/knarcraft/stargate/command/CommandStarGate.java +++ b/src/main/java/net/knarcraft/stargate/command/CommandStarGate.java @@ -15,7 +15,7 @@ import org.jetbrains.annotations.NotNull; */ public class CommandStarGate implements CommandExecutor { - private Stargate plugin; + private final Stargate plugin; /** * Instantiates the stargate command diff --git a/src/main/java/net/knarcraft/stargate/listener/BungeeCordListener.java b/src/main/java/net/knarcraft/stargate/listener/BungeeCordListener.java index 700fda4..0d58dda 100644 --- a/src/main/java/net/knarcraft/stargate/listener/BungeeCordListener.java +++ b/src/main/java/net/knarcraft/stargate/listener/BungeeCordListener.java @@ -23,7 +23,7 @@ public class BungeeCordListener implements PluginMessageListener { * @param message

The message received from the plugin

*/ @Override - public void onPluginMessageReceived(@NotNull String channel, @NotNull Player unused, @NotNull byte[] message) { + public void onPluginMessageReceived(@NotNull String channel, @NotNull Player unused, byte[] message) { //Ignore plugin messages if bungee support is not enabled or some other plugin message is received if (!Stargate.enableBungee || !channel.equals("BungeeCord")) { return; diff --git a/src/main/java/net/knarcraft/stargate/portal/Gate.java b/src/main/java/net/knarcraft/stargate/portal/Gate.java index cd1afb4..3f43583 100644 --- a/src/main/java/net/knarcraft/stargate/portal/Gate.java +++ b/src/main/java/net/knarcraft/stargate/portal/Gate.java @@ -28,13 +28,13 @@ public class Gate { //Gate materials private Material portalOpenBlock; private Material portalClosedBlock; - private Material portalButton; + private final Material portalButton; // Economy information - private int useCost; - private int createCost; - private int destroyCost; - private boolean toOwner; + private final int useCost; + private final int createCost; + private final int destroyCost; + private final boolean toOwner; /** * Instantiates a new gate diff --git a/src/main/java/net/knarcraft/stargate/portal/GateHandler.java b/src/main/java/net/knarcraft/stargate/portal/GateHandler.java index 5e62635..fb8867c 100644 --- a/src/main/java/net/knarcraft/stargate/portal/GateHandler.java +++ b/src/main/java/net/knarcraft/stargate/portal/GateHandler.java @@ -25,9 +25,9 @@ public class GateHandler { private static final Character EXIT = '*'; private static final Character CONTROL_BLOCK = '-'; - private static Material defaultPortalBlockOpen = Material.NETHER_PORTAL; - private static Material defaultPortalBlockClosed = Material.AIR; - private static Material defaultButton = Material.STONE_BUTTON; + private static final Material defaultPortalBlockOpen = Material.NETHER_PORTAL; + private static final Material defaultPortalBlockClosed = Material.AIR; + private static final Material defaultButton = Material.STONE_BUTTON; private static final HashMap gates = new HashMap<>(); private static final HashMap> controlBlocks = new HashMap<>(); @@ -162,7 +162,7 @@ public class GateHandler { int useCost = readConfig(config, fileName, "usecost", -1); int createCost = readConfig(config, fileName, "createcost", -1); int destroyCost = readConfig(config, fileName, "destroycost", -1); - boolean toOwner = (config.containsKey("toowner") ? Boolean.valueOf(config.get("toowner")) : EconomyHandler.toOwner); + boolean toOwner = (config.containsKey("toowner") ? Boolean.parseBoolean(config.get("toowner")) : EconomyHandler.toOwner); Gate gate = new Gate(fileName, new GateLayout(layout), types, portalOpenBlock, portalClosedBlock, portalButton, useCost, createCost, destroyCost, toOwner); diff --git a/src/main/java/net/knarcraft/stargate/portal/GateLayout.java b/src/main/java/net/knarcraft/stargate/portal/GateLayout.java index dfb5ab2..c7b1e81 100644 --- a/src/main/java/net/knarcraft/stargate/portal/GateLayout.java +++ b/src/main/java/net/knarcraft/stargate/portal/GateLayout.java @@ -16,7 +16,7 @@ import java.util.List; */ public class GateLayout { - private Character [][] layout; + private final Character [][] layout; private final List exits = new ArrayList<>(); private RelativeBlockVector[] entrances = new RelativeBlockVector[0]; private RelativeBlockVector[] border = new RelativeBlockVector[0]; diff --git a/src/main/java/net/knarcraft/stargate/portal/Portal.java b/src/main/java/net/knarcraft/stargate/portal/Portal.java index f252002..c386d1b 100644 --- a/src/main/java/net/knarcraft/stargate/portal/Portal.java +++ b/src/main/java/net/knarcraft/stargate/portal/Portal.java @@ -59,11 +59,11 @@ public class Portal { private String destination; private String lastDestination = ""; private String network; - private String ownerName; + private final String ownerName; private UUID ownerUUID; private boolean verified; private boolean fixed; - private Map options; + private final Map options; // In-use information private Player player; diff --git a/src/main/java/net/knarcraft/stargate/portal/PortalHandler.java b/src/main/java/net/knarcraft/stargate/portal/PortalHandler.java index d3b420e..cb61cbd 100644 --- a/src/main/java/net/knarcraft/stargate/portal/PortalHandler.java +++ b/src/main/java/net/knarcraft/stargate/portal/PortalHandler.java @@ -931,7 +931,7 @@ public class PortalHandler { } } PortalHandler.unregisterPortal(portal, false); - Stargate.log.info(Stargate.getString("prefix") + "Destroying stargate at " + portal.toString()); + Stargate.log.info(Stargate.getString("prefix") + "Destroying stargate at " + portal); } /** diff --git a/src/test/java/net/knarcraft/stargate/portal/GateLayoutTest.java b/src/test/java/net/knarcraft/stargate/portal/GateLayoutTest.java index 537df09..94afc86 100644 --- a/src/test/java/net/knarcraft/stargate/portal/GateLayoutTest.java +++ b/src/test/java/net/knarcraft/stargate/portal/GateLayoutTest.java @@ -15,7 +15,6 @@ import be.seeseemelk.mockbukkit.MockBukkit; import java.util.ArrayList; import java.util.List; -import java.util.Set; public class GateLayoutTest { @@ -40,7 +39,7 @@ public class GateLayoutTest { expected.add(new RelativeBlockVector(1, 3, 0)); expected.add(new RelativeBlockVector(2, 3, 0)); - Set exits = layout.getExits().keySet(); + List exits = layout.getExits(); exits.forEach((blockVector) -> assertTrue(expected.contains(blockVector))); }