2.2.048 hotfix for spear manager exception Fixes #5237

Author: nossr50

Changelog.txt

@@ -1,3 +1,6 @@
+Version 2.2.048
+    Fixed error when loading Spears skill manager on older Minecraft versions
+
 Version 2.2.047
     Fix bug where off-hand spears damage could be attributed to various combat skills and trigger their abilities
 

pom.xml

@@ -4,7 +4,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>com.gmail.nossr50.mcMMO</groupId>
     <artifactId>mcMMO</artifactId>
-    <version>2.2.047</version>
+    <version>2.2.048</version>
     <name>mcMMO</name>
     <url>https://github.com/mcMMO-Dev/mcMMO</url>
     <scm>

src/main/java/com/gmail/nossr50/datatypes/player/McMMOPlayer.java

@@ -181,7 +181,6 @@ private void initSkillManagers() {
         }
     }
 
-    //TODO: Add test
     private void initManager(PrimarySkillType primarySkillType) throws InvalidSkillException {
         final var version = mcMMO.getCompatibilityManager().getMinecraftGameVersion();
 
@@ -203,20 +202,12 @@ private void initManager(PrimarySkillType primarySkillType) throws InvalidSkillE
             case TRIDENTS -> new TridentsManager(this);
             case UNARMED -> new UnarmedManager(this);
             case WOODCUTTING -> new WoodcuttingManager(this);
-
-            case MACES -> version.isAtLeast(1, 21, 0)
-                    ? new MacesManager(this)
-                    : null; // keep current behavior: no manager on older versions
-
-            case SPEARS -> version.isAtLeast(1, 21, 11)
-                    ? new SpearsManager(this)
-                    : null; // same here
+            case MACES -> version.isAtLeast(1, 21, 0) ? new MacesManager(this) : null;
+            case SPEARS -> version.isAtLeast(1, 21, 11) ? new SpearsManager(this) : null;
         };
 
         if (manager != null) {
             skillManagers.put(primarySkillType, manager);
-        } else {
-            throw new InvalidSkillException("No valid skill manager for skill: " + primarySkillType);
         }
     }