OtherDrops

226k Downloads

compatibility with "Autoplant" - drop: SAPLING@PLANTED

Zarius opened this issue ยท 5 comments

commented

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).

commented

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

commented

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

commented

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

commented

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.

commented

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