mojira.dev
MC-265901

Comparator sometimes doesn't update when crafting with the Crafter

When the Crafter gets a short Redstone impulse(at most 4 redstone ticks) and the correct Construction(It didn't always work) is built, then the Comparator keeps the signal.

What I expected to happen was...:
The Comparator updates and stops being powered.

What actually happened was...:
The Comparator doesn't update and keeps being powered.

Steps to Reproduce:
1. Build the Construction, which is shown in the image.
2. Place a Recipe in the Crafter.
3. Press the Button.
4. Don't open the barrel. This will update the comparator.

Linked issues

Attachments

Comments

[Mod] Jingy

I can confirm this

PR0CESS

Code Analysis:

net.minecraft.world.level.block.CrafterBlock#dispenseFrom()

// Current code
serverLevel.setBlock(blockPos, blockState.setValue(CRAFTING, true), UPDATE_CLIENTS);

// Should be
serverLevel.setBlock(blockPos, blockState.setValue(CRAFTING, true), UPDATE_ALL);

Currently the update is `2` (UPDATE_CLIENTS) but it should be `3` (UPDATE_ALL) as block updates need to be done

migrated

This also happens for containers being filled by the crafter.

I have a crafter that is filling up a chest and a comparator that is reading the contents of the chest, but the comparator doesn't update its strength as the chest fills up.

Edit: This bug can only be reproduced when the chest is blocked by a block above (so you can't open the chest) not sure if this requires a separate ticket or not.

After removing the block you can open the chest but the comparator will still not update:

[media]

Panda4994

Yeah, that's worth an own report. But it's already tracked in MC-8340 and MC-100302 🙂

Haph2
[media]

Can confirm. Attached reproduction method. Place any recipe in crafter and update note block to recreate.

migrated

Just to know, is the crafter's cooldown shorter than others item?

The red arrow is short when powered using the observer than anything else but there is no application to this, right?

It does not craft faster?

ErfinderLabyrinth

Panda4994

1126916

Community Consensus

Expansion B

Important

Block states, Redstone

23w42a

23w43a

Retrieved