[1.13.2] Certain subscribed events not firing

[edma0]

Thanks in advance for pointing me in the right direction here!

 

What is happening?

I'm updating my mod from 1.10.2 to 1.13.2 and everything is working as expected except for the events.

Some are working perfectly and others are never fired.

I have commented the resulting behaviour in the below code.

I've likely missed something pretty obvious and would really appreciate any help.

 

How are you checking if the event is firing?

I'm running the debug configuration for both client only and server + client, placing breakpoints and stepping through the code.

 

What version of Forge are you working with?

forge-1.13.2-25.0.107-mdk

 

The code I am using to reproduce the issue:

 

First, in the main mod class, I register the EventsCommon class instance and call the proxy to register the others.

In the main mod class:

  Reveal hidden contents

private void init(FMLCommonSetupEvent event)
{
    MinecraftForge.EVENT_BUS.register(EventsCommon.instance);
    proxy.hookEvents();
}

 

 

EventsCommon:

  Reveal hidden contents

 

public class EventsCommon
{
    public static EventsCommon instance = new EventsCommon();

    @SubscribeEvent
    public void eventBlockDestroyed(BlockEvent.BreakEvent event)
    {
        // Works perfectly
    }

    @SubscribeEvent
    public void eventBlockPlaced(BlockEvent.PlaceEvent event)
    {
        // Never fired
    }
}

 

 

Then in the ClientProxy and ServerProxy hookEvents method, I register the EventsClient and EventsServer respectively.

CommonProxy:

  Reveal hidden contents

public class CommonProxy
{
    public void hookEvents() { }
}

 

 

ClientProxy:

  Reveal hidden contents

public class ClientProxy extends CommonProxy
{
    @Override
    public void hookEvents()
    {
        super.hookEvents();
        MinecraftForge.EVENT_BUS.register(EventsClient.instance);
    }
}

 

 

ServerProxy:

  Reveal hidden contents

public class ServerProxy extends CommonProxy
{
    @Override
    public void hookEvents()
    {
        super.hookEvents();
        MinecraftForge.EVENT_BUS.register(EventsServer.instance);
    }
}

 

 

 

Finally, the EventsClient and EventsServer classes. The EventsCommon class is posted above:

EventsClient:

  Reveal hidden contents

public class EventsClient
{
    public static EventsClient instance = new EventsClient();
	
    @SubscribeEvent
    public void eventrenderWorldLast(RenderWorldLastEvent event)
    {
        // Works perfectly
    }
}

 

 

EventsServer:

  Reveal hidden contents

public class EventsServer
{
    public static EventsServer instance = new EventsServer();
    
    @SubscribeEvent
    public void eventEntityJoining(EntityJoinWorldEvent event)
    {
        // Never fired
    }
}

 

 

Edited April 4, 2019 by edma0
Enclosed code blocks in spoiler tags to clean up post

[Cadiboo]

This is a lovely, comprehensive and well laid out post, regardless of code style. Personally I think you should just use the static event subscribing system as you can specify sides and not worry about registering the subscribers conditionally yourself (you can specify a Side in the annotation).

[edma0]

Thank you for your suggestion, I took your advice looked into the @Mod.EventBusSubscriber annotation.

That will make things very neat, I am glad I looked into it further, awesome tip.

 

I have implemented the change but I am still having no luck with BlockEvent.PlaceEvent.

It just isn't firing, but is registered in an identical fashion to BlockEvent.BreakEvent.

However BlockEvent.BreakEvent is working perfectly.

 

So I am singling out these two events as they are registered in the same way, but one works and the other does not.

Here is the code, in the Main mod class I have explicitly assigned the value and bus fields of the annotation to be on the safe side (rather than leaving them as default):

  Reveal hidden contents

@Mod.EventBusSubscriber(value={Dist.CLIENT, Dist.DEDICATED_SERVER}, bus=Mod.EventBusSubscriber.Bus.FORGE)
public static class CommonEvents
{
    @SubscribeEvent
    public static void onBlockBreak(final BlockEvent.BreakEvent event)
    {
        // Working perfectly
    }

    @SubscribeEvent
    public static void onBlockPlace(final BlockEvent.PlaceEvent event)
    {
        // Never fired
    }
}

 

 

[crackedEgg]

I can't find where BlockEvent.PlaceEvent is used in forge code. 

 

rg BlockEvent.PlaceEvent
minecraftforge/event/ForgeEventFactory.java
125:import net.minecraftforge.event.world.BlockEvent.PlaceEvent;

minecraftforge/common/ForgeHooks.java
635:            BlockEvent.PlaceEvent placeEvent = null;

 

Only two places in forge code that this shows up. That may be why its not firing.

 

[Cadiboo]

In my forge version (#100) it’s used in a method in ForgeEventFactory. For me method is called from vanilla code. I’ll see if I can reproduce this issue when I get home

[edma0]

Just updated to 25.0.108 to ensure the issue had not already been resolved.

Ah, yeah if the logic is similar to BlockEvent.BreakEvent then I think I see the issue.

 

There is a function with the following signature commented out with the task:

/* TODO: Talk to Sponge folk about World rollbacks.

@line 602 of net.minecraftforge.common.ForgeHooks.class

public static EnumActionResult onPlaceItemIntoWorld(@Nonnull ItemUseContext context)

 

In this function (that is now commented out) it seems the BlockEvent.PlaceEvent was generated and a call to ForgeEventFactory.onPlayerBlockPlaced event was made.

 

Edit:
Yeah it seems to be an open issue on the issue tracker, kicking myself for missing it.

https://github.com/MinecraftForge/MinecraftForge/issues/5476

Edited April 5, 2019 by edma0