FarmersDelight icon indicating copy to clipboard operation
FarmersDelight copied to clipboard

[1.18.2]: Cutting board allows the use of zero-durabiliy tools

Open LordXerus opened this issue 1 year ago • 1 comments

Description

The cutting board allows usage of zero-durability tools to do recipes successfully. Notable tools include Tinkers' Construct tools or tools that run on FE/RF. These tools remain and can be repaired after they reach zero-durability. In my particular case, I am able to shear leather boots with a broken(presumably zero durability) Tinkers' Construct Kama (which can act as shears).

Steps to reproduce

  1. Obtain Kama, cutting board, shearable item.
  2. Attempt to shear the item with zero-durability shear-like tool
  3. Shear completes successfully

Mod list

[✔️] appleskin-forge-mc1.18.2-2.4.1 [✔️] cofh_core-1.18.2-9.2.1.39 [✔️] comforts-forge-1.18.2-5.0.0.6 [✔️] dyenamics-2.0.0 [✔️] FarmersDelight-1.18.2-1.2.0 [✔️] HammerLib-1.18.2-18.2.16 [✔️] jei-1.18.2-forge-10.2.1.1004 [✔️] Mantle-1.18.2-1.9.45 [✔️] Patchouli-1.18.2-71.1 [✔️] rangedpumps-0.9.2 [✔️] refinedstorage-1.10.5 [✔️] rubidium-0.5.6 [✔️] SolarFluxReborn-1.18.2-18.2.4 [✔️] TConstruct-1.18.2-3.6.4.113 [✔️] thermal_cultivation-1.18.2-9.2.0.16 [✔️] thermal_dynamics-1.18.2-9.2.1b.13 [✔️] thermal_expansion-1.18.2-9.2.0.20 [✔️] thermal_foundation-1.18.2-9.2.0.46 [✔️] thermal_innovation-1.18.2-9.2.0.17 [✔️] thermal_integration-1.18.2-9.2.0.16 [✔️] thermal_locomotion-1.18.2-9.2.0.13

Logs

No response

Forge

  • [X] I am using the Forge version of Farmer's Delight

Minimal instance

  • [ ] I have tested this on a minimal instance

Performance and shader mods

  • [ ] I have tested this without performance or shader mods

LordXerus avatar May 21 '23 07:05 LordXerus

This is a tough one... Although I could definitely add a durability check to Cutting Board actions (which would cover some mods), some of those tool mods implement their own systems, which don't always utilize the Damage NBT value.

I just tested a quick implementation (checking if the tool is equal or above max damage), and it still didn't cover TCon or Thermal Innovation. I assume those tools are using other ways of handling "broken/uncharged" states, which I'm not sure I can consult from my end.

Another alternative is for such mods to "remove" their ToolActions from a broken/uncharged tool; this would make them invalid to cutting recipes.

I'll try contacting the devs to see if there's a way. At the very least, I could add the basic check.

vectorwing avatar Jun 24 '23 20:06 vectorwing