From 16c37d08063b2e9f5b40035656010124007d198f Mon Sep 17 00:00:00 2001
From: Oleg Agafonov <jaydi85@gmail.com>
Date: Sat, 13 Jul 2019 17:59:35 +0400
Subject: [PATCH] * Thousand-Year Storm - added logs and card hint info about
 storm count value;

---
 .../src/mage/cards/t/ThousandYearStorm.java   | 86 +++++++++++++++++--
 1 file changed, 81 insertions(+), 5 deletions(-)

diff --git a/Mage.Sets/src/mage/cards/t/ThousandYearStorm.java b/Mage.Sets/src/mage/cards/t/ThousandYearStorm.java
index 77f82d9ddd..67db063283 100644
--- a/Mage.Sets/src/mage/cards/t/ThousandYearStorm.java
+++ b/Mage.Sets/src/mage/cards/t/ThousandYearStorm.java
@@ -1,17 +1,23 @@
 package mage.cards.t;
 
 import mage.abilities.Ability;
+import mage.abilities.Mode;
 import mage.abilities.common.SpellCastControllerTriggeredAbility;
+import mage.abilities.dynamicvalue.DynamicValue;
+import mage.abilities.effects.Effect;
 import mage.abilities.effects.OneShotEffect;
+import mage.abilities.hint.ValueHint;
 import mage.cards.CardImpl;
 import mage.cards.CardSetInfo;
 import mage.constants.CardType;
 import mage.constants.Outcome;
 import mage.constants.WatcherScope;
+import mage.constants.Zone;
 import mage.filter.common.FilterInstantOrSorcerySpell;
 import mage.game.Game;
 import mage.game.events.GameEvent;
 import mage.game.stack.Spell;
+import mage.players.Player;
 import mage.watchers.Watcher;
 
 import java.util.HashMap;
@@ -28,9 +34,7 @@ public final class ThousandYearStorm extends CardImpl {
         super(ownerId, setInfo, new CardType[]{CardType.ENCHANTMENT}, "{4}{U}{R}");
 
         // Whenever you cast an instant or sorcery spell, copy it for each other instant and sorcery spell you've cast before it this turn. You may choose new targets for the copies.
-        this.addAbility(new SpellCastControllerTriggeredAbility(
-                new ThousandYearStormEffect(), new FilterInstantOrSorcerySpell(), false, true
-        ), new ThousandYearWatcher());
+        this.addAbility(new ThousandYearStormAbility());
     }
 
     public ThousandYearStorm(final ThousandYearStorm card) {
@@ -43,11 +47,48 @@ public final class ThousandYearStorm extends CardImpl {
     }
 }
 
+class ThousandYearStormAbility extends SpellCastControllerTriggeredAbility {
+
+    String stormCountInfo = null;
+
+    public ThousandYearStormAbility() {
+        super(Zone.BATTLEFIELD, new ThousandYearStormEffect(), new FilterInstantOrSorcerySpell(), false, true);
+        this.addHint(new ValueHint("You've cast instant and sorcery this turn", ThousandYearSpellsCastThatTurnValue.instance));
+        this.addWatcher(new ThousandYearWatcher());
+    }
+
+    public ThousandYearStormAbility(final ThousandYearStormAbility ability) {
+        super(ability);
+        this.stormCountInfo = ability.stormCountInfo;
+    }
+
+    @Override
+    public boolean checkTrigger(GameEvent event, Game game) {
+        // save storm count info, real count will be calculated to stack ability in resolve effect only
+        if (super.checkTrigger(event, game)) {
+            int stormCount = ThousandYearSpellsCastThatTurnValue.instance.calculate(game, this, null);
+            stormCountInfo = " (<b>storm count: " + Math.max(0, stormCount - 1) + "</b>) ";
+            return true;
+        }
+        return false;
+    }
+
+    @Override
+    public String getRule() {
+        return "Whenever you cast an instant or sorcery spell, copy it for each other instant and sorcery spell you've cast before it this turn"
+                + (stormCountInfo != null ? stormCountInfo : "") + ". You may choose new targets for the copies.";
+    }
+
+    @Override
+    public ThousandYearStormAbility copy() {
+        return new ThousandYearStormAbility(this);
+    }
+}
+
 class ThousandYearStormEffect extends OneShotEffect {
 
     public ThousandYearStormEffect() {
         super(Outcome.Benefit);
-        this.staticText = "copy it for each other instant and sorcery spell you've cast before it this turn. You may choose new targets for the copies";
     }
 
     public ThousandYearStormEffect(final ThousandYearStormEffect effect) {
@@ -62,7 +103,8 @@ class ThousandYearStormEffect extends OneShotEffect {
     @Override
     public boolean apply(Game game, Ability source) {
         Spell spell = game.getSpellOrLKIStack(getTargetPointer().getFirst(game, source));
-        if (spell != null) {
+        Player controller = spell != null ? game.getPlayer(spell.getControllerId()) : null;
+        if (spell != null && controller != null) {
             ThousandYearWatcher watcher = game.getState().getWatcher(ThousandYearWatcher.class);
             if (watcher != null) {
                 String stateSearchId = spell.getId().toString() + source.getSourceId().toString();
@@ -81,6 +123,11 @@ class ThousandYearStormEffect extends OneShotEffect {
         }
         return false;
     }
+
+    @Override
+    public String getText(Mode mode) {
+        return "copy it for each other instant and sorcery spell you've cast before it this turn. You may choose new targets for the copies";
+    }
 }
 
 class ThousandYearWatcher extends Watcher {
@@ -131,3 +178,32 @@ class ThousandYearWatcher extends Watcher {
     }
 
 }
+
+enum ThousandYearSpellsCastThatTurnValue implements DynamicValue {
+
+    instance;
+
+    @Override
+    public int calculate(Game game, Ability sourceAbility, Effect effect) {
+        ThousandYearWatcher watcher = game.getState().getWatcher(ThousandYearWatcher.class);
+        if (watcher == null) {
+            return 0;
+        }
+        return watcher.getAmountOfSpellsPlayerCastOnCurrentTurn(sourceAbility.getControllerId());
+    }
+
+    @Override
+    public ThousandYearSpellsCastThatTurnValue copy() {
+        return instance;
+    }
+
+    @Override
+    public String toString() {
+        return "X";
+    }
+
+    @Override
+    public String getMessage() {
+        return "You cast an instant or sorcery spell in this turn";
+    }
+}