Register burnables properly on fabric

[artemisSystem]

Fix #76

This also creates a <wood_item>s_that_burn tag for each wood item and use that for registration, so that warped and crimson ones don't burn (like all their wooden items in vanilla)

[artemisSystem]

@lilypuree do you have some time to look over this PR please? It fixes the quilt incompatibility

[Parkanizor]

Implementing this quick fix would be much appreciated. I'd build the fixed version myself but am dumb with computers and have a step fail every time :(

[Fuzzhamster5]

@lilypuree please could you do this as my quilt modpack REALLY needs this mod

[TimDev9492]

Bump

[Jothbot]

bump

[Fuzss]

@artemisSystem Your implementation does not work, you cannot use tags. Fuel registration happens during mod setup, while tags are initialized from the current data pack during world loading. During mod setup tags are always empty, so you are essentially registering nothing to the fuel registry.

[artemisSystem]

are you sure? did you test it? it's been a while but i think i remember it working perfectly. i'm assuming what's happening is it's registering the tag itself, and it's looked up in-world when needed

[Fuzss]

Nvm, it seems like Fabric Api handles that case by clearing caches every time tags are loaded.
I just noticed that when trying TheWinABagel's fork (assuming they didn't make an error in adapting your PR) that only beams do work as fuel, palisades, seats and supports don't. Not sure what's the cause then and if your PR is affected, too.

[unilock]

@Fuzss Looks like TheWinABagel's fork only adds the relevant block tags, not item tags. (TheWinABagel@e13290b#diff-f1afe8d18a5d2cf47951c181edce2c948583348d3ec780f1841fc9d8b1111af3)
(@TheWinABagel)

[TheWinABagel]

Whoops, will fix and upload a new version. I really need to figure out git one of these days...