[1.8] Can't figure out why I have a null pointer

[Guest]

I am passing the World and EntityPlayer.playerLocation arguments from onItemRightClick() to CivilizationObjectFactory.createPlot() to instantiate a new Plot instance. When I call World.getChunkFromBlockCoords() while passing the BlockPos I got from onItemRightClick() as an argument, I get a null pointer exception. I don't see why the BlockPos variable would be null for any reason, please help.

 

java.lang.NullPointerException: Unexpected error
at net.minecraft.world.World.getChunkFromBlockCoords(World.java:349)
at deletethis.civilization.CivilizationObjectFactory.createPlot(CivilizationObjectFactory.java:26)
at deletethis.civilization.item.ItemTownBook.onItemRightClick(ItemTownBook.java:106)

 

https://github.com/deletethis-github/Civilization/blob/master/src/main/java/deletethis/civilization/CivilizationObjectFactory.java#L26

https://github.com/deletethis-github/Civilization/blob/master/src/main/java/deletethis/civilization/item/ItemTownBook.java#L106

[Choonster]

If you look at the usages of the

playerLocation

field you'll see it's only used for sleeping, it's not usually set to the player's current position.

 

To get an

Entity

's position as a

BlockPos

, use

Entity#getPosition

or the

BlockPos(Entity)

constructor.