Ulduar flame leviathan (normal mode)

src/strategy/actions/ListSpellsAction.cpp

@@ -139,6 +139,9 @@ std::vector<std::pair<uint32, std::string>> ListSpellsAction::GetSpellList(std::
     {
         if (itr->second->State == PLAYERSPELL_REMOVED || !itr->second->Active)
             continue;
+        
+        if (!(itr->second->specMask & bot->GetActiveSpecMask()))
+            continue;
 
         SpellInfo const* spellInfo = sSpellMgr->GetSpellInfo(itr->first);
         if (!spellInfo)

src/strategy/actions/MovementActions.cpp

@@ -881,9 +881,9 @@ bool MovementAction::IsMovingAllowed(Unit* target)
     if (bot->GetMapId() != target->GetMapId())
         return false;
 
-    float distance = sServerFacade->GetDistance2d(bot, target);
-    if (!bot->InBattleground() && distance > sPlayerbotAIConfig->reactDistance)
-        return false;
+    // float distance = sServerFacade->GetDistance2d(bot, target);
+    // if (!bot->InBattleground() && distance > sPlayerbotAIConfig->reactDistance)
+    //     return false;
 
     return IsMovingAllowed();
 }
@@ -893,9 +893,10 @@ bool MovementAction::IsMovingAllowed(uint32 mapId, float x, float y, float z)
     // removed sqrt as means distance limit was effectively 22500 (ReactDistance<63>)
     // leaving it commented incase we find ReactDistance limit causes problems
     // float distance = sqrt(bot->GetDistance(x, y, z));
-    float distance = bot->GetDistance(x, y, z);
-    if (!bot->InBattleground() && distance > sPlayerbotAIConfig->reactDistance)
-        return false;
+    
+    // Remove react distance limit
+    // if (!bot->InBattleground())
+    //     return false;
 
     return IsMovingAllowed();
 }
@@ -923,6 +924,7 @@ bool MovementAction::IsWaitingForLastMove(MovementPriority priority)
     if (lastMove.lastdelayTime + lastMove.msTime > getMSTime())
         return true;
 
+
     return false;
 }
 

src/strategy/actions/UseMeetingStoneAction.cpp

@@ -175,6 +175,12 @@ bool SummonAction::Teleport(Player* summoner, Player* player)
     //         }
     //     }
     // }
+    if (player->GetVehicle())
+    {
+        botAI->TellError("You cannot summon me while I'm on a vehicle");
+        return false;
+    }
+
     if (!summoner->IsBeingTeleported() && !player->IsBeingTeleported())
     {
         float followAngle = GetFollowAngle();
@@ -222,6 +228,7 @@ bool SummonAction::Teleport(Player* summoner, Player* player)
                 }
 
                 player->GetMotionMaster()->Clear();
+                AI_VALUE(LastMovement&, "last movement").clear();
                 player->TeleportTo(mapId, x, y, z, 0);
                 return true;
             }

src/strategy/actions/VehicleActions.cpp

@@ -7,8 +7,11 @@
 
 #include "BattlegroundIC.h"
 #include "ItemVisitors.h"
+#include "ObjectDefines.h"
 #include "Playerbots.h"
+#include "QuestValues.h"
 #include "ServerFacade.h"
+#include "Unit.h"
 #include "Vehicle.h"
 
 // TODO methods to enter/exit vehicle should be added to BGTactics or MovementAction (so that we can better control
@@ -20,6 +23,21 @@ bool EnterVehicleAction::Execute(Event event)
     // do not switch vehicles yet
     if (bot->GetVehicle())
         return false;
+    
+    Player* master = botAI->GetMaster();
+    // Triggered by a chat command
+    if (event.getOwner() && master && master->GetTarget())
+    {
+        Unit* vehicleBase = botAI->GetUnit(master->GetTarget());
+        if (!vehicleBase)
+            return false;
+        Vehicle* veh = vehicleBase->GetVehicleKit();
+        if (vehicleBase->IsVehicle() && veh && veh->GetAvailableSeatCount())
+        {
+            return EnterVehicle(vehicleBase, false);
+        }
+        return false;
+    }
 
     GuidVector npcs = AI_VALUE(GuidVector, "nearest vehicles");
     for (GuidVector::iterator i = npcs.begin(); i != npcs.end(); i++)
@@ -27,6 +45,9 @@ bool EnterVehicleAction::Execute(Event event)
         Unit* vehicleBase = botAI->GetUnit(*i);
         if (!vehicleBase)
             continue;
+        
+        if (vehicleBase->HasUnitFlag(UNIT_FLAG_NOT_SELECTABLE))
+            continue;
 
         // dont let them get in the cannons as they'll stay forever and do nothing useful
         // dont let them in catapult they cant use them at all
@@ -44,27 +65,36 @@ bool EnterVehicleAction::Execute(Event event)
         if (vehicleBase->GetVehicleKit()->IsVehicleInUse())
             continue;
 
-        float dist = sServerFacade->GetDistance2d(bot, vehicleBase);
-        if (dist > 40.0f)
-            continue;
-
-        if (dist > INTERACTION_DISTANCE)
-            return MoveTo(vehicleBase);
-
-        bot->EnterVehicle(vehicleBase);
-
-        if (!bot->IsOnVehicle(vehicleBase))
-            return false;
-
-        // dismount because bots can enter vehicle on mount
-        WorldPacket emptyPacket;
-        bot->GetSession()->HandleCancelMountAuraOpcode(emptyPacket);
-        return true;
+        if (EnterVehicle(vehicleBase, true))
+            return true;
     }
 
     return false;
 }
 
+bool EnterVehicleAction::EnterVehicle(Unit* vehicleBase, bool moveIfFar)
+{
+    float dist = sServerFacade->GetDistance2d(bot, vehicleBase);
+    if (dist > 40.0f)
+        return false;
+
+    if (dist > INTERACTION_DISTANCE && !moveIfFar)
+        return false;
+
+    if (dist > INTERACTION_DISTANCE)
+        return MoveTo(vehicleBase);
+    // Use HandleSpellClick instead of Unit::EnterVehicle to handle special vehicle script (ulduar)
+    vehicleBase->HandleSpellClick(bot);
+
+    if (!bot->IsOnVehicle(vehicleBase))
+        return false;
+
+    // dismount because bots can enter vehicle on mount
+    WorldPacket emptyPacket;
+    bot->GetSession()->HandleCancelMountAuraOpcode(emptyPacket);
+    return true;
+}
+
 bool LeaveVehicleAction::Execute(Event event)
 {
     Vehicle* myVehicle = bot->GetVehicle();

src/strategy/actions/VehicleActions.h

@@ -17,6 +17,8 @@ public:
     EnterVehicleAction(PlayerbotAI* botAI, std::string const& name = "enter vehicle") : MovementAction(botAI, name) {}
 
     bool Execute(Event event) override;
+protected:
+    bool EnterVehicle(Unit* vehicleBase, bool moveIfFar);
 };
 
 class LeaveVehicleAction : public MovementAction