Made logger optional

Author: MackenzieMolloy

src/main/java/net/mackenziemolloy/shopguiplus/sellgui/SellGUI.java

@@ -54,8 +54,6 @@ public void onEnable() {
         generateFiles();
         setupMetrics();
         setupUpdates();
-
-        fileLogger = Logger.getLogger("SellGUIFileLogger");
         initLogger();
 
         logger.info("*-*");
@@ -75,6 +73,9 @@ public void checkCompatibility() {
     }
 
     public void initLogger() {
+        if (this.configuration.get("options.transaction_log.enabled") == null || !this.configuration.getBoolean("options.transaction_log.enabled")) return;
+
+        fileLogger = Logger.getLogger("SellGUIFileLogger");
         File log = FileUtils.loadFile("transaction.log");
         FileHandler handler = null;
 
@@ -91,6 +92,14 @@ public void initLogger() {
         fileLogger.setUseParentHandlers(false);
     }
 
+    public void closeLogger() {
+        if (fileLogger == null) return;
+        if (fileLogger.getHandlers().length == 0) return;
+
+        fileLogger.getHandlers()[0].close();
+        fileLogger = null;
+    }
+
     public CommentedConfiguration getConfiguration() {
         return this.configuration;
     }

src/main/java/net/mackenziemolloy/shopguiplus/sellgui/command/CommandSellGUI.java

@@ -133,6 +133,10 @@ private boolean commandReload(CommandSender sender) {
                 return;
             }
 
+            // This looks painful, I know.
+            if (this.plugin.getConfiguration().get("options.transaction_log.enabled") != null && !this.plugin.getConfiguration().getBoolean("options.transaction_log.enabled")) this.plugin.closeLogger();
+            else if(this.plugin.fileLogger == null) this.plugin.initLogger();
+
             sendMessage(sender, "reloaded_config");
             if(sender instanceof Player) {
                 Player player = (Player) sender;
@@ -600,7 +604,7 @@ private void onGuiClose(Player player, InventoryCloseEvent event, Set<Integer> i
             }
 
             /* Subject to deprecation */
-            plugin.fileLogger.info(player.getName() + " (" + player.getUniqueId() + ") sold: {" + HexColorUtility.purgeAllColor(receiptList.stream().collect(Collectors.joining(", "))) + "}");
+            if (plugin.fileLogger != null) plugin.fileLogger.info(player.getName() + " (" + player.getUniqueId() + ") sold: {" + HexColorUtility.purgeAllColor(receiptList.stream().collect(Collectors.joining(", "))) + "}");
 
 
             if(configuration.getBoolean("options.sell_titles")) {

src/main/java/net/mackenziemolloy/shopguiplus/sellgui/utility/LogFormatter.java

@@ -10,7 +10,7 @@
 
 public class LogFormatter extends Formatter {
 
-  private final DateFormat dateFormat = new SimpleDateFormat(SellGUI.getInstance().getConfiguration().getString("options.date_format"));
+  private final DateFormat dateFormat = new SimpleDateFormat(SellGUI.getInstance().getConfiguration().getString("options.transaction_log.date_format"));
 
   @Override
   public String format(LogRecord logRecord) {

src/main/resources/config.yml

@@ -6,7 +6,8 @@
 #    |____/|_____|_____|_____\____|\___/|___|
 #           ( By Mackenzie Molloy )
 
-# Support Discord | https://mackenziemolloy.net/discord
+#  Please contact me if you have any issues with the plugin - happy to help.
+#  - Support Discord | https://mackenziemolloy.net/discord
 
 #
 # Permissions:
@@ -23,8 +24,16 @@ options:
   receipt_type: 1
   sell_titles: true
 
-  # Date format of transactions logging (http://docs.oracle.com/javase/7/docs/api/java/text/SimpleDateFormat.html)
-  date_format: "yyyy/MM/dd HH:mm:ss"
+  #
+  # This is the feature where all sell transactions made via
+  # the SellGUI Sell Menu are logged to a file.
+  #
+
+  transaction_log:
+    # Should the Transaction logging feature be enabled?
+    enabled: true
+    # Date format of transactions logging (http://docs.oracle.com/javase/7/docs/api/java/text/SimpleDateFormat.html)
+    date_format: "yyyy/MM/dd HH:mm:ss"
 
   #
   # Whether prices should be rounded to 2 decimal places, or not.