Add better defaults than null

Author: Pablete1234

src/main/java/in/twizmwaz/cardinal/Cardinal.java

@@ -183,7 +183,7 @@ public static MatchThread getMatchThread(@NonNull CommandSender who) {
         return matchThread;
       }
     }
-    return null;
+    throw new IllegalStateException("Player is not in a match thread");
   }
 
   /**

src/main/java/in/twizmwaz/cardinal/module/spawn/SpawnModule.java

@@ -42,6 +42,7 @@
 import in.twizmwaz.cardinal.module.kit.KitModule;
 import in.twizmwaz.cardinal.module.region.Region;
 import in.twizmwaz.cardinal.module.region.RegionModule;
+import in.twizmwaz.cardinal.module.region.type.PointRegion;
 import in.twizmwaz.cardinal.module.region.type.modifications.PointProviderRegion;
 import in.twizmwaz.cardinal.module.team.Team;
 import in.twizmwaz.cardinal.module.team.TeamModule;
@@ -301,8 +302,9 @@ private Spawn getDefaultSpawn(@NonNull Match match) {
         return spawn;
       }
     }
-    // This should never happen as the match will not load without a default spawn.
-    return null;
+    // This should never happen as the match will not load without a default spawn. But just in case, use world spawn.
+    return new Spawn(true, null, false, false, false, false, false,
+        Lists.newArrayList(new PointRegion(match, match.getWorld().getSpawnLocation())));
   }
 
   /**