Farmer's Delight

Farmer's Delight

121M Downloads

[1.21.1 only] Cooking Pot Display Slot maxes out at 99

MerchantCalico opened this issue ยท 2 comments

commented

Minecraft version

1.21.1

Farmer's Delight version

1.2.8

NeoForge version

21.1.82

Description

The Cooking Pot's result display is allowed to store 99 items instead of 64.

This is because NeoForge's item handler slot limit maxes out at 99 (the actual max stack size), which the Cooking Pot result will use with the result slot, as the item stack's max count is ignored there.

This only happens on 1.21.1, because of the component change.

Steps to reproduce

  1. Get ingredients for any Cooking Pot recipe up to 99 times.
  2. Increase tick rate to 10000 (easier for testing)
  3. See that you can have 99 items in your Cooking Pot.

Mod list

Mod List:
appleskin-neoforge-mc1.21-3.0.5.jar |AppleSkin |appleskin |3.0.5+mc1.21 |Manifest: NOSIGNATURE
CraftTweaker-neoforge-1.21-20.0.21.jar |CraftTweaker |crafttweaker |20.0.21 |Manifest: NOSIGNATURE
main |Farmer's Delight |farmersdelight |1.2.8 |Manifest: NOSIGNATURE
jei-1.21-neoforge-19.8.2.99.jar |Just Enough Items |jei |19.8.2.99 |Manifest: NOSIGNATURE
neoforge-21.1.82-minecraft-merged.jar |Minecraft |minecraft |1.21.1 |Manifest: NOSIGNATURE
neoforge-21.1.82-minecraft-merged.jar |NeoForge |neoforge |21.1.82 |Manifest: NOSIGNATURE

Logs

No response

Minimal instance

  • I have tested this on a minimal instance

Performance and shader mods

  • I am using performance or shader mods
commented

Incredible stuff. That's a deep cooking pot...

Image

Alright, I'll fix it in a bit.

commented

Fixed on commit ef24895.