[DK] Nazgrim's Conquest (allegedly) now triggers off free cost rune spenders

Author: nyterage

engine/class_modules/sc_death_knight.cpp

@@ -12704,20 +12704,20 @@ double death_knight_t::resource_loss( resource_e resource_type, double amount, g
     if ( talent.rune_mastery.ok() )
       buffs.rune_mastery->trigger();
 
+    if ( talent.rider.nazgrims_conquest.ok() && buffs.apocalyptic_conquest->check() )
+    {
+      debug_cast<buffs::apocalyptic_conquest_buff_t*>( buffs.apocalyptic_conquest )->nazgrims_conquest +=
+          as<int>( amount );
+      invalidate_cache( CACHE_STRENGTH );
+    }
+
     // Proc Chance does not appear to be in data, using testing data that is current as of 4/19/2024
     if ( talent.rider.riders_champion.ok() && rng().roll( 0.2 ) )
       summon_rider( spell.summon_whitemane_2->duration() );
 
     // Effects that require the player to actually spend runes
     if ( actual_amount > 0 )
     {
-      if ( talent.rider.nazgrims_conquest.ok() && buffs.apocalyptic_conquest->check() &&
-           action->data().id() != spec.remorseless_winter->id() )
-      {
-        debug_cast<buffs::apocalyptic_conquest_buff_t*>( buffs.apocalyptic_conquest )->nazgrims_conquest +=
-            as<int>( amount );
-        invalidate_cache( CACHE_STRENGTH );
-      }
     }
   }
 

engine/sim/proc_rng.hpp

@@ -281,6 +281,8 @@ struct accumulated_rng_t : public proc_rng_t
 
   void reset( reset_type_e reset_type ) override;
   int trigger( action_state_t* = nullptr ) override;
+  void set_chance( double c )
+  { proc_chance = c; }
 };
 
 // Threshold RNG rng helper class ==========================