moving onupdate to worldscript

src/mod_autofish.cpp

@@ -1,86 +1,101 @@
 #include "ScriptMgr.h"
 #include "Player.h"
 #include "GameObject.h"
-#include "WorldSession.h"
 #include "ObjectAccessor.h"
-#include "CellImpl.h"
+#include "Config.h"
-#include "GridNotifiers.h"
-#include "GridNotifiersImpl.h"
 #include "LootMgr.h"
 
-class mod_autofish_PlayerScript : public PlayerScript
+namespace
 {
-public:
+    bool sEnabled = true;
-    mod_autofish_PlayerScript() : PlayerScript("mod_autofish_PlayerScript") {}
+    uint32 sTickMs = 200;
+    float sScanRange = 30.0f;
+    std::vector<uint32> sBobberEntries;
 
-    void OnUpdate(Player* player, uint32) override
+    std::vector<uint32> ParseEntryList(std::string const& csv)
     {
-        if (!player->IsAlive() || player->IsInCombat() || player->GetMap()->IsBattlegroundOrArena())
+        std::vector<uint32> out;
-            return;
+        std::stringstream ss(csv);
-
+        std::string item;
-        Acore::GameObjectInRangeCheck check(player, player, 10.0f);
+        while (std::getline(ss, item, ','))
-        Acore::GameObjectSearcher<Acore::GameObjectInRangeCheck> searcher(player, foundBobber, check);
-        player->VisitNearbyGridObject(10.0f, searcher);
-
-        if (!foundBobber)
-            return;
-
-        GameObject* go = foundBobber;
-        if (go->GetOwnerGUID() != player->GetGUID())
         {
-            foundBobber = nullptr;
+            item.erase(0, item.find_first_not_of(" \t\r\n"));
-            return;
+            item.erase(item.find_last_not_of(" \t\r\n") + 1);
+            if (!item.empty())
+                out.push_back(uint32(std::stoul(item)));
+        }
+        return out;
     }
 
-        if (go->GetGoState() != GO_STATE_READY)
+    void TryAutoFish(Player* plr)
     {
-            foundBobber = nullptr;
+        if (!plr || !plr->IsInWorld())
             return;
-        }
 
-        player->SendLoot(go->GetGUID(), LOOT_FISHING);
+        std::list<GameObject*> nearList;
+        for (auto entry : sBobberEntries)
+            plr->GetGameObjectListWithEntryInGrid(nearList, entry, sScanRange);
+
+        for (GameObject* go : nearList)
+        {
+            if (!go || go->GetOwnerGUID() != plr->GetGUID())
+                continue;
+
+            if (go->getLootState() == GO_READY)
+            {
+                plr->SendLoot(go->GetGUID(), LOOT_FISHING);
+
                 Loot* loot = &go->loot;
                 for (uint32 i = 0; i < loot->items.size(); ++i)
-            player->StoreLootItem(i, loot);
+                {
-        player->SendLootRelease(go->GetGUID());
+                    if (LootItem* li = loot->LootItemInSlot(i, plr))
-
+                        plr->AutoStoreLootItem(i, loot);
-        foundBobber = nullptr;
                 }
 
-private:
+                plr->SendLootRelease(go->GetGUID());
-    static GameObject* foundBobber;
+                go->SetLootState(GO_JUST_DEACTIVATED);
-};
+                return;
+            }
+        }
+    }
+}
 
-GameObject* mod_autofish_PlayerScript::foundBobber = nullptr;
+class AutoFish_WorldScript : public WorldScript
-
-class mod_autofish_WorldScript : public WorldScript
 {
 public:
-    mod_autofish_WorldScript() : WorldScript("mod_autofish_WorldScript") {}
+    AutoFish_WorldScript() : WorldScript("AutoFish_WorldScript") {}
 
-    void OnGameObjectCreate(GameObject* go) override
+    void OnAfterConfigLoad(bool) override
     {
-        if (go->GetGoType() == GAMEOBJECT_TYPE_FISHINGNODE)
+        sEnabled = sConfigMgr->GetOption<bool>("AutoFish.Enabled", true);
-            lastBobber = go;
+        sTickMs = sConfigMgr->GetOption<uint32>("AutoFish.TickMs", 200u);
+        sScanRange = sConfigMgr->GetOption<float>("AutoFish.ScanRange", 30.0f);
+        sBobberEntries = ParseEntryList(sConfigMgr->GetOption<std::string>("AutoFish.BobberEntries", "35591"));
     }
 
-    static GameObject* GetLastBobber() { return lastBobber; }
+    void OnUpdate(uint32 diff) override
-
-private:
-    static GameObject* lastBobber;
-};
-
-GameObject* mod_autofish_WorldScript::lastBobber = nullptr;
-
-class mod_autofish_GridNotifier : public Acore::GameObjectSearcher<Acore::GameObjectInRangeCheck>
     {
-public:
+        if (!sEnabled)
-    mod_autofish_GridNotifier(Player* p, GameObject*& result, Acore::GameObjectInRangeCheck const& check)
+            return;
-    : Acore::GameObjectSearcher<Acore::GameObjectInRangeCheck>(p, result, check) {}
-};
 
-void Addmod_autofishScripts()
+        static uint32 acc = 0;
+        acc += diff;
+        if (acc < sTickMs)
+            return;
+        acc = 0;
+
+        auto const& players = ObjectAccessor::GetPlayers();
+        for (auto const& kv : players)
         {
-    new mod_autofish_PlayerScript();
+            Player* plr = kv.second;
-    new mod_autofish_WorldScript();
+            if (!plr || !plr->IsInWorld() || plr->IsGameMaster())
+                continue;
+            TryAutoFish(plr);
+        }
+    }
+};
+
+void AddSC_autofish_world()
+{
+    new AutoFish_WorldScript();
 }