Animefan8888

Well since I am not the best when it comes to this, Natura has a github, and it is a mod you are using as a reference. Take this it is dangerous to go alone. https://github.com/progwml6/Natura/blob/master/src/main/java/mods/natura/dimension/NetheriteChunkProvider.java *edit It might be something as simple as checking if the y level is less than a number and set the block you want though.

If you didn't see the disregard, DimensionType doesn't exist in 1.8.9, but if you are talking about my last post you do need to unregister the provider because it doesn't override. DimensionManager.registerProvider()

Oh thank you Cool Alias I completely forgot what goes where when dealing with Events thanks for the refresher, now if you'll excuse me I've got some reading to refresh on.

Why are you still using 1.6.4? And Instead of MinecraftForge.EVENT_BUS.register(new EventHandlerClass) use FMLCommonHandler.instance().bus().register(new EventHandlerClass);

What Minecraft version are you using.

You should also unregister the ProviderType.

DimensionType.register(); I believe just use the same ID as the nether. Disregard this, unless I am wrong and that does exist in 1.8.9.

Now that I look at your code you are checking to see if the entity that died is an instance of EntityPlayer not entity Zombie. Sorry I read that post wrong. ANd to check if it was the player that killed it, use the damagesource object.

In your initialization method you need to call MinecraftForge.EVENT_BUS.register(new EventHandlerClass());

You also need to override the dimension type for the Nether.

Have you tried to do any debugging and have you looked into unregistering a vanilla dimension cause I'm not sure if that is possible as it would cause a huge problem. Plus DimensionManager.registerDimension() is only ever called in DimensionMessageHandler in the forge source. Not sure when the packet is sent though.

GameRegistry.registerFuelHandler(YourFuelHandler);

No it can be accessed on the server, I believe the reason for this is because they use the gameprofile(UUID from the profile)of the player to save nbt.

Happy modding to you, and I'm glad I could help. write Solved in the topic subject.

All entities need to be registered with the EntityRegistry, but no instead of public static Entity altarlisttier1(ItemStack item) { for (Entry<ItemStack, Entity> entry : altarlisttier1.entrySet()) { if (ItemStack.areItemsEqual(item, (ItemStack)entry.getKey())) { return entry.getValue(); } else { return null; } } return null; } Some thing like public static Entity getEntityFromItemStackInAltar(World world, ItemStack stack) { // Only needs either instanceof or == if (stack != null && (stack.getItem() instanceof ItemClass || stack.getItem() == Items.diamond)) { return new Entity(world); } }

Also it seems when you create that single instance, thank you for pointing that out Choonster, you are passing in a null parameter for world.

New Crash Report please.

Is your Void Beast a hostile entity, if so you might be better to extend something that doesn't extend EntityAgeable

For one you are spawning the entity even if it is null. EntityLiving entityliving = spawn instanceof EntityLiving ? (EntityLiving)spawn : null; spawn.setLocationAndAngles(pos.getX(), pos.getY()+1.25f, pos.getZ(), worldObj.rand.nextFloat() * 360.0F, 0.0F); AnvilChunkLoader.spawnEntity(spawn, worldObj);

Include the whole Entity class please.

Where is the GameProfile initialized? How is it initialized? And After looking at the crash report, it may be mo villages, unless this is happening in dev environment.

I don't think the Village has the players UUID saved and is throwing that error. And this doesn't look like it is writing anything, it looks like it is reading NBTTagList nbttaglist1 = compound.getTagList("Players", 10); for (int j = 0; j < nbttaglist1.tagCount(); ++j) { NBTTagCompound nbttagcompound1 = nbttaglist1.getCompoundTagAt(j); if (nbttagcompound1.hasKey("UUID") && this.worldObj != null && this.worldObj.getMinecraftServer() != null) { PlayerProfileCache playerprofilecache = this.worldObj.getMinecraftServer().getPlayerProfileCache(); GameProfile gameprofile = playerprofilecache.getProfileByUUID(UUID.fromString(nbttagcompound1.getString("UUID"))); if (gameprofile != null) { this.playerReputation.put(gameprofile.getName(), Integer.valueOf(nbttagcompound1.getInteger("S"))); } } else { this.playerReputation.put(nbttagcompound1.getString("Name"), Integer.valueOf(nbttagcompound1.getInteger("S"))); } }

Taken Directly from Game Profile, you should insert not equal to null checks. /** * Gets the unique ID of this game profile. * <p /> * This may be null for partial profile data if constructed manually. * * @return ID of the profile */ public UUID getId() { return id; } /** * Gets the display name of this game profile. * <p /> * This may be null for partial profile data if constructed manually. * * @return Name of the profile */ public String getName() { return name; } Which line is line 39 in your server tick event btw.

Why are you calling this every tick? Are you making sure there is a player? Is there another Phase? Wouldn't it be easier to use Player Tick? And if you don't mind me asking why are you writing a village to a players nbt?

So obviously both the UUID and Name of the player seems to not exist, could it be that you are calling CreateVillageNBT on the client side only or server side only. Where is CreateVillageNBT called.