You should add Static.assertPlayerNonNull here, so that it will throw a CRE instead of a NPE on line 721, if this is run from console. Many functions don't do this though, so I'm going to have to go back and add it to several functions anyways (there's a unit test that covers this) so you don't need to worry about it just yet, but in the future, do keep this in mind.