Jump to content

Fire RegistryEvent.Register<IRecipe> after loading JSON recipes


cad97

Recommended Posts

If my debug-crawling through the startup code is accurate, the current order is something like (trimmed to relevant items):

Quote
  • beginMinecraftLoading
    • LoaderState.PREINITIALIZATION
    • GameData::fireRegistryEvents
      • RegistryEvent.Register<Block>
      • RegistryEvent.Register<Item>
      • RegistryEvent.Register<IRecipe>
  • finishMinecraftLoading
    • "Initializing mods Phase 2"
      • CraftingHelper::loadRecipes -- loads in mod JSON recipes
      • LoaderState.INITIALIZATION

The effect of this is that during the IRecipe registration event, vanilla JSON recipes are in the registry but forge-loaded ones are not.

It would be more consistent to load all JSON recipes before firing the programmatic registration event.

 

What's the reason I want this?

During the recipe registration event, it is possible to remove vanilla JSON recipes:

@SubscribeEvent
public static void registerRecipes(RegistryEvent.Register<IRecipe> event)
{
  ((IForgeRegistryModifiable<IRecipe>)event.getRegistry()).remove(new ResourceLocation("minecraft:anvil"));
}

With this listener registered, the minecraft:anvil recipe disappears without a trace.

Unfortunately, because mod JSON recipes' loading is far off at this point, it is not possible to remove those, because they don't exist yet.

 

Note that doing this during FMLInitializationEvent (via ForgeRegistries.RECIPES of course) removes the recipe but leads to a game crash when trying to render the recipe book.

Spoiler

[16:58:39] [main/FATAL]: Reported exception thrown!
net.minecraft.util.ReportedException: Updating screen events
	at net.minecraft.client.Minecraft.runTick(Minecraft.java:1874) ~[Minecraft.class:?]
	at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:1171) ~[Minecraft.class:?]
	at net.minecraft.client.Minecraft.run(Minecraft.java:436) [Minecraft.class:?]
	at net.minecraft.client.main.Main.main(Main.java:118) [Main.class:?]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_121]
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_121]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_121]
	at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_121]
	at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.12.jar:?]
	at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.12.jar:?]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_121]
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_121]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_121]
	at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_121]
	at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:97) [start/:?]
	at GradleStart.main(GradleStart.java:26) [start/:?]
Caused by: java.lang.IndexOutOfBoundsException: bitIndex < 0: -1
	at java.util.BitSet.get(BitSet.java:623) ~[?:1.8.0_121]
	at net.minecraft.stats.RecipeBook.containsRecipe(RecipeBook.java:35) ~[RecipeBook.class:?]
	at net.minecraft.client.gui.recipebook.RecipeList.canCraft(RecipeList.java:44) ~[RecipeList.class:?]
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.lambda$updateCollections$0(GuiRecipeBook.java:176) ~[GuiRecipeBook.class:?]
	at java.util.ArrayList.forEach(ArrayList.java:1249) ~[?:1.8.0_121]
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.updateCollections(GuiRecipeBook.java:174) ~[GuiRecipeBook.class:?]
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.initVisuals(GuiRecipeBook.java:111) ~[GuiRecipeBook.class:?]
	at net.minecraft.client.gui.inventory.GuiCrafting.actionPerformed(GuiCrafting.java:142) ~[GuiCrafting.class:?]
	at net.minecraft.client.gui.GuiScreen.mouseClicked(GuiScreen.java:494) ~[GuiScreen.class:?]
	at net.minecraft.client.gui.inventory.GuiContainer.mouseClicked(GuiContainer.java:360) ~[GuiContainer.class:?]
	at net.minecraft.client.gui.inventory.GuiCrafting.mouseClicked(GuiCrafting.java:124) ~[GuiCrafting.class:?]
	at net.minecraft.client.gui.GuiScreen.handleMouseInput(GuiScreen.java:611) ~[GuiScreen.class:?]
	at net.minecraft.client.gui.GuiScreen.handleInput(GuiScreen.java:576) ~[GuiScreen.class:?]
	at net.minecraft.client.Minecraft.runTick(Minecraft.java:1861) ~[Minecraft.class:?]
	... 15 more
[16:58:39] [main/INFO] [STDOUT]: [net.minecraft.init.Bootstrap:printToSYSOUT:629]: ---- Minecraft Crash Report ----
// Oh - I know what I did wrong!

Time: 7/7/17 4:58 PM
Description: Updating screen events

java.lang.IndexOutOfBoundsException: bitIndex < 0: -1
	at java.util.BitSet.get(BitSet.java:623)
	at net.minecraft.stats.RecipeBook.containsRecipe(RecipeBook.java:35)
	at net.minecraft.client.gui.recipebook.RecipeList.canCraft(RecipeList.java:44)
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.lambda$updateCollections$0(GuiRecipeBook.java:176)
	at java.util.ArrayList.forEach(ArrayList.java:1249)
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.updateCollections(GuiRecipeBook.java:174)
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.initVisuals(GuiRecipeBook.java:111)
	at net.minecraft.client.gui.inventory.GuiCrafting.actionPerformed(GuiCrafting.java:142)
	at net.minecraft.client.gui.GuiScreen.mouseClicked(GuiScreen.java:494)
	at net.minecraft.client.gui.inventory.GuiContainer.mouseClicked(GuiContainer.java:360)
	at net.minecraft.client.gui.inventory.GuiCrafting.mouseClicked(GuiCrafting.java:124)
	at net.minecraft.client.gui.GuiScreen.handleMouseInput(GuiScreen.java:611)
	at net.minecraft.client.gui.GuiScreen.handleInput(GuiScreen.java:576)
	at net.minecraft.client.Minecraft.runTick(Minecraft.java:1861)
	at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:1171)
	at net.minecraft.client.Minecraft.run(Minecraft.java:436)
	at net.minecraft.client.main.Main.main(Main.java:118)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
	at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:97)
	at GradleStart.main(GradleStart.java:26)


A detailed walkthrough of the error, its code path and all known details is as follows:
---------------------------------------------------------------------------------------

-- Head --
Thread: Client thread
Stacktrace:
	at java.util.BitSet.get(BitSet.java:623)
	at net.minecraft.stats.RecipeBook.containsRecipe(RecipeBook.java:35)
	at net.minecraft.client.gui.recipebook.RecipeList.canCraft(RecipeList.java:44)
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.lambda$updateCollections$0(GuiRecipeBook.java:176)
	at java.util.ArrayList.forEach(ArrayList.java:1249)
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.updateCollections(GuiRecipeBook.java:174)
	at net.minecraft.client.gui.recipebook.GuiRecipeBook.initVisuals(GuiRecipeBook.java:111)
	at net.minecraft.client.gui.inventory.GuiCrafting.actionPerformed(GuiCrafting.java:142)
	at net.minecraft.client.gui.GuiScreen.mouseClicked(GuiScreen.java:494)
	at net.minecraft.client.gui.inventory.GuiContainer.mouseClicked(GuiContainer.java:360)
	at net.minecraft.client.gui.inventory.GuiCrafting.mouseClicked(GuiCrafting.java:124)
	at net.minecraft.client.gui.GuiScreen.handleMouseInput(GuiScreen.java:611)
	at net.minecraft.client.gui.GuiScreen.handleInput(GuiScreen.java:576)

-- Affected screen --
Details:
	Screen name: net.minecraft.client.gui.inventory.GuiCrafting

-- Affected level --
Details:
	Level name: MpServer
	All players: 1 total; [EntityPlayerSP['CAD97'/120, l='MpServer', x=206.76, y=73.00, z=340.80]]
	Chunk stats: MultiplayerChunkCache: 605, 605
	Level seed: 0
	Level generator: ID 00 - default, ver 1. Features enabled: false
	Level generator options: 
	Level spawn location: World: (222,64,354), Chunk: (at 14,4,2 in 13,22; contains blocks 208,0,352 to 223,255,367), Region: (0,0; contains chunks 0,0 to 31,31, blocks 0,0,0 to 511,255,511)
	Level time: 46123 game time, 6000 day time
	Level dimension: 0
	Level storage version: 0x00000 - Unknown?
	Level weather: Rain time: 0 (now: false), thunder time: 0 (now: false)
	Level game mode: Game mode: creative (ID 1). Hardcore: false. Cheats: false
	Forced entities: 51 total; [EntityPlayerSP['CAD97'/120, l='MpServer', x=206.76, y=73.00, z=340.80], EntityBat['Bat'/22, l='MpServer', x=133.25, y=18.10, z=299.87], EntitySkeleton['Skeleton'/23, l='MpServer', x=150.22, y=20.00, z=287.53], EntitySkeleton['Skeleton'/25, l='MpServer', x=132.50, y=51.00, z=407.28], EntitySkeleton['Skeleton'/26, l='MpServer', x=129.48, y=52.00, z=411.74], EntityCreeper['Creeper'/27, l='MpServer', x=140.25, y=50.00, z=402.64], EntitySkeleton['Skeleton'/28, l='MpServer', x=141.50, y=51.00, z=402.70], EntityCreeper['Creeper'/34, l='MpServer', x=151.42, y=34.00, z=267.80], EntityCreeper['Creeper'/35, l='MpServer', x=147.41, y=35.00, z=278.16], EntitySpider['Spider'/36, l='MpServer', x=148.98, y=34.00, z=268.01], EntitySpider['Spider'/37, l='MpServer', x=148.13, y=20.00, z=286.56], EntitySkeleton['Skeleton'/38, l='MpServer', x=159.45, y=44.00, z=360.69], EntityBat['Bat'/39, l='MpServer', x=157.25, y=48.10, z=369.75], EntityCreeper['Creeper'/40, l='MpServer', x=154.39, y=52.00, z=396.85], EntityCreeper['Creeper'/41, l='MpServer', x=148.80, y=52.00, z=397.53], EntityCreeper['Creeper'/42, l='MpServer', x=146.70, y=37.00, z=419.83], EntityBat['Bat'/44, l='MpServer', x=151.88, y=37.00, z=413.09], EntitySkeleton['Skeleton'/48, l='MpServer', x=162.45, y=44.00, z=325.75], EntityEnderman['Enderman'/49, l='MpServer', x=178.51, y=38.00, z=341.71], EntitySkeleton['Skeleton'/50, l='MpServer', x=160.82, y=56.00, z=395.58], EntityCreeper['Creeper'/51, l='MpServer', x=173.16, y=42.00, z=407.59], EntityBat['Bat'/52, l='MpServer', x=175.56, y=52.10, z=410.75], EntityCreeper['Creeper'/57, l='MpServer', x=184.18, y=31.00, z=334.51], EntitySkeleton['Skeleton'/58, l='MpServer', x=181.47, y=47.00, z=320.27], EntityBat['Bat'/186, l='MpServer', x=233.59, y=23.10, z=287.42], EntityBat['Bat'/59, l='MpServer', x=189.70, y=33.10, z=342.75], EntityBat['Bat'/187, l='MpServer', x=239.45, y=29.28, z=290.68], EntityCreeper['Creeper'/60, l='MpServer', x=176.16, y=32.00, z=343.39], EntityBat['Bat'/61, l='MpServer', x=187.75, y=56.10, z=341.25], EntityBat['Bat'/62, l='MpServer', x=183.73, y=58.10, z=346.25], EntityCreeper['Creeper'/63, l='MpServer', x=188.83, y=56.00, z=349.62], EntityZombie['Zombie'/64, l='MpServer', x=200.92, y=55.00, z=348.51], EntityCreeper['Creeper'/65, l='MpServer', x=185.38, y=50.15, z=340.49], EntityCreeper['Creeper'/66, l='MpServer', x=188.70, y=53.00, z=340.30], EntityCreeper['Creeper'/67, l='MpServer', x=187.59, y=46.00, z=380.49], EntityBat['Bat'/68, l='MpServer', x=159.66, y=44.04, z=369.87], EntitySkeleton['Skeleton'/69, l='MpServer', x=187.53, y=50.00, z=400.80], EntitySkeleton['Skeleton'/70, l='MpServer', x=181.02, y=58.00, z=410.50], EntitySkeleton['Skeleton'/71, l='MpServer', x=180.30, y=58.00, z=410.49], EntitySkeleton['Skeleton'/73, l='MpServer', x=198.76, y=61.00, z=342.50], EntityBat['Bat'/74, l='MpServer', x=205.17, y=30.10, z=387.93], EntityZombie['Zombie'/76, l='MpServer', x=220.69, y=17.00, z=264.77], EntityCreeper['Creeper'/77, l='MpServer', x=215.04, y=20.00, z=280.30], EntityCreeper['Creeper'/78, l='MpServer', x=212.78, y=19.00, z=276.43], EntityCreeper['Creeper'/79, l='MpServer', x=218.20, y=24.00, z=273.55], EntityCreeper['Creeper'/80, l='MpServer', x=215.70, y=20.00, z=281.00], EntityBat['Bat'/84, l='MpServer', x=232.38, y=20.10, z=285.37], EntityEnderman['Enderman'/86, l='MpServer', x=245.50, y=20.00, z=285.56], EntitySkeleton['Skeleton'/87, l='MpServer', x=240.72, y=22.00, z=295.50], EntitySkeleton['Skeleton'/88, l='MpServer', x=250.77, y=25.00, z=303.53], EntitySkeleton['Skeleton'/89, l='MpServer', x=276.91, y=48.00, z=282.54]]
	Retry entities: 0 total; []
	Server brand: fml,forge
	Server type: Integrated singleplayer server
Stacktrace:
	at net.minecraft.client.multiplayer.WorldClient.addWorldInfoToCrashReport(WorldClient.java:456)
	at net.minecraft.client.Minecraft.addGraphicsAndWorldToCrashReport(Minecraft.java:2863)
	at net.minecraft.client.Minecraft.run(Minecraft.java:457)
	at net.minecraft.client.main.Main.main(Main.java:118)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
	at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:97)
	at GradleStart.main(GradleStart.java:26)

-- System Details --
Details:
	Minecraft Version: 1.12
	Operating System: Windows 10 (amd64) version 10.0
	Java Version: 1.8.0_121, Oracle Corporation
	Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
	Memory: 843080520 bytes (804 MB) / 1253048320 bytes (1195 MB) up to 3801088000 bytes (3625 MB)
	JVM Flags: 0 total; 
	IntCache: cache: 1, tcache: 1, allocated: 12, tallocated: 94
	FML: MCP 9.40 Powered by Forge 14.21.1.2387 7 mods loaded, 7 mods active
	States: 'U' = Unloaded 'L' = Loaded 'C' = Constructed 'H' = Pre-initialized 'I' = Initialized 'J' = Post-initialized 'A' = Available 'D' = Disabled 'E' = Errored
	UCHIJAAAA	minecraft{1.12} [Minecraft] (minecraft.jar) 
	UCHIJAAAA	mcp{9.19} [Minecraft Coder Pack] (minecraft.jar) 
	UCHIJAAAA	FML{8.0.99.99} [Forge Mod Loader] (forgeSrc-1.12-14.21.1.2387.jar) 
	UCHIJAAAA	forge{14.21.1.2387} [Minecraft Forge] (forgeSrc-1.12-14.21.1.2387.jar) 
	UCHIJAAAA	forgelin{1.4.2} [Shadowfacts' Forgelin] (Forgelin-1.5.0.jar) 
	UCHIJAAAA	spawnercraftkt{<%version%>} [SpawnerCraft] (SpawnerCraft_main) 
	UCHIJAAAA	jei{4.7.1.69} [Just Enough Items] (jei_1.12-4.7.1.69.jar) 
	Loaded coremods (and transformers): 
ForgelinPlugin (Forgelin-1.5.0.jar)
  
	GL info: ' Vendor: 'NVIDIA Corporation' Version: '4.5.0 NVIDIA 382.05' Renderer: 'GeForce GTX 860M/PCIe/SSE2'
	Launched Version: 1.12
	LWJGL: 2.9.4
	OpenGL: GeForce GTX 860M/PCIe/SSE2 GL version 4.5.0 NVIDIA 382.05, NVIDIA Corporation
	GL Caps: Using GL 1.3 multitexturing.
Using GL 1.3 texture combiners.
Using framebuffer objects because OpenGL 3.0 is supported and separate blending is supported.
Shaders are available because OpenGL 2.1 is supported.
VBOs are available because OpenGL 1.5 is supported.

	Using VBOs: Yes
	Is Modded: Definitely; Client brand changed to 'fml,forge'
	Type: Client (map_client.txt)
	Resource Packs: 
	Current Language: English (US)
	Profiler Position: N/A (disabled)
	CPU: 8x Intel(R) Core(TM) i7-4810MQ CPU @ 2.80GHz

(Forgelin is a language adapter for Kotlin, NOT a coremod.)

 

A feature to properly remove JSON loaded recipes programmatically (based on config settings) would render this unnecessary.

Link to comment
Share on other sites

It is not intended to allow you to remove recipes during the Register event. It is intended to be json driven. It just hasn't been implemented yet.

But yes, the Register<IRecipe> event will eventually be fired multiple times, as it sits the current state is not determined and you should only REGISTER your shit in that event not remove others.

  • Like 1

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • I'm using Modrinth as a launcher for a forge modpack on 1.20.1, and can't diagnose the issue on the crash log myself. Have tried repairing the Minecraft instillation as well as removing a few mods that have been problematic for me in the past to no avail. Crash log is below, if any further information is necessary let me know. Thank you! https://paste.ee/p/k6xnS
    • Hey folks. I am working on a custom "Mecha" entity (extended from LivingEntity) that the player builds up from blocks that should get modular stats depending on the used blocks. e.g. depending on what will be used for the legs, the entity will have a different jump strength. However, something unexpected is happening when trying to override a few of LivingEntity's functions and using my new own "Mecha" specific fields: instead of their actual instance-specific value, the default value is used (0f for a float, null for an object...) This is especially strange as when executing with the same entity from a point in the code specific to the mecha entity, the correct value is used. Here are some code snippets to better illustrate what I mean: /* The main Mecha class, cut down for brevity */ public class Mecha extends LivingEntity { protected float jumpMultiplier; //somewhere later during the code when spawning the entity, jumpMultiplier is set to something like 1.5f //changing the access to public didn't help @Override //Overridden from LivingEntity, this function is only used in the jumpFromGround() function, used in the aiStep() function, used in the LivingEntity tick() function protected float getJumpPower() { //something is wrong with this function //for some reason I can't correctly access the fields and methods from the instanciated entity when I am in one of those overridden protected functions. this is very annoying LogUtils.getLogger().info(String.valueOf(this.jumpMultiplier))) //will print 0f return this.jumpMultiplier * super.getJumpPower(); } //The code above does not operate properly. Written as is, the entity will not jump, and adding debug logs shows that when executing the code, the value of this.jumpMultiplier is 0f //in contrast, it will be the correct value when done here: @Override public void tick() { super.tick(); //inherited LivingEntity logic //Custom logic LogUtils.getLogger().info(String.valueOf(this.jumpMultiplier))) //will print 1.5f } } My actual code is slightly different, as the jumpMuliplier is stored in another object (so I am calling "this.legModule.getJumpPower()" instead of the float), but even using a simple float exactly like in the code above didn't help. When running my usual code, the object I try to use is found to be null instead, leading to a crash from a nullPointerException. Here is the stacktrace of said crash: The full code can be viewed here. I have found a workaround in the case of jump strength, but have already found the same problem for another parameter I want to do, and I do not understand why the code is behaving as such, and I would very much like to be able to override those methods as intended - they seemed to work just fine like that for vanilla mobs... Any clues as to what may be happening here?
    • Please delete post. Had not noticed the newest edition for 1.20.6 which resolves the issue.
    • https://paste.ee/p/GTgAV Here's my debug log, I'm on 1.18.2 with forge 40.2.4 and I just want to get it to work!! I cant find any mod names in the error part and I would like some help from the pros!! I have 203 mods at the moment.
  • Topics

×
×
  • Create New...

Important Information

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