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] Hunger parameter and change to damageattacker parameter. #236

Open clucky opened 11 years ago

clucky commented 11 years ago

I noticed that there is a damageattacker parameter but not a damagevicitim. That being said, for ease of memory and relevance to other parameters, damageattacker should be changed to damage.attacker and there should be damage.victim damage.world and damage.server in addition to it.

I am wanting players to gradually loose hunger as they are mining stone. I figure this can either be done by doing hunger.attacker: -0.05 or by doing hunger: -1.0 alongside chance: 0.05%. As with damage, it should have hunger.attacker hunger.victim hunger.world and hunger.server.

Anyway, thank you for your consideration and hopefully I will get a chance to see these features added into OtherDrops. :)

Zarius commented 11 years ago

damageattacker is for supporting old configs.

You'll see here: https://github.com/Zarius/Bukkit-OtherBlocks/blob/master/src/com/gmail/zariust/otherdrops/parameters/actions/DamageAction.java?source=c#L39 that there is damage, damage.attacker, damage.victim, damage.world & damage.server already :)

Hunger is there too but not quite finalised (currently it only sets the hunger at a specific level which doesn't help.