diff --git a/src/me/ryanhamshire/GriefPrevention/DataStore.java b/src/me/ryanhamshire/GriefPrevention/DataStore.java index da6b800..951c902 100644 --- a/src/me/ryanhamshire/GriefPrevention/DataStore.java +++ b/src/me/ryanhamshire/GriefPrevention/DataStore.java @@ -1436,6 +1436,7 @@ public abstract class DataStore this.addDefault(defaults, Messages.AlreadyUnderSiegePlayer, "{0} is already under siege. Join the party!", "0: defending player"); this.addDefault(defaults, Messages.NotSiegableThere, "{0} isn't protected there.", "0: defending player"); this.addDefault(defaults, Messages.SiegeTooFarAway, "You're too far away to siege.", null); + this.addDefault(defaults, Messages.NoSiegeYourself, "You cannot siege yourself, don't be silly", null); this.addDefault(defaults, Messages.NoSiegeDefenseless, "That player is defenseless. Go pick on somebody else.", null); this.addDefault(defaults, Messages.AlreadyUnderSiegeArea, "That area is already under siege. Join the party!", null); this.addDefault(defaults, Messages.NoSiegeAdminClaim, "Siege is disabled in this area.", null); diff --git a/src/me/ryanhamshire/GriefPrevention/GriefPrevention.java b/src/me/ryanhamshire/GriefPrevention/GriefPrevention.java index 7401ba2..5846661 100644 --- a/src/me/ryanhamshire/GriefPrevention/GriefPrevention.java +++ b/src/me/ryanhamshire/GriefPrevention/GriefPrevention.java @@ -1695,7 +1695,7 @@ public class GriefPrevention extends JavaPlugin //if no parameter, just tell player cost per block and balance if(args.length != 1) { - GriefPrevention.sendMessage(player, TextMode.Info, Messages.BlockPurchaseCost, String.valueOf(GriefPrevention.instance.config_economy_claimBlocksPurchaseCost), String.valueOf(GriefPrevention.economy.getBalance(player))); + GriefPrevention.sendMessage(player, TextMode.Info, Messages.BlockPurchaseCost, String.valueOf(GriefPrevention.instance.config_economy_claimBlocksPurchaseCost), String.valueOf(GriefPrevention.economy.getBalance(player.getName()))); return false; } @@ -1720,7 +1720,7 @@ public class GriefPrevention extends JavaPlugin } //if the player can't afford his purchase, send error message - double balance = economy.getBalance(player); + double balance = economy.getBalance(player.getName()); double totalCost = blockCount * GriefPrevention.instance.config_economy_claimBlocksPurchaseCost; if(totalCost > balance) { @@ -1731,7 +1731,7 @@ public class GriefPrevention extends JavaPlugin else { //withdraw cost - economy.withdrawPlayer(player, totalCost); + economy.withdrawPlayer(player.getName(), totalCost); //add blocks playerData.setBonusClaimBlocks(playerData.getBonusClaimBlocks() + blockCount); @@ -1806,7 +1806,7 @@ public class GriefPrevention extends JavaPlugin { //compute value and deposit it double totalValue = blockCount * GriefPrevention.instance.config_economy_claimBlocksSellValue; - economy.depositPlayer(player, totalValue); + economy.depositPlayer(player.getName(), totalValue); //subtract blocks playerData.setBonusClaimBlocks(playerData.getBonusClaimBlocks() - blockCount); @@ -2398,6 +2398,13 @@ public class GriefPrevention extends JavaPlugin { return false; } + + // First off, you cannot siege yourself, that's just + // silly: + if (attacker.getName().equals( defender.getName() )) { + GriefPrevention.sendMessage(player, TextMode.Err, Messages.NoSiegeYourself); + return true; + } //victim must not have the permission which makes him immune to siege if(defender.hasPermission("griefprevention.siegeimmune")) diff --git a/src/me/ryanhamshire/GriefPrevention/Messages.java b/src/me/ryanhamshire/GriefPrevention/Messages.java index 6c52435..abcbeb6 100644 --- a/src/me/ryanhamshire/GriefPrevention/Messages.java +++ b/src/me/ryanhamshire/GriefPrevention/Messages.java @@ -66,6 +66,7 @@ public enum Messages AlreadySieging, NotSiegableThere, SiegeTooFarAway, + NoSiegeYourself, NoSiegeDefenseless, AlreadyUnderSiegePlayer, AlreadyUnderSiegeArea,