Commit Graph

31985 Commits

Author SHA1 Message Date
Agetian
fcad3618e0 - Fixed a NPE when trying to load a Planar Conquest when no conquest has been started yet and the profile folder was not yet created. 2016-12-21 07:58:11 +00:00
Agetian
9e22822dad - Preparing Mobile Forge release 1.5.58.006 (incremental). 2016-12-21 05:12:44 +00:00
Agetian
33c5f4d158 - Correctly identify the paid X for the purpose of building stack descriptions (e.g. fixes Repeal stack desc). 2016-12-20 19:20:45 +00:00
Agetian
7cf2c7e383 - Enforce deck construction limits in the deck editor by unique card name instead of by unique PaperCard. 2016-12-20 17:04:22 +00:00
Agetian
4b46daf963 - AI should not confirm the AF Tap action if nothing at all was targeted by it (prevents e.g. the AI pouring all available mana into Waxmane Baku). 2016-12-20 13:59:53 +00:00
Agetian
ae608246d1 - Reverting the lifelink change for now, better solution needed. 2016-12-20 10:18:06 +00:00
Agetian
3fc05d6940 - Resetting targets not required when changing ValidTgts for Detonate. 2016-12-20 08:40:14 +00:00
Agetian
4d491a6bdf - A [hacky] fix for Detonate that only allows targeting CMC 0 artifacts if Detonate is cast without mana cost (e.g. on Cascade) with X=0.
- Ideally should be done via Annouce$ X by announcing X before starting to cast the spell and presetting it to 0 when cast without mana cost, but that causes massive issues with the AI.
2016-12-20 08:37:04 +00:00
Agetian
d23964106d - Attempting to fix interaction between lifelink on combat damage and replacement effects like Gisela, Blade of Goldnight. 2016-12-20 08:35:06 +00:00
Agetian
c255cb5285 - Fixed copying a SA with no mana cost not setting the WithoutManaCost param which is needed by e.g. Play effect AI to properly determine that the spell should be played without paying its mana cost. 2016-12-20 05:22:37 +00:00
Agetian
7e4ab8d0bb - Fixed Grip of Chaos NPEing when Maze of Ith is used in its presence (and other potential similar NPEs). 2016-12-19 16:47:45 +00:00
Agetian
9f17f2700d - Chandra, Torch of Defiance should show the exiled card to the ability activator prior to asking whether the player wants to play it or not. 2016-12-19 06:06:30 +00:00
Agetian
66da481f45 - Fixed Kalonian Hydra. 2016-12-19 04:56:49 +00:00
Hanmac
875198bed5 TriggerChangesZone: use ArrayUtils.contains to check for String in String[] 2016-12-18 21:15:05 +00:00
Agetian
d07a0913ad - Allow multiple Destinations for TriggerChangesZone (fixes Reyhan, Last of the Abzan). 2016-12-18 18:46:09 +00:00
Hanmac
a2759ace2c Arcbound Fiend: use second mode for MoveCounter, no need for extra Pump Effect for Tgt 2016-12-18 06:35:39 +00:00
Hanmac
0000700320 CountersMoveAi: total refacor of AI class, now it has better logic for Tgt -> Defined and Source -> Tgt and for without Tgt 2016-12-18 06:33:40 +00:00
Hanmac
90ffab69de ClashAi & ManifestAi: use shorter form now for looking at TopCard of Library 2016-12-18 06:28:59 +00:00
Agetian
e8f30ace1e - Formatting imports. 2016-12-18 05:30:24 +00:00
Agetian
9352b7f3ce - A more transparent fix for Kalitas, Traitor of Ghet which does not affect cards such as Aurification, Curse of Stalked Prey and others.
- Adds an extra parameter to Card which stores the last known zone information for the card when it is copied for the purpose of LKI. Thus, things such as the zone and the timestamp of the LKI copy are not affected.
- If the card is not a LKI copy, the last known zone info is assumed to be the current zone the card is in.
2016-12-18 05:29:23 +00:00
Agetian
281e5f12c0 - Hopefully a better solution for the LKI problem: add a special flag to show that the card is a temporary copy and should not be considered as being in play 2016-12-17 20:12:08 +00:00
Agetian
b50183e7c4 - Copy the timstamp over to the LKI copy of a card (may be necessary to "prove" that the card and its LKI copy are the same, e.g. for the purpose of some triggers). 2016-12-17 19:47:07 +00:00
Hanmac
08b984ff4c cards: update cards using UpkeepCost 2016-12-17 18:05:44 +00:00
Hanmac
d8c97d9c4e CardFactoryUtil: moved "At the beginning of your upkeep, sacrifice" as "UpkeepCost" in CardFactoryUtil
delete Upkeep class because its not needed anymore
2016-12-17 18:03:43 +00:00
Agetian
4b10704390 - When setting cloned SA targets in CharmEffect::ChainAbilities, call setTargets last in order not to forget if there were several targets selected (fixes Abzan Charm AI failing to correctly divide and allocate counters to two targets). 2016-12-17 17:31:57 +00:00
Agetian
141f66d54d - Reset targets for ChangeZoneAi SameName logic (hopefully fixes Declaration in Stone AI targeting errors). 2016-12-17 15:05:57 +00:00
Agetian
1da0063310 - Somewhat better structuring of ChangeZoneAll AI targeting check. 2016-12-17 14:41:19 +00:00
Agetian
08a6d2557f - Reverting RemAIDeck mark on Paradigm Shift, back to RemRandomDeck only. 2016-12-17 14:36:05 +00:00
Agetian
e09d57562f - A better idea: for non-targeted AF ChangeZoneAll, just return "true" for now without trying to target it. This allows to keep Paradigm Shift as non-RemAIDeck. 2016-12-17 14:35:45 +00:00
Agetian
493d8625bf - The AI can't use Paradigm Shift, marking as RemAIDeck. 2016-12-17 14:26:14 +00:00
Agetian
8977d1ad8f - AI should not attempt targeting AF ChangeZoneAll which does not need any targets (e.g. Timetwister, Memory Jar, etc., which the current AI can't use). 2016-12-17 14:16:02 +00:00
Hanmac
2b055b84fb CardFactoryUtil: fixed Manifest 2016-12-17 11:04:05 +00:00
Agetian
256d0bd455 - Further updated the cards from the previous commit.
- Removed DB Cleanup from cards that do not utilize it anywhere else.
- Rewrote the cards that used RememberedController (from LKI) but do not care about the destruction of the creature to use TargetedController instead.
2016-12-17 08:44:18 +00:00
Agetian
c4908fc0c0 - Removed RememberLKI from cards updated in the previous commit.
- Fixed Death's Caress
2016-12-17 08:34:51 +00:00
Agetian
4ca2ca7441 - Update cards that have an extra ability which does not care whether the targeted creature was destroyed or not (indestructibe, regenerated, etc.) to use TargetedLKI instead of RememberedLKI. 2016-12-17 08:27:57 +00:00
Hanmac
ab84992837 phthisis: no need to use remember at all 2016-12-17 08:19:45 +00:00
Agetian
c105059785 - Using TargetedLKI for Phthisis so that it does not care about the creature being destroyed or not. 2016-12-17 08:09:18 +00:00
Agetian
eaf4e94851 - Reverting the RememberLKI change for now, breaks many other cards. 2016-12-17 08:08:03 +00:00
Agetian
3585177898 - Minor formatting fix. 2016-12-17 06:32:15 +00:00
Agetian
7dc9437752 - [Experimental]: do not invoke the achievement update in the game thread (which creates a race condition when there are multiple achievements of different types to be awarded, freezing the game completely). 2016-12-17 06:31:51 +00:00
Agetian
3f2b5425b9 - AB Destroy should remember LKI if instructed to via RememberLKI parameter even if the creature is not destroyed (for the sake of subabilities that may reference that LKI). Fixes Phthisis + regenerating a creature. 2016-12-17 06:23:37 +00:00
Agetian
0879a7fb85 - Fixed Slave of Bolas. 2016-12-16 19:42:51 +00:00
Agetian
c789fc32f7 - List most likely can't be null at this point since it's instantiated, so just check for empty. 2016-12-16 19:02:30 +00:00
Agetian
4aec9031d3 - NPE prevention measure in ControlGainAi (potentially trying to sort an empty list when the AI has nothing to target). 2016-12-16 19:01:52 +00:00
Agetian
99b730ebc7 - NPE prevention in Charm effect (note: if the AI is trying a Charm effect while it hasn't chosen anything, then there's also an issue elsewhere in the logic, may need further examination/correction). 2016-12-16 18:59:14 +00:00
Agetian
f439fe52c4 - Fixed Everflowing Chalice NPEing after being cast via Delay. 2016-12-16 18:49:34 +00:00
Agetian
d3951c0061 - Added an extra check for Genju of the <X> AI. 2016-12-16 12:02:19 +00:00
Agetian
f076b765de - Removed an extra blank line. 2016-12-16 10:53:58 +00:00
Agetian
b46aabf6c6 - Prevent the AI from animating a land enchanted with an animating aura (i.e. Genju of the Cedars) by tapping the land that is to be animated. 2016-12-16 10:53:20 +00:00
Agetian
c2907aa915 - [Experimental] attempt to fix Kalitas, Traitor of Ghet interaction with mass removal by reading LKI about the card's last known zone on a Moved replacement effect event. 2016-12-16 07:07:11 +00:00