yolinlin/mod-playerbots
1
0
Fork 0
mirror of https://github.com/mod-playerbots/mod-playerbots synced 2025-11-29 15:58:20 +08:00
Code Issues Packages Projects Releases Wiki Activity

Allowed tank bots to reassign roles if human main tank is affected by fused armor

Browse Source
This commit is contained in:
avirar
2024-12-22 13:37:33 +11:00
committed by GitHub
parent 39c6fb8daa
commit a6de87f6fc
1 changed files with 36 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

50
src/strategy/raids/ulduar/RaidUlduarActions.cpp
View File

@@ -993,30 +993,52 @@ bool RazorscaleHarpoonAction::isUseful()
bool RazorscaleFuseArmorAction::isUseful() bool RazorscaleFuseArmorAction::isUseful()
{ {
if (!botAI->IsMainTank(bot)) // If this bot cannot tank at all, no need to do anything
if (!botAI->IsTank(bot))
return false; return false;
Aura* fuseArmor = bot->GetAura(RazorscaleBossHelper::SPELL_FUSEARMOR); // If this bot is the main tank AND has Fuse Armor at the threshold, return true immediately
if (!fuseArmor) if (botAI->IsMainTank(bot))
{
Aura* fuseArmor = bot->GetAura(RazorscaleBossHelper::SPELL_FUSEARMOR);
if (fuseArmor && fuseArmor->GetStackAmount() >= RazorscaleBossHelper::FUSEARMOR_THRESHOLD)
return true;
}
// Otherwise, check if there's any other main tank with high Fuse Armor
Group* group = bot->GetGroup();
if (!group)
return false; return false;
return fuseArmor->GetStackAmount() >= RazorscaleBossHelper::FUSEARMOR_THRESHOLD; for (GroupReference* gref = group->GetFirstMember(); gref; gref = gref->next())
{
Player* member = gref->GetSource();
if (!member)
continue;
if (botAI->IsMainTank(member) && member != bot)
{
Aura* fuseArmor = member->GetAura(RazorscaleBossHelper::SPELL_FUSEARMOR);
if (fuseArmor && fuseArmor->GetStackAmount() >= RazorscaleBossHelper::FUSEARMOR_THRESHOLD)
{
// There is another main tank with high Fuse Armor
return true;
}
}
}
return false;
} }
bool RazorscaleFuseArmorAction::Execute(Event event) bool RazorscaleFuseArmorAction::Execute(Event event)
{ {
if (!botAI->IsMainTank(bot)) // We already know from isUseful() that:
{ // 1) This bot can tank, AND
return false; // 2) There is at least one main tank (possibly this bot) with Fuse Armor >= threshold.
}
RazorscaleBossHelper bossHelper(botAI); RazorscaleBossHelper bossHelper(botAI);
bossHelper.AssignRolesBasedOnHealth();
// Check if the bot is still the main tank after reassignment // Attempt to reassign the roles based on health/Fuse Armor debuff
if (botAI->IsMainTank(bot)) bossHelper.AssignRolesBasedOnHealth();
{
return false;
}
return true; return true;
} }