diff options
author | snowleo <schneeleo@gmail.com> | 2011-11-04 01:23:50 +0100 |
---|---|---|
committer | snowleo <schneeleo@gmail.com> | 2011-11-04 01:24:01 +0100 |
commit | 1d95f1c6d26df1d291268abbe222d58a55aab634 (patch) | |
tree | 03f99893735a61a98dd2489d33d52370645f3828 | |
parent | 6e4bf932ecf89890bd3f586c49c8a0acde8f7970 (diff) | |
download | Essentials-1d95f1c6d26df1d291268abbe222d58a55aab634.tar Essentials-1d95f1c6d26df1d291268abbe222d58a55aab634.tar.gz Essentials-1d95f1c6d26df1d291268abbe222d58a55aab634.tar.lz Essentials-1d95f1c6d26df1d291268abbe222d58a55aab634.tar.xz Essentials-1d95f1c6d26df1d291268abbe222d58a55aab634.zip |
Fix bug in sign state loop
-rw-r--r-- | Essentials/src/com/earth2me/essentials/signs/SignProtection.java | 30 |
1 files changed, 20 insertions, 10 deletions
diff --git a/Essentials/src/com/earth2me/essentials/signs/SignProtection.java b/Essentials/src/com/earth2me/essentials/signs/SignProtection.java index 157535cd9..ced8443a6 100644 --- a/Essentials/src/com/earth2me/essentials/signs/SignProtection.java +++ b/Essentials/src/com/earth2me/essentials/signs/SignProtection.java @@ -35,10 +35,15 @@ public class SignProtection extends EssentialsSign protected boolean onSignCreate(final ISign sign, final User player, final String username, final IEssentials ess) throws SignException, ChargeException { sign.setLine(3, "§4" + username); - if (hasAdjacentBlock(sign.getBlock()) && isBlockProtected(sign.getBlock(), player, username, true) != SignProtectionState.NOT_ALLOWED) + if (hasAdjacentBlock(sign.getBlock())) { - sign.setLine(3, "§1" + username); - return true; + final SignProtectionState state = isBlockProtected(sign.getBlock(), player, username, true); + if (state == SignProtectionState.NOSIGN || state == SignProtectionState.OWNER + || player.isAuthorized("essentials.signs.protection.override")) + { + sign.setLine(3, "§1" + username); + return true; + } } player.sendMessage(Util.i18n("signProtectInvalidLocation")); return false; @@ -183,11 +188,6 @@ public class SignProtection extends EssentialsSign SignProtectionState retstate = SignProtectionState.NOSIGN; for (SignProtectionState state : signs.values()) { - - if (state == SignProtectionState.OWNER) - { - return state; - } if (state == SignProtectionState.ALLOWED) { retstate = state; @@ -197,6 +197,16 @@ public class SignProtection extends EssentialsSign retstate = state; } } + if (!secure || retstate == SignProtectionState.NOSIGN) + { + for (SignProtectionState state : signs.values()) + { + if (state == SignProtectionState.OWNER) + { + return state; + } + } + } return retstate; } @@ -300,7 +310,7 @@ public class SignProtection extends EssentialsSign player.sendMessage(Util.format("noDestroyPermission", block.getType().toString().toLowerCase())); return false; } - + @Override public boolean onBlockBreak(final Block block, final IEssentials ess) { @@ -324,7 +334,7 @@ public class SignProtection extends EssentialsSign return state == SignProtectionState.NOSIGN; } - + @Override public boolean onBlockIgnite(final Block block, final IEssentials ess) { |