Rename methods

This commit is contained in:
filoghost 2020-07-01 09:40:46 +02:00
parent decb0eb039
commit f0aa5347ee
6 changed files with 54 additions and 54 deletions

View File

@ -131,13 +131,13 @@ public class AdvancedIcon extends ConfigurableIconImpl {
@Override
public boolean onClick(Inventory inventory, Player player) {
// Check all the requirements
boolean hasAllRequirements = Requirement.checkAll(player, clickPermission, requiredMoney, requiredExpLevel, requiredItems);
boolean hasAllRequirements = Requirement.hasAll(player, clickPermission, requiredMoney, requiredExpLevel, requiredItems);
if (!hasAllRequirements) {
return closeOnClick;
}
// If all requirements are satisfied, take their cost
boolean takenAllCosts = Requirement.takeCostAll(player, clickPermission, requiredMoney, requiredExpLevel, requiredItems);
boolean takenAllCosts = Requirement.takeAllCosts(player, clickPermission, requiredMoney, requiredExpLevel, requiredItems);
if (!takenAllCosts) {
return closeOnClick;
}

View File

@ -51,7 +51,7 @@ public class PermissionChecker implements Requirement {
@Override
public boolean check(Player player) {
public boolean hasCost(Player player) {
if (hasPermission(player)) {
return true;
} else {

View File

@ -34,7 +34,7 @@ public class RequiredExpLevel implements Requirement {
}
@Override
public boolean check(Player player) {
public boolean hasCost(Player player) {
if (player.getLevel() < levels) {
player.sendMessage(ChestCommands.getLang().no_exp.replace("{levels}", Integer.toString(levels)));
return false;

View File

@ -38,7 +38,7 @@ public class RequiredItems implements Requirement {
}
@Override
public boolean check(Player player) {
public boolean hasCost(Player player) {
boolean missingItems = false;
for (RequiredItem item : items) {

View File

@ -36,7 +36,7 @@ public class RequiredMoney implements Requirement {
}
@Override
public boolean check(Player player) {
public boolean hasCost(Player player) {
if (!VaultEconomyHook.INSTANCE.isEnabled()) {
player.sendMessage(ChatColor.RED + "This action has a price, but Vault with a compatible economy plugin was not found. For security, the action has been blocked. Please inform the staff.");
return false;

View File

@ -18,13 +18,13 @@ import org.bukkit.entity.Player;
public interface Requirement {
boolean check(Player player);
boolean hasCost(Player player);
boolean takeCost(Player player);
public static boolean checkAll(Player player, Requirement... requirements) {
static boolean hasAll(Player player, Requirement... requirements) {
for (Requirement requirement : requirements) {
if (requirement != null && !requirement.check(player)) {
if (requirement != null && !requirement.hasCost(player)) {
return false;
}
}
@ -32,7 +32,7 @@ public interface Requirement {
return true;
}
public static boolean takeCostAll(Player player, Requirement... requirements) {
static boolean takeAllCosts(Player player, Requirement... requirements) {
for (Requirement requirement : requirements) {
if (requirement != null) {
boolean success = requirement.takeCost(player);