Code cleanup

Author: tinyoverflow

src/main/java/me/tinyoverflow/privatestatus/AddressRepository.java

@@ -2,9 +2,7 @@
 
 import me.tinyoverflow.privatestatus.events.AddAddressEvent;
 import org.bukkit.OfflinePlayer;
-import org.jetbrains.annotations.NotNull;
 
-import javax.sound.sampled.Line;
 import java.net.InetAddress;
 import java.net.UnknownHostException;
 import java.time.LocalDateTime;
@@ -28,35 +26,6 @@ public AddressRepository(Logger logger)
         this.zoneOffset = OffsetDateTime.now().getOffset();
     }
 
-    /**
-     * Loads data from a map consisting of the UUID string as the key and the base64 encoded address as the value.
-     *
-     * @param configMap Map of key => value pairs from the configuration file.
-     */
-    public void fromMap(@NotNull Map<String, Object> configMap)
-    {
-        // Make sure we work with a fresh map.
-        storage.clear();
-
-        // Process each and every item inside the section.
-        for (Map.Entry<String, Object> entry : configMap.entrySet()) {
-            String address = entry.getKey();
-            LocalDateTime expiration = LocalDateTime.ofEpochSecond((Long) entry.getValue(), 0, zoneOffset);
-
-            // Decode the stored address and add it to the repository.
-            try {
-                InetAddress inetAddress = InetAddress.getByName(address);
-                storage.put(inetAddress, expiration);
-            }
-            catch (UnknownHostException e) {
-                logger.warning("Unknown host found in config. Skipping: " + address);
-            }
-        }
-
-        // Notify the user about the loading state.
-        logger.info("Loaded " + storage.size() + " addresses from configuration.");
-    }
-
     /**
      * Loads known addresses from a {@code List<Map<String, Object>>}.
      *
@@ -87,25 +56,6 @@ public void fromList(List<Map<String, Object>> data)
         logger.info("Loaded " + storage.size() + " addresses from configuration.");
     }
 
-    /**
-     * Returns the data as a string map.
-     *
-     * @return The map with the UUID string as the key and the base64 encoded address as the value.
-     */
-    public Map<String, Long> toMap()
-    {
-        Map<String, Long> addressList = new HashMap<>();
-
-        for (Map.Entry<InetAddress, LocalDateTime> entry : storage.entrySet()) {
-            InetAddress inetAddress = entry.getKey();
-            LocalDateTime localDateTime = entry.getValue();
-
-            addressList.put(inetAddress.getHostAddress(), localDateTime.toEpochSecond(zoneOffset));
-        }
-
-        return addressList;
-    }
-
     public List<Map<String, Object>> toList()
     {
         List<Map<String, Object>> addressList = new ArrayList<>();
@@ -148,6 +98,7 @@ public void add(OfflinePlayer player, InetAddress inetAddress, LocalDateTime exp
         }
     }
 
+    @SuppressWarnings("unchecked")
     public Map<InetAddress, LocalDateTime> getAll()
     {
         return (Map<InetAddress, LocalDateTime>) storage.clone();

src/main/java/me/tinyoverflow/privatestatus/PrivateStatus.java

@@ -2,7 +2,6 @@
 
 import com.destroystokyo.paper.event.server.PaperServerListPingEvent;
 import me.tinyoverflow.privatestatus.jobs.PruneExpiredAddressesJob;
-import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
@@ -13,7 +12,10 @@
 import java.net.InetAddress;
 import java.net.InetSocketAddress;
 import java.time.LocalDateTime;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
 import java.util.logging.Level;
 
 public class PrivateStatus extends JavaPlugin implements Listener
@@ -45,6 +47,7 @@ public void onLoad()
     }
 
     @Override
+    @SuppressWarnings("unchecked")
     public void onEnable()
     {
         getServer().getPluginManager().registerEvents(this, this);
@@ -57,10 +60,10 @@ public void onEnable()
         // Load addresses from configuration file, if the section exists.
         List<?> data = getConfig().getList(CONFIG_KNOWN_ADDRESSES, new ArrayList<>());
         repository.fromList((List<Map<String, Object>>) data);
-//        ConfigurationSection configurationSection = getConfig().getConfigurationSection(CONFIG_KNOWN_ADDRESSES);
-//        if (configurationSection != null) {
-//            repository.fromMap(configurationSection.getValues(false));
-//        }
+        //        ConfigurationSection configurationSection = getConfig().getConfigurationSection(CONFIG_KNOWN_ADDRESSES);
+        //        if (configurationSection != null) {
+        //            repository.fromMap(configurationSection.getValues(false));
+        //        }
 
         // Initialize Metrics
         PrivateStatusMetrics metrics = new PrivateStatusMetrics(this, 19291);