Jump to content

Recommended Posts

Posted

I am working within a GeoArmorItem's class (from GeckoLib), attempting to add contextual animations to an armor, and now have found and understand the logic necessary to check the conditions and change animation accordingly. Just one problem: I subscribed all code that relies on the player object to onPlayerRendered, so I can make sure I'm not getting any unexpected errors from trying to use things that don't exist yet. (before doing this, MC would crash when loading the title menu). Now, this in itself isn't a problem, but onPlayerRendered is, as far as my testing has proven, never called, so my conditional sprinting boolean never is true.

How do I make onPlayerRendered work here, or am I doing this wrong?

full class: https://pastebin.com/jWpQHwmJ 

 

Posted
@EventBusSubscriber(modid = MODID) // Automatic registration
public class EventHandlingClass {

    @SubscribeEvent // method handles an event
    public static void handleAnEvent(SomeEvent event) {
        ...
    }
}

If you don't use @EventBusSubscriber and static methods, you have to explicitly register the class, instance or method.

https://docs.minecraftforge.net/en/latest/concepts/events/

  • Thanks 1

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Posted
Quote

If you don't use @EventBusSubscriber and static methods, you have to explicitly register the class, instance or method.

This works perfectly now after making the method (and its fields) static and adding the "automatic registration" line above the class.
I knew I was close to getting this to work but considering I was reading up on the event bus, this is kinda embarrassing that I didn't pick this up honestly. Thank you so much for the help!

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.