fix(Core/AI): Reduce movement leash extension interval. (#19923)

* Init.

* Use GetGameTime() instead of system call.

Co-Authored-By: Anton Popovichenko <walkline.ua@gmail.com>

* Update leash timer if rooted.

* Remove unnecessary checks, add extension for fanning.

More parity with VMaNGOS implementation.

---------

Co-authored-by: Anton Popovichenko <walkline.ua@gmail.com>

src/server/game/Entities/Creature/Creature.cpp

@@ -3678,7 +3678,7 @@ bool Creature::IsNotReachableAndNeedRegen() const
 std::shared_ptr<time_t> const& Creature::GetLastLeashExtensionTimePtr() const
 {
     if (m_lastLeashExtensionTime == nullptr)
-        m_lastLeashExtensionTime = std::make_shared<time_t>(time(nullptr));
+        m_lastLeashExtensionTime = std::make_shared<time_t>(GameTime::GetGameTime().count());
     return m_lastLeashExtensionTime;
 }
 
@@ -3699,7 +3699,7 @@ time_t Creature::GetLastLeashExtensionTime() const
 
 void Creature::UpdateLeashExtensionTime()
 {
-    (*GetLastLeashExtensionTimePtr()) = time(nullptr);
+    (*GetLastLeashExtensionTimePtr()) = GameTime::GetGameTime().count();
 }
 
 bool Creature::CanPeriodicallyCallForAssistance() const

src/server/game/Movement/MovementGenerators/TargetedMovementGenerator.cpp

@@ -78,8 +78,11 @@ bool ChaseMovementGenerator<T>::DoUpdate(T* owner, uint32 time_diff)
     {
         owner->StopMoving();
         _lastTargetPosition.reset();
-        if (Creature* cOwner2 = owner->ToCreature())
+        if (cOwner)
-            cOwner2->SetCannotReachTarget();
+        {
+            cOwner->UpdateLeashExtensionTime();
+            cOwner->SetCannotReachTarget();
+        }
         return true;
     }
 
@@ -128,8 +131,8 @@ bool ChaseMovementGenerator<T>::DoUpdate(T* owner, uint32 time_diff)
             {
                 i_recalculateTravel = false;
                 i_path = nullptr;
-                if (Creature* cOwner2 = owner->ToCreature())
+                if (cOwner)
-                    cOwner2->SetCannotReachTarget();
+                    cOwner->SetCannotReachTarget();
                 owner->StopMoving();
                 owner->SetInFront(target);
                 MovementInform(owner);
@@ -143,22 +146,25 @@ bool ChaseMovementGenerator<T>::DoUpdate(T* owner, uint32 time_diff)
     {
         i_recalculateTravel = false;
         i_path = nullptr;
-        if (Creature* cOwner2 = owner->ToCreature())
+        if (cOwner)
-            cOwner2->SetCannotReachTarget();
+            cOwner->SetCannotReachTarget();
         owner->ClearUnitState(UNIT_STATE_CHASE_MOVE);
         owner->SetInFront(target);
         MovementInform(owner);
     }
 
-    // Mobs should chase you infinitely if you stop and wait every few seconds.
+    if (owner->movespline->Finalized())
-    i_leashExtensionTimer.Update(time_diff);
+    { // Mobs should chase you infinitely if you stop and wait every few seconds.
-    if (i_leashExtensionTimer.Passed())
+        i_leashExtensionTimer.Update(time_diff);
-    {
+        if (i_leashExtensionTimer.Passed())
-        i_leashExtensionTimer.Reset(5000);
+        {
-        if (owner->HasUnitState(UNIT_STATE_CHASE) && owner->movespline->Finalized() && !target->isMoving())
+            i_leashExtensionTimer.Reset(1500);
-            if (Creature* creature = owner->ToCreature())
+            if (cOwner)
-                creature->UpdateLeashExtensionTime();
+                cOwner->UpdateLeashExtensionTime();
+        }
     }
+    else if (i_recalculateTravel)
+        i_leashExtensionTimer.Reset(1500);
 
     // if the target moved, we have to consider whether to adjust
     if (!_lastTargetPosition || target->GetPosition() != _lastTargetPosition.value() || mutualChase != _mutualChase || !owner->IsWithinLOSInMap(target))

src/server/game/Movement/MovementGenerators/TargetedMovementGenerator.h

@@ -39,7 +39,7 @@ class ChaseMovementGenerator : public MovementGeneratorMedium<T, ChaseMovementGe
 {
 public:
     ChaseMovementGenerator(Unit* target, Optional<ChaseRange> range = {}, Optional<ChaseAngle> angle = {})
-        : TargetedMovementGeneratorBase(target), i_leashExtensionTimer(5000), i_path(nullptr), i_recheckDistance(0), i_recalculateTravel(true), _range(range), _angle(angle) {}
+        : TargetedMovementGeneratorBase(target), i_leashExtensionTimer(1500), i_path(nullptr), i_recheckDistance(0), i_recalculateTravel(true), _range(range), _angle(angle) {}
     ~ChaseMovementGenerator() { }
 
     MovementGeneratorType GetMovementGeneratorType() { return CHASE_MOTION_TYPE; }