search

friendlyhj / ZenRecipeReloading

Recipe hot reloading for crafttweaker 1.12
MIT License
16 stars 2 forks source link

Errors with Extended Crafting & Artisan Worktables #7

Closed TheComputerizer closed 11 months ago

TheComputerizer commented 11 months ago

It seems like this mod causes issues with the TableCrafting#remove action, even if the script file is explicitly defined as #notreloadable this is the error I get for that. The script I used to test is a simple mods.extendedcrafting.TableCrafting;.remove(<extendedcrafting:material:24>). When I try using /ct reload the command fails with this mixin error

On a separate note, I get this weird truncated class file error with my basic worktable script.

Both seem to work fine without the mod and /ct syntax doesn't complain about anything

friendlyhj commented 11 months ago
  1. Are your Extended Crafting is nomifactory version?
  2. Try this version zenrecipereload-1.1.3.zip
TheComputerizer commented 11 months ago

I wasn't aware of the nomi version of extended crafting and updating to that seems to have fixed the first issue. The truncated class file thing seems to still be an issue, but I'm not 100% sure whether its my script breaking or not

friendlyhj commented 11 months ago

Released v1.1.4 to fix the first issue. This mod should only compat with the nomifactory version.

The stacktrace shows the error is from your script. But I can't find what is wrong. Can you update complie bytecode? Add debug preprocessor, then it will be output at generated dir.

TheComputerizer commented 11 months ago

I figured out it didn't seem to like my my tableExtra util method, maybe since it had too many parameters. The initial script loading and reloading seems to be working as intended now that I've updated to the nomifactory version of extended crafting and ZRR 1.1.4, but running /ct jeiReload seems to give me a class cast exception related to entity age. Not sure how viable debugging that would be since it could just be a compatibility error but since it doesn't seem to be related to the initial issues I'll close it