search

Zarius / Bukkit-OtherBlocks

OtherBlocks (now known as OtherDrops) plugin for Bukkit (dev build: https://www.mediafire.com/?i6ows1g6kus2o0l)
http://dev.bukkit.org/server-mods/otherdrops/
GNU General Public License v3.0
17 stars 17 forks source link

compatibility with "Autoplant" - drop: SAPLING@PLANTED #31

Closed Zarius closed 11 years ago

Zarius commented 13 years ago

Autoplant is meant to plant a new sapling whenever you chop down the tree. If you have modified the block with otherblocks however and replace the default drop then the "blockbreak" event is cancelled and the block is changed to air by the time that Autoplant gets it.

This applies to other plugins that rely on checking blockbreak events (eg. LogBlock) and I can't see any way around this other than using a specific API to inform the particular plugin of the intended event (as done with LogBlock) or completely disabling the ability to stop the default drop (which defaults a large part of the purpose of this plugin).

Zarius commented 13 years ago

Alternative way (for lack of any better method) is to allow a drop: SAPLING@PLANTED option. Still don't get the benefit of the other autoplant options (protection, etc) but better than nothing.

Zarius commented 13 years ago

Need to test but I think that setting OtherBlocks priority to high should allow Autoplant to work.

Zarius commented 11 years ago

Obselete now - Autoplant is inactive & OtherDrops blockbreak priority is highest so shouldn't be a problem.

CelticMinstrel commented 11 years ago

Um... I realize this is a bit late, but wouldn't "drop: SAPLING@PLANTED" be kinda similar to "replacementblock: SAPLING"?

Zarius commented 11 years ago

Good point, although I'm not sure that replacementblock: was in the code when this issue was posted.