Creators-of-Create / Create

[Forge Mod] Building Tools and Aesthetic Technology
MIT License
2.91k stars 906 forks source link

[1.20.1] Unable to register more than one Fan Processing Type #6111

Open DakotaPride opened 8 months ago

DakotaPride commented 8 months ago

Describe the Bug

Attempting to register two different Fan Processing Types results in a crash occurring. The crash report/latest.log files show that the "FanProcessingTypeRegistry.insertSortedType" method is at fault for the issue, from the "FanProcessingTypeRegistry.register" method.

I asked about it in the discord, but nobody could figure out why exactly the issue was happening.

If any more information needs to be provided, I'll gladly provide it, just let me know!

image image

Reproduction Steps

  1. In a development environment, attempt to register 2 or more fan processing types utilizing the base create mod registration as a reference/basis.
  2. Run the game

Expected Result

For the game to load properly with both fan processing types functioning properly.

Screenshots and Videos

No response

Crash Report or Log

https://pastebin.com/FLPJJfeF

Operating System

Windows 11

Mod Version

0.5.1e

Minecraft Version

1.20.1

Forge Version

47.1.46

Other Mods

No response

Additional Context

No response

DakotaPride commented 8 months ago

I found the issue, didn't realize that the "getPriority" method needed to be unique. I couldn't find any documentation on creating a custom fan processing type, so this might be a good idea to include considering that not a single person had noticed this.

DakotaPride commented 8 months ago
image