Leaderboard

Seriously, Use the event not the static registries. Timing of registration is important. So stop doing it in random places. I was tempted to name the event 'RegisterYourStuffHereAndStopBreakingEverythingElseSeriouslyStopRegisteringYourCrapInRandomPlacesItBreaksThingsAndHoldsUsBackFromBeingAbleToDoCoolThingsThatWeveBeenWantingToDoForFourYearsNow' but I decided against that as it was a bit long

Thanks!. I am thinking about updating this to 1.10.2. I am also looking into making a Mac version.

RegisterShitHereAndStopBreakingStuffEvent would have worked just fine.

It needs to be RegistryEvent.Register not just the base class more info https://mcforge.readthedocs.io/en/latest/concepts/registries/

You have a typo.

You have copied code from someone else. Your CommonProxy class does not contain a preInit(FMLPreInitalizationEvent event) function, yet you are trying to call it. Of course there is no error in the CommonProxy class, this function doesn't exist and there's no reason that your CommonProxy thinks it should. This is the same reasing calling Math.CalculatePie() throws an error. There is also no reason this method should exist either: anything you might do in your common proxy you can do in your main mod class. See Code Style Problem #2. item.getRegistryName(), you somehow inserted a comma instead of a period, because that function takes 2 parameters. Additionally, you should only be calling this code during the ModelRegistrationEvent.

If you own a domain, it'd be [tld].[domain].[packagename] So, do not use com.[username] unless you actually own [username].com. If you don't own a domain, just use your username as the base package.