Bukkit/CraftBukkit — pull request #645 by wilsonsd (2012-01-18 04:33:46+00:00)
Title: Throw BlockPlaceEvent when changing a single step to a double step.

BUKKIT-394 fix.

Issue BUKKIT-394: When a player places a step on top of another step
to form a double step, no BlockPlaceEvent is thrown.

Reason for bug: When a player places a step on top of a single step,
ItemStep.a line 38 detects this and prevents the call to ItemBlock.a
on line 46. But ItemBlock.a is where CraftBukkit creates and fires
the BlockPlaceEvent.

Fix: Remove the check in ItemStep.a. Rather than detecting the new
double-step in ItemStep, we'll detect it in ItemBlock.a. This way, we
create a BlockPlaceEvent in the usual way.

This pull request is the same fix as PR 607, but I've rebased the branch to the current (as of 1/17/2012) head of CraftBukkit and retested.
https://github.com/Bukkit/CraftBukkit/pull/607