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

[Feature Request] Addition of replacementmob #239

Closed clucky closed 11 years ago

clucky commented 11 years ago

I am wanting to do the following, however, when I do it, it replaces the fence the chicken was standing on with air. I figured a simple solution to this would be to replace the entity instead of the block.

Before:

otherdrops: 
    CHICKEN:
      - tool: MONSTER_EGG@0
        action: RIGHT_CLICK
        consumetool: 1
        replacementblock: air
        flag: UNIQUE
        drop: MONSTER_EGG@THIS
        message: ["&bAll right! &bYou caught &c%v&b!"]
      - tool: [ANY_TOOL, ANY_OBJECT]
        drop: default

After:

otherdrops: 
    CHICKEN:
      - tool: MONSTER_EGG@0
        action: RIGHT_CLICK
        consumetool: 1
        replacementmob: air
        flag: UNIQUE
        drop: MONSTER_EGG@THIS
        message: ["&bAll right! &bYou caught &c%v&b!"]
      - tool: [ANY_TOOL, ANY_OBJECT]
        drop: default
CultistO commented 11 years ago

I thought I had made a ticket about this, but I guess I only mentioned it on the main page. You can use "damage.victim: 100" to remove the mob without removing surrounding blocks. The problem is that they will then drop their default drop along with your custom one.

We discussed adding something like damage.victim.remove to solve the problem, which I think is a better alternative.

Zarius commented 11 years ago

Fixed in d1d18a70fa3a124d8b41e86a18dbd404ad44c6bd. replacemenblock: AIR on an entity will now only remove the entity - other replacement blocks work as usual, as does replacementblock: AIR on a blocktarget.