Merge pull request #3105 from DominicJamesWhite/patch-1

Update bone_shield.go
wowsims · May 20, 2023 · de054c3 · de054c3
2 parents 42502d2 + c9e5a22
commit de054c3
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/sim/deathknight/bone_shield.go b/sim/deathknight/bone_shield.go
@@ -28,12 +28,14 @@ func (dk *Deathknight) registerBoneShieldSpell() {
 			dk.BoneShieldAura.SetStacks(sim, dk.BoneShieldAura.MaxStacks)
 		},
 		OnSpellHitTaken: func(aura *core.Aura, sim *core.Simulation, spell *core.Spell, result *core.SpellResult) {
-			if sim.CurrentTime > stackRemovalCd+2*time.Second {
-				stackRemovalCd = sim.CurrentTime
+			if result.Landed() {
+				if sim.CurrentTime > stackRemovalCd+2*time.Second {
+					stackRemovalCd = sim.CurrentTime
 
-				aura.RemoveStack(sim)
-				if aura.GetStacks() == 0 {
-					aura.Deactivate(sim)
+					aura.RemoveStack(sim)
+					if aura.GetStacks() == 0 {
+						aura.Deactivate(sim)
+					}
 				}
 			}
 		},