[DiscordArchive] I think in master the arcane charges are buffs and not power?
[DiscordArchive] I think in master the arcane charges are buffs and not power?
Archived author: Drikish • Posted: 2025-01-11T18:49:06.988000+00:00
Original source
I think in master the arcane charges are buffs and not power?
Archived author: Drikish • Posted: 2025-01-11T18:49:30.508000+00:00
Original source
nope take that back
Archived author: Drikish • Posted: 2025-01-11T18:49:40.376000+00:00
Original source
```c++
// Consume all arcane charges
if (int32 arcaneCharges = -caster->ModifyPower(POWER_ARCANE_CHARGES, -caster->GetMaxPower(POWER_ARCANE_CHARGES), false))
if (AuraEffect const* auraEffect = caster->GetAuraEffect(SPELL_MAGE_ARCANE_BARRAGE_R3, EFFECT_0, caster->GetGUID()))
caster->CastSpell(caster, SPELL_MAGE_ARCANE_BARRAGE_ENERGIZE, { SPELLVALUE_BASE_POINT0, arcaneCharges * auraEffect->GetAmount() / 100 });
```
Master ^
Archived author: Northstrider • Posted: 2025-01-11T18:49:51.793000+00:00
Original source
also you don't need to store the _primaryTarget, you have GetExplTargetUnit for that
Archived author: Drikish • Posted: 2025-01-11T18:50:09.798000+00:00
Original source
That comes from master also
Archived author: Drikish • Posted: 2025-01-11T18:50:13.742000+00:00
Original source
```c++
void MarkPrimaryTarget(SpellEffIndex /*effIndex*/)
{
_primaryTarget = GetHitUnit()->GetGUID();
}
```
Archived author: Drikish • Posted: 2025-01-11T18:50:27.463000+00:00
Original source
https://github.com/TrinityCore/TrinityCo...e.cpp#L177
Archived author: Drikish • Posted: 2025-01-11T18:51:04.914000+00:00
Original source
This handles whether or not the primary target is hit and if not it reduces the damage by effectIndex 1
Archived author: Drikish • Posted: 2025-01-11T18:51:26.506000+00:00
Original source
But it doesn't seem to handle Resonance which Shauren pointed out still exists so maybe thats just handled by the aura system automatically?
Archived author: Northstrider • Posted: 2025-01-11T18:52:05.807000+00:00
Original source
it doesn't handle the talent because it's not implemented. There is a 'new' hook to modify pct damage bonus