Resolve 9 FindBugs bugs

Resolved:
 - team.getExpectedPlayers() is a collection of String, not Player
 - setConsoleWorld(String) might pass null into setConsoleWorld(World) which then calls world.getUID(). This is replaced with setting explicit null (the startup value).
 - (x3) SportBukkitUtil must be prepared to encounter Errors as well as Exceptions and ignore them too
 - (x3) Renaming Exception parameters to 'ignored' when they are ignored
 - Comment out unused player in unit test

Unresolved:
 - The fields of MatchParams and TeamInfo are never written by code
 - CombatListener is guaranteed to reference vapl, despite there being a nullcheck there