Fix incorrect block updates for doors

[zavdav]

📌 Pull Request

Description

Correctly fixes the issue described in #184 where placing a door next to an opened trap door would cause the bottom half of the door to be broken and a door item to be dropped on the floor. When powering the remaining upper half of the door with a redstone torch, the torch would be removed and another door item would be dropped.

The actual cause of this issue is a CraftBukkit change in ItemDoor.java which sends a block update from the perspective of a redstone wire, in order to instantly trigger redstone updates for the door. This behavior was later reverted in CraftBukkit.

Related Issues / Discussions

• Related to Fix Door duplication bug #184

Motivation and Context

#184 introduced a bug where if a door is broken at the top half, the bottom half is not removed, allowing for half-door blocks to be created.

Type of Change

Please tick all that apply:

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ⚡ Performance improvement
• 🧩 New feature (non-breaking)
• 🔧 Refactor / cleanup (no functional changes)
• 🔥 Crash / exploit fix
• 📚 Documentation update
• ❗ Breaking change (may affect plugins or server behavior)

Testing Performed

• Compiled successfully with mvn clean package
• Tested on a Beta 1.7.3 server
• Existing functionality verified
• Edge cases considered

Test details:

Tested on local server, verified the duplication glitch to be fixed and the correct removal of the entire door block to take place when breaking it at the top half and bottom half.

Compatibility Considerations

• No known compatibility impact
• Potential plugin impact (described below)
• Network / protocol (Netcode) behavior changed
• Configuration change required

Checklist

Please confirm the following:

• No unnecessary formatting or whitespace-only changes
• Changes are compatible with Minecraft Beta 1.7.3
• No dependencies have been added without discussion with the RetroMC team

[RhysB]

Tested and confirmed