conf/playerbots.conf.dist

@@ -519,7 +519,7 @@ AiPlayerbot.AutoGearScoreLimit = 0
 # "taxi" (bots may use all flight paths, though they will not actually learn them)
 # "raid" (bots use cheats implemented into raid strategies)
 # To use multiple cheats, separate them by commas below (e.g., to enable all, use "gold,health,mana,power,raid,taxi")
-# Default: food, taxi, and raid are enabled
+# Default: taxi and raid are enabled
 AiPlayerbot.BotCheats = "food,taxi,raid"
 
 #
@@ -1603,10 +1603,10 @@ AiPlayerbot.PremadeSpecLink.11.6.80 = 05320021--230033312031500531353013251
 #
 #
 
-# Applies automatically refreshing buffs to bots simulating effects of spells, flasks, food, runes, etc.
+# Applies a permanent buff to all bots simulating effects of spells, flasks, food, runes, etc.
 # Requires sending the command "nc +worldbuff" in chat to a bot (or a group of bots) to enable
 # Each entry in the matrix should be formatted as follows: Entry:FactionID,ClassID,SpecID,MinimumLevel,MaximumLevel:SpellID1,SpellID2,etc.;
-# FactionID may be set to 0 for the entry to apply buffs to bots of either faction
+# Use 0 for any field to make it agnostic (e.g., 0 for FactionID means the entry will apply buffs to either faction)
 # The default entries create a cross-faction list of level 80 buffs for each implemented pve spec from the "Premade Specs" section
 # The default entries may be deleted or modified, and new custom entries may be added
 

src/strategy/actions/BattleGroundJoinAction.cpp

@@ -688,7 +688,7 @@ bool BGLeaveAction::Execute(Event event)
 
     WorldPacket packet(CMSG_BATTLEFIELD_PORT, 20);
     packet << type << unk2 << (uint32)_bgTypeId << unk << uint8(0);
-    bot->GetSession()->QueuePacket(new WorldPacket(packet));
+    bot->GetSession()->HandleBattleFieldPortOpcode(packet);
 
     if (IsRandomBot)
         botAI->SetMaster(nullptr);
@@ -917,7 +917,7 @@ bool BGStatusAction::Execute(Event event)
 
                     WorldPacket packet(CMSG_BATTLEFIELD_PORT, 20);
                     packet << type << unk2 << (uint32)_bgTypeId << unk << action;
-                    bot->GetSession()->QueuePacket(new WorldPacket(packet));
+                    bot->GetSession()->HandleBattleFieldPortOpcode(packet);
 
                     botAI->ResetStrategies(false);
                     if (!bot->GetBattleground())
@@ -972,7 +972,7 @@ bool BGStatusAction::Execute(Event event)
             WorldPacket packet(CMSG_BATTLEFIELD_PORT, 20);
             action = 0;
             packet << type << unk2 << (uint32)_bgTypeId << unk << action;
-            bot->GetSession()->QueuePacket(new WorldPacket(packet));
+            bot->GetSession()->HandleBattleFieldPortOpcode(packet);
 
             botAI->ResetStrategies(!IsRandomBot);
             botAI->GetAiObjectContext()->GetValue<uint32>("bg type")->Set(0);
@@ -1039,7 +1039,7 @@ bool BGStatusAction::Execute(Event event)
 
         WorldPacket packet(CMSG_BATTLEFIELD_PORT, 20);
         packet << type << unk2 << (uint32)_bgTypeId << unk << action;
-        bot->GetSession()->QueuePacket(new WorldPacket(packet));
+        bot->GetSession()->HandleBattleFieldPortOpcode(packet);
 
         botAI->ResetStrategies(false);
         if (!bot->GetBattleground())

src/strategy/shaman/ShamanActions.cpp

@@ -93,26 +93,19 @@ bool CastSpiritWalkAction::Execute(Event event)
 
 bool SetTotemAction::Execute(Event event)
 {
-    const size_t spellIdsCount = sizeof(totemSpellIds) / sizeof(uint32);
-    if (spellIdsCount == 0)
-        return false;  // early return
+    size_t spellIdsCount = sizeof(totemSpellIds) / sizeof(uint32);
 
     uint32 totemSpell = 0;
-
-    // Iterate backwards to prioritize the highest-rank totem spell the bot knows
-    for (size_t i = spellIdsCount; i-- > 0;)
+    for (int i = spellIdsCount - 1; i >= 0; --i)
     {
-        const uint32 spellId = totemSpellIds[i];
-        if (bot->HasSpell(spellId))
+        if (bot->HasSpell(totemSpellIds[i]))
         {
-            totemSpell = spellId;
+            totemSpell = totemSpellIds[i];
             break;
         }
     }
-
-    if (totemSpell == 0)
+    if (!totemSpell)
         return false;
-
     bot->addActionButton(actionButtonId, totemSpell, ACTION_BUTTON_SPELL);
     return true;
 }

src/strategy/shaman/ShamanTriggers.cpp

@@ -424,7 +424,6 @@ bool SetTotemTrigger::IsActive()
 {
     if (!bot->HasSpell(SPELL_CALL_OF_THE_ELEMENTS))
         return false;
-
     if (!bot->HasSpell(requiredSpellId))
         return false;
 
@@ -432,20 +431,13 @@ bool SetTotemTrigger::IsActive()
     if (!button || button->GetType() != ACTION_BUTTON_SPELL || button->GetAction() == 0)
         return true;
 
-    const size_t totemSpellIdsCount = sizeof(totemSpellIds) / sizeof(uint32);
-    if (totemSpellIdsCount == 0)
+    size_t totemSpellIdsCount = sizeof(totemSpellIds) / sizeof(uint32);
+    for (size_t i = 0; i < totemSpellIdsCount; ++i)
     {
-        return false;
-    }
-
-    for (int i = (int)totemSpellIdsCount - 1; i >= 0; --i)
-    {
-        const uint32 spellId = totemSpellIds[i];
-        if (bot->HasSpell(spellId))
+        if (button->GetAction() == totemSpellIds[i])
         {
-            return button->GetAction() != spellId;
+            return false;
         }
     }
-
-    return false;
+    return true;
 }