[Solved]PopulateChunkEvent.Populate Never Fires In Overworld

[jredfox]

@SubscribeEvent
    public void populate(PopulateChunkEvent.Populate event){
		System.out.println("hie");
	}

@Mod.EventHandler
	public void init(FMLInitializationEvent e)
	{
		 MinecraftForge.EVENT_BUS.register(this);
	}

What am I doing wrong I read the code in the chunk provider it occurs 100 times but, I do not see any output if I remove the .populate it then works but, I need it to only fire for the populate for spawners there is a dungeon right next to my spawn with the seed of 2 and it's not printing.

 

Ok this was solved by changeing the event buss to terrain

Edited February 27, 2018 by jredfox

[jabelar]

Event handler methods need to be static.

[Draco18s]

19 minutes ago, jabelar said:

Event handler methods need to be static.

This is not true. This is only true if you register the class or use the annotation. If you register a class instance (such as jredfox has done) then making the methods static is wrong.

[jabelar]

Maybe it's because I'm viewing this on the phone, but it is not clear what instance his "this" refers to. Jredfox can you post your whole code?

[Draco18s]

1 hour ago, jabelar said:

Maybe it's because I'm viewing this on the phone, but it is not clear what instance his "this" refers to. Jredfox can you post your whole code?

The class instance that the code is in. Which considering he is getting logging messages when using the PopulateChunkEvent super-class, that part's fine.

[jredfox]

5 hours ago, Draco18s said:

The class instance that the code is in. Which considering he is getting logging messages when using the PopulateChunkEvent super-class, that part's fine.

Then why does this method never fire the chunks fired several dozen dungeons If I revert to just pupulate it works but, populate populate what I actually need it's never firing 

 

is it in the wrong bus or is this a bug?

Edited February 27, 2018 by jredfox

[jabelar]

Well it should be the terraingen bus that you register your handler too.

 

However, I just tried it out and you are right. I can get the PopulateChunkEvent to fire (a lot) but can't get the PopulateChunkEvent.Populate to fire at all -- I tested this simply by printing out a console statement in each case.

 

Weird. I think you can agree that the event is supposed to be fired according to the chunk generator code...

 

If I simply handle the PopulateChunkEvent and print out the event (toString) I get Pre and Post but no populate:

[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Pre@70579ed5
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Post@3d9125df
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Pre@499013c9
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Post@7bb934e4
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Pre@37d4bc70
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Post@3dfd888
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Pre@4a989d61
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Post@78b48ec2
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Pre@6a119a98
[20:07:43] [Server thread/INFO] [STDOUT]: [com.blogspot.jabelarminecraft.examplemod.TerrainGenEventHandler:onEvent:30]: Populate Eventnet.minecraftforge.event.terraingen.PopulateChunkEvent$Post@177714ca

 

Edited February 27, 2018 by jabelar

[jredfox]

4 minutes ago, jabelar said:

Well it should be the terraingen bus that you register your handler too.

 

However, I just tried it out and you are right. I can get the PopulateChunkEvent to fire (a lot) but can't get the PopulateChunkEvent.Populate to fire at all -- I tested this simply by printing out a console statement in each case.

 

Weird. I think you can agree that the event is supposed to be fired according to the chunk generator code...

What happens when you switched to terrain bus did it work?

[jredfox]

yes switching the event bus did make it fire wonder why nobody thought of that first I am noob

[jabelar]

3 minutes ago, jredfox said:

What happens when you switched to terrain bus did it work?

By the way I just edited above post to show my console output -- shows that pre and post events are getting handled, but not populate.

 

I just realized that Pre and Post are posted to the main event bus, but Populate is on terrain gen bus. Give me a second, but you can try too, I think that is the problem -- register on the terraingen bus.

 

[jredfox]

4 minutes ago, jabelar said:

By the way I just edited above post to show my console output -- shows that pre and post events are getting handled, but not populate.

 

I just realized that Pre and Post are posted to the main event bus, but Populate is on terrain gen bus. Give me a second, but you can try too, I think that is the problem -- register on the terraingen bus.

 

I got this part working could you help me on the other forum again the wither skeletons are not spawner just blank spawners with that code

[jabelar]

Okay, see you on the other thread shortly. I'm on a work call at the moment but will try...