Skip to content

Commit

Permalink
[shaman] Flame Shock is affected by Master of the Elements
Browse files Browse the repository at this point in the history
track Flash of Lightning only if it is selected
  • Loading branch information
Bloodmallet committed Apr 26, 2023
1 parent 3fde152 commit fcbc792
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions engine/class_modules/sc_shaman.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6786,6 +6786,10 @@ struct flame_shock_t : public shaman_spell_t
{
parse_options( options_str );

if ( p()->is_ptr() ) {
affected_by_master_of_the_elements = true;
}

// Ensure Flame Shock is single target, since Simulationcraft naively interprets a
// Max Targets value on a spell to mean "aoe this many targets"
aoe = 0;
Expand Down Expand Up @@ -9921,6 +9925,10 @@ void shaman_t::trigger_splintered_elements( action_t* secondary )

void shaman_t::trigger_flash_of_lightning()
{
if ( !talent.flash_of_lightning.ok() ) {
return;
}

if ( !talent.stormkeeper.enabled() && !talent.stormkeeper2.enabled() &&
!talent.storm_elemental.enabled() && !talent.totemic_recall.enabled() )
{
Expand Down

0 comments on commit fcbc792

Please sign in to comment.