Use Java logger

Unfortunately slf4j is not available on Spigot

Related: 7689bca

Author: TheMolkaPL

src/main/java/pl/craftserve/radiation/CraftserveListener.java

@@ -24,16 +24,16 @@
 import org.bukkit.event.Listener;
 import org.bukkit.event.player.PlayerJoinEvent;
 import org.bukkit.plugin.Plugin;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import java.net.InetAddress;
 import java.net.UnknownHostException;
 import java.util.Locale;
 import java.util.Objects;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 public class CraftserveListener implements Listener {
-    static final Logger logger = LoggerFactory.getLogger(CraftserveListener.class);
+    static final Logger logger = Logger.getLogger(CraftserveListener.class.getName());
 
     private static final String PERMISSION = "craftserveradiation.ad";
     private static final String TEXT = ChatColor.GREEN + "Polecamy korzystanie z hostingu " +
@@ -50,7 +50,7 @@ public void enable() {
         try {
             shouldAdvertise = this.shouldAdvertise();
         } catch (UnknownHostException e) {
-            logger.error("Could not resolve local host.", e);
+            logger.log(Level.SEVERE, "Could not resolve local host.", e);
             shouldAdvertise = true;
         }
 

src/main/java/pl/craftserve/radiation/LugolsIodinePotion.java

@@ -40,8 +40,6 @@
 import org.bukkit.plugin.Plugin;
 import org.bukkit.potion.PotionData;
 import org.bukkit.potion.PotionType;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import pl.craftserve.radiation.nms.RadiationNmsBridge;
 
 import java.text.MessageFormat;
@@ -54,9 +52,11 @@
 import java.util.StringJoiner;
 import java.util.concurrent.TimeUnit;
 import java.util.function.Predicate;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 public class LugolsIodinePotion implements Listener, Predicate<ItemStack> {
-    static final Logger logger = LoggerFactory.getLogger(LugolsIodinePotion.class);
+    static final Logger logger = Logger.getLogger(LugolsIodinePotion.class.getName());
 
     private static final byte TRUE = 1;
 
@@ -147,7 +147,7 @@ private void broadcastConsumption(Player player, int durationSeconds) {
         Objects.requireNonNull(player, "player");
 
         String name = this.config.name();
-        logger.info("{} has consumed {} with a duration of {} seconds", player.getName(), name, durationSeconds);
+        logger.info(player.getName() + " has consumed " + name + " with a duration of " + durationSeconds + " seconds");
 
         this.config.drinkMessage().ifPresent(rawMessage -> {
             String message = ChatColor.RED + MessageFormat.format(rawMessage, player.getDisplayName() + ChatColor.RESET, name);

src/main/java/pl/craftserve/radiation/MetricsHandler.java

@@ -23,16 +23,16 @@
 import org.bukkit.event.HandlerList;
 import org.bukkit.event.Listener;
 import org.bukkit.plugin.Plugin;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import pl.craftserve.metrics.pluginmetricslite.MetricSubmitEvent;
 import pl.craftserve.metrics.pluginmetricslite.MetricsLite;
 
 import java.util.Map;
 import java.util.Objects;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 public class MetricsHandler implements Listener  {
-    static final Logger logger = LoggerFactory.getLogger(MetricsHandler.class);
+    static final Logger logger = Logger.getLogger(MetricsHandler.class.getName());
 
     private final Plugin plugin;
     private final Server server;
@@ -54,15 +54,15 @@ public void start() {
         try {
             MetricsLite.start(this.plugin);
         } catch (Throwable throwable) {
-            logger.error("Could not start metrics.", throwable);
+            logger.log(Level.SEVERE, "Could not start metrics.", throwable);
         }
     }
 
     public void stop() {
         try {
             MetricsLite.stopIfRunning(this.plugin);
         } catch (Throwable throwable) {
-            logger.error("Could not stop metrics.", throwable);
+            logger.log(Level.SEVERE, "Could not stop metrics.", throwable);
         }
 
         HandlerList.unregisterAll(this);

src/main/java/pl/craftserve/radiation/Radiation.java

@@ -42,8 +42,6 @@
 import org.bukkit.potion.PotionEffect;
 import org.bukkit.potion.PotionEffectType;
 import org.bukkit.scheduler.BukkitRunnable;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import java.text.MessageFormat;
 import java.util.ArrayList;
@@ -56,9 +54,11 @@
 import java.util.Set;
 import java.util.UUID;
 import java.util.function.Predicate;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 public class Radiation implements Listener {
-    static final Logger logger = LoggerFactory.getLogger(Radiation.class);
+    static final Logger logger = Logger.getLogger(Radiation.class.getName());
 
     private final Set<UUID> affectedPlayers = new HashSet<>(128);
 
@@ -124,7 +124,7 @@ private void broadcastEscape(Player player) {
         Objects.requireNonNull(player, "player");
 
         String radiationId = this.config.id();
-        logger.info("{} has escaped to \"{}\" radiation zone at {}", player.getName(), radiationId, player.getLocation());
+        logger.info(player.getName() + " has escaped to \"" + radiationId + "\" radiation zone at " + player.getLocation());
 
         this.config.enterMessage().ifPresent(rawMessage -> {
             String message = ChatColor.RED + MessageFormat.format(rawMessage, player.getDisplayName() + ChatColor.RESET, radiationId);

src/main/java/pl/craftserve/radiation/RadiationPlugin.java

@@ -37,8 +37,6 @@
 import org.bukkit.configuration.MemoryConfiguration;
 import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.plugin.java.JavaPlugin;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import pl.craftserve.radiation.nms.RadiationNmsBridge;
 import pl.craftserve.radiation.nms.V1_14ToV1_15NmsBridge;
 
@@ -53,9 +51,11 @@
 import java.util.TreeSet;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 public final class RadiationPlugin extends JavaPlugin {
-    static final Logger logger = LoggerFactory.getLogger(RadiationPlugin.class);
+    static final Logger logger = Logger.getLogger(RadiationPlugin.class.getName());
 
     private static final char COLOR_CODE = '&';
 
@@ -83,7 +83,7 @@ public static String colorize(String input) {
 
     private RadiationNmsBridge initializeNmsBridge() {
         String serverVersion = RadiationNmsBridge.getServerVersion(this.getServer());
-        logger.info("Detected server version: {}", serverVersion);
+        logger.info("Detected server version: " + serverVersion);
 
         switch (serverVersion) {
             case "v1_14_R1":
@@ -114,7 +114,7 @@ public void onEnable() {
         try {
             this.radiationNmsBridge = this.initializeNmsBridge();
         } catch (Exception e) {
-            logger.error("Failed to launch {}. Plausibly your server version is unsupported.", this.getName(), e);
+            logger.log(Level.SEVERE, "Failed to launch " + this.getName() + ". Plausibly your server version is unsupported.", e);
             this.setEnabled(false);
             return;
         }
@@ -132,7 +132,7 @@ public void onEnable() {
         try {
             this.config = new Config(config);
         } catch (InvalidConfigurationException e) {
-            logger.error("Could not load configuration file.", e);
+            logger.log(Level.SEVERE, "Could not load configuration file.", e);
             this.setEnabled(false);
             return;
         }
@@ -166,7 +166,7 @@ public void onEnable() {
 
         Set<String> radiationIds = new TreeSet<>(Comparator.naturalOrder());
         radiationIds.addAll(this.activeRadiations.keySet());
-        logger.info("Loaded and enabled {} radiation(s): {}", this.activeRadiations.size(), String.join(", ", radiationIds));
+        logger.info("Loaded and enabled " + this.activeRadiations.size() + " radiation(s): " + String.join(", ", radiationIds));
 
         this.craftserveListener.enable();
         this.metricsHandler.start();
@@ -242,7 +242,7 @@ private boolean migrate(ConfigurationSection section, int protocol) {
         Objects.requireNonNull(section, "section");
 
         if (protocol > CURRENT_PROTOCOL_VERSION) {
-            logger.error("Your configuration file's protocol version \"{}\" is invalid. Are you trying to load it using a newer version of the plugin?", protocol);
+            logger.severe("Your configuration file's protocol version \"" + protocol + "\" is invalid. Are you trying to load it using a newer version of the plugin?");
             return false;
         }
 
@@ -279,7 +279,7 @@ private boolean migrate(ConfigurationSection section, int protocol) {
             AtomicBoolean logged = new AtomicBoolean();
             section.getStringList("world-names").forEach(worldName -> {
                 if (logged.compareAndSet(false, true)) {
-                    logger.warn(
+                    logger.warning(
                             "Enabling in legacy region-name mode! The plugin will try to automatically migrate to the new flag-based system.\n" +
                             "If everything went fine please completely remove your config.yml file.");
                 }
@@ -336,7 +336,7 @@ private void migrateFromRegionId(String worldName, String regionId) {
 
         World world = this.getServer().getWorld(worldName);
         if (world == null) {
-            logger.warn(error + "the world is unloaded.");
+            logger.warning(error + "the world is unloaded.");
             return;
         }
 
@@ -346,19 +346,19 @@ private void migrateFromRegionId(String worldName, String regionId) {
 
         RegionContainer regionContainer = matcher.getRegionContainer();
         if (regionContainer == null) {
-            logger.warn(error + "region container is not present.");
+            logger.warning(error + "region container is not present.");
             return;
         }
 
         RegionManager regionManager = regionContainer.get(BukkitAdapter.adapt(world));
         if (regionManager == null) {
-            logger.warn(error + "region manager for the world is not present.");
+            logger.warning(error + "region manager for the world is not present.");
             return;
         }
 
         ProtectedRegion legacyRegion = regionManager.getRegion(regionId);
         if (legacyRegion == null) {
-            logger.warn(error + "legacy region is not present.");
+            logger.warning(error + "legacy region is not present.");
             return;
         }
 
@@ -372,7 +372,7 @@ private void migrateFromRegionId(String worldName, String regionId) {
         }
 
         global.setFlag(this.radiationFlag, true);
-        logger.info("Region {} in world {} has been successfully migrated to the new flag-based system.", regionId, worldName);
+        logger.info("Region " + regionId + " in world " + worldName + " has been successfully migrated to the new flag-based system.");
     }
 
     //

src/main/java/pl/craftserve/radiation/nms/V1_14ToV1_15NmsBridge.java

@@ -18,17 +18,17 @@
 
 import org.bukkit.Material;
 import org.bukkit.NamespacedKey;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import pl.craftserve.radiation.LugolsIodinePotion;
 
 import java.lang.reflect.Array;
 import java.lang.reflect.Method;
 import java.text.MessageFormat;
 import java.util.Objects;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 public class V1_14ToV1_15NmsBridge implements RadiationNmsBridge {
-    static final Logger logger = LoggerFactory.getLogger(V1_14ToV1_15NmsBridge.class);
+    static final Logger logger = Logger.getLogger(V1_14ToV1_15NmsBridge.class.getName());
 
     private final Class<?> itemClass;
     private final Class<?> iRegistryClass;