Uploaded image for project: 'Minecraft: Java Edition'
  1. Minecraft: Java Edition
  2. MC-264986

Item count of fuel slot affects the way smelting recipe book moves ingredients

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • 24w33a
    • 1.20.1, 23w33a, 1.21
    • None
    • Confirmed
    • Crafting
    • Normal
    • Platform

      The recipe book place system doesn't not ignore the fuel slot like it does the result slot. This relates to MC-243057 but is a separate issue.

      When the game goes to calculate the number of items to place in a crafting grid (in the furnace's case, its just 1x1) it iterates over slots in the grid + 1 and then ignores the result slot. This doesn't ignore the fuel slot so the count of the fuel is taken into account.

      This manifests itself in game as the following:

      1. Place a furnace with a 8 coal blocks in the fuel slot.
      2. Get a stack of stone bricks in your inventory and put half into the input slot.
      3. Click (without shift) the cracked stone brick recipe in the book.

      Observe how the input stack of stone bricks is decreased down to the fuel slot count + 1.

      This also manifests itself another way:

      1. Place a furnace with 64 coal blocks in the fuel slot.
      2. Get a stack of stone bricks in your inventory and put half into the input slot.
      3. Click (with or without shift) the cracked stone brick recipe in the book.

      Observe how nothing changes despite clearly having more stone bricks in your inventory to add to the input slot.

      A possible fix would be in the for loops iterating over the "grids" in both ServerPlaceRecipe#handleRecipeClicked and ServerPlaceRecipe#getStackSize to skip the AbstractFurnaceMenu#FUEL_SLOT if the menu in an instance of AbstractFurnaceMenu. I'm sure there is an alternative solution that isn't as "special case-y" as that one, probably some method in AbstractContainerMenu that returns the slots to check.

            boq [Mojang] Bartosz Bok
            Machine Maker Jake Potrebic
            Votes:
            5 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:
              CHK: