ShatteredDawn
diff --git a/‎src/Ai/Base/Actions/AcceptBattlegroundInvitationAction.cpp‎
Lines changed: 3 additions & 3 deletions b/‎src/Ai/Base/Actions/AcceptBattlegroundInvitationAction.cpp‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎src/Ai/Base/Actions/AddLootAction.cpp‎
Lines changed: 1 addition & 1 deletion b/‎src/Ai/Base/Actions/AddLootAction.cpp‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Ai/Base/Actions/AreaTriggerAction.cpp‎
Lines changed: 1 addition & 1 deletion b/‎src/Ai/Base/Actions/AreaTriggerAction.cpp‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Ai/Base/Actions/AutoMaintenanceOnLevelupAction.cpp‎
Lines changed: 5 additions & 4 deletions b/‎src/Ai/Base/Actions/AutoMaintenanceOnLevelupAction.cpp‎
Lines changed: 5 additions & 4 deletions
diff --git a/‎src/Ai/Base/Actions/BankAction.cpp‎
Lines changed: 1 addition & 1 deletion b/‎src/Ai/Base/Actions/BankAction.cpp‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Ai/Base/Actions/BattleGroundJoinAction.cpp‎
Lines changed: 4 additions & 15 deletions b/‎src/Ai/Base/Actions/BattleGroundJoinAction.cpp‎
Lines changed: 4 additions & 15 deletions
diff --git a/‎src/Ai/Base/Actions/BattleGroundTactics.cpp‎
Lines changed: 15 additions & 20 deletions b/‎src/Ai/Base/Actions/BattleGroundTactics.cpp‎
Lines changed: 15 additions & 20 deletions
diff --git a/‎src/Ai/Base/Actions/BossAuraActions.cpp‎
Lines changed: 4 additions & 4 deletions b/‎src/Ai/Base/Actions/BossAuraActions.cpp‎
Lines changed: 4 additions & 4 deletions
diff --git a/‎src/Ai/Base/Actions/CancelChannelAction.cpp‎
Lines changed: 3 additions & 1 deletion b/‎src/Ai/Base/Actions/CancelChannelAction.cpp‎
Lines changed: 3 additions & 1 deletion
diff --git a/‎src/Ai/Base/Actions/CastCustomSpellAction.cpp‎
Lines changed: 1 addition & 1 deletion b/‎src/Ai/Base/Actions/CastCustomSpellAction.cpp‎
Lines changed: 1 addition & 1 deletion
@@ -6,9 +6,9 @@
 #include "AcceptBattlegroundInvitationAction.h"
 
 #include "Event.h"
-#include "Playerbots.h"
+#include "PlayerbotAI.h"
 
-bool AcceptBgInvitationAction::Execute(Event event)
+bool AcceptBgInvitationAction::Execute(Event)
 {
     uint8 type = 0;                      // arenatype if arena
     uint8 unk2 = 0;                      // unk, can be 0x0 (may be if was invited?) and 0x1
@@ -18,9 +18,9 @@ bool AcceptBgInvitationAction::Execute(Event event)
 
     WorldPacket packet(CMSG_BATTLEFIELD_PORT, 20);
     packet << type << unk2 << (uint32)bgTypeId_ << unk << action;
-    // packet << bgTypeId_ << action;
     bot->GetSession()->HandleBattleFieldPortOpcode(packet);
 
     botAI->ResetStrategies();
+
     return true;
 }
@@ -22,7 +22,7 @@ bool AddLootAction::Execute(Event event)
     return AI_VALUE(LootObjectStack*, "available loot")->Add(guid);
 }
 
-bool AddAllLootAction::Execute(Event event)
+bool AddAllLootAction::Execute(Event)
 {
     bool added = false;
 
 
@@ -58,7 +58,7 @@ bool ReachAreaTriggerAction::Execute(Event event)
     return true;
 }
 
-bool AreaTriggerAction::Execute(Event event)
+bool AreaTriggerAction::Execute(Event)
 {
     LastMovement& movement = context->GetValue<LastMovement&>("last area trigger")->Get();
 
 
@@ -1,19 +1,20 @@
 #include "AutoMaintenanceOnLevelupAction.h"
 
-#include "GuildMgr.h"
+#include "SpellMgr.h"
+
 #include "PlayerbotAIConfig.h"
 #include "PlayerbotFactory.h"
-#include "Playerbots.h"
 #include "RandomPlayerbotMgr.h"
 #include "SharedDefines.h"
 #include "BroadcastHelper.h"
 
-bool AutoMaintenanceOnLevelupAction::Execute(Event event)
+bool AutoMaintenanceOnLevelupAction::Execute(Event)
 {
     AutoPickTalents();
     AutoLearnSpell();
     AutoUpgradeEquip();
     AutoTeleportForLevel();
+
     return true;
 }
 
@@ -72,7 +73,7 @@ void AutoMaintenanceOnLevelupAction::LearnSpells(std::ostringstream* out)
         LearnQuestSpells(out);
 }
 
-void AutoMaintenanceOnLevelupAction::LearnTrainerSpells(std::ostringstream* out)
+void AutoMaintenanceOnLevelupAction::LearnTrainerSpells(std::ostringstream*)
 {
     PlayerbotFactory factory(bot, bot->GetLevel());
     factory.InitSkills();
 
@@ -27,7 +27,7 @@ bool BankAction::Execute(Event event)
     return false;
 }
 
-bool BankAction::ExecuteBank(std::string const text, Unit* bank)
+bool BankAction::ExecuteBank(std::string const text, Unit*)
 {
     if (text.empty() || text == "?")
     {
 
@@ -13,9 +13,8 @@
 #include "PlayerbotAI.h"
 #include "Playerbots.h"
 #include "PositionValue.h"
-#include "UpdateTime.h"
 
-bool BGJoinAction::Execute(Event event)
+bool BGJoinAction::Execute(Event)
 {
     uint32 queueType = AI_VALUE(uint32, "bg type");
     if (!queueType)  // force join to fill bg
@@ -25,8 +24,6 @@ bool BGJoinAction::Execute(Event event)
 
         BattlegroundQueueTypeId queueTypeId = (BattlegroundQueueTypeId)bgList[urand(0, bgList.size() - 1)];
         BattlegroundTypeId bgTypeId = BattlegroundMgr::BGTemplateId(queueTypeId);
-        BattlegroundBracketId bracketId;
-        bool isArena = false;
         bool isRated = false;
 
         Battleground* bg = sBattlegroundMgr->GetBattlegroundTemplate(bgTypeId);
@@ -38,12 +35,8 @@ bool BGJoinAction::Execute(Event event)
         if (!pvpDiff)
             return false;
 
-        bracketId = pvpDiff->GetBracketId();
-
         if (ArenaType type = ArenaType(BattlegroundMgr::BGArenaType(queueTypeId)))
         {
-            isArena = true;
-
             std::vector<uint32>::iterator i = find(ratedList.begin(), ratedList.end(), queueTypeId);
             if (i != ratedList.end())
                 isRated = true;
@@ -409,8 +402,6 @@ bool BGJoinAction::JoinQueue(uint32 type)
 
     bracketId = pvpDiff->GetBracketId();
 
-    uint32 BracketSize = bg->GetMaxPlayersPerTeam() * 2;
-    uint32 TeamSize = bg->GetMaxPlayersPerTeam();
     TeamId teamId = bot->GetTeamId();
 
     // check if already in queue
@@ -487,8 +478,6 @@ bool BGJoinAction::JoinQueue(uint32 type)
     if (isArena)
     {
         isArena = true;
-        BracketSize = type * 2;
-        TeamSize = type;
         isRated = botAI->GetAiObjectContext()->GetValue<uint32>("arena type")->Get();
 
         if (joinAsGroup)
@@ -653,7 +642,7 @@ bool FreeBGJoinAction::shouldJoinBg(BattlegroundQueueTypeId queueTypeId, Battleg
     return false;
 }
 
-bool BGLeaveAction::Execute(Event event)
+bool BGLeaveAction::Execute(Event)
 {
     if (!(bot->InBattlegroundQueue() || bot->InBattleground()))
         return false;
@@ -1064,7 +1053,7 @@ bool BGStatusAction::Execute(Event event)
     return true;
 }
 
-bool BGStatusCheckAction::Execute(Event event)
+bool BGStatusCheckAction::Execute(Event)
 {
     if (bot->IsBeingTeleported())
         return false;
@@ -1080,7 +1069,7 @@ bool BGStatusCheckAction::Execute(Event event)
 
 bool BGStatusCheckAction::isUseful() { return bot->InBattlegroundQueue(); }
 
-bool BGStrategyCheckAction::Execute(Event event)
+bool BGStrategyCheckAction::Execute(Event)
 {
     bool inside_bg = bot->InBattleground() && bot->GetBattleground();
     ;
 
@@ -1364,8 +1364,8 @@ std::string const BGTactics::HandleConsoleCommandPrivate(WorldSession* session,
                 }
             }
         }
-        uint32 min = 0u;
-        uint32 max = vPaths->size() - 1;
+        int64_t min = 0u;
+        int64_t max = vPaths->size() - 1;
         if (num >= 0)  // num specified or found
         {
             if (num > max)
@@ -1557,7 +1557,7 @@ bool BGTactics::eyJumpDown()
 //
 // actual bg tactics below
 //
-bool BGTactics::Execute(Event event)
+bool BGTactics::Execute(Event)
 {
     Battleground* bg = bot->GetBattleground();
     if (!bg)
@@ -2185,16 +2185,6 @@ bool BGTactics::selectObjective(bool reset)
                 case 3:  // Balanced
                     defendersProhab = 3;
                     break;
-                case 4:
-                case 5:
-                case 6:
-                case 7:  // Heavy Offense
-                    defendersProhab = 1;
-                    break;
-                case 8:
-                case 9:  // Heavy Defense
-                    defendersProhab = 6;
-                    break;
             }
 
             if (enemyStrategy == WS_STRATEGY_DEFENSIVE)
@@ -2226,9 +2216,14 @@ bool BGTactics::selectObjective(bool reset)
                     target.Relocate(enemyFC->GetPositionX(), enemyFC->GetPositionY(), enemyFC->GetPositionZ());
             }
             // Graveyard Camping if in lead
-            else if (!hasFlag && role < 8 &&
-                (team == TEAM_ALLIANCE && allianceScore == 2 && hordeScore == 0) ||
-                (team == TEAM_HORDE && hordeScore == 2 && allianceScore == 0))
+            else if (
+                !hasFlag
+                && role < 8
+                && (
+                    (team == TEAM_ALLIANCE && allianceScore == 2 && hordeScore == 0)
+                    || (team == TEAM_HORDE && hordeScore == 2 && allianceScore == 0)
+                )
+            )
             {
                 if (team == TEAM_ALLIANCE)
                     SetSafePos(WS_GY_CAMPING_HORDE, 10.0f);
@@ -2497,7 +2492,6 @@ bool BGTactics::selectObjective(bool reset)
             EYBotStrategy strategyHorde = static_cast<EYBotStrategy>(GetBotStrategyForTeam(bg, TEAM_HORDE));
             EYBotStrategy strategyAlliance = static_cast<EYBotStrategy>(GetBotStrategyForTeam(bg, TEAM_ALLIANCE));
             EYBotStrategy strategy = (team == TEAM_ALLIANCE) ? strategyAlliance : strategyHorde;
-            EYBotStrategy enemyStrategy = (team == TEAM_ALLIANCE) ? strategyHorde : strategyAlliance;
 
             auto IsOwned = [&](uint32 nodeId) -> bool
             { return eyeOfTheStormBG->GetCapturePointInfo(nodeId)._ownerTeamId == team; };
@@ -3231,7 +3225,6 @@ bool BGTactics::selectObjectiveWp(std::vector<BattleBotPath*> const& vPaths)
     if (bgType == BATTLEGROUND_RB)
         bgType = bg->GetBgTypeID(true);
 
-    PositionMap& posMap = context->GetValue<PositionMap&>("position")->Get();
     PositionInfo pos = context->GetValue<PositionMap&>("position")->Get()["bg objective"];
     if (!pos.isSet())
         return false;
@@ -3326,7 +3319,9 @@ bool BGTactics::selectObjectiveWp(std::vector<BattleBotPath*> const& vPaths)
 
         // don't pick path where bot is already closest to the paths closest point to target (it means path cant lead it
         // anywhere) don't pick path where closest point is too far away
-        if (closestPointIndex == (reverse ? 0 : path->size() - 1) || closestPointDistToBot > botDistanceLimit)
+        const int64_t pathSize = path->size() - 1;
+
+        if (closestPointIndex == (reverse ? 0 : pathSize) || closestPointDistToBot > botDistanceLimit)
             continue;
 
         // creates a score based on dist-to-bot and dist-to-destination, where lower is better, and dist-to-bot is more
@@ -4249,7 +4244,7 @@ bool BGTactics::IsLockedInsideKeep()
     return false;
 }
 
-bool ArenaTactics::Execute(Event event)
+bool ArenaTactics::Execute(Event)
 {
     if (!bot->InBattleground())
     {
 
@@ -18,7 +18,7 @@ bool BossFireResistanceAction::isUseful()
     return bossFireResistanceTrigger.IsActive();
 }
 
-bool BossFireResistanceAction::Execute(Event event)
+bool BossFireResistanceAction::Execute(Event)
 {
     PaladinFireResistanceStrategy paladinFireResistanceStrategy(botAI);
     botAI->ChangeStrategy(ADD_STRATEGY_CHAR + paladinFireResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);
@@ -32,7 +32,7 @@ bool BossFrostResistanceAction::isUseful()
     return bossFrostResistanceTrigger.IsActive();
 }
 
-bool BossFrostResistanceAction::Execute(Event event)
+bool BossFrostResistanceAction::Execute(Event)
 {
     PaladinFrostResistanceStrategy paladinFrostResistanceStrategy(botAI);
     botAI->ChangeStrategy(ADD_STRATEGY_CHAR + paladinFrostResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);
@@ -46,7 +46,7 @@ bool BossNatureResistanceAction::isUseful()
     return bossNatureResistanceTrigger.IsActive();
 }
 
-bool BossNatureResistanceAction::Execute(Event event)
+bool BossNatureResistanceAction::Execute(Event)
 {
     HunterNatureResistanceStrategy hunterNatureResistanceStrategy(botAI);
     botAI->ChangeStrategy(ADD_STRATEGY_CHAR + hunterNatureResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);
@@ -60,7 +60,7 @@ bool BossShadowResistanceAction::isUseful()
     return bossShadowResistanceTrigger.IsActive();
 }
 
-bool BossShadowResistanceAction::Execute(Event event)
+bool BossShadowResistanceAction::Execute(Event)
 {
     PaladinShadowResistanceStrategy paladinShadowResistanceStrategy(botAI);
     botAI->ChangeStrategy(ADD_STRATEGY_CHAR + paladinShadowResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);
 
@@ -7,12 +7,14 @@
 #include "Player.h"
 #include "PlayerbotAI.h"
 
-bool CancelChannelAction::Execute(Event event)
+bool CancelChannelAction::Execute(Event)
 {
     if (bot->GetCurrentSpell(CURRENT_CHANNELED_SPELL))
     {
         bot->InterruptSpell(CURRENT_CHANNELED_SPELL);
+
         return true;
     }
+
     return false;
 }
@@ -334,7 +334,7 @@ bool CastRandomSpellAction::castSpell(uint32 spellId, WorldObject* wo)
         return botAI->CastSpell(spellId, wo->GetPositionX(), wo->GetPositionY(), wo->GetPositionZ());
 }
 
-bool DisEnchantRandomItemAction::Execute(Event event)
+bool DisEnchantRandomItemAction::Execute(Event)
 {
     std::vector<Item*> items =
         AI_VALUE2(std::vector<Item*>, "inventory items", "usage " + std::to_string(ITEM_USAGE_DISENCHANT));
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ bool AddLootAction::Execute(Event event)`
`22`	`22`	`return AI_VALUE(LootObjectStack*, "available loot")->Add(guid);`
`23`	`23`	`}`
`24`	`24`
`25`		`-bool AddAllLootAction::Execute(Event event)`
	`25`	`+bool AddAllLootAction::Execute(Event)`
`26`	`26`	`{`
`27`	`27`	`bool added = false;`
`28`	`28`
Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ bool ReachAreaTriggerAction::Execute(Event event)`
`58`	`58`	`return true;`
`59`	`59`	`}`
`60`	`60`
`61`		`-bool AreaTriggerAction::Execute(Event event)`
	`61`	`+bool AreaTriggerAction::Execute(Event)`
`62`	`62`	`{`
`63`	`63`	`LastMovement& movement = context->GetValue<LastMovement&>("last area trigger")->Get();`
`64`	`64`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ bool BankAction::Execute(Event event)`
`27`	`27`	`return false;`
`28`	`28`	`}`
`29`	`29`
`30`		`-bool BankAction::ExecuteBank(std::string const text, Unit* bank)`
	`30`	`+bool BankAction::ExecuteBank(std::string const text, Unit*)`
`31`	`31`	`{`
`32`	`32`	`if (text.empty() \|\| text == "?")`
`33`	`33`	`{`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ bool BossFireResistanceAction::isUseful()`
`18`	`18`	`return bossFireResistanceTrigger.IsActive();`
`19`	`19`	`}`
`20`	`20`
`21`		`-bool BossFireResistanceAction::Execute(Event event)`
	`21`	`+bool BossFireResistanceAction::Execute(Event)`
`22`	`22`	`{`
`23`	`23`	`PaladinFireResistanceStrategy paladinFireResistanceStrategy(botAI);`
`24`	`24`	`botAI->ChangeStrategy(ADD_STRATEGY_CHAR + paladinFireResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);`
`@@ -32,7 +32,7 @@ bool BossFrostResistanceAction::isUseful()`
`32`	`32`	`return bossFrostResistanceTrigger.IsActive();`
`33`	`33`	`}`
`34`	`34`
`35`		`-bool BossFrostResistanceAction::Execute(Event event)`
	`35`	`+bool BossFrostResistanceAction::Execute(Event)`
`36`	`36`	`{`
`37`	`37`	`PaladinFrostResistanceStrategy paladinFrostResistanceStrategy(botAI);`
`38`	`38`	`botAI->ChangeStrategy(ADD_STRATEGY_CHAR + paladinFrostResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);`
`@@ -46,7 +46,7 @@ bool BossNatureResistanceAction::isUseful()`
`46`	`46`	`return bossNatureResistanceTrigger.IsActive();`
`47`	`47`	`}`
`48`	`48`
`49`		`-bool BossNatureResistanceAction::Execute(Event event)`
	`49`	`+bool BossNatureResistanceAction::Execute(Event)`
`50`	`50`	`{`
`51`	`51`	`HunterNatureResistanceStrategy hunterNatureResistanceStrategy(botAI);`
`52`	`52`	`botAI->ChangeStrategy(ADD_STRATEGY_CHAR + hunterNatureResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);`
`@@ -60,7 +60,7 @@ bool BossShadowResistanceAction::isUseful()`
`60`	`60`	`return bossShadowResistanceTrigger.IsActive();`
`61`	`61`	`}`
`62`	`62`
`63`		`-bool BossShadowResistanceAction::Execute(Event event)`
	`63`	`+bool BossShadowResistanceAction::Execute(Event)`
`64`	`64`	`{`
`65`	`65`	`PaladinShadowResistanceStrategy paladinShadowResistanceStrategy(botAI);`
`66`	`66`	`botAI->ChangeStrategy(ADD_STRATEGY_CHAR + paladinShadowResistanceStrategy.getName(), BotState::BOT_STATE_COMBAT);`
Original file line number	Diff line number	Diff line change
`@@ -7,12 +7,14 @@`
`7`	`7`	`#include "Player.h"`
`8`	`8`	`#include "PlayerbotAI.h"`
`9`	`9`
`10`		`-bool CancelChannelAction::Execute(Event event)`
	`10`	`+bool CancelChannelAction::Execute(Event)`
`11`	`11`	`{`
`12`	`12`	`if (bot->GetCurrentSpell(CURRENT_CHANNELED_SPELL))`
`13`	`13`	`{`
`14`	`14`	`bot->InterruptSpell(CURRENT_CHANNELED_SPELL);`
	`15`	`+`
`15`	`16`	`return true;`
`16`	`17`	`}`
	`18`	`+`
`17`	`19`	`return false;`
`18`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -334,7 +334,7 @@ bool CastRandomSpellAction::castSpell(uint32 spellId, WorldObject* wo)`
`334`	`334`	`return botAI->CastSpell(spellId, wo->GetPositionX(), wo->GetPositionY(), wo->GetPositionZ());`
`335`	`335`	`}`
`336`	`336`
`337`		`-bool DisEnchantRandomItemAction::Execute(Event event)`
	`337`	`+bool DisEnchantRandomItemAction::Execute(Event)`
`338`	`338`	`{`
`339`	`339`	`std::vector<Item*> items =`
`340`	`340`	`AI_VALUE2(std::vector<Item*>, "inventory items", "usage " + std::to_string(ITEM_USAGE_DISENCHANT));`