Jump to content

[Solved]Proper custom forge events


FlameAtronach93

Recommended Posts

I'm back.

 

I can't find out what I'm doing wrong. This is the class I'm using:

 

 

 

import net.minecraft.block.Block;
import net.minecraft.client.renderer.RenderBlocks;
import net.minecraftforge.common.ForgeDirection;
import net.minecraftforge.event.Event;

public class RenderBlockEvent extends Event {
    
    final public RenderBlocks context;
    
    /**
     * Stores the block that is about to be rendered.
     */
    final public Block block;
    
    
    public RenderBlockEvent( RenderBlocks renderBlocks, Block block ) {
        context = renderBlocks;
        this.block = block;
    }
    
    
    /**
     * Event triggered before a block is rendered by the RenderBlocks class.
     * 
     * @author Derija
     */
    static public class Before extends RenderBlockEvent {
        
        public Before( RenderBlocks renderBlocks, Block block ) {
            super(renderBlocks, block);
        }
        
    }
}

 

 

There are a couple more nested classes like the one shown.

 

The exception is:

java.lang.NoSuchMethodException: com.wod.crystalpower.coremod.events.RenderBlockEvent$Before.<init>()

at java.lang.Class.getConstructor0(Unknown Source)

at java.lang.Class.getConstructor(Unknown Source)

at net.minecraftforge.event.EventBus.register(EventBus.java:69)

at net.minecraftforge.event.EventBus.register(EventBus.java:53)

at com.wod.crystalpower.client.ClientProxy.<init>(ClientProxy.java:72)

... more

Which is ultimately caused by registering an event listener for these events.

 

Now obviously Forge is trying to get the default constructor which I don't have. So I'm confused as to why forge events in general work since none of them provides a default constructor themselves except for the base Event class...

 

Example:

 

 

public class BiomeEvent extends Event
{
    public final BiomeGenBase biome;

    public BiomeEvent(BiomeGenBase biome)
    {
        this.biome = biome;
    }
    
    public static class CreateDecorator extends BiomeEvent
    {
        public final BiomeDecorator originalBiomeDecorator;
        public BiomeDecorator newBiomeDecorator;
        
        public CreateDecorator(BiomeGenBase biome, BiomeDecorator original)
        {
            super(biome);
            originalBiomeDecorator = original;
            newBiomeDecorator = original;
        }
    }
    // ... more derived events.

 

 

 

I don't think there is any event register. I could implement a default constructor that would assign dummy values to those finals, but if Forge doesn't have to, I'd like to know why...

 

What am I missing?

 

Sincerely,

 

Flame

 

 

SOLUTION

 

Solution was as simple as this: do not exclude the event from the class transformers. In my case, it looked like this previously:

@MCVersion("1.5.2")
@TransformerExclusions({ "com.wod.crystalpower.coremod." })
public class CPCoreMod implements IFMLLoadingPlugin {
    // method overrides ...
}

 

Since my events are in package com.wod.crystalpower.coremod.events and Forge uses those strings as parameter of String.beginsWith, this would exclude my events from the net.minecraftforge.transformers.EventTransformer which is used to create the default constructor Forge uses when registering events. The relevant line of code now looks like:

 

@TransformerExclusions({ "com.wod.crystalpower.coremod.CPCoreMod", "com.wod.crystalpower.coremod.asm." });

 

Credits for the solution of the problem go to LexManos. Thank you!

Link to comment
Share on other sites

Are you doing this in a coremod and are you excluding your event from the transformers?

Because that'll break events, Please refer to The EventTransformer

To see some of the magic behind out event system.

 

Both aspects considered. I am excluding the entire coremod from class transforming.

 

Edit: Wait, does that mean I'm not supposed to exclude my event from the transformers? That would explain everything. I'll try it out, thanks!

 

Edit 2: Purfectly worked! Thanks a lot! ;D

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.