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

Armor after washing in the cauldron immediately put on the player

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Resolution: Unresolved
    • Affects Version/s: Minecraft 1.11.2, Minecraft 17w18b, Minecraft 1.12 Pre-Release 1, Minecraft 1.12 Pre-Release 2, Minecraft 1.12 Pre-Release 3, Minecraft 1.12 Pre-Release 4, Minecraft 1.12 Pre-Release 5, Minecraft 1.12 Pre-Release 6, Minecraft 1.12 Pre-Release 7, Minecraft 1.12, Minecraft 1.12.1 Pre-Release 1, Minecraft 1.12.1, Minecraft 1.12.2 Pre-Release 1, Minecraft 1.12.2 Pre-Release 2, Minecraft 1.12.2, Minecraft 17w43a, Minecraft 17w43b, Minecraft 17w45a, Minecraft 17w45b, Minecraft 17w46a, Minecraft 17w47a, Minecraft 17w47b, Minecraft 17w49a, Minecraft 17w49b, Minecraft 17w50a, Minecraft 1.13-pre6, Minecraft 1.13, Minecraft 18w30b, Minecraft 1.13.1, Minecraft 1.13.2, Minecraft 18w45a, Minecraft 18w46a, 1.14.4, 19w34a, 19w37a, 19w39a, 19w41a, 19w42a, 1.15 Pre-release 6, 20w15a, 20w28a
    • Fix Version/s: None
    • Confirmation Status:
      Confirmed
    • Category:
      (Unassigned)

      Description

      Steps to reproduce

      1. Place the filled cauldron:
        /setblock ~1 ~ ~ cauldron[level=3]
      2. Get dyed armor.
        /give @p leather_chestplate{display:{color:0}} 1
      3. Check if there is any item in the chestplate slot.
      4. Click on the cauldron with leather chestplate.
        Also, when you wash the armor, there will not be any animation of the hand (before 19w37a).

      Code analysis

      This is caused by the check if the world is a client or a server (!worldIn.isRemote). Removing it fixes the issue.

      BlockCauldron.onBlockActivated(world, pos, state, playerIn, hand, facing, x, y, z)
      if (itemarmor.getArmorMaterial() == ItemArmor.ArmorMaterial.LEATHER && itemarmor.hasColor(itemstack) && !worldIn.isRemote)
      {
          itemarmor.removeColor(itemstack);
          //...
          return true;
      }
      

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              Den28Rus user33
              Votes:
              10 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                CHK: