diff --git a/src/main/java/fr/xephi/authme/AuthMe.java b/src/main/java/fr/xephi/authme/AuthMe.java index 775b22aa..655a1f33 100644 --- a/src/main/java/fr/xephi/authme/AuthMe.java +++ b/src/main/java/fr/xephi/authme/AuthMe.java @@ -10,6 +10,7 @@ import fr.xephi.authme.cache.backup.JsonCache; import fr.xephi.authme.cache.limbo.LimboCache; import fr.xephi.authme.cache.limbo.LimboPlayer; import fr.xephi.authme.command.CommandHandler; +import fr.xephi.authme.command.CommandInitializer; import fr.xephi.authme.converter.Converter; import fr.xephi.authme.converter.ForceFlatToSqlite; import fr.xephi.authme.datasource.*; @@ -28,6 +29,7 @@ import fr.xephi.authme.settings.*; import fr.xephi.authme.util.GeoLiteAPI; import fr.xephi.authme.util.StringUtils; import fr.xephi.authme.util.Utils; +import fr.xephi.authme.util.Wrapper; import net.minelink.ctplus.CombatTagPlus; import org.apache.logging.log4j.LogManager; import org.bukkit.Bukkit; @@ -70,8 +72,9 @@ public class AuthMe extends JavaPlugin { private static AuthMe plugin; private static Server server; + private static Wrapper wrapper = Wrapper.getInstance(); - public Management management; + private Management management; public NewAPI api; public SendMailSSL mail; public DataManager dataManager; @@ -431,8 +434,7 @@ public class AuthMe extends JavaPlugin { * Set up the command handler. */ private void setupCommandHandler() { - this.commandHandler = new CommandHandler(false); - this.commandHandler.init(); + this.commandHandler = new CommandHandler(CommandInitializer.getBaseCommands()); } /** @@ -955,13 +957,14 @@ public class AuthMe extends JavaPlugin { @Override public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) { - // Get the command handler, and make sure it's valid - CommandHandler commandHandler = this.getCommandHandler(); - if (commandHandler == null) + // Make sure the command handler has been initialized + if (commandHandler == null) { + wrapper.getLogger().severe("AuthMe command handler is not available"); return false; + } - // Handle the command, return the result - return commandHandler.onCommand(sender, cmd, commandLabel, args); + // Handle the command + return commandHandler.processCommand(sender, commandLabel, args); } /** diff --git a/src/main/java/fr/xephi/authme/ConsoleLogger.java b/src/main/java/fr/xephi/authme/ConsoleLogger.java index 89031804..fee14296 100644 --- a/src/main/java/fr/xephi/authme/ConsoleLogger.java +++ b/src/main/java/fr/xephi/authme/ConsoleLogger.java @@ -2,7 +2,6 @@ package fr.xephi.authme; import com.google.common.base.Throwables; import fr.xephi.authme.settings.Settings; -import fr.xephi.authme.util.StringUtils; import fr.xephi.authme.util.Wrapper; import java.io.IOException; @@ -17,8 +16,10 @@ import java.util.Date; */ public final class ConsoleLogger { + private static final String NEW_LINE = System.getProperty("line.separator"); + private static final DateFormat DATE_FORMAT = new SimpleDateFormat("[MM-dd HH:mm:ss]"); + private static Wrapper wrapper = Wrapper.getInstance(); - private static final DateFormat df = new SimpleDateFormat("[MM-dd HH:mm:ss]"); private ConsoleLogger() { // Service class @@ -57,11 +58,11 @@ public final class ConsoleLogger { */ private static void writeLog(String message) { String dateTime; - synchronized (df) { - dateTime = df.format(new Date()); + synchronized (DATE_FORMAT) { + dateTime = DATE_FORMAT.format(new Date()); } try { - Files.write(Settings.LOG_FILE.toPath(), (dateTime + ": " + message + StringUtils.newline).getBytes(), + Files.write(Settings.LOG_FILE.toPath(), (dateTime + ": " + message + NEW_LINE).getBytes(), StandardOpenOption.APPEND, StandardOpenOption.CREATE); } catch (IOException ignored) { @@ -77,6 +78,6 @@ public final class ConsoleLogger { if (!Settings.useLogging) { return; } - writeLog("" + Throwables.getStackTraceAsString(ex)); + writeLog(Throwables.getStackTraceAsString(ex)); } } diff --git a/src/main/java/fr/xephi/authme/api/API.java b/src/main/java/fr/xephi/authme/api/API.java index e849e795..1f1ac69d 100644 --- a/src/main/java/fr/xephi/authme/api/API.java +++ b/src/main/java/fr/xephi/authme/api/API.java @@ -170,7 +170,7 @@ public class API { */ @Deprecated public static void forceLogin(Player player) { - instance.management.performLogin(player, "dontneed", true); + instance.getManagement().performLogin(player, "dontneed", true); } /** diff --git a/src/main/java/fr/xephi/authme/api/NewAPI.java b/src/main/java/fr/xephi/authme/api/NewAPI.java index cfe4508f..908f75df 100644 --- a/src/main/java/fr/xephi/authme/api/NewAPI.java +++ b/src/main/java/fr/xephi/authme/api/NewAPI.java @@ -170,7 +170,7 @@ public class NewAPI { * @param player * player */ public void forceLogin(Player player) { - plugin.management.performLogin(player, "dontneed", true); + plugin.getManagement().performLogin(player, "dontneed", true); } /** @@ -179,7 +179,7 @@ public class NewAPI { * @param player * player */ public void forceLogout(Player player) { - plugin.management.performLogout(player); + plugin.getManagement().performLogout(player); } /** @@ -189,7 +189,7 @@ public class NewAPI { * @param password String */ public void forceRegister(Player player, String password) { - plugin.management.performRegister(player, password, null); + plugin.getManagement().performRegister(player, password, null); } /** @@ -198,6 +198,6 @@ public class NewAPI { * @param player * player */ public void forceUnregister(Player player) { - plugin.management.performUnregister(player, "", true); + plugin.getManagement().performUnregister(player, "", true); } } diff --git a/src/main/java/fr/xephi/authme/command/CommandArgumentDescription.java b/src/main/java/fr/xephi/authme/command/CommandArgumentDescription.java index 0440ddc0..f846b789 100644 --- a/src/main/java/fr/xephi/authme/command/CommandArgumentDescription.java +++ b/src/main/java/fr/xephi/authme/command/CommandArgumentDescription.java @@ -1,23 +1,22 @@ package fr.xephi.authme.command; /** + * Wrapper for the description of a command argument. */ public class CommandArgumentDescription { - // TODO: Allow argument to consist of infinite parts.