Mark Langen
18663f0a7a
Amonket Aftermath ability and card frame changes Completed
...
* Aftermath Ability implementation complete (At least until we see comprehensive rules that contradict the way I assumed it will work)
* Aftermath Card Frame rendering complete
* Normal Split and Fuse Split card frame rendering complete
* Amonket Split card CMC changes NOT made, but left for a separate commit
2017-04-04 00:29:54 -06:00
Mark Langen
a96a7f89f5
Work in progress changes for Amonket Aftermath ability.
...
* Card Rendering has support for Aftermath Split card frames
* Card Rendering has support for Split cards
* Aftermath ability work in progress
2017-04-03 04:15:25 -06:00
ingmargoudt
211d433ea9
playertype, enum singleton
2017-04-02 22:09:20 +02:00
Derek M
b19170f34f
Merge pull request #3045 from SickBoyWi/master
...
Implemented Soul Burn card for the two sets it's a part of: Ice Age a…
2017-04-02 12:17:32 -04:00
LevelX2
097a8ce0dd
Some rework to end turn logic and removing of stack objects (related to #2977 ).
2017-04-02 15:18:50 +02:00
LevelX2
15c1c7f8c0
* Fixed a problem with flashback ability and variable non mana costs (related to #2436 ).
2017-04-02 10:11:25 +02:00
ingmargoudt
594ffa2754
rewrote some streams and optionals
2017-04-01 22:45:40 +02:00
LevelX2
d0bf77cedf
* Fixed a problem with replacement effects that add counters were applied more than once to the same EnteresTheBattlefield event ( fixes #2872 ).
2017-04-01 17:43:17 +02:00
Johnny.Hastings@gmail.com
2be3f20320
Implemented Soul Burn card for the two sets it's a part of: Ice Age and Invasion.
2017-03-31 18:33:17 -05:00
ingmargoudt
884ae83791
Merge pull request #3042 from ingmargoudt/card_functions
...
move static function to CardImpl, and other api fixes
2017-03-31 18:52:37 +02:00
LevelX2
db25867067
* Fixed a problem of VariableCostImpl with stack abilities.
2017-03-31 17:30:41 +02:00
Achilles
c968a03ff8
- Fixes all but the Conflagrate issue in regards to FlashbackAbility. The CMC issue works with the Flashback cards as well. Conflagrate will require more testing to see what is going on.
2017-03-30 21:11:15 -05:00
ingmargoudt
6f8f10fcc3
move static function to CardImpl, and other api fixes
2017-03-30 20:09:47 +02:00
fireshoes
1bf2d1782c
[MPS-AKH] Added Amonkhet Masterpiece reprints. [MM3] Token updates
2017-03-29 10:42:43 -05:00
fireshoes
55b433ae36
[AKH] Added 3/27 spoilers to mtg-cards-data.txt. Implemented several of the new cards.
2017-03-28 22:25:29 -05:00
Derek M
1dcd3d0a81
Merge branch 'master' into type_api
2017-03-27 19:13:45 -04:00
Jeff
aa81a6d663
- LoseAbilityAllEffect() now works with rule 611.2c.
2017-03-27 17:17:56 -05:00
Jeff
c4dc0222d8
- Fixed Indomitable Creativity. Bug #3029
2017-03-27 16:42:32 -05:00
ingmargoudt
5cd1b5880f
fix supertype on the client
2017-03-27 22:20:19 +02:00
ingmargoudt
0879298e92
refactor cards to use supertype enum
2017-03-27 21:56:33 +02:00
ingmargoudt
84559457d3
refactor supertype to enum
2017-03-27 21:51:04 +02:00
Clint Herron
70e4752cfd
Merge remote-tracking branch 'upstream/master' into CMH-GuiltyConscienceAndBackfire
2017-03-27 08:37:27 -04:00
Achilles
3a2c828fb8
- Fixed Rivals' Duel.
2017-03-26 18:25:45 -05:00
Achilles
c2bcb59c82
- Added Barrin's Spite. Fixed a targeting issue.
2017-03-25 22:41:03 -05:00
Clint Herron
402beb3b49
Merge remote-tracking branch 'upstream/master' into CMH-GuiltyConscienceAndBackfire
2017-03-25 22:07:23 -04:00
Achilles
26dbee6f2d
- Added BecomesCreatureAttachedWithActivatedAbilityOrSpellEffect(). Converted all the Genju cards to use it. This class is subject to rule 611.2c.
2017-03-25 17:30:14 -05:00
Achilles
0639b3b75b
- Reverted BecomesCreatureAttachedEffect(). It broke cards that are not subject to Rule 611.2c like Ensoul Artifact. A new class called BecomesCreatureAttachedWithActivatedAbilityOrSpellEffect() will be made to fix the Genju of the Spires bug. Yes, that is a long name. If someone has a better suggestion, please let it be known.
2017-03-25 17:02:47 -05:00
Achilles
ff489446fe
- Fixed BecomesCreatureAttachedEffect(). Rule 611.2c. Fixes https://github.com/magefree/mage/issues/3011
2017-03-25 13:11:08 -05:00
Clint Herron
40d318ce09
Merge remote-tracking branch 'upstream/master' into CMH-GuiltyConscienceAndBackfire
2017-03-25 10:46:20 -04:00
Derek M
1a9c5fa469
Merge pull request #3014 from drmDev/bug/sphereCosts
...
fixes #3012 manaCosts not null check and way to retrieve it
2017-03-24 08:06:30 -04:00
Clint Herron
bce7e690a8
Merge remote-tracking branch 'upstream/master' into CMH-GuiltyConscienceAndBackfire
2017-03-23 19:26:34 -04:00
Clint Herron
3c81275ab1
Implementing Guilty Conscience and Backfire, as well as expanding capabilities of trigers that occur when an enchanted creature deals damage to a player, and adding an effect to deal damage to an enchanted creature.
2017-03-23 19:23:28 -04:00
drmDev
57efb10003
fixing tokens exiled at end of combat - not at end step
2017-03-23 11:21:38 -04:00
drmDev
097d60f5fc
fixes #3012 manaCosts not null check and way to retrieve it
2017-03-23 10:25:21 -04:00
ingmargoudt
5ae4ddce07
Merge branch 'master' into phantoms
2017-03-23 13:43:34 +01:00
vraskulin
d3214c65e4
Merge remote-tracking branch 'fork/master'
2017-03-22 14:51:31 +03:00
vraskulin
d93061c0bc
Equals should be compared with == but not equals()
2017-03-22 14:50:50 +03:00
ingmargoudt
b154bf58f9
phantom effect code duplication
2017-03-22 10:25:57 +01:00
LevelX2
7693f3fd59
Added Ogre Leadfoot.
2017-03-21 11:37:22 +01:00
vraskulin
f37c823aa8
String + string concatenation can be replaced by string + char concetenation
2017-03-20 14:00:18 +03:00
drmDev
311e41abd2
refactored exiletokensAtEndStep
2017-03-19 17:54:07 -04:00
drmDev
c109fc8a0e
Implemented card Lightning Coils and test for it
2017-03-19 12:23:58 -04:00
LevelX2
441b6bc2e9
Fixed AI handling of cost reduction of reductions defined with "up to" causing game locks.
2017-03-19 11:20:37 +01:00
Jeff
6cb77bfb8f
- Fix for bug #2977 . Perhaps Levelx knows of a better solution. It works fine in my testing.
...
Bound // Determined + Child of Alara = No Death Trigger
2017-03-17 14:54:14 -05:00
LevelX2
68d398c6fb
Merge pull request #2968 from ingmargoudt/singleton-as-enum
...
rewrote singleton to enum where applicable
2017-03-16 17:02:31 +01:00
Clint Herron
3eb9be1b18
Adding check to Flagbearer requirement that only checks for spells that are cast and abilities that are activated -- choosing targets from triggered abilities is exempt.
2017-03-16 03:30:07 -04:00
ingmargoudt
234cfe9519
rewrote singleton to enum where applicable
2017-03-14 16:32:47 +01:00
LevelX2
962f05bbcb
* Offering - Fixed that offering could not be activated as instant, cancel during offering mana payment now restores also the sacrificed permanent.
2017-03-12 11:29:53 +01:00
LevelX2
1b2636d0cd
* Arachnuns Web - Fixed condition handling. Some renaming.
2017-03-12 10:36:51 +01:00
ingmargoudt
1f215cdbe3
removed autoboxing :
...
Boolean -> boolean
Integer -> int
Float -> float
2017-03-11 15:17:14 +01:00
ingmargoudt
4b5378c423
pool.count() == 0 ---> pool.isEmpty()
2017-03-11 13:52:46 +01:00
LevelX2
c042d50ec7
Fixed a bug that fizzling spell copies let also wrongly fizzle the original spell on the stack the copy was made from.
2017-03-11 12:00:05 +01:00
Nathaniel Brandes
900d68f77d
Fix CMC for flashbacked cards.
...
This should fix Conflagate/Chalice of the Void interaction.
2017-03-09 23:32:42 -08:00
Nathaniel Brandes
d7d700f66a
Some clarification around loyalty. Added unit test.
2017-03-09 02:30:52 -08:00
Justin Herlehy
1a2724c9b2
Revert: Fix planeswalker loyalty counters.
...
This change was incorrect.
Gatherer Ruling:
10/1/2005: Planeswalkers will enter the battlefield with double the normal amount of loyalty counters. However, if you activate an ability whose cost has you put loyalty counters on a planeswalker, the number you put on isn’t doubled. This is because those counters are put on as a cost, not as an effect.
2017-03-09 01:14:10 -08:00
Nathaniel Brandes
52fa369024
Fix planeswalker loyalty counters.
...
They should be added like normal counters, firing events that can be
intercepted by cards like doubling season.
2017-03-08 23:57:11 -08:00
Nathaniel Brandes
4a9eddb724
Properly fix targeting.
...
The notTarget should only invalidate the canBeTargetedBy return.
The Filter should still be relevant.
2017-03-08 20:59:01 -08:00
Nathaniel Brandes
495e20ae1a
Fix target defender cases that don't target.
2017-03-08 20:44:02 -08:00
Nathaniel Brandes
ce1f4a3bf8
Phyrexian mana now correctly a payment choice.
...
Per rule 601.2b, it is not determined at the pay costs step,
but at the "choice" step, long before costs are determined.
This fixes trinisphere interactions and should be consistent with the
rules.
2017-03-08 20:03:28 -08:00
Nathaniel Brandes
3bebdd65bc
Fixed crewing staying if the object changed zones.
2017-03-08 01:01:10 -08:00
Achilles
31ba751c66
- Fix for Spellskite dealing with a targeted source spell/ability that has multiple targets.
2017-03-07 22:14:18 -06:00
ingmargoudt
7e26a7a141
commandzone is public
2017-03-07 23:19:05 +01:00
ingmargoudt
c5002983e3
changed static zone function to enum method
2017-03-07 22:02:15 +01:00
ingmargoudt
1bc8e2248b
changed enum equals to ==, removed contains check for set
2017-03-07 21:47:11 +01:00
ingmargoudt
1caf3a6be4
change getLibrary().size() > 0 to hasCards()
2017-03-07 17:28:26 +01:00
Nathaniel Brandes
d6e4ef793e
Fix delve.
...
Delve should not target the cards it exiles.
2017-03-07 00:03:37 -08:00
ingmargoudt
90c6a69335
fixed a count in Battlefield, changed loops to streams
2017-03-06 12:25:14 +01:00
ingmargoudt
3d029d6ad5
put comparing logic in counttype, remove all switches
2017-03-06 11:46:22 +01:00
ingmargoudt
6d4c388c5e
small change in gender enum
2017-03-06 10:18:16 +01:00
ingmargoudt
5387f06898
cards now use the new CountType
2017-03-05 23:29:51 +01:00
ingmargoudt
80580d69e9
Replace checktype enum for Cardtype enum
2017-03-05 20:04:09 +01:00
ingmargoudt
b5e8c88153
remove all contains(cardtype.XX) references, instead use the default interface method
2017-03-05 09:14:33 +01:00
Nathaniel Brandes
58d0a8eb2a
Remove empty function incorrectly checked in.
2017-03-04 23:08:54 -08:00
Nathaniel Brandes
1d3ebb749d
Fix omniscience effect for colorless mana.
...
Also unified the tests for omniscience into a single file, and
put the effect into a single class so that Tamiyo and Omniscience
can share implementations.
2017-03-04 23:04:11 -08:00
Justin Herlehy
8cfe66e29c
Fix Ichorid to set notTarget flag.
...
Fixes issued #2906
2017-03-04 16:20:17 -08:00
ingmargoudt
5a57731968
added new helper methods to test for CardType, to get rid of the contains(CardType.XXX) everywhere, put the logic of that in one place and use the interface call
2017-03-04 00:16:36 +01:00
ingmargoudt
372584f7ad
changed cardtype from list to enumset
2017-03-03 17:04:32 +01:00
ingmargoudt
d01aed42ed
change enum equals to ==
2017-03-01 15:05:44 +01:00
vraskulin
0bc9cf91a8
Minor improvements - StringBuilder now have initial value
2017-02-28 15:26:07 +03:00
vraskulin
11dc1e10f1
Few lambda's and removed explicit type from creating lists and hashmaps
2017-02-28 13:46:57 +03:00
vraskulin
0a3c95dde5
Inner classes should be static. It reduces memory usage and gives perfomance boost + DeckImporter rollback
2017-02-28 12:40:12 +03:00
vraskulin
ad7d3c8078
equals "" should be replaced with isEmpty
2017-02-28 11:45:15 +03:00
vraskulin
3600d03e2c
StringBuilder in a class field may cause memory leaks, it's not GCed, so it's a better way to store all errors just in string field
2017-02-28 11:37:31 +03:00
ingmargoudt
afbf5ce2b6
Merge pull request #2891 from kubikrubikvkube/master
...
Nulls to optional. Also there was bug with AetherChaser
2017-02-27 17:51:35 +01:00
Jeff
79983fbe26
- Extra check to verify a player is in game.
2017-02-27 08:41:07 -06:00
vraskulin
498c8cf60c
Made utility classes final to explicitly forbid it's inheritance
2017-02-27 17:03:38 +03:00
vraskulin
f309717616
Merge remote-tracking branch 'fork/master'
2017-02-27 16:33:47 +03:00
vraskulin
af3986df46
All final strings became static - it will give a little perfomance boost, due to not instantiating same string every time
2017-02-27 16:33:32 +03:00
doncarton
5eaaa67d1e
Nulls to optional. Also there was bug with AetherChaser "you may pay" was not set to optional
2017-02-26 15:15:54 +03:00
LevelX2
d4415e2eff
Merge pull request #2887 from aastrand/ignorelist
...
Add client-side user ignore list
2017-02-26 09:49:43 +01:00
LevelX2
f06ee144f7
* Fixed AddCardTypeSourceEffect with limited duration and object reference was applied on wrong objects (fixes failing crew test).
2017-02-25 12:05:43 +01:00
Anders Åstrand
f68460603a
Add client-side user ignore list
...
This commit adds a client-side per-server ignore list.
* Whispers and talk from an ignored user is hidden
* Ignored users cannot join your tables
* Ignored users tables are hidden
2017-02-22 22:16:09 +01:00
ingmargoudt
48a1fe8602
Merge pull request #2880 from ingmargoudt/master
...
put the writing of equipped and enchanted in the attachmenttype
2017-02-19 23:59:25 +01:00
ingmargoudt
d0fa00f3af
put the writing of equipped and enchanted in the attachmenttype
2017-02-19 23:58:32 +01:00
Achilles
0372eb12e2
- Little fix for a watcher used for Ephara, God of the Polis.
2017-02-19 16:42:32 -06:00
ingmargoudt
972ed6a3f2
changed enum comparison to ==.
...
isDependentTo returns empty set rather than null
2017-02-19 23:38:47 +01:00
JOAC69
845e406d63
Fix failing tests that were doing Object.Equels where == was needed
2017-02-19 02:31:04 -06:00
LevelX2
899046cf16
* Gratuitous Violence - FIxed that if the damage of a creature with lifelink was doubeled only the non doubled amount was added to life.
2017-02-18 01:26:02 +01:00
LevelX2
e6be0577bb
Fixed a bug of FirstSpellCastThisTurnWatcher ( fixes #2871 ).
2017-02-17 23:23:33 +01:00
ingmargoudt
cf23d0905b
mentioned the wrong hashmap (confusing names!!!)
2017-02-16 15:14:44 +01:00
ingmargoudt
14c459b924
solving review issues
2017-02-16 14:03:25 +01:00
ingmargoudt
4aa51210f9
refactor of watchers to use map functions
2017-02-15 23:19:40 +01:00
spjspj
fbbfc6c611
Merge pull request #2869 from spjspj/master
...
Limit deck to 1000 cards
2017-02-16 08:07:17 +11:00
spjspj
b65c1180e4
Limit deck to 1000 cards
2017-02-16 08:06:23 +11:00
ingmargoudt
a1bc43d33e
Merge pull request #2868 from kubikrubikvkube/master
...
Functional interfaces annotation and bugfixes
2017-02-15 20:27:21 +01:00
vraskulin
a15220d51e
Bugfixes with != instead of equals and default toString calls
2017-02-15 19:52:20 +03:00
vraskulin
752392fc46
Functional interfaces should be annotated as functional interfaces
2017-02-15 19:23:12 +03:00
Plopman
cafd002634
Fix {Q} and Thousand year elixir. Fix #2349
2017-02-14 23:04:57 +01:00
ingmargoudt
26e16be772
fix counters
2017-02-13 21:45:37 +01:00
ingmargoudt
3a152ab3d6
rewrote enum comparisons, iterator to removeIf, added some stream and filters
2017-02-11 22:37:00 +01:00
Jeff
fc3ecdf8d0
- Added Auratouched Mage and test suite.
2017-02-10 10:15:56 -06:00
Jeff
1e82f0750a
Fixed FlashbackTest and made modifications to CascadeAbility allowing it to pass its test suite.
2017-02-09 10:20:21 -06:00
spjspj
0ff7ff686d
Merge pull request #2848 from spjspj/master
...
Allow lands with AnyColorManaAbility to be have automatic payments done
2017-02-09 18:26:14 +11:00
spjspj
a08816123c
Allow lands with AnyColorManaAbility to be have automatic payments done
2017-02-09 18:20:51 +11:00
LevelX2
166c6a3590
* Sunforger - Fixed activated unattach ability.
2017-02-06 22:55:52 +01:00
LevelX2
b6a66f1aba
Fixed syntax error of cascadeAbility.
2017-02-06 22:02:44 +01:00
LevelX2
b8cc0120cb
Merge origin/master
2017-02-06 21:21:16 +01:00
vraskulin
1e64ecfe07
Merge remote-tracking branch 'fork/master'
2017-02-06 14:36:24 +03:00
vraskulin
076045d862
Fixed merge conflict
2017-02-06 14:36:09 +03:00
vraskulin
0557b5e89c
list.size > 0 changed to !list.isEmpty
...
We care about if it's empty or not, not about it's size
2017-02-06 14:32:51 +03:00
vraskulin
f60ebfbb1f
All 1-character strings converted to primitives
...
"b" + "r" now changed to 'b' + 'w'. It's more straight-forward, and may cause perfomance improvements - character primitives allocation is faster and less expensive than string creation.
2017-02-06 14:32:51 +03:00
vraskulin
31589778ca
Try/finally refactored to try with resources
2017-02-06 14:32:11 +03:00
kubikrubikvkube
6a114ac902
Merge branch 'master' into master
2017-02-06 14:29:59 +03:00
LevelX2
b3597d64a0
Merge origin/master
2017-02-05 13:54:24 +01:00
LevelX2
b86acbe0da
Some minor fixes.
2017-02-05 13:53:36 +01:00
spjspj
72f4778506
Merge pull request #2837 from spjspj/master
...
Add Canadian Highlander Deck/Game implementation.
2017-02-05 23:25:53 +11:00
spjspj
c98bf163be
Add Canadian Highlander Deck/Game implementation. Also implements the double-mulligan-on-6,5,4,3,2,1 rule.
2017-02-05 23:23:51 +11:00
LevelX2
d71ebdcea9
Merge origin/master
2017-02-05 11:41:20 +01:00
LevelX2
3525e46514
XMage 1.4.21V2
2017-02-05 11:39:43 +01:00
spjspj
9fe53e52c2
Merge pull request #2836 from spjspj/master
...
Add field for server console
2017-02-05 14:54:28 +11:00
spjspj
fa97e4647e
Add field for server console
2017-02-05 14:11:05 +11:00
LevelX2
ff6d79829e
* Fixed a bug that damage was wrongly still assigned to players during combat that already left the game.
2017-02-04 23:56:03 +01:00
LevelX2
1d404ee57f
Fixed a problem that the TAPPED_FOR_MANA event was not always send ( fixed #2826 ).
2017-02-04 17:56:04 +01:00
LevelX2
0dacf2a6fa
* Fixed that the cascade converted mana costs comparison did not work correctly with split spells.
2017-02-04 14:26:17 +01:00
LevelX2
227143aacf
* Entwine - Fixed that for spells cast without mana costs entwine could wrongly be used for free.
2017-02-04 14:24:30 +01:00
LevelX2
067b72dcce
* Added test.
2017-02-03 21:36:33 +01:00
LevelX2
a5299e3a30
* Fixed a bug that card art of cards with multiple arts (e.g. FEM) were not displayed during game play.
2017-02-03 20:45:47 +01:00
emerald000
f4c9ac53db
Added Battlefield Scrounger, Death Spark and Rasputin Dreamweaver.
2017-01-31 11:06:49 -05:00
LevelX2
9ec65dc480
* Green Sun's Zenith - Fixed a bug of the used Search effect (also affecting the other colors).
2017-01-31 16:20:21 +01:00
LevelX2
034ef22468
* Fixed PreventDamageByTargetEffect to handle delayed spell damge ( fixes #2822 ).
2017-01-30 21:19:17 +01:00
LevelX2
0a1356d40e
* Fixes to test and fix for issue #2816
2017-01-29 11:16:04 +01:00
LevelX2
2cf15830d7
* Some fixes to Hivestone and BecomesSubtyoeAllEffect.
2017-01-29 11:15:09 +01:00
LevelX2
e9d46db772
* Scrapheap Scrounger - Fixed that the exile cost was not mandatory.
2017-01-28 12:10:38 +01:00
vraskulin
fc54c0156c
list.size > 0 changed to !list.isEmpty
...
We care about if it's empty or not, not about it's size
2017-01-27 16:05:11 +03:00
vraskulin
0de8bd2f70
All 1-character strings converted to primitives
...
"b" + "r" now changed to 'b' + 'w'. It's more straight-forward, and may cause perfomance improvements - character primitives allocation is faster and less expensive than string creation.
2017-01-27 15:57:26 +03:00
goblin
7b40803b4f
fix the issue
...
Alter the constructor to create the object with setTargetPointer, like
the other constructors.
I hope nothing relies on it being false. Tests don't show anything breaking.
2017-01-27 13:26:31 +01:00
vraskulin
7b35e25347
Try/finally refactored to try with resources
2017-01-26 20:58:40 +03:00
LevelX2
082210a4bd
Xmage 1.4.21V1
2017-01-25 00:14:05 +01:00
LevelX2
ca0485075c
Merge pull request #2810 from Galatolol/master
...
Added 5 cards
2017-01-24 23:12:22 +01:00
LevelX2
cec9d00a59
* Fixed that non creature spells with morph could not be countered with Remove if cast with morph.
2017-01-24 23:08:47 +01:00
