Commit Graph

3359 Commits

Author SHA1 Message Date
Hanmac
877b0abd66 add Sentry Breadcrumb 2018-11-03 18:02:18 +01:00
Hanmac
70e2714cf3 Affinity as Keyword 2018-11-01 15:26:48 +01:00
Michael Kamensky
85aaf5c548 Fixed a double Taps trigger on Convoke/Improvise. 2018-10-31 09:10:10 +00:00
Agetian
3e91c95dcf - When making token copies of permanents, set the copied permanent so that its CMC can be calculated. 2018-10-26 22:58:16 +03:00
Agetian
f4db360265 - Prevent NPEs caused by the AI when testing for the new style AI hints from within AI methods on Card objects, which is dangerous because Card.getRules() is not guaranteed to be non-null, and may indeed be null for objects such as tokens (especially noticeable in Momir Basic and MoJhoSto).
- Currently relegated 99% of AI calls to those getAIHints tests to a wrapper method which checks for a non-null getRules.
- The card predicate has to test for non-null directly to avoid adding an unnecessary dependency on the AI module (the alternative would be to add the wrapper methods to the Card object, but that'll clutter it even more and the AI hints belong to the AI side of things, not the card itself).
2018-10-26 17:16:12 +03:00
Chris H
ff4622954f Fix NPEs reported by Sentry 2018-10-20 22:07:57 -04:00
Michael Kamensky
4b52ff05d5 Update direct references to RemAIDeck/RemRandomDeck to use the new format 2018-10-19 12:13:31 +00:00
Michael Kamensky
d12e6430a6 Merge branch 'epicRework' into 'master'
Epic rework

See merge request core-developers/forge!1018
2018-10-16 04:12:53 +00:00
swordshine
2e0a0c5560 Merge branch 'migrate-full-cost' into 'master'
Convert FullCost to PermanentCreature

See merge request core-developers/forge!1020
2018-10-16 03:56:08 +00:00
Chris H
a80396ae97 Convert FullCost to PermanentCreature 2018-10-15 22:18:23 -04:00
Hanmac
6c2dfcffff Epic rework 2018-10-15 22:05:13 +02:00
Blacksmith
b11830a12a [maven-release-plugin] prepare for next development iteration 2018-10-15 15:42:16 +00:00
Blacksmith
a8fbfce51f [maven-release-plugin] prepare release forge-1.6.17 2018-10-15 15:42:12 +00:00
swordshine
3143751550 Merge branch 'fix-lich' into 'master'
Fix Underrealm Lich revealing a card going to hand.

See merge request core-developers/forge!1013
2018-10-14 08:15:38 +00:00
Agetian
8653338f56 - Underrealm Lich should not reveal a card going to hand. 2018-10-14 08:57:06 +03:00
Michael Kamensky
7142751ef3 Merge branch 'add-jump-start-to-flashback' into 'master'
Improve flashback zone display

See merge request core-developers/forge!1012
2018-10-14 03:40:12 +00:00
Chris H
3889c75e32 Improve flashback zone display 2018-10-13 19:14:19 -04:00
Agetian
46c764d5ab - Temporarily revert a change in CostPartMana to avoid breaking Convoke, Improvise, and Delve. 2018-10-13 21:02:47 +03:00
Agetian
2a1a753182 - Minor mistype fix in SurveilEffect. 2018-10-13 19:48:05 +03:00
Jamin Collins
51bc066238 Merge branch 'fix+gonti' into 'master'
Fixing Mana types being inappropriately referenced in relation to Matrices

See merge request core-developers/forge!883
2018-10-12 23:13:33 +00:00
Michael Kamensky
bfa43bf157 Merge branch 'patch' into 'master'
Remaining C18 cards

See merge request core-developers/forge!1001
2018-10-11 11:12:20 +00:00
swordshine
e78028a254 - Updated Genesis Storm 2018-10-11 16:41:41 +08:00
Hanmac
b519896855 CloneEffect: now with NewName and GainThisAbility 2018-10-11 08:09:48 +02:00
swordshine
6c9ba7694d - Added some cards 2018-10-11 08:54:09 +08:00
Chris H
8ac960168d Cross my fingers 2018-10-09 21:43:48 -04:00
swordshine
a2761c868c - Updated more scripts 2018-10-09 23:14:28 +08:00
swordshine
68de368c46 - Commander Storm cards 2018-10-09 20:51:00 +08:00
swordshine
4089aefcde - migrate Aminatou, the Fateshifter 2018-10-09 15:55:06 +08:00
swordshine
375c1e0da7 - Added Aminatou, the Fateshifter 2018-10-09 12:03:29 +08:00
Chris H
29ff9d3c92 Pull ManaConversion out to be reused for SpellAbilityStackInstance 2018-10-07 22:07:53 -04:00
swordshine
e7b793261e - Removed an used import 2018-10-08 08:38:27 +08:00
swordshine
9d5332b95b - Added Coveted Jewel 2018-10-07 22:53:02 +08:00
swordshine
c2d0648655 - Added Venerated Loxodon 2018-10-07 19:29:27 +08:00
swordshine
6f0871724f - Added Beamsplitter Mage 2018-10-07 18:24:23 +08:00
swordshine
7ea35cabb4 - Added Thousand-Year Storm 2018-10-07 14:29:40 +08:00
swordshine
98c8feac99 - Added Mnemonic Betrayal 2018-10-05 14:13:20 +08:00
swordshine
85a5fe380d - Fixed Commander tax 2018-10-05 14:10:21 +08:00
swordshine
c77f9fe3ce - Added Discovery // Dispersal 2018-10-04 14:26:25 +08:00
swordshine
ece38aed9d - Added Treasure Nabber 2018-10-03 23:43:32 +08:00
Michael Kamensky
7c0eb68ed5 Merge branch 'tokenCardText' into 'master'
Token card text

Closes #716

See merge request core-developers/forge!961
2018-10-02 07:18:45 +00:00
Hanmac
45ee00f10b Player: canPlayLand use the mayPlay of the Land Ability 2018-09-30 16:41:22 +02:00
Hanmac
a298dcf3bc TokenScript: make CardTextChange working again
SpellAbility: changeTextIntrinsic to sub abilities
2018-09-30 15:10:12 +02:00
KrazyTheFox
f0c2cf8577 Fix up pom files 2018-09-29 16:41:06 -04:00
Blacksmith
1e46f88ef5 [maven-release-plugin] prepare for next development iteration 2018-09-29 19:53:15 +00:00
Blacksmith
5192fc3f59 [maven-release-plugin] prepare release forge-1.6.16 2018-09-29 19:53:10 +00:00
Michael Kamensky
2ab59849a8 Merge branch 'guilds_fxes' into 'master'
Experimental Frenzy

See merge request core-developers/forge!954
2018-09-28 09:27:23 +00:00
tehdiplomat
c1df1da195 Apparently unused imports is a compile error? 2018-09-27 08:21:48 -04:00
Jamin W. Collins
32ab0aadb0 fix NPE when there is nothing to choose from
With the existing code, if a player cast Surgical Extraction targetting
a card that only existed in the target player's graveyard (no copies in
the players hand or library), an NPE would be thrown.

Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
2018-09-26 17:26:03 -06:00
Michael Kamensky
3f675c0102 Merge branch 'guildofravnica' into 'master'
Added Etrata the Silencer, including new HIT counters

See merge request core-developers/forge!949
2018-09-26 06:13:24 +00:00
Jamin W. Collins
79267ac29f avoid CME in checkStaticAbilities()
Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
2018-09-25 21:41:38 -06:00