BlockDispenseEvent is called twice since 26.3 for most blocks
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 55/100
Research direction
Start in DispenserBlock around line 121 and trace SulfurCubeBlockDispenseItemBehavior.execute into dispenseBlock and CraftEventFactory.handleBlockDispenseEvent. Check how the fallback through DefaultDispenseItemBehavior triggers the event, then reproduce with an event listener that logs each dispense. Done means one BlockDispenseEvent per dispense, with the sulfur cube behavior and fallback still working.
Written by the indexing model from the issue text.
Description
Expected behavior
BlockDispenseEvent is only called once when a dispenser throws out a block
Observed/Actual behavior
When a block is dispensed from a dispenser, it used to go through the usual DefaultDispenseItemBehavior.
With the Sulfur Cubes that changed for most blocks (DispenserBlock line 121):
} else if (itemStack.is(ItemTags.SULFUR_CUBE_SWALLOWABLE)) {
return SulfurCubeBlockDispenseItemBehavior.INSTANCE;
}
SulfurCubeBlockDispenseItemBehavior#execute (vanilla) falls back to super.execute if no Sulfur Cube took the item:
@Override
protected ItemStack execute(final BlockSource source, final ItemStack dispensed) {
return dispenseBlock(source.level(), source.pos().relative(source.state().getValue(DispenserBlock.FACING)), dispensed, source, this) // Paper - Call BlockDispenseEvent
? dispensed
: super.execute(source, dispensed);
}
Paper calls handleBlockDispenseEvent at the start of dispenseBlock, before it checks whether a Sulfur Cube is in front:
ItemStack result = org.bukkit.craftbukkit.event.CraftEventFactory.handleBlockDispenseEvent(source, pos, dispensed, currentBehavior);
If there is none, dispenseBlock returns false and vanilla falls back to super.execute, which fires the event again in DefaultDispenseItemBehavior.spawnItem.
The same applies to TNT: the TNT behavior also calls SulfurCubeBlockDispenseItemBehavior.dispenseBlock first and then fires its own BlockDispenseEvent. The TNT is only primed once, but the event is called twice for a single dispense.
sequenceDiagram
autonumber
participant D as DispenserBlock
participant S as SulfurCubeBlockDispenseItemBehavior
participant DD as DefaultDispenseItemBehavior (super)
participant F as CraftEventFactory
participant L as Plugin Listener
D->>D: getDefaultDispenseMethod(itemStack)
Note over D: itemStack.is(SULFUR_CUBE_SWALLOWABLE)<br/>→ SulfurCubeBlockDispenseItemBehavior.INSTANCE
D->>S: dispense(source, stack)
S->>S: execute(source, stack)
S->>S: dispenseBlock(level, pos, stack, source, this)
S->>F: handleBlockDispenseEvent(source, pos, stack, this)
F->>L: BlockDispenseEvent (1st call)
L-->>F: not cancelled
F-->>S: null
S->>S: getEntitiesOfClass(SulfurCube, AABB(pos))
Note over S: no Sulfur Cube in front<br/>→ dispenseBlock returns false
S->>DD: super.execute(source, stack)
DD->>DD: spawnItem(level, stack, 6, direction, position, source)
DD->>L: BlockDispenseEvent (2nd call)
L-->>DD: not cancelled
DD->>DD: level.addFreshEntity(itemEntity)
Steps/models to reproduce
Watch the BlockDispenseEvent with a event listener and see two logs for the same dispense.
Plugin and Datapack List
Paper version
This server is running Paper version 26.3-209-main@03af8b4 (2026-10-20T21:49:44Z) (Implementing API version 26.3.build.209-stable)
Other
No response
- Dominant language
- Java
- Stars
- 12.7k
- Forks
- 3.5k
- Avg merge
- 2d 23h
- Merged PRs (30d)
- 48
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from PaperMC/Paper
-
type: bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
status: needs triage
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
PaperMC/Paper#14375 · 1 comment ·
Maintainers usually reply within 1 day
-
Add .getIgniter() to ExplosiveMinecartPossibly taken @WouterGritter claimed this 34 days ago. Openstatus: needs triage
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
PaperMC/Paper#14097 · 1 comment ·
Maintainers usually reply within 1 day
-
BlockType#isOccluding is incorrectly handledPossibly taken @ryoji-ikeda claimed this 178 days ago. Openstatus: accepted
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
PaperMC/Paper#13790 · 5 comments ·
Maintainers usually reply within 1 day
-
status: needs triage version: 26.3
Difficulty 3/5 1-2 days Newbie friendliness 72/100
PaperMC/Paper#14412 · 2 comments ·
Maintainers usually reply within 1 day
Similar issues
-
human-required sync-failed
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Azure/osdu-spi-file#63 ·
Maintainers usually reply within 1 day
-
frontend quick win
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
CodeForPhilly/benefit-decision-toolkit#571 ·
Maintainers usually reply within 1 day
-
approved
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
checkstyle/checkstyle#22129 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
Add Hardcharger JDBC driverPossibly taken @prmoore77 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day