OtherDrops

226k Downloads

[Feature Request] Hunger parameter and change to damageattacker parameter.

clucky opened this issue ยท 1 comments

commented

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

commented

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.