WolfyScript / CustomCrafting

Spigot based plugin to create your own custom Recipes & Items. Designed to be easy to use with in-game GUI and tons of configuration possibilities.
https://modrinth.com/plugin/customcrafting
GNU General Public License v3.0
113 stars 36 forks source link

Furnace recipes causing issues with Slimefun4 #221

Closed LcyDev closed 1 year ago

LcyDev commented 1 year ago

What happened?

When you use an Enhanced Furnace of Slimefun4 and try to use recipes made in Custom Crafting, everything's fine until the last item gets smelted.

image

Once that happens, an error will pop up in the console.

How can the bug be reproduced?

  1. Install Slimefun4.
  2. Make a CC furnace recipe.
  3. /sf give <player> ENHANCED_FURNACE
  4. Try smelting the custom recipe in it.
  5. Check the console for errors.

Expected Behaviour?

This shouldn't happen

Plugin version

CustomCrafting-4.16.5.1 - WolfyUtils-4.16.9.6

Server software

Purpur (1.19.3) b1900

Relevant log output

https://paste.gg/p/ConaII_/ba285dbaa2654c33a3544a0e8b714ba9

https://paste.gg/p/ConaII_/ce0b4a7588a34a96a7b233f099e54ec7

Submit issue

LcyDev commented 1 year ago

https://github.com/Slimefun/Slimefun4/issues/3741

J3fftw1 commented 1 year ago

This issue is fixed in https://github.com/Slimefun/Slimefun4/pull/3756 For some reason CustomCrafting adds a null item. We still should have checked for it in the first place but still weird a null item is being added.

WolfyScript commented 1 year ago

This issue is fixed in Slimefun/Slimefun4#3756 For some reason CustomCrafting adds a null item. We still should have checked for it in the first place but still weird a null item is being added.

Yeah, I don't know why it happens, I guess it can be null by desing (at least that's what the API says). I had to also check if the smelting item is null, because another plugin caused the same issue before.
https://github.com/WolfyScript/CustomCrafting/blob/73bebaa50537ac36ff0bd492375eb067d9014242/src/main/java/me/wolfyscript/customcrafting/utils/cooking/SmeltAPIAdapter.java#L92

J3fftw1 commented 1 year ago

im guessing your check happens later then we check the smelting for some reason. Might also just be spigot jank