GameObject: Despawn/Refill chests 5 min after first opening even if n…

…ot looted

src/game/Entities/GameObject.cpp

@@ -98,7 +98,7 @@ GameObject::GameObject() : WorldObject(),
     m_isInUse = false;
     m_reStockTimer = 0;
     m_rearmTimer = 0;
-    m_despawnTimer = 0;
+    m_despawnTimer = TimePoint::max();
 
     m_delayedActionTimer = 0;
 
@@ -548,9 +548,7 @@ void GameObject::Update(const uint32 diff)
                 case GAMEOBJECT_TYPE_CHEST:
                     if (m_loot)
                     {
-                        if (m_loot->IsChanged())
-                            m_despawnTimer = time(nullptr) + 5 * MINUTE; // TODO:: need to add a define?
-                        else if (m_despawnTimer != 0 && m_despawnTimer <= time(nullptr))
+                        if (m_despawnTimer <= GetMap()->GetCurrentClockTime())
                             m_lootState = GO_JUST_DEACTIVATED;
 
                         m_loot->Update();
@@ -655,7 +653,7 @@ void GameObject::Update(const uint32 diff)
                     SetLootState(GO_READY);
                     return; // SetLootState and return because go is treated as "burning flag" due to GetGoAnimProgress() being 100 and would be removed on the client
                 case GAMEOBJECT_TYPE_CHEST:
-                    m_despawnTimer = 0;
+                    m_despawnTimer = TimePoint::max();
                     // consumable confirmed to override chest restock
                     if (!m_goInfo->chest.consumable && m_goInfo->chest.chestRestockTime)
                     {
@@ -774,6 +772,11 @@ void GameObject::Heartbeat()
         AI()->OnHeartbeat();
 }
 
+void GameObject::SetChestDespawn()
+{
+    m_despawnTimer = GetMap()->GetCurrentClockTime() + std::chrono::minutes(5);
+}
+
 void GameObject::Refresh()
 {
     // not refresh despawned not casted GO (despawned casted GO destroyed in all cases anyway)

src/game/Entities/GameObject.h

@@ -820,6 +820,7 @@ class GameObject : public WorldObject
         uint32 GetRespawnDelay() const { return m_respawnDelay; }
         void SetRespawnDelay(uint32 delay, bool once = false) { m_respawnDelay = delay; m_respawnOverriden = true; m_respawnOverrideOnce = once; }
         void SetForcedDespawn() { m_forcedDespawn = true; };
+        void SetChestDespawn();
         void Refresh();
         void Delete();
 
@@ -1006,7 +1007,7 @@ class GameObject : public WorldObject
         // Used for chest type
         bool m_isInUse;                                     // only one player at time are allowed to open chest
         time_t m_reStockTimer;                              // timer to refill the chest
-        time_t m_despawnTimer;                              // timer to despawn the chest if something changed in it
+        TimePoint m_despawnTimer;                           // timer to despawn the chest if something changed in it
 
         void TriggerSummoningRitual();
         void TriggerDelayedAction();

src/game/Loot/LootMgr.cpp

@@ -1327,7 +1327,7 @@ void Loot::Release(Player* player)
                 {
                     if (!IsLootedForAll())
                     {
-                        updateClients = true;
+                        go->SetChestDespawn(); // chests despawn after 5 min even if nothing looted
                         break;
                     }