Upstream/NoCheatPlus
1
0
Fork 0
mirror of https://github.com/NoCheatPlus/NoCheatPlus.git synced 2024-10-06 10:27:26 +02:00
Code Issues Projects Releases Wiki Activity

Fixed recursion within error handling of Permissions plugin

Browse Source
This commit is contained in:
Evenprime 2011-04-08 09:43:47 +02:00
parent 9ef5282905
commit 8e04bdece9
2 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
plugin.yml
View File

@ -3,7 +3,7 @@ name: NoCheat
author: Evenprime
main: cc.co.evenprime.bukkit.nocheat.NoCheat
version: 0.8
version: 0.8.1
commands:
nocheat:

15
src/cc/co/evenprime/bukkit/nocheat/NoCheat.java
View File

@ -46,6 +46,8 @@ public class NoCheat extends JavaPlugin {
public final AirbuildCheck airbuildCheck;
private NoCheatConfiguration config;
private boolean exceptionWithPermissions = false;
// Permissions 2.x, if available
private PermissionHandler permissions;
@ -251,7 +253,6 @@ public class NoCheat extends JavaPlugin {
if(config.chatLevel.intValue() <= l.intValue()) {
for(Player player : getServer().getOnlinePlayers()) {
if(hasPermission(player, "nocheat.notify")) {
System.out.println("logged to chat");
player.sendMessage("["+l.getName()+"] " + message);
}
}
@ -285,9 +286,15 @@ public class NoCheat extends JavaPlugin {
return false;
}
catch(Throwable e) {
log(Level.SEVERE, "Asking Permissions-Plugin if "+player.getName()+" has permission "+permission+" caused the Exception: "+e.toString() + " . Full error message recorded to log-file");
for(StackTraceElement s : e.getStackTrace()) {
config.logger.log(Level.SEVERE, s.toString());
if(!this.exceptionWithPermissions) {
// Prevent spam and recursion by definitely doing this only once
this.exceptionWithPermissions = true;
String logtext = "Asking Permissions-Plugin if "+player.getName()+" has permission "+permission+" caused an Exception "+ e.getMessage() + ". Please review your permissions config file. This message is only displayed once, the player is considered to not have that permission from now on. The full stack trace is written into the nocheat logfile.";
log(Level.SEVERE, logtext);
for(StackTraceElement s : e.getStackTrace()) {
config.logger.log(Level.SEVERE, s.toString());
}
}
return false;
}