Fix Azgalor melee Rain of Fire target swapping

src/Ai/Raid/HyjalSummit/Action/RaidHyjalSummitActions.cpp

@@ -750,14 +750,21 @@ bool AzgalorMeleeGetOutOfFireAction::Execute(Event /*event*/)
     if (!azgalor || !IsBotInsideActiveAzgalorRainOfFire(bot, 17.0f))
         return false;
 
-    if (Unit* doomguard = AI_VALUE2(Unit*, "find target", "lesser doomguard"))
+    Unit* doomguard = AI_VALUE2(Unit*, "find target", "lesser doomguard");
+    bool isTargetingDoomguard =
+        doomguard && (bot->GetVictim() == doomguard || bot->GetTarget() == doomguard->GetGUID());
+    if (isTargetingDoomguard)
     {
-        if (bot->GetVictim() != doomguard)
-            return Attack(doomguard);
+        if (bot->GetVictim() != azgalor || bot->GetTarget() != azgalor->GetGUID())
+            return Attack(azgalor);
 
-        return true;
+        return false;
     }
 
+    if (doomguard &&
+        (bot->GetVictim() != doomguard || bot->GetTarget() != doomguard->GetGUID()))
+        return Attack(doomguard);
+
     return MoveAway(azgalor, 5.0f);
 }
 

src/Ai/Raid/HyjalSummit/Util/RaidHyjalSummitHelpers.cpp

@@ -13,34 +13,29 @@
 
 namespace HyjalSummitHelpers
 {
-    namespace
+    std::unordered_map<ObjectGuid, RainOfFireData>* GetActiveAzgalorRainOfFire(uint32 instanceId)
     {
-        constexpr uint32 AZGALOR_RAIN_OF_FIRE_DURATION = 10000;
+        auto instanceIt = rainOfFirePosition.find(instanceId);
+        if (instanceIt == rainOfFirePosition.end())
+            return nullptr;
 
-        std::unordered_map<ObjectGuid, RainOfFireData>* GetActiveAzgalorRainOfFire(uint32 instanceId)
+        uint32 now = getMSTime();
+        auto& dynObjMap = instanceIt->second;
+        for (auto it = dynObjMap.begin(); it != dynObjMap.end(); )
         {
-            auto instanceIt = rainOfFirePosition.find(instanceId);
-            if (instanceIt == rainOfFirePosition.end())
-                return nullptr;
-
-            uint32 now = getMSTime();
-            auto& dynObjMap = instanceIt->second;
-            for (auto it = dynObjMap.begin(); it != dynObjMap.end(); )
-            {
-                if (getMSTimeDiff(it->second.spawnTime, now) >= AZGALOR_RAIN_OF_FIRE_DURATION)
-                    it = dynObjMap.erase(it);
-                else
-                    ++it;
-            }
-
-            if (dynObjMap.empty())
-            {
-                rainOfFirePosition.erase(instanceIt);
-                return nullptr;
-            }
+            if (getMSTimeDiff(it->second.spawnTime, now) >= AZGALOR_RAIN_OF_FIRE_DURATION)
+                it = dynObjMap.erase(it);
+            else
+                ++it;
+        }
 
-            return &dynObjMap;
+        if (dynObjMap.empty())
+        {
+            rainOfFirePosition.erase(instanceIt);
+            return nullptr;
         }
+
+        return &dynObjMap;
     }
 
     // General

src/Ai/Raid/HyjalSummit/Util/RaidHyjalSummitHelpers.h

@@ -90,13 +90,15 @@ namespace HyjalSummitHelpers
     extern const Position AZGALOR_TANK_FINAL_POSITION;
     extern const Position AZGALOR_DOOMGUARD_POSITION;
     extern std::unordered_map<ObjectGuid, TankPositionState> azgalorTankStep;
+    constexpr uint32 AZGALOR_RAIN_OF_FIRE_DURATION = 10000;
     struct RainOfFireData
     {
         Position position;
         uint32 spawnTime;
     };
     extern std::unordered_map<uint32, std::unordered_map<ObjectGuid, RainOfFireData>> rainOfFirePosition;
     TankPositionState GetAzgalorTankPositionState(PlayerbotAI* botAI, Player* bot);
+    std::unordered_map<ObjectGuid, RainOfFireData>* GetActiveAzgalorRainOfFire(uint32 instanceId);
     bool IsBotInsideActiveAzgalorRainOfFire(Player* bot, float radius);
     bool AnyGroupMemberHasDoom(Player* bot);