From 7a76a3b005e9b4786d7869edb79d9b1f7f4a0886 Mon Sep 17 00:00:00 2001
From: xenohedron <xenohedron@users.noreply.github.com>
Date: Sun, 2 Jun 2024 03:49:07 -0400
Subject: [PATCH] remove superfluous usage of isPhasedOutIndirectly

---
 Mage.Sets/src/mage/cards/s/SmashingSuccess.java           | 2 +-
 .../abilities/effects/common/DestroySourceEffect.java     | 4 +---
 .../abilities/effects/common/DestroyTargetEffect.java     | 8 +-------
 3 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/Mage.Sets/src/mage/cards/s/SmashingSuccess.java b/Mage.Sets/src/mage/cards/s/SmashingSuccess.java
index 4ea579052e3..2242d968a92 100644
--- a/Mage.Sets/src/mage/cards/s/SmashingSuccess.java
+++ b/Mage.Sets/src/mage/cards/s/SmashingSuccess.java
@@ -65,7 +65,7 @@ class SmashingSuccessEffect extends OneShotEffect {
     @Override
     public boolean apply(Game game, Ability source) {
         Permanent permanent = game.getPermanent(source.getFirstTarget());
-        if (permanent != null && permanent.isPhasedIn() && !permanent.isPhasedOutIndirectly()) {
+        if (permanent != null && permanent.isPhasedIn()) {
             if (permanent.isArtifact(game)) {
                 if (permanent.destroy(source, game, false)) {
                     Token token = new TreasureToken();
diff --git a/Mage/src/main/java/mage/abilities/effects/common/DestroySourceEffect.java b/Mage/src/main/java/mage/abilities/effects/common/DestroySourceEffect.java
index 73fb8e4012b..5cbee674863 100644
--- a/Mage/src/main/java/mage/abilities/effects/common/DestroySourceEffect.java
+++ b/Mage/src/main/java/mage/abilities/effects/common/DestroySourceEffect.java
@@ -36,9 +36,7 @@ public class DestroySourceEffect extends OneShotEffect {
     @Override
     public boolean apply(Game game, Ability source) {
         Permanent permanent = source.getSourcePermanentIfItStillExists(game);
-        if (permanent != null
-                && permanent.isPhasedIn()
-                && !permanent.isPhasedOutIndirectly()) {
+        if (permanent != null && permanent.isPhasedIn()) {
             permanent.destroy(source, game, noRegen);
             return true;
         }
diff --git a/Mage/src/main/java/mage/abilities/effects/common/DestroyTargetEffect.java b/Mage/src/main/java/mage/abilities/effects/common/DestroyTargetEffect.java
index df1944b5bce..5271b8fcb5f 100644
--- a/Mage/src/main/java/mage/abilities/effects/common/DestroyTargetEffect.java
+++ b/Mage/src/main/java/mage/abilities/effects/common/DestroyTargetEffect.java
@@ -6,10 +6,6 @@ import mage.abilities.effects.OneShotEffect;
 import mage.constants.Outcome;
 import mage.game.Game;
 import mage.game.permanent.Permanent;
-import mage.target.Target;
-import mage.target.targetpointer.FirstTargetPointer;
-import mage.target.targetpointer.SecondTargetPointer;
-import mage.util.CardUtil;
 
 import java.util.UUID;
 
@@ -53,9 +49,7 @@ public class DestroyTargetEffect extends OneShotEffect {
         int affectedTargets = 0;
         for (UUID permanentId : getTargetPointer().getTargets(game, source)) {
             Permanent permanent = game.getPermanent(permanentId);
-            if (permanent != null
-                    && permanent.isPhasedIn()
-                    && !permanent.isPhasedOutIndirectly()) {
+            if (permanent != null && permanent.isPhasedIn()) {
                 permanent.destroy(source, game, noRegen);
                 affectedTargets++;
             }