Attempting a manual coord refresh 3 seconds after teleport

src/teleport-to-node.cpp

@@ -4,17 +4,18 @@
 #include "ChatCommand.h"
 #include "Player.h"
 #include "GameObject.h"
-#include "ObjectAccessor.h"
+#include "BasicEvent.h"
-#include "Timer.h"
 
 #include <vector>
 #include <limits>
+#include <cstdint>
 
 using namespace Acore::ChatCommands;
 
 static constexpr float TELEPORT_DISTANCE = 250.0f;
-static constexpr float Z_BUMP = 1.0f;
+static constexpr float Z_BUMP = 1.5f;
-static constexpr uint32 RECHECK_INTERVAL_MS = 5000;
+static constexpr float MIN_RETELEPORT_DIST = 1.0f;
+static constexpr uint32 RECHECK_DELAY_MS = 3000;
 
 static const std::vector<uint32> kVeinEntries = {
     324,1610,1667,1731,1732,1733,1734,2054,2055,3763,3764,19903,
@@ -25,7 +26,7 @@ static const std::vector<uint32> kVeinEntries = {
     189978,189979,189980,189981,195036,
 };
 
-static GameObject* FindNearestVein(Player* player, float maxRange)
+static GameObject* FindNearestReadyVein(Player* player, float maxRange)
 {
     GameObject* nearest = nullptr;
     float nearestDist = std::numeric_limits<float>::infinity();
@@ -34,40 +35,72 @@ static GameObject* FindNearestVein(Player* player, float maxRange)
     {
         if (GameObject* go = player->FindNearestGameObject(entry, maxRange))
         {
-            float dist = player->GetDistance(go);
+            if (go->getLootState() != LootState::GO_READY)
-            if (dist <= maxRange && dist < nearestDist)
+                continue;
+            if (go->GetGoState() != GOState::GO_STATE_READY)
+                continue;
+
+            float d = player->GetDistance(go);
+            if (d <= maxRange && d < nearestDist)
             {
                 nearest = go;
-                nearestDist = dist;
+                nearestDist = d;
             }
         }
     }
     return nearest;
 }
 
-static void DumpNearbyNodes(Player* player)
+class TeleNodeRecheckEvent : public BasicEvent
 {
-    ChatHandler(player->GetSession()).PSendSysMessage("Nearby node coordinates:");
+public:
-    for (uint32 entry : kVeinEntries)
+    TeleNodeRecheckEvent(Player* player, uint64 lastNodeGuidRaw)
+        : _player(player), _lastNodeGuidRaw(lastNodeGuidRaw) { }
+
+    bool Execute(uint64 /*time*/, uint32 /*diff*/) override
     {
-        if (GameObject* go = player->FindNearestGameObject(entry, TELEPORT_DISTANCE))
+        if (!_player || !_player->IsInWorld())
+            return true;
+
+        if (GameObject* node = FindNearestReadyVein(_player, TELEPORT_DISTANCE))
         {
-            ChatHandler(player->GetSession()).PSendSysMessage(
+            if (node->GetGUID().GetRawValue() != _lastNodeGuidRaw)
-                "Entry %u -> Map %u X: %.2f Y: %.2f Z: %.2f",
+            {
-                entry, go->GetMapId(), go->GetPositionX(), go->GetPositionY(), go->GetPositionZ()
+                if (_player->GetDistance(node) >= MIN_RETELEPORT_DIST)
+                {
+                    _player->TeleportTo(
+                        node->GetMapId(),
+                        node->GetPositionX(),
+                        node->GetPositionY(),
+                        node->GetPositionZ() + Z_BUMP,
+                        node->GetOrientation()
                     );
                 }
             }
         }
+        return true;
+    }
 
-static void TeleportToNearest(Player* player)
+private:
+    Player* _player;
+    uint64 _lastNodeGuidRaw;
+};
+
+static bool DoTeleNode(Player* player)
 {
     if (!player)
-        return;
+        return false;
 
-    GameObject* node = FindNearestVein(player, TELEPORT_DISTANCE);
+    if (player->IsInCombat())
-    if (node)
     {
+        ChatHandler(player->GetSession()).PSendSysMessage("You can't use this while in combat.");
+        return true;
+    }
+
+    if (GameObject* node = FindNearestReadyVein(player, TELEPORT_DISTANCE))
+    {
+        uint64 lastGuid = node->GetGUID().GetRawValue();
+
         player->TeleportTo(
             node->GetMapId(),
             node->GetPositionX(),
@@ -75,41 +108,15 @@ static void TeleportToNearest(Player* player)
             node->GetPositionZ() + Z_BUMP,
             node->GetOrientation()
         );
-        ChatHandler(player->GetSession()).PSendSysMessage("Teleported to nearest node.");
+
+        player->m_Events.AddEvent(new TeleNodeRecheckEvent(player, lastGuid),
+                                  player->m_Events.CalculateTime(RECHECK_DELAY_MS));
     }
     else
     {
-        ChatHandler(player->GetSession()).PSendSysMessage("No node found within range.");
+        ChatHandler(player->GetSession()).PSendSysMessage("No mining nodes found within range.");
-    }
     }
 
-class TeleNodeRunnable : public BasicEvent
-{
-public:
-    explicit TeleNodeRunnable(Player* player) : _player(player) { }
-    bool Execute(uint64 /*time*/, uint32 /*diff*/) override
-    {
-        if (!_player || !_player->IsInWorld())
-            return true;
-
-        DumpNearbyNodes(_player);
-        TeleportToNearest(_player);
-        _player->m_Events.AddEvent(new TeleNodeRunnable(_player), _player->m_Events.CalculateTime(RECHECK_INTERVAL_MS));
-        return true;
-    }
-
-private:
-    Player* _player;
-};
-
-static bool HandleTeleNode(ChatHandler* handler)
-{
-    Player* player = handler->GetSession()->GetPlayer();
-    if (!player)
-        return false;
-
-    player->m_Events.AddEvent(new TeleNodeRunnable(player), player->m_Events.CalculateTime(0));
-    handler->PSendSysMessage("telenode started: re-evaluating every 5 seconds.");
     return true;
 }
 
@@ -126,6 +133,23 @@ public:
         };
         return commandTable;
     }
+
+private:
+    static bool HandleTeleNode(ChatHandler* handler)
+    {
+        if (!handler)
+            return false;
+
+        Player* player = handler->GetSession() ? handler->GetSession()->GetPlayer() : nullptr;
+        if (!player)
+        {
+            handler->SendSysMessage("Player only.");
+            handler->SetSentErrorMessage(true);
+            return false;
+        }
+
+        return DoTeleNode(player);
+    }
 };
 
 void AddSC_telenode_commandscript()