From 934b24661551e7f43652602216cf08f0cb457460 Mon Sep 17 00:00:00 2001 From: RonanCraft Date: Mon, 14 Mar 2022 14:51:16 -0400 Subject: [PATCH] short hand coding changes + kingdoms updated to 1.12.3 --- BetterRTPAddons/pom.xml | 2 +- pom.xml | 4 +- .../SuperRonanCraft/BetterRTP/BetterRTP.java | 2 +- .../BetterRTP/player/commands/Commands.java | 2 +- .../BetterRTP/player/events/Join.java | 4 +- .../BetterRTP/player/rtp/RTP.java | 50 ++++++------- .../BetterRTP/player/rtp/RTPLoader.java | 8 +- .../player/rtp/RTPPluginValidation.java | 4 +- .../player/rtp/RTPSetupInformation.java | 19 +++-- .../BetterRTP/player/rtp/RTPTeleport.java | 6 +- .../references/helpers/HelperRTP.java | 6 +- .../references/settings/Settings.java | 17 +++-- .../references/settings/SoftDepends.java | 75 ++++--------------- 13 files changed, 78 insertions(+), 121 deletions(-) diff --git a/BetterRTPAddons/pom.xml b/BetterRTPAddons/pom.xml index c996ac8..0703b3b 100644 --- a/BetterRTPAddons/pom.xml +++ b/BetterRTPAddons/pom.xml @@ -109,7 +109,7 @@ me.SuperRonanCraft BetterRTP - 3.2.3 + 3.2.4 provided diff --git a/pom.xml b/pom.xml index ae7ee21..74b6421 100644 --- a/pom.xml +++ b/pom.xml @@ -7,7 +7,7 @@ me.SuperRonanCraft BetterRTP jar - 3.2.3 + 3.2.4 1.8 @@ -258,7 +258,7 @@ com.github.cryptomorin kingdoms - 1.11.15 + 1.12.3 provided diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/BetterRTP.java b/src/main/java/me/SuperRonanCraft/BetterRTP/BetterRTP.java index 4438d21..b4d3754 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/BetterRTP.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/BetterRTP.java @@ -127,7 +127,7 @@ public class BetterRTP extends JavaPlugin { } public static void debug(String str) { - if (getInstance().getSettings().debug) + if (getInstance().getSettings().isDebug()) getInstance().getLogger().info(str); } } diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/player/commands/Commands.java b/src/main/java/me/SuperRonanCraft/BetterRTP/player/commands/Commands.java index 74995ef..39a7c4d 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/player/commands/Commands.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/player/commands/Commands.java @@ -35,7 +35,7 @@ public class Commands { } public void registerCommand(RTPCommand cmd, boolean forced) { - if (!cmd.isDebugOnly() || pl.getSettings().debug || forced) //If debug only, can it be enabled? + if (!cmd.isDebugOnly() || pl.getSettings().isDebug() || forced) //If debug only, can it be enabled? commands.add(cmd); } diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/player/events/Join.java b/src/main/java/me/SuperRonanCraft/BetterRTP/player/events/Join.java index 54b271f..53bf5cf 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/player/events/Join.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/player/events/Join.java @@ -30,8 +30,8 @@ public class Join { //RTP on first join private void rtpOnFirstJoin(Player p) { - if (getPl().getSettings().rtpOnFirstJoin_Enabled && !p.hasPlayedBefore()) - HelperRTP.tp(p, Bukkit.getConsoleSender(), getPl().getSettings().rtpOnFirstJoin_World, null, RTP_TYPE.JOIN); //Console is sender to override delays + if (getPl().getSettings().isRtpOnFirstJoin_Enabled() && !p.hasPlayedBefore()) + HelperRTP.tp(p, Bukkit.getConsoleSender(), getPl().getSettings().getRtpOnFirstJoin_World(), null, RTP_TYPE.JOIN); //Console is sender to override delays } private BetterRTP getPl() { diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTP.java b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTP.java index 68988e3..dcef87f 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTP.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTP.java @@ -56,54 +56,54 @@ public class RTP { } public WorldPlayer getPlayerWorld(RTPSetupInformation setup_info) { - WorldPlayer pWorld = new WorldPlayer(setup_info.player, Bukkit.getWorld(setup_info.world)); + WorldPlayer pWorld = new WorldPlayer(setup_info.getPlayer(), Bukkit.getWorld(setup_info.getWorld())); //Locations - if (setup_info.location == null) { + if (setup_info.getLocation() == null) { List locationsList = new ArrayList<>(); for (Map.Entry location_set : worldLocations.entrySet()) { RTPWorld location = location_set.getValue(); - if (location.getWorld().getName().equals(setup_info.world)) + if (location.getWorld().getName().equals(setup_info.getWorld())) locationsList.add(location_set.getValue()); } if (!locationsList.isEmpty()) { RTPWorld location = locationsList.size() > 1 ? locationsList.get((new Random()).nextInt(locationsList.size() - 1)) : locationsList.get(0); - setup_info.location = (WorldLocations) location; + setup_info.setLocation((WorldLocations) location); } } - if (setup_info.location != null) { + if (setup_info.getLocation() != null) { String setup_name = null; for (Map.Entry location_set : worldLocations.entrySet()) { RTPWorld location = location_set.getValue(); - if (location == setup_info.location) { + if (location == setup_info.getLocation()) { setup_name = location_set.getKey(); break; } } - pWorld.setup(setup_name, setup_info.location, setup_info.location.getBiomes(), setup_info.personalized); + pWorld.setup(setup_name, setup_info.getLocation(), setup_info.getLocation().getBiomes(), setup_info.isPersonalized()); } if (!pWorld.isSetup()) { //Custom World - if (customWorlds.containsKey(setup_info.world)) { + if (customWorlds.containsKey(setup_info.getWorld())) { RTPWorld cWorld = customWorlds.get(pWorld.getWorld().getName()); - pWorld.setup(null, cWorld, setup_info.biomes, setup_info.personalized); + pWorld.setup(null, cWorld, setup_info.getBiomes(), setup_info.isPersonalized()); } //Default World else - pWorld.setup(null, defaultWorld, setup_info.biomes, setup_info.personalized); + pWorld.setup(null, defaultWorld, setup_info.getBiomes(), setup_info.isPersonalized()); } //World type WORLD_TYPE world_type; //World rtp type - if (this.world_type.containsKey(setup_info.world)) - world_type = this.world_type.get(setup_info.world); + if (this.world_type.containsKey(setup_info.getWorld())) + world_type = this.world_type.get(setup_info.getWorld()); else { world_type = WORLD_TYPE.NORMAL; - this.world_type.put(setup_info.world, world_type); //Defaults this so the error message isn't spammed - getPl().getLogger().warning("Seems like the world `" + setup_info.world + "` does not have a `WorldType` declared. " + + this.world_type.put(setup_info.getWorld(), world_type); //Defaults this so the error message isn't spammed + getPl().getLogger().warning("Seems like the world `" + setup_info.getWorld() + "` does not have a `WorldType` declared. " + "Please add/fix this in the config.yml file! " + "This world will be treated as an overworld!"); } @@ -112,22 +112,22 @@ public class RTP { } public void start(RTPSetupInformation setup_info) { - RTP_SettingUpEvent setup = new RTP_SettingUpEvent(setup_info.player); + RTP_SettingUpEvent setup = new RTP_SettingUpEvent(setup_info.getPlayer()); Bukkit.getPluginManager().callEvent(setup); if (setup.isCancelled()) { return; } - String world_name = setup_info.world; - Player p = setup_info.player; - CommandSender sendi = setup_info.sender; + String world_name = setup_info.getWorld(); + Player p = setup_info.getPlayer(); + CommandSender sendi = setup_info.getSender(); // Locations - if (setup_info.location != null) { - WorldLocations location = setup_info.location; + if (setup_info.getLocation() != null) { + WorldLocations location = setup_info.getLocation(); world_name = location.getWorld().getName(); - setup_info.world = world_name; - setup_info.biomes = location.getBiomes(); + setup_info.setWorld(world_name); + setup_info.setBiomes(location.getBiomes()); } // Check overrides @@ -146,7 +146,7 @@ public class RTP { } if (overriden.containsKey(world_name)) { world_name = overriden.get(world_name); - setup_info.world = world_name; + setup_info.setWorld(world_name); } // Not forced and has 'betterrtp.world.' if (sendi == p && !getPl().getPerms().getAWorld(sendi, world_name)) { @@ -162,7 +162,7 @@ public class RTP { // Economy if (!getPl().getEco().hasBalance(sendi, pWorld)) return; - rtp(sendi, pWorld, setup_info.delay, setup_info.rtp_type, setup_info.cooldown); + rtp(sendi, pWorld, setup_info.isDelay(), setup_info.getRtp_type(), setup_info.isCooldown()); } private void rtp(CommandSender sendi, WorldPlayer pWorld, boolean delay, RTP_TYPE type, boolean cooldown) { @@ -175,7 +175,7 @@ public class RTP { //Setup player rtp methods RTPPlayer rtpPlayer = new RTPPlayer(p, this, pWorld, type); // Delaying? Else, just go - if (getPl().getSettings().delayEnabled && delay) { + if (getPl().getSettings().isDelayEnabled() && delay) { new RTPDelay(sendi, rtpPlayer, delayTime, cancelOnMove, cancelOnDamage); } else { teleport.beforeTeleportInstant(sendi, p); diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPLoader.java b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPLoader.java index 583f998..df79ea4 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPLoader.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPLoader.java @@ -22,7 +22,7 @@ public class RTPLoader { for (Map m : map) for (Map.Entry entry : m.entrySet()) { customWorlds.put(entry.getKey().toString(), new WorldCustom(entry.getKey().toString())); - if (getPl().getSettings().debug) + if (getPl().getSettings().isDebug()) BetterRTP.debug("- Custom World '" + entry.getKey() + "' registered"); } } catch (Exception e) { @@ -38,7 +38,7 @@ public class RTPLoader { for (Map m : override_map) for (Map.Entry entry : m.entrySet()) { overriden.put(entry.getKey().toString(), entry.getValue().toString()); - if (getPl().getSettings().debug) + if (getPl().getSettings().isDebug()) getPl().getLogger().info("- Override '" + entry.getKey() + "' -> '" + entry.getValue() + "' added"); if (Bukkit.getWorld(entry.getValue().toString()) == null) getPl().getLogger().warning("The world `" + entry.getValue() + "` doesn't seem to exist! Please update `" + entry.getKey() + "'s` override! Maybe there are capital letters?"); @@ -75,7 +75,7 @@ public class RTPLoader { getPl().getLogger().info("- World Type failed for '" + entry.getKey() + "' is it loaded?"); }*/ } - if (getPl().getSettings().debug) + if (getPl().getSettings().isDebug()) for (String world : world_type.keySet()) BetterRTP.debug("- World Type for '" + world + "' set to '" + world_type.get(world) + "'"); } catch (Exception e) { @@ -95,7 +95,7 @@ public class RTPLoader { WorldLocations location = new WorldLocations(entry.getKey().toString()); if (location.isValid()) { worlds.put(entry.getKey().toString(), location); - if (getPl().getSettings().debug) + if (getPl().getSettings().isDebug()) BetterRTP.debug("- Location '" + entry.getKey() + "' registered"); } } diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPPluginValidation.java b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPPluginValidation.java index 4c740de..36c9e34 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPPluginValidation.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPPluginValidation.java @@ -161,7 +161,7 @@ public class RTPPluginValidation { //Safe locations depending on enabled depende // https://www.spigotmc.org/resources/kingdomsx.77670/ private boolean getKingdomsx(Location loc) { boolean result = true; - if (getDepends().isKingdomsX()) + if (getDepends().isKingdomsx()) try { org.kingdoms.constants.land.Land land = org.kingdoms.constants.land.Land.getLand(loc); result = land == null || !land.isClaimed(); @@ -176,7 +176,7 @@ public class RTPPluginValidation { //Safe locations depending on enabled depende // https://www.spigotmc.org/resources/hclaims.90540/ (Local Repo) private boolean gethClaims(Location loc) { boolean result = true; - if (getDepends().ishClaims()) + if (getDepends().isHClaims()) try { result = ClaimAPI.getInstance().isClaimed(loc); } catch (Exception e) { diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPSetupInformation.java b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPSetupInformation.java index d889033..73cb9c3 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPSetupInformation.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPSetupInformation.java @@ -1,5 +1,7 @@ package me.SuperRonanCraft.BetterRTP.player.rtp; +import lombok.Getter; +import lombok.Setter; import me.SuperRonanCraft.BetterRTP.references.rtpinfo.worlds.WorldLocations; import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @@ -8,14 +10,15 @@ import java.util.List; public class RTPSetupInformation { //Will provide information to setup an RTP attempt - public String world; - public CommandSender sender; - public Player player; - public boolean personalized, cooldown; - public List biomes; - public WorldLocations location; - public boolean delay; - public RTP_TYPE rtp_type; + @Getter @Setter private String world; + @Getter private final CommandSender sender; + @Getter private final Player player; + @Getter private final boolean personalized; + @Getter private final boolean cooldown; + @Getter @Setter private List biomes; + @Getter @Setter private WorldLocations location; + @Getter private final boolean delay; + @Getter private final RTP_TYPE rtp_type; public RTPSetupInformation(String world, CommandSender sender, Player player, boolean personalized) { this(world, sender, player, personalized, null, false, null, null); diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPTeleport.java b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPTeleport.java index abc29d1..a21eda7 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPTeleport.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/player/rtp/RTPTeleport.java @@ -67,7 +67,7 @@ public class RTPTeleport { getPl().getpInfo().getRtping().remove(p); //No longer rtp'ing //Save respawn location if first join if (type == RTP_TYPE.JOIN) //RTP Type was Join - if (BetterRTP.getInstance().getSettings().rtpOnFirstJoin_SetAsRespawn) //Save as respawn is enabled + if (BetterRTP.getInstance().getSettings().isRtpOnFirstJoin_SetAsRespawn()) //Save as respawn is enabled p.setBedSpawnLocation(loc, true); //True means to force a respawn even without a valid bed } }); @@ -132,7 +132,7 @@ public class RTPTeleport { private List> getChunks(Location loc) { //List all chunks in range to load List> asyncChunks = new ArrayList<>(); - int range = Math.round(Math.max(0, Math.min(16, getPl().getSettings().preloadRadius))); + int range = Math.round(Math.max(0, Math.min(16, getPl().getSettings().getPreloadRadius()))); for (int x = -range; x <= range; x++) for (int z = -range; z <= range; z++) { Location locLoad = new Location(loc.getWorld(), loc.getX() + (x * 16), loc.getY(), loc.getZ() + (z * 16)); @@ -158,7 +158,7 @@ public class RTPTeleport { } private boolean sendStatusMessage() { - return getPl().getSettings().statusMessages; + return getPl().getSettings().isStatusMessages(); } private BetterRTP getPl() { diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/references/helpers/HelperRTP.java b/src/main/java/me/SuperRonanCraft/BetterRTP/references/helpers/HelperRTP.java index 04c48c9..a629588 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/references/helpers/HelperRTP.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/references/helpers/HelperRTP.java @@ -35,7 +35,7 @@ public class HelperRTP { if (ignoreCooldown || isCoolingDown(sendi, player)) { //Is Cooling down boolean delay = false; if (!ignoreDelay && sendi == player) //Forced? - if (getPl().getSettings().delayEnabled && getPl().getSettings().delayTime > 0) //Delay enabled? + if (getPl().getSettings().isDelayEnabled() && getPl().getSettings().getDelayTime() > 0) //Delay enabled? if (!getPl().getPerms().getBypassDelay(player)) //Can bypass? delay = true; //player.sendMessage("Cooldown applies: " + cooldownApplies(sendi, player)); @@ -69,8 +69,8 @@ public class HelperRTP { return false; } else { //Normal cooldown long Left = cooldownHandler.timeLeft(cooldownData); - if (getPl().getSettings().delayEnabled && !getPl().getPerms().getBypassDelay(sendi)) - Left = Left + getPl().getSettings().delayTime; + if (getPl().getSettings().isDelayEnabled() && !getPl().getPerms().getBypassDelay(sendi)) + Left = Left + getPl().getSettings().getDelayTime(); if (Left > 0) { //Still cooling down getPl().getText().getCooldown(sendi, String.valueOf(Left)); diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/Settings.java b/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/Settings.java index 47e59a8..0910dc7 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/Settings.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/Settings.java @@ -1,18 +1,19 @@ package me.SuperRonanCraft.BetterRTP.references.settings; +import lombok.Getter; import me.SuperRonanCraft.BetterRTP.references.file.FileBasics; import me.SuperRonanCraft.BetterRTP.BetterRTP; public class Settings { - public boolean debug; - public boolean delayEnabled; - public int delayTime; - public boolean rtpOnFirstJoin_Enabled; - public String rtpOnFirstJoin_World; - public boolean rtpOnFirstJoin_SetAsRespawn; - public boolean statusMessages; //Send more information about rtp - public int preloadRadius; //Amount of chunks to load around a safe rtp location (clamped (0 - 16)) + @Getter private boolean debug; + @Getter private boolean delayEnabled; + @Getter private int delayTime; + @Getter private boolean rtpOnFirstJoin_Enabled; + @Getter private String rtpOnFirstJoin_World; + @Getter private boolean rtpOnFirstJoin_SetAsRespawn; + @Getter private boolean statusMessages; //Send more information about rtp + @Getter private int preloadRadius; //Amount of chunks to load around a safe rtp location (clamped (0 - 16)) //Dependencies private final SoftDepends depends = new SoftDepends(); public boolean protocolLibSounds; diff --git a/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/SoftDepends.java b/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/SoftDepends.java index 6c16bfd..9c3ceee 100644 --- a/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/SoftDepends.java +++ b/src/main/java/me/SuperRonanCraft/BetterRTP/references/settings/SoftDepends.java @@ -1,5 +1,6 @@ package me.SuperRonanCraft.BetterRTP.references.settings; +import lombok.Getter; import me.SuperRonanCraft.BetterRTP.references.file.FileBasics; import me.SuperRonanCraft.BetterRTP.BetterRTP; import org.bukkit.Bukkit; @@ -21,66 +22,18 @@ public class SoftDepends { private boolean respect_ultimateClaims = false; private boolean respect_pueblos = false; //RETURNABLES - private boolean worldguard = false; - private boolean griefprevention = false; - private boolean towny = false; - private boolean redProtect = false; - private boolean factionsUUID = false; - private boolean lands = false; - private boolean residence = false; - private boolean kingdomsx = false; - private boolean hClaims = false; - private boolean griefDefender = false; - private boolean ultimateClaims = false; - private boolean pueblos = false; - - public boolean isWorldguard() { - return worldguard; - } - - public boolean isGriefprevention() { - return griefprevention; - } - - public boolean isTowny() { - return towny; - } - - public boolean isRedProtect() { - return redProtect; - } - - public boolean isFactionsUUID() { - return factionsUUID; - } - - public boolean isLands() { - return lands; - } - - public boolean isResidence() { - return residence; - } - - public boolean isKingdomsX() { - return kingdomsx; - } - - public boolean ishClaims() { - return hClaims; - } - - public boolean isGriefDefender() { - return griefDefender; - } - - public boolean isUltimateClaims() { - return ultimateClaims; - } - - public boolean isPueblos() { - return pueblos; - } + @Getter private boolean worldguard = false; + @Getter private boolean griefprevention = false; + @Getter private boolean towny = false; + @Getter private boolean redProtect = false; + @Getter private boolean factionsUUID = false; + @Getter private boolean lands = false; + @Getter private boolean residence = false; + @Getter private boolean kingdomsx = false; + @Getter private boolean hClaims = false; + @Getter private boolean griefDefender = false; + @Getter private boolean ultimateClaims = false; + @Getter private boolean pueblos = false; void load() { FileBasics.FILETYPE config = BetterRTP.getInstance().getFiles().getType(FileBasics.FILETYPE.CONFIG); @@ -184,7 +137,7 @@ public class SoftDepends { } private void debug(String str) { - if (BetterRTP.getInstance().getSettings().debug) + if (BetterRTP.getInstance().getSettings().isDebug()) BetterRTP.getInstance().getLogger().log(Level.INFO, str); } }