Warrior: Proper stance logic for Sweeping Strikes (#128)

* Warrior: Prebuilt arms weapon specialization talents

* Warrior: Proper stance logic for Sweeping Strikes

Stance Dance:
Arms Warrior pools rage in Battle Stance until Sweeping Strikes is castable until returning back to Berserk Stance

* PvE/PvP/Raid logic + Code cleanup

* Ignore mortal strike and heroic strike while pooling rage

* Update ArmsWarriorStrategy.cpp

* Disable Battle Stance spam if SS is on cooldown

playerbot/strategy/warrior/ArmsWarriorStrategy.cpp

@@ -1033,4 +1033,116 @@ void ArmsWarriorCcRaidStrategy::InitNonCombatTriggers(std::list<TriggerNode*>& t
     WarriorCcRaidStrategy::InitNonCombatTriggers(triggers);
 }
 
-#endif
+#endif
+
+class WarriorSweepingStrikesPveMultiplier : public Multiplier
+{
+public:
+    WarriorSweepingStrikesPveMultiplier(PlayerbotAI* ai) : Multiplier(ai, "aoe arms pve") {}
+
+    float GetValue(Action* action) override
+    {
+        // Disable Berserker Stance
+        const std::string& actionName = action->getName();
+        if ((actionName == "berserker stance" ||
+            actionName == "whirlwind" ||
+            actionName == "mortal strike" ||
+            actionName == "heroic strike" ||
+            actionName == "cleave") &&
+            AI_VALUE2(bool, "trigger active", "melee light aoe") &&
+            bot->HasSpell(12292) &&
+            bot->IsSpellReady(12292) &&
+            !bot->HasAura(12292))
+        {
+            return 0.0f;
+        }
+
+        // Disable Battle Stance spam if SS is on CD
+        if ((actionName == "battle stance") &&
+            AI_VALUE2(bool, "trigger active", "melee light aoe") &&
+            bot->HasSpell(12292) &&
+            !bot->IsSpellReady(12292))
+        {
+            return 0.0f;
+        }
+
+        return 1.0f;
+    }
+};
+
+class WarriorSweepingStrikesRaidMultiplier : public Multiplier
+{
+public:
+    WarriorSweepingStrikesRaidMultiplier(PlayerbotAI* ai) : Multiplier(ai, "aoe arms raid") {}
+
+    float GetValue(Action* action) override
+    {
+        // Disable Berserker Stance
+        const std::string& actionName = action->getName();
+        if ((actionName == "berserker stance" ||
+            actionName == "whirlwind" ||
+            actionName == "mortal strike" ||
+            actionName == "heroic strike" ||
+            actionName == "cleave") &&
+            AI_VALUE2(bool, "trigger active", "melee light aoe") &&
+            bot->HasSpell(12292) &&
+            bot->IsSpellReady(12292) &&
+            !bot->HasAura(12292))
+        {
+            return 0.0f;
+        }
+
+        // Disable Battle Stance spam if SS is on CD
+        if ((actionName == "battle stance") &&
+            AI_VALUE2(bool, "trigger active", "melee light aoe") &&
+            bot->HasSpell(12292) &&
+            !bot->IsSpellReady(12292))
+        {
+            return 0.0f;
+        }
+
+        return 1.0f;
+    }
+};
+
+class WarriorSweepingStrikesPvpMultiplier : public Multiplier
+{
+public:
+    WarriorSweepingStrikesPvpMultiplier(PlayerbotAI* ai) : Multiplier(ai, "aoe arms pvp") {}
+
+    float GetValue(Action* action) override
+    {
+        // Disable Berserker Stance
+        const std::string& actionName = action->getName();
+        if ((actionName == "berserker stance" ||
+            actionName == "whirlwind" ||
+            actionName == "mortal strike" ||
+            actionName == "heroic strike" ||
+            actionName == "cleave") &&
+            AI_VALUE2(bool, "trigger active", "melee light aoe") &&
+            bot->HasSpell(12292) &&
+            bot->IsSpellReady(12292) &&
+            !bot->HasAura(12292))
+        {
+            return 0.0f;
+        }
+
+        // Disable Battle Stance spam if SS is on CD
+        if ((actionName == "battle stance") &&
+            AI_VALUE2(bool, "trigger active", "melee light aoe") &&
+            bot->HasSpell(12292) &&
+            !bot->IsSpellReady(12292))
+        {
+            return 0.0f;
+        }
+
+        return 1.0f;
+    }
+};
+
+void ArmsWarriorAoeStrategy::InitCombatMultipliers(std::list<Multiplier*>& multipliers)
+{
+    multipliers.push_back(new WarriorSweepingStrikesPveMultiplier(ai));
+    multipliers.push_back(new WarriorSweepingStrikesRaidMultiplier(ai));
+    multipliers.push_back(new WarriorSweepingStrikesPvpMultiplier(ai));
+}

playerbot/strategy/warrior/ArmsWarriorStrategy.h

@@ -71,6 +71,7 @@ namespace ai
 
     protected:
         virtual void InitCombatTriggers(std::list<TriggerNode*>& triggers) override;
+        void InitCombatMultipliers(std::list<Multiplier*>& multipliers);
         virtual void InitNonCombatTriggers(std::list<TriggerNode*>& triggers) override;
     };