From d27e764e24919a4f63ee5df96cba435a253b7ba3 Mon Sep 17 00:00:00 2001
From: DustinLin <39450497+DustinLin@users.noreply.github.com>
Date: Sun, 4 Aug 2024 12:14:16 -0700
Subject: [PATCH] [Bug] Adding check to apply attr on moves that dont fail
 (parting shot) (#3195)

---
 src/phases.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/phases.ts b/src/phases.ts
index 3c4d7f1ce44..362f5f22cc2 100644
--- a/src/phases.ts
+++ b/src/phases.ts
@@ -2998,7 +2998,7 @@ export class MoveEffectPhase extends PokemonPhase {
           }
 
           applyAttrs.push(new Promise(resolve => {
-            applyFilteredMoveAttrs((attr: MoveAttr) => attr instanceof MoveEffectAttr && attr.trigger === MoveEffectTrigger.PRE_APPLY && (!attr.firstHitOnly || firstHit) && (!attr.lastHitOnly || lastHit),
+            applyFilteredMoveAttrs((attr: MoveAttr) => attr instanceof MoveEffectAttr && attr.trigger === MoveEffectTrigger.PRE_APPLY && (!attr.firstHitOnly || firstHit) && (!attr.lastHitOnly || lastHit) && hitResult !== HitResult.NO_EFFECT,
               user, target, move).then(() => {
               if (hitResult !== HitResult.FAIL) {
                 const chargeEffect = !!move.getAttrs(ChargeAttr).find(ca => ca.usedChargeEffect(user, this.getTarget(), move));