[Balance] star cascade procs on impact and doesn't proc off procs

Author: gastank

engine/class_modules/sc_druid.cpp

@@ -7084,6 +7084,15 @@ struct ap_generator_t : public druid_spell_t
 
     if ( cast_state( s )->dream_burst )
       p()->active.dream_burst->execute_on_target( s->target );
+
+    // has an icd to prevent multiple procs, but we can just check only on the main target instead
+    if ( !proc && s->chain_target == 0 && rng().roll( cascade_chance ) )
+    {
+      if ( time_to_execute > 0_ms )
+        p()->spell_queued.star_cascade = target;
+      else
+        p()->active.star_cascade->execute_on_target( target );
+    }
   }
 
   void record_data( action_state_t* s ) override
@@ -7100,20 +7109,6 @@ struct ap_generator_t : public druid_spell_t
     }
   }
 
-  double gain_energize_resource( resource_e rt, double a, gain_t* g ) override
-  {
-    auto ret = druid_spell_t::gain_energize_resource( rt, a, g );
-    if ( ret && rng().roll( cascade_chance ) )
-    {
-      if ( time_to_execute > 0_ms )
-        p()->spell_queued.star_cascade = target;
-      else
-        p()->active.star_cascade->execute_on_target( target );
-    }
-
-    return ret;
-  }  
-
   void reset() override { druid_spell_t::reset(); dreamstate = false; }
 
   void cancel() override { druid_spell_t::cancel(); dreamstate = false; }