conf/playerbots.conf.dist

@@ -588,9 +588,6 @@ AiPlayerbot.LimitTalentsExpansion = 0
 # Default: 1 (enabled)
 AiPlayerbot.EnableRandomBotTrading = 1
 
-# Configure message prefixes which will be excluded in analysis in trade action to open trade window
-AiPlayerbot.TradeActionExcludedPrefixes = "RPLL_H_,DBMv4,{звезда} Questie,{rt1} Questie"
-
 #
 #
 #
@@ -1958,13 +1955,13 @@ AiPlayerbot.AllowedLogFiles = ""
 ####################################################################################################
 # A list of gameObject GUID's that are not allowed for bots to interact with.
 #
-AiPlayerbot.DisallowedGameObjects = 176213,17155,2656,74448,19020,3719,3658,3705,3706,105579,75293,17155,2857,179490
+AiPlayerbot.DisallowedGameObjects = 176213,17155,2656,74448,19020,3719,3658,3705,3706,105579,75293,17155
 #
 # List of GUID's:
 # QuestItems:
 # 176213 = Blood of Heroes, 17155 = Defias Gunpowder, 2656 = Waterlogged Envelope
 # Chests:
-# Large Solid Chest = 74448, Box of Assorted Parts = 19020, Food Crate = 3719, Water Barrel = 3658, Barrel of Milk = 3705, Barrel of sweet Nectar = 3706, Tattered Chest = 105579, Large bettered Chest = 75293, Solid Chest = 2857, Battered Foodlocker = 179490
+# Large Solid Chest = 74448, Box of Assorted Parts = 19020, Food Crate = 3719, Water Barrel = 3658, Barrel of Milk = 3705, Barrel of sweet Nectar = 3706, Tattered Chest = 105579, Large bettered Chest = 75293
 # Feel free to edit and help to complete.
 #
 ####################################################################################################

src/PlayerbotAI.cpp

@@ -4430,48 +4430,9 @@ void PlayerbotAI::RemoveShapeshift()
     // RemoveAura("tree of life");
 }
 
-// Mirrors Blizzard’s GetAverageItemLevel rules :
-// https://wowpedia.fandom.com/wiki/API_GetAverageItemLevel
-uint32 PlayerbotAI::GetEquipGearScore(Player* player)
-{
-    constexpr uint8 TOTAL_SLOTS = 17;                      // every slot except Body & Tabard
-    uint32 sumLevel = 0;
-
-    /* ---------- 0.  Detect “ignore off-hand” situations --------- */
-    Item* main = player->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_MAINHAND);
-    Item* off  = player->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_OFFHAND);
-
-    bool ignoreOffhand = false;                           // true → divisor = 16
-    if (main)
-    {
-        bool twoHand = (main->GetTemplate()->InventoryType == INVTYPE_2HWEAPON);
-        if (twoHand && !player->HasAura(SPELL_TITAN_GRIP))
-            ignoreOffhand = true;                         // classic 2-hander
-    }
-    else if (!off)                                        // both hands empty
-        ignoreOffhand = true;
-
-    /* ---------- 1.  Sum up item-levels -------------------------- */
-    for (uint8 slot = EQUIPMENT_SLOT_START; slot < EQUIPMENT_SLOT_END; ++slot)
-    {
-        if (slot == EQUIPMENT_SLOT_BODY || slot == EQUIPMENT_SLOT_TABARD)
-            continue;                                     // Blizzard never counts these
-
-        if (ignoreOffhand && slot == EQUIPMENT_SLOT_OFFHAND)
-            continue;                                     // skip off-hand in 2-H case
-
-        if (Item* it = player->GetItemByPos(INVENTORY_SLOT_BAG_0, slot))
-            sumLevel += it->GetTemplate()->ItemLevel;     // missing items add 0
-    }
-
-    /* ---------- 2.  Divide by 17 or 16 -------------------------- */
-    const uint8 divisor = ignoreOffhand ? TOTAL_SLOTS - 1 : TOTAL_SLOTS; // 16 or 17
-    return sumLevel / divisor;
-}
-
 // NOTE : function rewritten as flags "withBags" and "withBank" not used, and _fillGearScoreData sometimes attribute
 // one-hand/2H Weapon in wrong slots 
-/*uint32 PlayerbotAI::GetEquipGearScore(Player* player)
+uint32 PlayerbotAI::GetEquipGearScore(Player* player)
 {
     // This function aims to calculate the equipped gear score
  
@@ -4493,11 +4454,11 @@ uint32 PlayerbotAI::GetEquipGearScore(Player* player)
             if (!player->HasAura(SPELL_TITAN_GRIP) && mh_type == INVTYPE_2HWEAPON && i == SLOT_MAIN_HAND)
                 sum += item->GetTemplate()->ItemLevel;
 	    }
-    } 
+    }
 
     uint32 gs = uint32(sum / count);
     return gs;
-}*/
+}
 
 /*uint32 PlayerbotAI::GetEquipGearScore(Player* player, bool withBags, bool withBank)
 {

src/PlayerbotAIConfig.cpp

@@ -4,7 +4,9 @@
  */
 
 #include "PlayerbotAIConfig.h"
+
 #include <iostream>
+
 #include "Config.h"
 #include "PlayerbotDungeonSuggestionMgr.h"
 #include "PlayerbotFactory.h"
@@ -463,8 +465,6 @@ bool PlayerbotAIConfig::Initialize()
 
     LoadListString<std::vector<std::string>>(sConfigMgr->GetOption<std::string>("AiPlayerbot.AllowedLogFiles", ""),
                                              allowedLogFiles);
-    LoadListString<std::vector<std::string>>(sConfigMgr->GetOption<std::string>("AiPlayerbot.TradeActionExcludedPrefixes", ""),
-                                             tradeActionExcludedPrefixes);
 
     worldBuffs.clear();
 

src/PlayerbotAIConfig.h

@@ -255,7 +255,6 @@ public:
     uint32 iterationsPerTick;
 
     std::mutex m_logMtx;
-    std::vector<std::string> tradeActionExcludedPrefixes;
     std::vector<std::string> allowedLogFiles;
     std::unordered_map<std::string, std::pair<FILE*, bool>> logFiles;
 

src/strategy/actions/ChatActionContext.h

@@ -73,7 +73,6 @@
 #include "UseItemAction.h"
 #include "UseMeetingStoneAction.h"
 #include "WhoAction.h"
-#include "WipeAction.h"
 #include "WtsAction.h"
 #include "OpenItemAction.h"
 #include "UnlockItemAction.h"
@@ -186,7 +185,6 @@ public:
         creators["join"] = &ChatActionContext::join;
         creators["lfg"] = &ChatActionContext::lfg;
         creators["calc"] = &ChatActionContext::calc;
-        creators["wipe"] = &ChatActionContext::wipe;
     }
 
 private:
@@ -292,7 +290,6 @@ private:
     static Action* tell_estimated_dps(PlayerbotAI* ai) { return new TellEstimatedDpsAction(ai); }
     static Action* join(PlayerbotAI* ai) { return new JoinGroupAction(ai); }
     static Action* calc(PlayerbotAI* ai) { return new TellCalculateItemAction(ai); }
-    static Action* wipe(PlayerbotAI* ai) { return new WipeAction(ai); }
 };
 
 #endif

src/strategy/actions/TradeAction.cpp

@@ -4,6 +4,7 @@
  */
 
 #include "TradeAction.h"
+
 #include "ChatHelper.h"
 #include "Event.h"
 #include "ItemCountValue.h"
@@ -14,8 +15,11 @@ bool TradeAction::Execute(Event event)
 {
     std::string const text = event.getParam();
 
+    // Table with prefixes to be excluded from analysis
+    static const std::vector<std::string> excludedPrefixes = {"RPLL_H_"};
+
     // If text starts with any excluded prefix, don't process it further.
-    for (const auto& prefix : sPlayerbotAIConfig->tradeActionExcludedPrefixes)
+    for (const auto& prefix : excludedPrefixes)
     {
         if (text.find(prefix) == 0)
             return false;

src/strategy/actions/WipeAction.cpp

@@ -1,18 +0,0 @@
-/*
- * Copyright (C) 2016+ AzerothCore <www.azerothcore.org>, released under GNU GPL v2 license, you may redistribute it
- * and/or modify it under version 2 of the License, or (at your option), any later version.
- */
-
-#include "PlayerbotAI.h"
-#include "WipeAction.h"
-
-bool WipeAction::Execute(Event event)
-{
-    Player* master = event.getOwner();
-
-    if (botAI->GetMaster()->GetGUID() != event.getOwner()->GetGUID())
-        return false;
-
-    bot->Kill(bot, bot);
-    return true;
-}

src/strategy/actions/WipeAction.h

@@ -1,24 +0,0 @@
-/*
- * Copyright (C) 2016+ AzerothCore <www.azerothcore.org>, released under GNU GPL v2 license, you may redistribute it
- * and/or modify it under version 2 of the License, or (at your option), any later version.
- */
-
-#ifndef _PLAYERBOT_WIPEACTION_H
-#define _PLAYERBOT_WIPEACTION_H
-
-#include "Action.h"
-
-class PlayerbotAI;
-
-class WipeAction : public Action
-{
-public:
-    WipeAction(PlayerbotAI* botAI) : Action(botAI, "wipe") {}
-
-    bool Execute(Event event) override;
-
-private:
-    std::string bossName;
-};
-
-#endif

src/strategy/generic/ChatCommandHandlerStrategy.cpp

@@ -100,8 +100,6 @@ void ChatCommandHandlerStrategy::InitTriggers(std::vector<TriggerNode*>& trigger
 	    new TriggerNode("unlock items", NextAction::array(0, new NextAction("unlock items", relevance), nullptr)));
     triggers.push_back(
 	    new TriggerNode("unlock traded item", NextAction::array(0, new NextAction("unlock traded item", relevance), nullptr)));
-    triggers.push_back(
-        new TriggerNode("wipe", NextAction::array(0, new NextAction("wipe", relevance), nullptr)));
 }
 
 ChatCommandHandlerStrategy::ChatCommandHandlerStrategy(PlayerbotAI* botAI) : PassTroughStrategy(botAI)

src/strategy/triggers/ChatTriggerContext.h

@@ -132,7 +132,6 @@ public:
         creators["disperse"] = &ChatTriggerContext::disperse;
         creators["calc"] = &ChatTriggerContext::calc;
         creators["qi"] = &ChatTriggerContext::qi;
-        creators["wipe"] = &ChatTriggerContext::wipe;
     }
 
 private:
@@ -244,7 +243,6 @@ private:
     static Trigger* disperse(PlayerbotAI* ai) { return new ChatCommandTrigger(ai, "disperse"); }
     static Trigger* calc(PlayerbotAI* ai) { return new ChatCommandTrigger(ai, "calc"); }
     static Trigger* qi(PlayerbotAI* botAI) { return new ChatCommandTrigger(botAI, "qi"); }
-    static Trigger* wipe(PlayerbotAI* botAI) { return new ChatCommandTrigger(botAI, "wipe"); }
 };
 
 #endif