[1.8][Solved] Event Handlers, adding chicken tempt item

[Cleverpanda]

I'm trying to get when a chicken is spawned and add an AI task to it so it will accept my item as a tempt item, but I'm having issues with the event handler I think. I've regestered it below:  It's a MinecraftForgeevent, I used the MinecraftForge bus, It's called server side; I added a check. But i'm not getting any console output from my code.... What am I doing wrong?

 

@EventHandler
    public void Init(FMLInitializationEvent event)
    {
    	if(event.getSide() == Side.SERVER)
    	{
    		System.out.println("Test-Server side");
                        MinecraftForge.EVENT_BUS.register(ChickenFollowHandler.class);
    	}

 

	@SubscribeEvent
public void onEntitySpawn(EntityJoinWorldEvent event) {
	System.out.println("something Spawned!");
	 if(event.entity instanceof EntityChicken){

		 EntityChicken chicken = (EntityChicken)event.entity;
		 chicken.tasks.addTask(8, new EntityAITempt(chicken, 1.0D, Corn.kernels, false));
	 }
}
}

[Draco18s]

You're passing a class reference for your event handler, the Forge event bus doesn't accept static references (until build 2014, and even then your handler method isn't static).  You need to pass

new ChickenFollowHandler()

instead.

[Cleverpanda]

hmm.. I did that and nothing is changed. I wasn't able to get any other info to help debug either.

[Choonster]

Were you testing this with the integrated (single-player) server or the dedicated server?

 

FMLStateEvent#getSide

returns the physical side that the code is running on, i.e. the client or dedicated server. This isn't the same as the logical side, i.e. the client thread or server thread. This page explains sides in more detail.

 

You almost never want to use

FMLStateEvent#getSide

. Anything that depends on the physical side should be in the appropriate proxy class, anything that depends on the logical side should check

World#isRemote

.

[Cleverpanda]

Thanks guys, the combination of both of those fixed the issue, ans I was pleasantly surprised when the chicken code worked as well.

 

I moved that print statement into the check for a chicken and it is executing twice per spawn, is this once for server and once for client?

Would this mean I should only execute the add AI task for the server to aid performance?

[coolAlias]

Yes, that's exactly right - AI tasks typically (always? I can't think of any counterexamples) only process on the server, so only add it there.