Hotfix after recent changes related to the refactoring of IsMovingAllowed (mod-playerbots#1933)

Hotfix for the issue where bots can't enter/move on transports
(elevators, zeppelins, ships, etc.). Change the logic to determine if
bot is on a vehicle. According to the current implementation in
AzerothCore, `GetVehicle()` is the most common approach for this.
Additionally, other checks related to the vehicle: `GetBase`, `IsAlive`,
`GetVehicleInfo`, and `GetSeatForPassenger` -- are processed inside
`IsInVehicle`.

This should be more than enough to determine if the bot is on a vehicle
and can/cant control it.

Issue: mod-playerbots#1927

Author: privatecore

src/strategy/actions/MovementActions.cpp

@@ -972,8 +972,7 @@ bool MovementAction::IsMovingAllowed()
     // Vehicle state: is in the vehicle and can control it (rare, content-specific).
     // We need to check charmed state AFTER vehicle one, cuz that's how it works:
     // passengers are set to charmed by vehicle with CHARM_TYPE_VEHICLE.
-    if ((bot->HasUnitMovementFlag(MOVEMENTFLAG_ONTRANSPORT) && !botAI->IsInVehicle(true)) ||
-        bot->IsCharmed())
+    if ((bot->GetVehicle() && !botAI->IsInVehicle(true)) || bot->IsCharmed())
         return false;
 
     return true;