From 9462eb061909580d4e0dfeca9e24ef854803adab Mon Sep 17 00:00:00 2001
From: ChanceSD <5256215+ChanceSD@users.noreply.github.com>
Date: Wed, 5 Apr 2023 16:20:00 +0100
Subject: [PATCH 01/16] Many changes
---
pvpmanager/pom.xml | 9 ++
.../NoChance/PvPManager/Commands/Newbie.java | 2 +-
.../me/NoChance/PvPManager/Commands/PvP.java | 2 +-
.../NoChance/PvPManager/Commands/PvPInfo.java | 7 +-
.../PvPManager/Commands/PvPStatus.java | 11 +-
.../{ => API}/BaseDependency.java | 2 +-
.../Dependencies/{ => API}/Dependency.java | 3 +-
.../{ => API}/DisguiseDependency.java | 2 +-
.../Dependencies/{ => API}/GodDependency.java | 2 +-
.../{ => API}/RegionDependency.java | 2 +-
.../WorldGuardDependency.java} | 4 +-
.../Dependencies/ForceToggleDependency.java | 5 +-
.../PvPManager/Dependencies/Hook.java | 1 +
.../Dependencies/Hooks/EssentialsHook.java | 2 +
.../Hooks/GriefPreventionHook.java | 31 ++++
.../Dependencies/Hooks/LibsDisguisesHook.java | 4 +-
.../Dependencies/Hooks/PlaceHolderAPI.java | 2 +-
.../Hooks/PlaceHolderAPIHook.java | 4 +-
.../Dependencies/Hooks/SimpleClansHook.java | 1 -
.../Dependencies/Hooks/VaultHook.java | 11 +-
.../Hooks/WorldGuardLegacyHook.java | 6 +-
.../Hooks/WorldGuardModernHook.java | 12 +-
.../Libraries/Metrics/CustomMetrics.java | 4 +-
.../Listeners/DebugEntityListener.java | 12 +-
.../PvPManager/Listeners/EntityListener.java | 17 ++-
.../Listeners/EntityListener1_9.java | 4 +-
.../PvPManager/Listeners/MoveListener.java | 2 +-
.../PvPManager/Listeners/MoveListener1_9.java | 3 +-
.../PvPManager/Listeners/PlayerListener.java | 63 ++++++--
.../PvPManager/Listeners/WGListener.java | 4 +-
.../Listeners/WGListenerLegacy.java | 4 +-
.../Managers/DependencyManager.java | 30 ++--
.../PvPManager/Managers/DisplayManager.java | 1 +
.../PvPManager/Managers/PlayerHandler.java | 49 +++---
.../PvPManager/Player/BasePlayer.java | 21 ++-
.../NoChance/PvPManager/Player/EcoPlayer.java | 4 +-
...ancelResult.java => ProtectionResult.java} | 5 +-
.../me/NoChance/PvPManager/PvPManager.java | 3 +-
.../java/me/NoChance/PvPManager/PvPlayer.java | 56 +++++--
.../PvPManager/Settings/Messages.java | 141 +++++++++++++-----
.../PvPManager/Settings/Settings.java | 56 +++++--
.../NoChance/PvPManager/Tasks/NewbieTask.java | 4 +
.../me/NoChance/PvPManager/Tasks/TagTask.java | 3 -
.../PvPManager/Utils/CombatUtils.java | 19 ++-
.../NoChance/PvPManager/Utils/DateUtil.java | 1 +
.../NoChance/PvPManager/Utils/MCVersion.java | 36 +++++
.../NoChance/PvPManager/Utils/NCDuration.java | 75 ++++++++++
.../NoChance/PvPManager/Utils/TimeUtil.java | 44 ++++++
.../player/nametag/BukkitNameTag.java | 35 +++--
pvpmanager/src/main/resources/config.yml | 15 +-
.../main/resources/locale/messages.properties | 115 +++++++-------
pvpmanager/src/main/resources/plugin.yml | 1 +
.../Listeners/EntityListenerTest.java | 12 +-
.../PvPManager/Managers/DependencyTest.java | 8 +-
.../me/NoChance/PvPManager/UpdaterTest.java | 2 +-
55 files changed, 707 insertions(+), 267 deletions(-)
rename pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/{ => API}/BaseDependency.java (92%)
rename pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/{ => API}/Dependency.java (82%)
rename pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/{ => API}/DisguiseDependency.java (78%)
rename pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/{ => API}/GodDependency.java (81%)
rename pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/{ => API}/RegionDependency.java (86%)
rename pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/{WorldGuardHook.java => API/WorldGuardDependency.java} (82%)
create mode 100644 pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/GriefPreventionHook.java
rename pvpmanager/src/main/java/me/NoChance/PvPManager/Player/{CancelResult.java => ProtectionResult.java} (83%)
create mode 100644 pvpmanager/src/main/java/me/NoChance/PvPManager/Utils/DateUtil.java
create mode 100644 pvpmanager/src/main/java/me/NoChance/PvPManager/Utils/MCVersion.java
create mode 100644 pvpmanager/src/main/java/me/NoChance/PvPManager/Utils/NCDuration.java
create mode 100644 pvpmanager/src/main/java/me/NoChance/PvPManager/Utils/TimeUtil.java
diff --git a/pvpmanager/pom.xml b/pvpmanager/pom.xml
index fa1847955..9a921876f 100644
--- a/pvpmanager/pom.xml
+++ b/pvpmanager/pom.xml
@@ -231,6 +231,15 @@
1.16.8.1.1
provided
+
+
+
+ com.github.TechFortress
+ GriefPrevention
+ 16.14.0
+ provided
+ true
+
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/Newbie.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/Newbie.java
index 9f24ce0ca..6a7d1e847 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/Newbie.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/Newbie.java
@@ -60,7 +60,7 @@ public final boolean onCommand(final CommandSender sender, final Command command
return true;
}
}
- } else if (!(sender instanceof Player)) {
+ } else {
sender.sendMessage("This command is only available for players.");
}
return false;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvP.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvP.java
index 61f6242b7..6176bbc04 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvP.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvP.java
@@ -100,7 +100,7 @@ private void togglePvPAdmin(final CommandSender sender, final String playerName,
final PvPlayer specifiedPlayer = ph.get(Bukkit.getPlayer(playerName));
specifiedPlayer.setPvP(toggle ? !specifiedPlayer.hasPvPEnabled() : state);
final String stateMessage = specifiedPlayer.hasPvPEnabled() ? Messages.getEnabled() : Messages.getDisabled();
- sender.sendMessage(Messages.getPvPToggleAdminChanged().replace("%p", playerName).replace("%state", stateMessage));
+ sender.sendMessage(Messages.getPvPToggleAdminChanged().replace("%p", playerName).replace("%state", stateMessage)); // TODO add replace variables
}
private void togglePvPAll(final CommandSender sender, final boolean state, final boolean toggle) {
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPInfo.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPInfo.java
index 97c71ef62..9bf4a8ee9 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPInfo.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPInfo.java
@@ -32,11 +32,12 @@ public final boolean onCommand(final CommandSender sender, final Command cmd, fi
sendInfo(sender, ph.get((Player) sender));
return true;
} else if (args.length == 1 && sender.hasPermission("pvpmanager.info.others")) {
- if (CombatUtils.isOnline(args[0])) {
- sendInfo(sender, ph.get(Bukkit.getPlayer(args[0])));
+ final String name = args[0];
+ if (CombatUtils.isOnline(name)) {
+ sendInfo(sender, ph.get(Bukkit.getPlayer(name)));
return true;
}
- sender.sendMessage(Messages.getErrorPlayerNotFound().replace("%p", args[0]));
+ sender.sendMessage(Messages.getErrorPlayerNotFound(name));
return true;
}
return false;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPStatus.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPStatus.java
index f3faf48d9..2265a8ea8 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPStatus.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/PvPStatus.java
@@ -32,16 +32,17 @@ public final boolean onCommand(final CommandSender sender, final Command cmd, fi
pvpPlayer.message(Messages.getSelfStatusEnabled());
return true;
} else if (args.length == 1 && sender.hasPermission("pvpmanager.pvpstatus.others")) {
- if (CombatUtils.isOnline(args[0])) {
- target = ph.get(Bukkit.getPlayer(args[0]));
+ final String name = args[0];
+ if (CombatUtils.isOnline(name)) {
+ target = ph.get(Bukkit.getPlayer(name));
if (!target.hasPvPEnabled()) {
- sender.sendMessage(Messages.getOthersStatusDisabled().replace("%p", args[0]));
+ sender.sendMessage(Messages.getOthersStatusDisabled(name));
return true;
}
- sender.sendMessage(Messages.getOtherStatusEnabled().replace("%p", args[0]));
+ sender.sendMessage(Messages.getOtherStatusEnabled(name));
return true;
}
- sender.sendMessage(Messages.getErrorPlayerNotFound().replace("%p", args[0]));
+ sender.sendMessage(Messages.getErrorPlayerNotFound(name));
return true;
}
return false;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/BaseDependency.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/BaseDependency.java
similarity index 92%
rename from pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/BaseDependency.java
rename to pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/BaseDependency.java
index 8cada20f9..471c3795c 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/BaseDependency.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/BaseDependency.java
@@ -1,4 +1,4 @@
-package me.NoChance.PvPManager.Dependencies;
+package me.NoChance.PvPManager.Dependencies.API;
import org.bukkit.ChatColor;
import org.bukkit.plugin.java.JavaPlugin;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Dependency.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/Dependency.java
similarity index 82%
rename from pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Dependency.java
rename to pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/Dependency.java
index 994af453b..eca1db052 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Dependency.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/Dependency.java
@@ -1,8 +1,9 @@
-package me.NoChance.PvPManager.Dependencies;
+package me.NoChance.PvPManager.Dependencies.API;
import org.bukkit.plugin.java.JavaPlugin;
import me.NoChance.PvPManager.Settings.Settings;
+import me.NoChance.PvPManager.Dependencies.Hook;
public interface Dependency {
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/DisguiseDependency.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/DisguiseDependency.java
similarity index 78%
rename from pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/DisguiseDependency.java
rename to pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/DisguiseDependency.java
index e89cee243..98c042cc4 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/DisguiseDependency.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/DisguiseDependency.java
@@ -1,4 +1,4 @@
-package me.NoChance.PvPManager.Dependencies;
+package me.NoChance.PvPManager.Dependencies.API;
import org.bukkit.entity.Player;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/GodDependency.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/GodDependency.java
similarity index 81%
rename from pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/GodDependency.java
rename to pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/GodDependency.java
index 6c0171fad..28fda8aad 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/GodDependency.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/GodDependency.java
@@ -1,4 +1,4 @@
-package me.NoChance.PvPManager.Dependencies;
+package me.NoChance.PvPManager.Dependencies.API;
import org.bukkit.entity.Player;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/RegionDependency.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/RegionDependency.java
similarity index 86%
rename from pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/RegionDependency.java
rename to pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/RegionDependency.java
index f7450dbda..acc2c1dec 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/RegionDependency.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/RegionDependency.java
@@ -1,4 +1,4 @@
-package me.NoChance.PvPManager.Dependencies;
+package me.NoChance.PvPManager.Dependencies.API;
import org.bukkit.Location;
import org.bukkit.entity.Player;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/WorldGuardHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/WorldGuardDependency.java
similarity index 82%
rename from pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/WorldGuardHook.java
rename to pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/WorldGuardDependency.java
index 4d63cd22b..cdd9dc210 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/WorldGuardHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/API/WorldGuardDependency.java
@@ -1,4 +1,4 @@
-package me.NoChance.PvPManager.Dependencies;
+package me.NoChance.PvPManager.Dependencies.API;
import java.util.Set;
@@ -10,7 +10,7 @@
import me.NoChance.PvPManager.Managers.PlayerHandler;
-public interface WorldGuardHook extends GroupDependency, RegionDependency {
+public interface WorldGuardDependency extends GroupDependency, RegionDependency {
public boolean hasAllowPvPFlag(Player player);
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/ForceToggleDependency.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/ForceToggleDependency.java
index ae8caae76..5a5076a91 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/ForceToggleDependency.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/ForceToggleDependency.java
@@ -2,13 +2,14 @@
import org.bukkit.entity.Player;
-import me.NoChance.PvPManager.Player.CancelResult;
+import me.NoChance.PvPManager.Dependencies.API.Dependency;
+import me.NoChance.PvPManager.Player.ProtectionResult;
public interface ForceToggleDependency extends Dependency {
public boolean shouldDisable(Player player);
- public boolean shouldDisable(Player attacker, Player defender, CancelResult reason);
+ public boolean shouldDisable(Player attacker, Player defender, ProtectionResult reason);
public default boolean shouldDisableProtection() {
return getConfigBoolean("No Protection In War", true);
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hook.java
index 5f52a4e9e..de869a3b1 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hook.java
@@ -18,6 +18,7 @@ public enum Hook {
SIMPLECLANS("SimpleClans"),
PLACEHOLDERAPI("PlaceholderAPI"),
LIBSDISGUISES("LibsDisguises"),
+ GRIEFPREVENTION("GriefPrevention"),
TOWNY("Towny"),
KINGDOMSX("Kingdoms");
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/EssentialsHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/EssentialsHook.java
index d63f50421..d99da93a0 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/EssentialsHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/EssentialsHook.java
@@ -8,6 +8,8 @@
import me.NoChance.PvPManager.Dependencies.BaseDependency;
import me.NoChance.PvPManager.Dependencies.GodDependency;
import me.NoChance.PvPManager.Dependencies.Hook;
+import me.NoChance.PvPManager.Dependencies.API.BaseDependency;
+import me.NoChance.PvPManager.Dependencies.API.GodDependency;
public class EssentialsHook extends BaseDependency implements AFKDependency, GodDependency {
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/GriefPreventionHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/GriefPreventionHook.java
new file mode 100644
index 000000000..b0074e07b
--- /dev/null
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/GriefPreventionHook.java
@@ -0,0 +1,31 @@
+package me.NoChance.PvPManager.Dependencies.Hooks;
+
+import me.NoChance.PvPManager.Dependencies.Hook;
+import me.NoChance.PvPManager.Dependencies.API.BaseDependency;
+import me.NoChance.PvPManager.Settings.Settings;
+import me.NoChance.PvPManager.Utils.Log;
+import me.ryanhamshire.GriefPrevention.GriefPrevention;
+
+public class GriefPreventionHook extends BaseDependency {
+
+ public GriefPreventionHook(final Hook hook) {
+ super(hook);
+ final GriefPrevention gp = (GriefPrevention) getPlugin();
+ if (gp.config_pvp_combatTimeoutSeconds > 0 || gp.config_pvp_punishLogout) {
+ Log.warning("GriefPrevention has some combat features without showing any feedback messages. "
+ + "Make sure to set PunishLogout to false and set CombatTimeoutSeconds to 0 seconds in GP config. "
+ + "Conflicts with those features often get wrongly blamed on PvPManager due to the lack of GP feedback messages.");
+ if (Settings.isKillOnLogout() && gp.config_pvp_punishLogout) {
+ Log.severe("You have kill on logout enabled in PvPManager, so we automatically disabled the same feature on GriefPrevention. "
+ + "Disable this setting in GP or PvPManager to remove this error, this is necessary as it could cause issues.");
+ gp.config_pvp_punishLogout = false;
+ }
+ }
+ }
+
+ @Override
+ public String onEnableMessage() {
+ return "GriefPrevention Found! Checking for conflicting settings";
+ }
+
+}
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/LibsDisguisesHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/LibsDisguisesHook.java
index 8fe7bd66e..245bc2c96 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/LibsDisguisesHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/LibsDisguisesHook.java
@@ -2,9 +2,9 @@
import org.bukkit.entity.Player;
-import me.NoChance.PvPManager.Dependencies.BaseDependency;
-import me.NoChance.PvPManager.Dependencies.DisguiseDependency;
import me.NoChance.PvPManager.Dependencies.Hook;
+import me.NoChance.PvPManager.Dependencies.API.BaseDependency;
+import me.NoChance.PvPManager.Dependencies.API.DisguiseDependency;
import me.chancesd.sdutils.utils.Log;
import me.libraryaddict.disguise.DisguiseAPI;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPI.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPI.java
index 3e221f1e5..125a3b1c9 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPI.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPI.java
@@ -59,7 +59,7 @@ public String onPlaceholderRequest(final Player player, final String identifier)
return String.valueOf(p.getNewbieTimeLeft() / 1000);
case "pvp_command_timeleft":
final int timeLeft = CombatUtils.getTimeLeft(p.getToggleTime(), Settings.getToggleCooldown());
- return String.valueOf(timeLeft > 0 ? timeLeft : 0);
+ return String.valueOf(Math.max(timeLeft, 0));
case "current_enemy":
final PvPlayer pPlayer = p.getEnemy();
return pPlayer != null ? pPlayer.getName() : "None";
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPIHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPIHook.java
index 4c5e8dc48..095cbb228 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPIHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/PlaceHolderAPIHook.java
@@ -1,13 +1,13 @@
package me.NoChance.PvPManager.Dependencies.Hooks;
import me.NoChance.PvPManager.PvPManager;
-import me.NoChance.PvPManager.Dependencies.BaseDependency;
import me.NoChance.PvPManager.Dependencies.DependencyException;
import me.NoChance.PvPManager.Dependencies.Hook;
+import me.NoChance.PvPManager.Dependencies.API.BaseDependency;
public class PlaceHolderAPIHook extends BaseDependency {
- final PlaceHolderAPI placeHolderAPI;
+ private final PlaceHolderAPI placeHolderAPI;
public PlaceHolderAPIHook(final Hook hook) {
super(hook);
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/SimpleClansHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/SimpleClansHook.java
index 2f0d67a67..cda83a7a1 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/SimpleClansHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/SimpleClansHook.java
@@ -2,7 +2,6 @@
import org.bukkit.entity.Player;
-import me.NoChance.PvPManager.Dependencies.BaseDependency;
import me.NoChance.PvPManager.Dependencies.DependencyException;
import me.NoChance.PvPManager.Dependencies.Hook;
import me.NoChance.PvPManager.Dependencies.GroupDependency;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/VaultHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/VaultHook.java
index e6fc6e7f0..fd88dc684 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/VaultHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/VaultHook.java
@@ -3,9 +3,9 @@
import org.bukkit.Bukkit;
import org.bukkit.plugin.RegisteredServiceProvider;
-import me.NoChance.PvPManager.Dependencies.BaseDependency;
import me.NoChance.PvPManager.Dependencies.DependencyException;
import me.NoChance.PvPManager.Dependencies.Hook;
+import me.NoChance.PvPManager.Dependencies.API.BaseDependency;
import net.milkbowl.vault.economy.Economy;
public class VaultHook extends BaseDependency {
@@ -19,13 +19,10 @@ public VaultHook(final Hook hook) {
private Economy setupEconomy() {
final RegisteredServiceProvider economyProvider = Bukkit.getServicesManager().getRegistration(Economy.class);
- Economy eco = null;
- if (economyProvider != null) {
- eco = economyProvider.getProvider();
- } else {
+ if (economyProvider != null)
+ return economyProvider.getProvider();
+ else
throw new DependencyException("Error loading Vault! No Economy plugin found", Hook.VAULT);
- }
- return eco;
}
@Override
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardLegacyHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardLegacyHook.java
index 78fb91145..12ef81827 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardLegacyHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardLegacyHook.java
@@ -14,14 +14,14 @@
import com.sk89q.worldguard.protection.flags.StateFlag.State;
import com.sk89q.worldguard.protection.regions.ProtectedRegion;
-import me.NoChance.PvPManager.Dependencies.BaseDependency;
import me.NoChance.PvPManager.Dependencies.DependencyException;
import me.NoChance.PvPManager.Dependencies.Hook;
-import me.NoChance.PvPManager.Dependencies.WorldGuardHook;
+import me.NoChance.PvPManager.Dependencies.API.BaseDependency;
+import me.NoChance.PvPManager.Dependencies.API.WorldGuardDependency;
import me.NoChance.PvPManager.Listeners.WGListenerLegacy;
import me.NoChance.PvPManager.Managers.PlayerHandler;
-public class WorldGuardLegacyHook extends BaseDependency implements WorldGuardHook {
+public class WorldGuardLegacyHook extends BaseDependency implements WorldGuardDependency {
private RegionQuery regionQuery;
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardModernHook.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardModernHook.java
index a8abd8016..8839f08d7 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardModernHook.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Dependencies/Hooks/WorldGuardModernHook.java
@@ -13,17 +13,17 @@
import com.sk89q.worldguard.protection.regions.RegionQuery;
import me.NoChance.PvPManager.PvPlayer;
-import me.NoChance.PvPManager.Dependencies.BaseDependency;
import me.NoChance.PvPManager.Dependencies.ForceToggleDependency;
import me.NoChance.PvPManager.Dependencies.Hook;
-import me.NoChance.PvPManager.Dependencies.WorldGuardHook;
+import me.NoChance.PvPManager.Dependencies.API.BaseDependency;
+import me.NoChance.PvPManager.Dependencies.API.WorldGuardDependency;
import me.NoChance.PvPManager.Listeners.WGListener;
import me.NoChance.PvPManager.Managers.PlayerHandler;
-import me.NoChance.PvPManager.Player.CancelResult;
+import me.NoChance.PvPManager.Player.ProtectionResult;
import me.NoChance.PvPManager.Settings.Messages;
import me.NoChance.PvPManager.Settings.Settings;
-public class WorldGuardModernHook extends BaseDependency implements WorldGuardHook, ForceToggleDependency {
+public class WorldGuardModernHook extends BaseDependency implements WorldGuardDependency, ForceToggleDependency {
private final RegionQuery regionQuery;
@@ -85,11 +85,11 @@ public boolean shouldDisable(final Player player) {
}
@Override
- public boolean shouldDisable(final Player damager, final Player defender, final CancelResult reason) {
+ public boolean shouldDisable(final Player damager, final Player defender, final ProtectionResult reason) {
if (hasAllowPvPFlag(defender) || containsRegionsAt(defender.getLocation(), Settings.getWorldguardOverridesList())) {
final PvPlayer attacker = PvPlayer.get(damager);
final PvPlayer attacked = PvPlayer.get(defender);
- if (reason == CancelResult.PVPDISABLED) {
+ if (reason == ProtectionResult.PVPDISABLED) {
disablePvP(attacker);
disablePvP(attacked);
} else {
diff --git a/pvpmanager/src/main/java/me/NoChance/PvPManager/Libraries/Metrics/CustomMetrics.java b/pvpmanager/src/main/java/me/NoChance/PvPManager/Libraries/Metrics/CustomMetrics.java
index be0b06ee1..330152b47 100644
--- a/pvpmanager/src/main/java/me/NoChance/PvPManager/Libraries/Metrics/CustomMetrics.java
+++ b/pvpmanager/src/main/java/me/NoChance/PvPManager/Libraries/Metrics/CustomMetrics.java
@@ -10,8 +10,8 @@
import org.bukkit.Bukkit;
import me.NoChance.PvPManager.PvPManager;
-import me.NoChance.PvPManager.Dependencies.Dependency;
import me.NoChance.PvPManager.Dependencies.Hook;
+import me.NoChance.PvPManager.Dependencies.API.Dependency;
import me.NoChance.PvPManager.Settings.Settings;
import me.chancesd.sdutils.metrics.Metrics;
@@ -36,7 +36,7 @@ public String call() {
metrics.addCustomChart(new Metrics.DrilldownPie("features", new Callable