Commit graph

13393 commits

Author SHA1 Message Date
LevelX2
bc42e48c4a * Conviction - Fixed that the activated ability had a mana costs of {U} instead of {W}. 2015-11-23 12:12:48 +01:00
Neil Gentleman
085b00499c fix Grinning Totem
the exiled card wouldn't be put in its owner's graveyard if Grinning Totem
changed zones before the delayed trigger

any cards using CardUtil.getCardExileZoneId for delayed effects are likely
broken in the same way
2015-11-22 18:40:46 -08:00
Neil Gentleman
230d1d37bd fix Grinning Totem, Thada Adel, Commune with Lava
opponent was able to cast the exiled cards

also changed Knacksaw Clique to use the same templating as Ornate Kanzashi
2015-11-22 18:40:38 -08:00
Neil Gentleman
99d33eb771 fix Erratic Explosion
wouldn't reveal any cards, since ec7c888

updated Erratic Mutation to match, but it wasn't broken
2015-11-22 18:38:26 -08:00
Neil Gentleman
f06de472a4 fix Hostility
was preventing damage to opponents regardless of source
2015-11-22 18:00:28 -08:00
Neil Gentleman
a90d0e5597 reuse shared code for picking basic land sets 2015-11-22 18:00:10 -08:00
LevelX2
05dd1daee6 * Added some information to the player tooltip in game (expericence counters, deck hash code, # of wins). 2015-11-22 11:07:48 +01:00
LevelX2
975105fc4f * Sandstone Oracle - Fixed rule spelling. 2015-11-22 11:07:11 +01:00
Neil Gentleman
702be67453 correct some dodgy sample deck names 2015-11-21 22:36:07 -08:00
Neil Gentleman
3540eaada4 fix some set boosters
- Masters Edition II & IV don't have basics, they used ME1 lands
   when drafting
 - Fate Reforged does have basics, they just didn't show up in boosters
 - Journey Into Nyx boosters have a basic land slot
 - Unlimited boosters do not
 - Portal Three Kingdoms boosters are only 10 cards
 - Starter 1999 boosters have two basic lands
2015-11-21 18:36:46 -08:00
Neil Gentleman
038d02e8d7 add test for non-standard booster generation 2015-11-21 18:35:10 -08:00
LevelX2
96bc3172f5 * Fixed a problem of returning cards from exile (e.g. Fiend Hunter) if the triggering permanent made multiple zone changes before the left battlefield triggered ability resolved to return the exiled permanents. 2015-11-21 23:41:41 +01:00
LevelX2
b84a315780 [C15] Fixed not correct working Commander 2015 decklists. 2015-11-21 22:56:14 +01:00
LevelX2
e3d686954a Rogue's Passage - Fixed wrong collectors number of C15 card. 2015-11-21 22:54:49 +01:00
poixen
c60a5daa2d Merge pull request #1381 from poixen/fix_add
Renamed addXXX to increaseXXX
2015-11-21 12:20:49 +01:00
poixen
c8fcbca9f9 Renamed addXXX to increaseXXX
+ add implies a variable addition rather than an increment.
2015-11-21 12:20:14 +01:00
poixen
b0da2f2260 Merge pull request #1378 from poixen/mana_cleanup
Removed minus subtraction limits
2015-11-21 12:06:02 +01:00
poixen
b08dfa48db Merge branch 'fix_mana' into mana_cleanup 2015-11-21 12:04:37 +01:00
poixen
a5242ff087 resolved conflicts 2015-11-21 12:02:55 +01:00
poixen
5c6bdfceb8 Merge pull request #1379 from poixen/manasymbol
Manasymbol
2015-11-21 11:47:28 +01:00
LevelX2
0d86e8e063 Xmage 1.4.5v0 2015-11-21 10:33:43 +01:00
Quercitron
f3019d70fd * Tin Street Hooligan - Fix that ETB trigger went on the stack even if G was not spent to cast it 2015-11-21 03:22:46 +03:00
emerald000
59fabeee20 Added missing Scion subtype. 2015-11-20 17:42:14 -05:00
poixen
f2972c5b8e Updated Phyrexian Mana
+ “A Phyrexian mana symbol represents a cost that can be paid either
with *one mana of its color* or by paying 2 life”. These symbols are
mono colored, as such I have made them so.
+ Added test cases for phyrexian mana symbols
2015-11-20 22:43:26 +01:00
poixen
6138af0ffb Added Snow Mana Test
+ Added test case for snow mana
2015-11-20 22:29:49 +01:00
poixen
09ea717ed3 Added Hybrid Mana Tests
+ Added hybrid mana test cases
2015-11-20 22:24:55 +01:00
poixen
de13fb5cba Removed redundant modifiers
+ Removed the private modifiers. They are not needed in an ENUM class.
2015-11-20 22:08:39 +01:00
poixen
f34e69308e Updated mono colored hybrid mana type
+ “107.4e. Hybrid mana symbols are also colored mana symbols.” I have
added the Type COLORED to each mono hybrid symbol
+ added tests for hybrid mana symbols
2015-11-20 22:07:24 +01:00
poixen
f6cc03ec44 Added W mana test case
Added U mana test case
Added B mana test case
Added R mana test case
Added G mana test case
2015-11-20 20:59:52 +01:00
poixen
56be135532 Removed minus subtraction limits
+ Removed minus subtraction limits to allow other areas of code to work
2015-11-20 17:22:48 +01:00
LevelX2
8a133a43d0 Added test for Mana.enough method. 2015-11-20 15:06:00 +01:00
Quercitron
7e58dc70d5 Update Swiss tournament pairing - For the last round, for each unpaired player pair against the highest ranked player they haven't played against 2015-11-20 02:54:50 +03:00
LevelX2
b30a6c4b15 * Cryptoplasm - Fixed that the copied creature did not have the triggered ability (fixes #1367). 2015-11-19 23:28:12 +01:00
LevelX2
684cb2d472 Merge pull request #1377 from magefree/revert-1374-mana_cleanup
Revert "Mana Class Overhaul"
2015-11-19 23:00:04 +01:00
LevelX2
c4ab5806e0 Revert "Mana Class Overhaul" 2015-11-19 22:59:57 +01:00
LevelX2
02c1b9f22f Merge pull request #1376 from magefree/revert-1375-mana_cleanup
Revert "Mana updates"
2015-11-19 22:59:33 +01:00
LevelX2
68ff09c219 Revert "Mana updates" 2015-11-19 22:59:15 +01:00
LevelX2
f4871fe489 Merge pull request #1375 from poixen/mana_cleanup
Mana updates
2015-11-19 22:37:14 +01:00
LevelX2
4d14743c39 * Fungal Behemoth - Fixed that not only counters on controlled permanents were counted. 2015-11-19 22:35:43 +01:00
poixen
05b841577a added subtraction logic to enough()
+ added subornation logic to enough(). We do not want to mix the public
subtract() call with this method. As doing so would mean that either a)
subtraction can go below 0 (this should not happen) or b) we break this
function with exceptions. This is a work around for these scenarios.
2015-11-19 17:42:59 +01:00
LevelX2
cd17436c4e [C15] Added the C15 deck lists. 2015-11-19 17:25:38 +01:00
poixen
2c617a6aaf Mana updates
+ calling subtraction will now throw an exception if you try and use
more mana than is available. This is better than setting it to 0.
Setting to 0 impose that you should still be allowed to perform the
action.
+ updated subtraction test to check for exception
+ subtractionCost() will not allow using mana that is not available,
same as subtract()
2015-11-19 17:09:40 +01:00
LevelX2
ec7c888044 Some fixes to put token onto battlefield handling of cards. Some minor reworks. 2015-11-19 15:52:45 +01:00
LevelX2
7bb5d1bb43 Merge pull request #1374 from poixen/mana_cleanup
Mana Class Overhaul
2015-11-19 14:37:39 +01:00
Quercitron
486d547f5e Fix Swiss pairing for the case when some players have left the tournament 2015-11-19 13:18:43 +03:00
poixen
1f5638539f Class Overhaul
+ added javadoc
+ added final to method variables that should not be mutated
+ Mana can no longer be initialised with minus values
+ added null assertions
+ renamed ‘addXXX’ to ‘increaseXXX’. Add implies that you are able to
add variable amounts.
+ subtraction can no longer allow minus values.
+ removed unused state
+ refactored classes use renamed/removed mana calls
+ added comment to contains(), needs to be looked into?
+ Added validation to setting of mana. Can no longer be set to minus
values. Will log if an attempt is made
+Added comment to static mana, this is unsafe.
2015-11-19 09:53:52 +01:00
LevelX2
85bb13b6d8 Merge pull request #1373 from nigelzor/more-cards
implement a bunch of simple cards
2015-11-19 09:41:56 +01:00
Neil Gentleman
3fdfbca85b Aysen Bureaucrats has art variants 2015-11-18 21:49:59 -08:00
Neil Gentleman
c01cfd0395 fix Meteor Blast card name 2015-11-18 21:49:58 -08:00
Neil Gentleman
dac723028c implement a bunch of simple cards 2015-11-18 21:05:14 -08:00