Upstream/NoCheatPlus
1
0
Fork 0
mirror of https://github.com/NoCheatPlus/NoCheatPlus.git synced 2024-11-06 18:50:54 +01:00
Code Issues Projects Releases Wiki Activity

Do not undo command changes.

Browse Source 
Instead just clear the list. It appears that this leads to recalculation
of permissions, which leads to freezing during disabling the plugin
having NPC plugins like Citizens 2 in place. Undoing the changes is not
that important anyway.
This commit is contained in:
asofold 2012-12-04 22:28:51 +01:00
parent cfa397fabc
commit 9838cf4393
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/fr/neatmonster/nocheatplus/NoCheatPlus.java
View File

@ -345,9 +345,12 @@ public class NoCheatPlus extends JavaPlugin implements NoCheatPlusAPI {
if (verbose) LogUtil.logInfo("[NoCheatPlus] Cleanup DataManager...");
dataMan.onDisable();
// Restore changed commands.
if (verbose) LogUtil.logInfo("[NoCheatPlus] Undo command changes...");
undoCommandChanges();
// Clear command changes list (compatibility issues with NPCs, leads to rrecalculation of perms).
changedCommands.clear();
changedCommands = null;
// // Restore changed commands.
// if (verbose) LogUtil.logInfo("[NoCheatPlus] Undo command changes...");
// undoCommandChanges();
// Cleanup the configuration manager.
if (verbose) LogUtil.logInfo("[NoCheatPlus] Cleanup ConfigManager...");
@ -361,6 +364,7 @@ public class NoCheatPlus extends JavaPlugin implements NoCheatPlusAPI {
/**
* Does not undo 100%, but restore old permission, permission-message, label (unlikely to be changed), permission default.
* @deprecated Leads to compatibility issues with NPC plugins such as Citizens 2, due to recalculation of permissions (specifically during disabling).
*/
public void undoCommandChanges() {
if (changedCommands != null){