fix: skyblock level mission bugs

Took 16 minutes

Author: Swofty-Developments

type.generic/src/main/java/net/swofty/types/generic/SkyBlockGenericLoader.java

@@ -390,11 +390,6 @@ public void initialize(MinecraftServer server) {
          */
         BlockAttribute.registerBlockAttributes();
 
-        /**
-         * Register SkyBlock levels
-         */
-        SkyBlockLevelCause.initializeCauses();
-
         /**
          * Register event
          */
@@ -522,6 +517,11 @@ public void initialize(MinecraftServer server) {
             }
         });
 
+        /**
+         * Register SkyBlock levels
+         */
+        SkyBlockLevelCause.initializeCauses();
+
         /**
          * Handle ConnectionManager
          */

type.generic/src/main/java/net/swofty/types/generic/data/datapoints/DatapointSkyBlockExperience.java

@@ -2,6 +2,7 @@
 
 import lombok.Getter;
 import lombok.NoArgsConstructor;
+import lombok.NonNull;
 import lombok.Setter;
 import net.swofty.commons.protocol.Serializer;
 import net.swofty.types.generic.data.Datapoint;
@@ -128,7 +129,7 @@ public boolean hasExperienceFor(SkyBlockLevelCauseAbstr cause) {
          *
          * @param cause the {@link SkyBlockLevelCauseAbstr} to be added.
          */
-        public void addExperience(SkyBlockLevelCauseAbstr cause) {
+        public void addExperience(@NonNull SkyBlockLevelCauseAbstr cause) {
             if (completedExperienceCauses.contains(cause)) return;
             String causeKey = SkyBlockLevelCause.getKey(cause);
             if (completedExperienceCauses.stream().anyMatch((c) -> SkyBlockLevelCause.getKey(c).equals(causeKey))) return;

type.generic/src/main/java/net/swofty/types/generic/levels/SkyBlockLevelCause.java

@@ -1,5 +1,6 @@
 package net.swofty.types.generic.levels;
 
+import lombok.NonNull;
 import lombok.SneakyThrows;
 import net.swofty.commons.item.ItemType;
 import net.swofty.types.generic.collection.CollectionCategories;
@@ -22,7 +23,6 @@
 public class SkyBlockLevelCause {
     private static final Map<String, SkyBlockLevelCauseAbstr> CAUSES = new HashMap<>();
 
-    @SneakyThrows
     public static void initializeCauses() {
         // Register all Skill causes
         for (SkillCategories category : SkillCategories.values()) {
@@ -96,8 +96,8 @@ public static FairySoulExchangeLevelCause getFairySoulExchangeCause(int exchange
     }
 
     public static MissionLevelCause getMissionCause(String missionKey) {
-        for (SkyBlockLevelCauseAbstr cause : CAUSES.values()) {
-            if (cause instanceof MissionLevelCause missionCause) {
+        for (String key : CAUSES.keySet()) {
+            if (CAUSES.get(key) instanceof MissionLevelCause missionCause) {
                 if (missionCause.getMissionKey().equals(missionKey)) {
                     return missionCause;
                 }
@@ -169,7 +169,7 @@ public static SkillLevelCause getSkillCause(SkillCategories category, int level)
         return CAUSES.get(key);
     }
 
-    public static String getKey(SkyBlockLevelCauseAbstr cause) {
+    public static String getKey(@NonNull SkyBlockLevelCauseAbstr cause) {
         for (Map.Entry<String, SkyBlockLevelCauseAbstr> entry : CAUSES.entrySet()) {
             if (entry.getValue() == cause) {
                 return entry.getKey();

type.generic/src/main/java/net/swofty/types/generic/mission/MissionData.java

@@ -4,6 +4,7 @@
 import lombok.Setter;
 import lombok.SneakyThrows;
 import net.swofty.types.generic.calendar.SkyBlockCalendar;
+import net.swofty.types.generic.levels.SkyBlockLevelCause;
 import net.swofty.types.generic.region.RegionType;
 import net.swofty.types.generic.user.SkyBlockPlayer;
 import org.jetbrains.annotations.Nullable;
@@ -93,6 +94,7 @@ public void endMission(String missionID) {
 
         ActiveMission activeMission = activeMissions.stream().filter(mission -> mission.getMissionID().equals(missionID)).findFirst().get();
         getMissionFromCache(missionID).onEnd(getSkyBlockPlayer(), activeMission.getCustomData(), activeMission);
+        getSkyBlockPlayer().getSkyBlockExperience().addExperience(SkyBlockLevelCause.getMissionCause(activeMission.getMissionID()));
         activeMission.setMissionEnded((int) SkyBlockCalendar.getElapsed());
         activeMissions.remove(activeMission);
         completedMissions.add(activeMission);

type.generic/src/main/java/net/swofty/types/generic/mission/missions/blacksmith/MissionMineCoal.java

@@ -22,7 +22,7 @@ public void onBlockBreak(CustomBlockBreakEvent event) {
             return;
         }
 
-        if (event.getMaterial().equals(Material.COAL)) {
+        if (event.getMaterial().equals(Material.COAL_ORE)) {
             MissionData.ActiveMission mission = data.getMission(this.getClass()).getKey();
             mission.setMissionProgress(mission.getMissionProgress() + 1);
             mission.checkIfMissionEnded(event.getPlayer());
@@ -48,7 +48,6 @@ public Map<String, Object> onStart(SkyBlockPlayer player, MissionData.ActiveMiss
     public void onEnd(SkyBlockPlayer player, Map<String, Object> customData, MissionData.ActiveMission mission) {
         mission.getObjectiveCompleteText(new ArrayList<>(List.of("§3100 §7Mining Experience", "§b5 SkyBlock XP"))).forEach(player::sendMessage);
         player.getSkills().increase(player, SkillCategories.MINING, 100D);
-        player.getSkyBlockExperience().addExperience(SkyBlockLevelCause.getMissionCause(getID()));
         player.getMissionData().startMission(MissionTalkToBlacksmithAgain.class);
     }