[Jenkins]Jenkins

Changes

Summary

  1. Make broken shields ineffective (#5079) (details)
  2. Fix firestarter not respecting dual handed logic (details)
  3. Fix firestarter and glowing immediately breaking the block when used as a left click modifier (details)
  4. Ensure all interaction hooks are disabled when on cooldown (details)
  5. Fix chestplates being unable to attack (#5080) (details)
  6. Reduce spilling cooldown since its a bit more reasonable after the potion nerf (details)
  7. Fix shields overlapping armor models in third person (details)
Commit 676973dab7e88e34105493df170f6379cc27ec09 by KnightMiner
Make broken shields ineffective (#5079)

Unfortunatelly have no way to make you stop blocking as vanilla deletes the shield if the use item is empty. I can however make it useless when broken at least
Also deduplicate a bit of tool damaging code
The file was modifiedsrc/main/java/slimeknights/tconstruct/library/tools/item/ModifiableItem.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/library/tools/item/ModifiableLauncherItem.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/library/tools/helper/ToolDamageUtil.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/tools/logic/InteractionHandler.java (diff)
Commit 0bb59bb8b85cb7222bc7b5c5de21834cadecb176 by KnightMiner
Fix firestarter not respecting dual handed logic
The file was modifiedsrc/main/java/slimeknights/tconstruct/tools/modifiers/ability/interaction/FirestarterModifier.java (diff)
Commit bbf96852e6ca98cc1dc0e38c8a159eab3d45f399 by KnightMiner
Fix firestarter and glowing immediately breaking the block when used as a left click modifier

As part of this fix, bows now have the proper hooks to be a harvest tool, if you just give them harvest logic in the tool defintion.
The file was modifiedsrc/main/java/slimeknights/tconstruct/tools/modifiers/ability/interaction/FirestarterModifier.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/library/tools/item/ModifiableItem.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/tools/modifiers/ability/tool/GlowingModifier.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/library/tools/helper/ToolHarvestLogic.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/library/tools/item/ModifiableLauncherItem.java (diff)
The file was modifiedsrc/main/java/slimeknights/tconstruct/tools/logic/InteractionHandler.java (diff)
Commit 8d93f893f31e870c93b69b145c7b8aa4a1c97267 by KnightMiner
Ensure all interaction hooks are disabled when on cooldown
The file was modifiedsrc/main/java/slimeknights/tconstruct/tools/logic/InteractionHandler.java (diff)
Commit ae60c1aec3b72eb31d63b75aa34e1b1b424a6e07 by KnightMiner
Fix chestplates being unable to attack (#5080)
The file was modifiedsrc/main/java/slimeknights/tconstruct/library/tools/helper/ToolAttackUtil.java (diff)
Commit 81a9a12c6883ed15b9f418693797e07d5bd41ea8 by KnightMiner
Reduce spilling cooldown since its a bit more reasonable after the potion nerf
The file was modifiedsrc/main/java/slimeknights/tconstruct/tools/modifiers/ability/tool/SpillingModifier.java (diff)
Commit 08ac5ac8ea18dde1d70923f08f9a2cd733a933e7 by KnightMiner
Fix shields overlapping armor models in third person
The file was modifiedsrc/main/resources/assets/tconstruct/models/item/base/shield.json (diff)