Ok one last thing I noticed. The Paladin and Ranger and Beastlord are actually set up so much better. The "stance" is used in determining when the heal needs to occur. For example, at balanced they will heal at 50%, but at efficient they heal at 25%. These numbers of course can be changed, and now that I switched Paladin with Cleric I am basically where I need to be and can make changes on the fly by just switching his stance which is exactly what I wanted.
The only thing next would be to stop him from healing himself so quickly, but the main issue is resolved. I really think Clerics were intended to be the same way and someone quit in the middle or made a change just for a big raid or something and either never changed it back or we ended up with older code.
Here is the code I am referring to and where it says "Cleric" initially said Paladin.
if( botCasterClass == CLERIC || botCasterClass == BEASTLORD || botCasterClass == RANGER) {
if(caster->HasGroup()) {
Group *g = caster->GetGroup();
float hpRatioToHeal = 25.0f;
switch(caster->GetBotStance()) {
case BotStanceReactive:
case BotStanceBalanced:
hpRatioToHeal = 50.0f;
break;
case BotStanceBurn:
case BotStanceBurnAE:
hpRatioToHeal = 20.0f;
break;
case BotStanceAggressive:
case BotStanceEfficient:
default:
hpRatioToHeal = 25.0f;
break;
|