Done.
Raging Blow should be fixed as well.
Thx for the quick fix, but it doesn't work like it should.
First of all the actual Raging Blow:
This Code checks if there are more than one enemies around AND if the Meat Cleaver buff is greater than one. But you only need one stack of Meat Cleaver to hit two enemies. With three stacks of it, you can hit up to four enemies, because one target gets always hit and with every stack an additional target gets blowed. Also the statement of "Unit.EnemyMeleeUnits.Count() > 1" is alway true if there are more than one units around, even if there are three or four. But the second condition "Buff.PlayerCountBuff("Meat Cleaver") > 1" will cause to cast the Raging Blow way to early.
PHP:
Spell.CastSpell("Raging Blow", ret => Me.HasAura(131116) && (Unit.EnemyMeleeUnits.Count() > 1 && Buff.PlayerCountBuff("Meat Cleaver") > 1 || Unit.EnemyMeleeUnits.Count() > 2 && Buff.PlayerCountBuff("Meat Cleaver") > 2), "Raging Blow AoE")
My code would look like this:
PHP:
Spell.CastSpell("Raging Blow", ret => Me.HasAura(131116) && ((Unit.EnemyMeleeUnits.Count() == 2 && Buff.PlayerCountBuff("Meat Cleaver") == 1) || (Unit.EnemyMeleeUnits.Count() == 3 && Buff.PlayerCountBuff("Meat Cleaver") == 2) || (Unit.EnemyMeleeUnits.Count() > 3 && Buff.PlayerCountBuff("Meat Cleaver") > 2)), "Raging Blow AoE")
Explanation:
- Check if there are exact two enemies near and if there is a stack of Meat Cleaver (one stack = hits two units). (OR)
- Check if there are exact three enemies near and if there are two stacks of Meat Cleaver (two stacks = hits three units). (OR)
- Check if there are more than three enemies near and if there are more than two stacks of Meat Cleaver (three stacks = hits four units).
The Cleave:
Imagine a fight against four enemies. The following code would cleave two of them at the cost of 20 rage if Deadly Calm ist active and we have more than 39 rage. But it would be better to save the rage and use Whirlwind more often, which gets us more stacks of Meat Cleaver and allows us to hard hit the whole group with a nice Raging Blow.
PHP:
Spell.CastSpell("Cleave", ret => Me.CurrentTarget != null && Unit.EnemyMeleeUnits.Count() > 1 && CLUSettings.Instance.UseAoEAbilities && ColossusSmashBloodthirstCooldown && Buff.PlayerHasBuff("Deadly Calm") && Me.CurrentRage >= (TalentManager.HasGlyph("Unending Rage") ? 60 : 40), "Cleave"),
The only useful usage of Cleave would be a situation against exact two enemies, because its does less damage than Whirlwind, but costs less rage if Deadly Calm is enabled. But even then, I would recommend to use a Whirlwind - Raging Blow combination since it does way more damage. Long story short: There isn't really a use for cleave.
And last but not least: I miss the use of Victory Rush / Impending Victory.
PS: I'm talking about the fury warrior thingy.