refactor: replacement effects don't use apply method now (#11484)

magefree · Nov 28, 2023 · 05e2cf1 · 05e2cf1
1 parent 8ad3238
commit 05e2cf1
Show file tree

Hide file tree

Showing 275 changed files with 165 additions and 1,539 deletions.
diff --git a/Mage.Sets/src/mage/cards/a/AbandonedSarcophagus.java b/Mage.Sets/src/mage/cards/a/AbandonedSarcophagus.java
@@ -74,11 +74,6 @@ public AbandonedSarcophagusReplacementEffect copy() {
         return new AbandonedSarcophagusReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player controller = game.getPlayer(source.getControllerId());

diff --git a/Mage.Sets/src/mage/cards/a/Abundance.java b/Mage.Sets/src/mage/cards/a/Abundance.java
@@ -56,11 +56,6 @@ public AbundanceReplacementEffect copy() {
         return new AbundanceReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player controller = game.getPlayer(event.getPlayerId());

diff --git a/Mage.Sets/src/mage/cards/a/AcolytesReward.java b/Mage.Sets/src/mage/cards/a/AcolytesReward.java
@@ -69,11 +69,6 @@ public void init(Ability source, Game game) {
         amount = DevotionCount.W.calculate(game, source, this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         boolean result = false;

diff --git a/Mage.Sets/src/mage/cards/a/AegisOfHonor.java b/Mage.Sets/src/mage/cards/a/AegisOfHonor.java
@@ -62,11 +62,6 @@ private AegisOfHonorEffect(final AegisOfHonorEffect card) {
     }
 
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean checksEventType(GameEvent event, Game game) {
         return event.getType() == GameEvent.EventType.DAMAGE_PLAYER;

diff --git a/Mage.Sets/src/mage/cards/a/AlhammarretsArchive.java b/Mage.Sets/src/mage/cards/a/AlhammarretsArchive.java
@@ -89,11 +89,6 @@ public AlhammarretsArchiveReplacementEffect copy() {
         return new AlhammarretsArchiveReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player controller = game.getPlayer(source.getControllerId());

diff --git a/Mage.Sets/src/mage/cards/a/AlmsCollector.java b/Mage.Sets/src/mage/cards/a/AlmsCollector.java
@@ -64,11 +64,6 @@ public AlmsCollectorReplacementEffect copy() {
         return new AlmsCollectorReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player controller = game.getPlayer(source.getControllerId());

diff --git a/Mage.Sets/src/mage/cards/a/AnafenzaTheForemost.java b/Mage.Sets/src/mage/cards/a/AnafenzaTheForemost.java
@@ -81,11 +81,6 @@ public AnafenzaTheForemostEffect copy() {
         return new AnafenzaTheForemostEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player controller = game.getPlayer(source.getControllerId());

diff --git a/Mage.Sets/src/mage/cards/a/AnthemOfRakdos.java b/Mage.Sets/src/mage/cards/a/AnthemOfRakdos.java
@@ -78,11 +78,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
         return game.getControllerId(event.getSourceId()).equals(source.getControllerId()) && HellbentCondition.instance.apply(game, source);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         event.setAmount(CardUtil.overflowMultiply(event.getAmount(), 2));

diff --git a/Mage.Sets/src/mage/cards/a/ArchmageAscension.java b/Mage.Sets/src/mage/cards/a/ArchmageAscension.java
@@ -80,11 +80,6 @@ public ArchmageAscensionReplacementEffect copy() {
         return new ArchmageAscensionReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player player = game.getPlayer(event.getPlayerId());

diff --git a/Mage.Sets/src/mage/cards/a/ArchonOfCoronation.java b/Mage.Sets/src/mage/cards/a/ArchonOfCoronation.java
@@ -79,11 +79,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
                 && source.isControlledBy(game.getMonarchId());
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return false;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         event.setAmount(0);

diff --git a/Mage.Sets/src/mage/cards/a/ArlinnThePacksHope.java b/Mage.Sets/src/mage/cards/a/ArlinnThePacksHope.java
@@ -81,11 +81,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
         return permanent != null && permanent.isControlledBy(source.getControllerId()) && permanent.isCreature(game);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return false;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Permanent target = ((EntersTheBattlefieldEvent) event).getTarget();

diff --git a/Mage.Sets/src/mage/cards/a/AvacynGuardianAngel.java b/Mage.Sets/src/mage/cards/a/AvacynGuardianAngel.java
@@ -109,11 +109,6 @@ private AvacynGuardianAngelPreventToCreaturePreventionEffect(AvacynGuardianAngel
         this.color = effect.color;
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean applies(GameEvent event, Ability source, Game game) {
         if (!super.applies(event, source, game)
@@ -175,11 +170,6 @@ private AvacynGuardianAngelPreventToPlayerPreventionEffect(AvacynGuardianAngelPr
         this.color = effect.color;
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean applies(GameEvent event, Ability source, Game game) {
         if (!super.applies(event, source, game) || !event.getTargetId().equals(getTargetPointer().getFirst(game, source))) {

diff --git a/Mage.Sets/src/mage/cards/b/BarbarianClass.java b/Mage.Sets/src/mage/cards/b/BarbarianClass.java
@@ -105,11 +105,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
         return source.isControlledBy(event.getPlayerId());
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return false;
-    }
-
     @Override
     public BarbarianClassEffect copy() {
         return new BarbarianClassEffect(this);

diff --git a/Mage.Sets/src/mage/cards/b/BenevolentUnicorn.java b/Mage.Sets/src/mage/cards/b/BenevolentUnicorn.java
@@ -63,11 +63,6 @@ public BenevolentUnicornEffect copy() {
         return new BenevolentUnicornEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         event.setAmount(event.getAmount() - 1);

diff --git a/Mage.Sets/src/mage/cards/b/BewitchingLeechcraft.java b/Mage.Sets/src/mage/cards/b/BewitchingLeechcraft.java
@@ -93,11 +93,6 @@ public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         return !permanentUntapping.getCounters(game).removeCounter(CounterType.P1P1, 1);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return false;
-    }
-
     @Override
     public boolean applies(GameEvent event, Ability source, Game game) {
         return game.getTurnStepType() == PhaseStep.UNTAP

diff --git a/Mage.Sets/src/mage/cards/b/Biophagus.java b/Mage.Sets/src/mage/cards/b/Biophagus.java
@@ -102,11 +102,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
         return permanent != null && mor.refersTo(permanent, game);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return false;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Permanent target = ((EntersTheBattlefieldEvent) event).getTarget();

diff --git a/Mage.Sets/src/mage/cards/b/BitterFeud.java b/Mage.Sets/src/mage/cards/b/BitterFeud.java
@@ -180,11 +180,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
                 && !sourcePlayerId.equals(targetPlayerId);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         event.setAmount(CardUtil.overflowMultiply(event.getAmount(), 2));

diff --git a/Mage.Sets/src/mage/cards/b/BloodOfTheMartyr.java b/Mage.Sets/src/mage/cards/b/BloodOfTheMartyr.java
@@ -59,11 +59,6 @@ public boolean checksEventType(GameEvent event, Game game) {
         return event.getType() == GameEvent.EventType.DAMAGE_PERMANENT;
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player controller = game.getPlayer(source.getControllerId());

diff --git a/Mage.Sets/src/mage/cards/b/BloodScrivener.java b/Mage.Sets/src/mage/cards/b/BloodScrivener.java
@@ -61,11 +61,6 @@ public BloodScrivenerReplacementEffect copy() {
         return new BloodScrivenerReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player player = game.getPlayer(event.getPlayerId());

diff --git a/Mage.Sets/src/mage/cards/b/BloodletterOfAclazotz.java b/Mage.Sets/src/mage/cards/b/BloodletterOfAclazotz.java
@@ -63,11 +63,6 @@ public BloodletterOfAclazotzEffect copy() {
         return new BloodletterOfAclazotzEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         event.setAmount(CardUtil.overflowMultiply(event.getAmount(), 2));
@@ -85,4 +80,3 @@ public boolean applies(GameEvent event, Ability source, Game game) {
                 && game.getOpponents(source.getControllerId()).contains(event.getPlayerId());
     }
 }
-
diff --git a/Mage.Sets/src/mage/cards/b/BoonReflection.java b/Mage.Sets/src/mage/cards/b/BoonReflection.java
@@ -53,11 +53,6 @@ public BoonReflectionEffect copy() {
         return new BoonReflectionEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         event.setAmount(CardUtil.overflowMultiply(event.getAmount(), 2));

diff --git a/Mage.Sets/src/mage/cards/b/BorealOutrider.java b/Mage.Sets/src/mage/cards/b/BorealOutrider.java
@@ -86,11 +86,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
         return spell != null && event.getTargetId().equals(spell.getCard().getId());
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return false;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Permanent creature = ((EntersTheBattlefieldEvent) event).getTarget();

diff --git a/Mage.Sets/src/mage/cards/b/BraceForImpact.java b/Mage.Sets/src/mage/cards/b/BraceForImpact.java
@@ -64,11 +64,6 @@ public BraceForImpactPreventDamageTargetEffect copy() {
         return new BraceForImpactPreventDamageTargetEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         GameEvent preventEvent = new PreventDamageEvent(event.getTargetId(), source.getSourceId(), source, source.getControllerId(), event.getAmount(), ((DamageEvent) event).isCombatDamage());

diff --git a/Mage.Sets/src/mage/cards/b/BranchingEvolution.java b/Mage.Sets/src/mage/cards/b/BranchingEvolution.java
@@ -74,11 +74,6 @@ public boolean applies(GameEvent event, Ability source, Game game) {
         return false;
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public BranchingEvolutionEffect copy() {
         return new BranchingEvolutionEffect(this);

diff --git a/Mage.Sets/src/mage/cards/b/BreathstealersCrypt.java b/Mage.Sets/src/mage/cards/b/BreathstealersCrypt.java
@@ -53,11 +53,6 @@ public BreathstealersCryptEffect copy() {
         return new BreathstealersCryptEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player player = game.getPlayer(event.getPlayerId());

diff --git a/Mage.Sets/src/mage/cards/b/BruvacTheGrandiloquent.java b/Mage.Sets/src/mage/cards/b/BruvacTheGrandiloquent.java
@@ -57,11 +57,6 @@ public BruvacTheGrandiloquentReplacementEffect copy() {
         return new BruvacTheGrandiloquentReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean checksEventType(GameEvent event, Game game) {
         return event.getType() == GameEvent.EventType.MILL_CARDS;

diff --git a/Mage.Sets/src/mage/cards/c/CallousGiant.java b/Mage.Sets/src/mage/cards/c/CallousGiant.java
@@ -58,11 +58,6 @@ public CallousGiantEffect copy() {
         return new CallousGiantEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         if(event.getAmount() <= 3)

diff --git a/Mage.Sets/src/mage/cards/c/CandlesGlow.java b/Mage.Sets/src/mage/cards/c/CandlesGlow.java
@@ -64,11 +64,6 @@ public CandlesGlowPreventDamageTargetEffect copy() {
         return new CandlesGlowPreventDamageTargetEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         GameEvent preventEvent = new PreventDamageEvent(event.getTargetId(), source.getSourceId(), source, source.getControllerId(), event.getAmount(), ((DamageEvent) event).isCombatDamage());

diff --git a/Mage.Sets/src/mage/cards/c/ChainsOfMephistopheles.java b/Mage.Sets/src/mage/cards/c/ChainsOfMephistopheles.java
@@ -55,11 +55,6 @@ public ChainsOfMephistophelesReplacementEffect copy() {
         return new ChainsOfMephistophelesReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player player = game.getPlayer(event.getPlayerId());

diff --git a/Mage.Sets/src/mage/cards/c/ChannelHarm.java b/Mage.Sets/src/mage/cards/c/ChannelHarm.java
@@ -58,11 +58,6 @@ public ChannelHarmEffect copy() {
         return new ChannelHarmEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         Player sourceController = game.getPlayer(source.getControllerId());

diff --git a/Mage.Sets/src/mage/cards/c/ChaosMoon.java b/Mage.Sets/src/mage/cards/c/ChaosMoon.java
@@ -149,11 +149,6 @@ public ChaosMoonEvenReplacementEffect copy() {
         return new ChaosMoonEvenReplacementEffect(this);
     }
 
-    @Override
-    public boolean apply(Game game, Ability source) {
-        return true;
-    }
-
     @Override
     public boolean replaceEvent(GameEvent event, Ability source, Game game) {
         ManaEvent manaEvent = (ManaEvent) event;