TheYeti/mcMMO — Issue #48: Exp gain in combat is based on the entity health, not on the damage done
Author: bm01
Created: 2012-01-26 20:57:28+00:00
State: open

In:
* Combat.combatChecks(), line 108
* Combat.getXp(), line 382
* Maybe elsewhere?

if(hpLeft < event.getDamage())
xpinc = event.getDamage();
else
xpinc = hpLeft;

Should logically be:

if(hpLeft < event.getDamage())
xpinc = hpLeft;
else
xpinc = event.getDamage();

This also explain why Unarmed is so easy to level up, and why you gain less exp per mob with a diamond sword than a wooden one.

By the way, a little off subject, but the event damage doesn't take sword enchants into account. With a diamond sword it will always be 7 (+ critical hit). That isn't the case with bow enchants however.