VikeStep/sprinkles_for_vanilla

ExplosionHandler NullPointerException when explosion.exploder is null

Closed this issue · 3 comments

Choonster commented

sprinkles_for_vanilla: 1.2.2 (offending code is still in 1.2.6)
FML log: http://paste.atlauncher.com/view/ea8fd3f3 (crash report starts on line 408)

When explosion.exploder is null in ExplosionHandler#onExplosionStart (line 59) with global -> explosions -> explosionLogging set to true, EntityList.getEntityString throws a NullPointerException.

Originally reported here.

VikeStep commented

I'll upload this update shortly. May I suggest disabling explosion logging by default. When you get to things like the dragon or wither it will become incredibly spammy. The only reason I included the explosion logging was for if you wished to add your own explosion rules and wished to know the name of the explosion to use.

Choonster commented

Thanks for the quick response.

Explosion logging was disabled by default in the pack, so the user must have enabled it themselves.

VikeStep commented

What could've also happened is that the server had it enabled. My mod has config syncing so it will use the server's values when possible.