MattCzyr / RealisticTorches

A Minecraft mod that causes torches to burn out after a configurable amount of time.
https://www.curseforge.com/minecraft/mc-mods/realistic-torches
Other
26 stars 12 forks source link

Can't Craft realistic torches #44

Open HyperCodec opened 3 years ago

HyperCodec commented 3 years ago

The realistic torches won't craft, and only the normal torch recipe works. Maybe you could simply modify the vanilla torch instead of making a new item?

voxelli commented 3 years ago

I believe this has to do with realistictorches:unlit_torch not being properly registered. I don't have my development env set up, but it looks like the problem is a simple typo: https://github.com/MattCzyr/RealisticTorches/blob/9e1e7046b48095f6673c3506962d6c928624e7f0/src/main/java/com/chaosthedude/realistictorches/items/RealisticTorchesItems.java#L36

There is an extra ).

Felix-FTW commented 2 years ago

The mod was working fine until I made a copy of my world, now I can't craft unlit torches

HyperCodec commented 2 years ago

The mod was working fine until I made a copy of my world, now I can't craft unlit torches

Do you know if this is fixed yet?

HyperCodec commented 2 years ago

This issue has been open for a year

Felix-FTW commented 1 year ago

yeah im pretty sure the mod is abandoned, it hasnt been updated past 1.16

i can probably get a guy to update it in the future if you have the money, look up arcane studios on discord, they have mod devs on there

in the meantime, look at the code i put in the comments or check out that "Restric Recipes" mod

On Sun, Aug 14, 2022, 3:25 PM HyperCodec @.***> wrote:

This issue has been open for a year

— Reply to this email directly, view it on GitHub https://github.com/MattCzyr/RealisticTorches/issues/44#issuecomment-1214460496, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVYIG3DF3F27WICFNA3OXJTVZFW4TANCNFSM46S3OXIA . You are receiving this because you commented.Message ID: @.***>

HyperCodec commented 1 year ago

yeah im pretty sure the mod is abandoned, it hasnt been updated past 1.16 i can probably get a guy to update it in the future if you have the money, look up arcane studios on discord, they have mod devs on there in the meantime, look at the code i put in the comments or check out that "Restric Recipes" mod

I could probably update the mod myself — it can't be that hard — rather than making someone else do it. The problem I see here is, if this is truly abandoned, no one will be there to accept a pull request.

MattCzyr commented 1 year ago

I could probably update the mod myself — it can't be that hard — rather than making someone else do it. The problem I see here is, if this is truly abandoned, no one will be there to accept a pull request.

I'm still around and able to accept the PR if someone wants to update this mod. I just haven't had the time to do it myself as I've been prioritizing my other mods for quite a while.

Felix-FTW commented 1 year ago

You should put what you just said in the description of the mod so people know what's up

On 10/11/22, Matt Czyr @.***> wrote:

I could probably update the mod myself — it can't be that hard — rather than making someone else do it. The problem I see here is, if this is truly abandoned, no one will be there to accept a pull request.

I'm still around and able to accept the PR if someone wants to update this mod. I just haven't had the time to do it myself as I've been prioritizing my other mods for quite a while.

--

Reply to this email directly or view it on GitHub:

https://github.com/MattCzyr/RealisticTorches/issues/44#issuecomment-1275476905

You are receiving this because you commented.

Message ID: @.***>

HyperCodec commented 1 year ago

I believe this has to do with realistictorches:unlit_torch not being properly registered. I don't have my development env set up, but it looks like the problem is a simple typo:

https://github.com/MattCzyr/RealisticTorches/blob/9e1e7046b48095f6673c3506962d6c928624e7f0/src/main/java/com/chaosthedude/realistictorches/items/RealisticTorchesItems.java#L36

There is an extra ).

I don't see how there is an extra ). Looks the same as the rest.

MattCzyr commented 1 year ago

I've got some time to come back to this mod, and I looked into this issue. I wasn't able to recreate it myself (in my testing, unlit torches can be crafted as intended), but I have an idea of what might be causing it for others. If either of you are still facing this issue, try this file and see if it persists:

https://www.dropbox.com/s/6lkoymjstb5hpu5/RealisticTorches-1.16.5-3.1.3-beta.jar?dl=0

Note, I also updated to the latest Forge version for 1.16.5, so make sure you're using the latest version.

HyperCodec commented 1 year ago

I've got some time to come back to this mod, and I looked into this issue. I wasn't able to recreate it myself (in my testing, unlit torches can be crafted as intended), but I have an idea of what might be causing it for others. If either of you are still facing this issue, try this file and see if it persists:

https://www.dropbox.com/s/6lkoymjstb5hpu5/RealisticTorches-1.16.5-3.1.3-beta.jar?dl=0

Note, I also updated to the latest Forge version for 1.16.5, so make sure you're using the latest version.

I'll go find out what version and such I am using and get back to you (should be on Monday bc busy tomorrow)