Commented new changes

Author: IllusionTheDev

SkyblockCore-Common/src/main/java/me/illusion/skyblockcore/common/event/SkyblockEvent.java

@@ -1,5 +1,8 @@
 package me.illusion.skyblockcore.common.event;
 
+/**
+ * This class is a basic tag class for skyblock events. It is expected that all skyblock events extend this class.
+ */
 public abstract class SkyblockEvent {
 
 }

SkyblockCore-Common/src/main/java/me/illusion/skyblockcore/common/event/impl/SkyblockPlatformEnabledEvent.java

@@ -3,6 +3,9 @@
 import me.illusion.skyblockcore.common.event.SkyblockEvent;
 import me.illusion.skyblockcore.common.platform.SkyblockPlatform;
 
+/**
+ * This event is called when the SkyblockPlatform is enabled.
+ */
 public class SkyblockPlatformEnabledEvent extends SkyblockEvent {
 
     private final SkyblockPlatform platform;
@@ -11,6 +14,11 @@ public SkyblockPlatformEnabledEvent(SkyblockPlatform plugin) {
         this.platform = plugin;
     }
 
+    /**
+     * Gets the SkyblockPlatform that was enabled.
+     *
+     * @return The SkyblockPlatform that was enabled.
+     */
     public SkyblockPlatform getPlatform() {
         return platform;
     }

SkyblockCore-Common/src/main/java/me/illusion/skyblockcore/common/event/listener/SkyblockEventListener.java

@@ -2,8 +2,18 @@
 
 import me.illusion.skyblockcore.common.event.SkyblockEvent;
 
+/**
+ * This interface is responsible for listening to skyblock events.
+ *
+ * @param <T> The event type.
+ */
 public interface SkyblockEventListener<T extends SkyblockEvent> {
 
+    /**
+     * This method is called when a skyblock event is called.
+     *
+     * @param event The event that was called.
+     */
     void onEvent(T event);
 
 }

SkyblockCore-Common/src/main/java/me/illusion/skyblockcore/common/event/manager/SkyblockEventManager.java

@@ -3,10 +3,26 @@
 import me.illusion.skyblockcore.common.event.SkyblockEvent;
 import me.illusion.skyblockcore.common.event.listener.SkyblockEventListener;
 
+/**
+ * This interface is responsible for managing skyblock events and their listeners.
+ */
 public interface SkyblockEventManager {
 
+    /**
+     * Subscribes a listener to a specific event class.
+     *
+     * @param eventClass The event class to subscribe to.
+     * @param listener   The listener to subscribe.
+     * @param <T>        The event type.
+     */
     <T extends SkyblockEvent> void subscribe(Class<T> eventClass, SkyblockEventListener<T> listener);
 
+    /**
+     * Unsubscribes a listener from a specific event class.
+     *
+     * @param event The event class to unsubscribe from.
+     * @param <T>   The event type.
+     */
     <T extends SkyblockEvent> void callEvent(T event);
 
 }

SkyblockCore-Common/src/main/java/me/illusion/skyblockcore/common/event/manager/SkyblockEventManagerImpl.java

@@ -5,9 +5,12 @@
 import me.illusion.skyblockcore.common.event.SkyblockEvent;
 import me.illusion.skyblockcore.common.event.listener.SkyblockEventListener;
 
+/**
+ * This class is a basic implementation of the SkyblockEventManager interface.
+ */
 public class SkyblockEventManagerImpl implements SkyblockEventManager {
 
-    private final Set<SkyblockEventHandler<?>> handlers = ConcurrentHashMap.newKeySet();
+    private final Set<SkyblockEventHandler<?>> handlers = ConcurrentHashMap.newKeySet(); // This could be better
 
     @Override
     public <T extends SkyblockEvent> void subscribe(Class<T> eventClass, SkyblockEventListener<T> listener) {
@@ -23,11 +26,16 @@ public <T extends SkyblockEvent> void callEvent(T event) {
                 continue;
             }
 
-            SkyblockEventHandler<T> castedHandler = (SkyblockEventHandler<T>) handler;
+            SkyblockEventHandler<T> castedHandler = (SkyblockEventHandler<T>) handler; // icky
             castedHandler.accept(event);
         }
     }
 
+    /**
+     * This is an internal class that represents a subscribed event handler.
+     *
+     * @param <T>
+     */
     private static class SkyblockEventHandler<T extends SkyblockEvent> {
 
         private final Class<T> eventClass;

SkyblockCore-Server/src/main/java/me/illusion/skyblockcore/server/event/island/SkyblockIslandEvent.java

@@ -3,6 +3,9 @@
 import me.illusion.skyblockcore.common.event.SkyblockEvent;
 import me.illusion.skyblockcore.server.island.SkyblockIsland;
 
+/**
+ * This class is a basic tag class for skyblock island events. It is expected that all skyblock island events extend this class.
+ */
 public abstract class SkyblockIslandEvent extends SkyblockEvent {
 
     private final SkyblockIsland island;
@@ -11,6 +14,11 @@ public SkyblockIslandEvent(SkyblockIsland island) {
         this.island = island;
     }
 
+    /**
+     * Gets the SkyblockIsland that was involved in this event.
+     *
+     * @return The SkyblockIsland that was involved in this event.
+     */
     public SkyblockIsland getIsland() {
         return island;
     }

SkyblockCore-Server/src/main/java/me/illusion/skyblockcore/server/event/island/SkyblockIslandLoadEvent.java

@@ -2,6 +2,9 @@
 
 import me.illusion.skyblockcore.server.island.SkyblockIsland;
 
+/**
+ * This event is called when a SkyblockIsland is loaded.
+ */
 public class SkyblockIslandLoadEvent extends SkyblockIslandEvent {
 
     public SkyblockIslandLoadEvent(SkyblockIsland island) {

SkyblockCore-Server/src/main/java/me/illusion/skyblockcore/server/event/island/SkyblockIslandUnloadEvent.java

@@ -2,6 +2,9 @@
 
 import me.illusion.skyblockcore.server.island.SkyblockIsland;
 
+/**
+ * This event is called when a SkyblockIsland is unloaded.
+ */
 public class SkyblockIslandUnloadEvent extends SkyblockIslandEvent {
 
     public SkyblockIslandUnloadEvent(SkyblockIsland island) {

SkyblockCore-Server/src/main/java/me/illusion/skyblockcore/server/island/AbstractIslandManager.java

@@ -10,6 +10,9 @@
 import me.illusion.skyblockcore.common.platform.SkyblockPlatform;
 import me.illusion.skyblockcore.common.profile.SkyblockProfileCache;
 
+/**
+ * Abstract implementation of the SkyblockIslandManager
+ */
 public abstract class AbstractIslandManager implements SkyblockIslandManager {
 
     protected final Map<UUID, SkyblockIsland> loadedIslands = new ConcurrentHashMap<>();

SkyblockCore-Server/src/main/java/me/illusion/skyblockcore/server/util/SkyblockLocation.java

@@ -1,5 +1,8 @@
 package me.illusion.skyblockcore.server.util;
 
+/**
+ * Represents a multi-platform location. It is expected that each platform makes an adapter for this class.
+ */
 public class SkyblockLocation {
 
     private final String world;

SkyblockCore-Spigot/src/main/java/me/illusion/skyblockcore/spigot/SkyblockSpigotPlugin.java

@@ -101,11 +101,17 @@ private void finishLoading() {
 
     }
 
+    /**
+     * Registers the default skyblock networks
+     */
     private void registerNetworks() {
         networkRegistry.register(new ComplexSkyblockNetwork(this));
         networkRegistry.register(new SimpleSkyblockNetwork(this));
     }
 
+    /**
+     * Initializes the cosmos setup
+     */
     private void initCosmos() {
         if (cosmosSetup != null) {
             throw new IllegalStateException("Cosmos setup already initialized!");
@@ -118,6 +124,11 @@ private void initCosmos() {
         cosmosSetup = cosmosSetupFile.getSetup();
     }
 
+    /**
+     * Sets the profile cache
+     *
+     * @param profileCache The profile cache
+     */
     public void setProfileCache(SkyblockProfileCache profileCache) {
         this.profileCache = profileCache;
     }

SkyblockCore-Spigot/src/main/java/me/illusion/skyblockcore/spigot/network/SkyblockNetworkRegistryImpl.java

@@ -4,10 +4,21 @@
 import me.illusion.skyblockcore.server.network.AbstractSkyblockNetworkRegistry;
 import me.illusion.skyblockcore.spigot.SkyblockSpigotPlugin;
 import me.illusion.skyblockcore.spigot.utilities.config.BukkitConfigurationAdapter;
+import org.bukkit.Bukkit;
 
 public class SkyblockNetworkRegistryImpl extends AbstractSkyblockNetworkRegistry {
 
+    private final SkyblockSpigotPlugin plugin;
+
     public SkyblockNetworkRegistryImpl(SkyblockSpigotPlugin plugin) {
         super(plugin, BukkitConfigurationAdapter.adapt(new YMLBase(plugin, "network-settings.yml").getConfiguration()));
+        this.plugin = plugin;
+    }
+
+    @Override
+    protected void failToEnable(String name) {
+        super.failToEnable(name);
+
+        Bukkit.getPluginManager().disablePlugin(plugin);
     }
 }