[1.12.2] @SubscribeEvent seems to be not working as it in 1.7.10 did

[nocot]

Hello,

I've just finally started to update my mod for 1.12.2 and I found my first problem. In 1.7.10 to make an event, I had to register event class (still not sure if it was completly nessesary)

@EventHandler
public static void Init(FMLInitializationEvent event)
{
  MinecraftForge.EVENT_BUS.register(test.class);
}

And subscribe an event in my class

public class test {
	@SubscribeEvent
	 public void onPlayerTick(PlayerTickEvent evt) {
        System.out.println("OK");
    }
}

And it used to work - at this example, printed "OK" to a console every tick. But when it comes to 1.12.2, for unknown for me reason, it is not possible anymore. It just shows nothing. Is there a simple solution for it? My actual goal is currently to print "OK" every tick.

 

Woops, my bad. Changed a title.

Edited July 3, 2018 by nocot

[V0idWa1k3r]

If you are registering a class object to the event bus your event handers must be static. To have non-static handlers working register an instance of your class to the bus.

https://mcforge.readthedocs.io/en/latest/events/intro/#static-event-handlers

[nocot]

Oh right, I made a mistake. It got to work now, thank you.

[jredfox]

  On 7/3/2018 at 7:20 PM, nocot said:

Oh right, I made a mistake. It got to work now, thank you.

Expand  

no static if you use subscribe event the new option has to be static but, the old has to be object oriented(no static). Also if you use the old you have to create an object from it not registering the class

Edited July 4, 2018 by jredfox

[larsgerrits]

  On 7/4/2018 at 7:07 AM, jredfox said:

no static if you use subscribe event the new option has to be static but, the old has to be object oriented(no static). Also if you use the old you have to create an object from it not registering the class

Expand  

Uuhhh.... what?

Look, you probably have the best intentions, but please, work on your sentences. Their impossible to decipher.

[jredfox]

  On 7/4/2018 at 1:20 PM, larsgerrits said:

Uuhhh.... what?

Look, you probably have the best intentions, but please, work on your sentences. Their impossible to decipher.

Expand  

you register you said class rather then an object. Take a look here at my code:

MinecraftForge.EVENT_BUS.register(new com.EvilNotch.lanessentials.events.EventHandler());

In that class we created a new object so we must keep the @SubsribeEvent methods object methods meaning no static in the method name.

 

To learn more about java objects methods either use youtube or take a class on computer science. This has nothing to do with forge. Basically static can be called from everywhere and object methods are used object.method() style. The benefit being that you can use the objects own variables in said method to do the said function of that method.
 

@SubscribeEvent
public void skinNo(PlayerLoggedOutEvent e)
{
   	System.out.println("event player logout fired"); 
}

if you do it the new way the methods need to be static and I am unsure if you register it with an object or class.

Edited July 4, 2018 by jredfox

[DaemonUmbra]

  On 7/4/2018 at 2:25 PM, jredfox said:

if you do it the new way the methods need to be static and I am unsure if you register it with an object or class.

Expand  

Read the docs before saying anything, please.

nocot is clearly trying to go the static eventhandler route and you are saying this method is wrong despite admitting not having done your research.

 

Edit: Thread locked because OP has stated that their issue has been solved.

Edited July 4, 2018 by DaemonUmbra
Softened my response