* add start of Bargain
Current version probably has a bunch of bugs related to zcc and copy.
* add Torch the Tower
* add Torch the Tower tests
* add better than nothing activationKey before tag cost tracking gets cleaned up
---------
Co-authored-by: Evan Kranzler <theelk801@gmail.com>
* Rework Whipgrass Entangler
Made a class for "Ability linked with an Effect", that also takes responsability of manually calling its effect's newId method.
* apply review & cleanup
* [CMM] Implement Teyo, Geometric Tactician
* rework as a RestrictionEffect (common class with Mystic Barrier; Pramikon, Sky Rampart)
* add checkMayAttackDefender test
* more Pramikon tests
* refactor SacrificeCostManaValue to be an enum.
* [CMM] Implement Demon of Fates Design
* Add Unit Tests, including one bug on alternative cost.
* fix alternativeCosts made from dynamicCost returning that they were not activated when paid.
* fix small issues, add hint
* cleanup tests and add a couple
* Capitalize enum instances
* Minor fixes
* simplify the ContinuousEffect
* use the ConditionPermanentHint made for the Demon
* fix text
* Implement [CMM] Hatchery Sliver
* limit ReplicateAbility to only its linked instance
* Add test for multiple instances of replicate
* add another test
* implements Forge Anew with unit tests
* fixes equip effect and adjusts test
* fixes effect to only apply on controller's turns
* changed test location to set
* fixes comparison, fixes wording of text to match oracle text
* fixes issues with noted cards and standardizes code formatting, adds tests to confirm bug is fixed
* updates tests to use StrictChooseMode
* cleans up effect implementation for listed cards
* adds comments for choices, fixes mistake in Misstep test
* updates filters, removes unnecessary text being set
* improve variable name
* add test for Jitte trample damage
* fix duplicated triggers when trampling over to player
* bring corresponding ability into alignment
* adjust authorship, comment
* extra deck cards not counted in deck size
* extra deck handling in deckbuilder
* move responsibility for extraDeckCard boolean to CardImpl
* remove redundant field copy
* Backup is not optional and its effect should be BoostCreature, not Detriment
* Add tests for Backup ability
* Add Mirror-Shield Hoplite source test
* Use Strict mode for the hoplite source test
* [LTC] Implement Feasting Hobbit
Refactor DevourEffect and its text generation, to be more permissive in the future.
* change DevourAbility to have the constructor parameters of DevourEffect
* only pluralize when there is more than 1 devoured permanent.
* added more unit tests than there are cards with devour
* public -> private filter of Caprichrome.
- fixed miss phase changed events and logs in some use cases;
- added source info in turn modification logs;
- added game logs for take and lost control of the spell (example: Word of Command)
- added game logs for skip step;
- added game logs for extra step;
- added game logs for skip phase;
* UI Changes
* Add new buffer time options
* Main functionality
* Final implementation
Also added player UI for when they are using their buffer time (timer turns green)
* Added Goldberry
* Slight optimizaztion
* Happy Path Test
* More unhappy tests
* Sanity check for Goldberry's counter choices
* Updated player.getMultiAmount to support individual constraints
* Some cleanup
Also modified ResourcefulDefense to use new multi amount api
* Updated logging
* Added hint for number of counters
* Fixed issue with Resourceful Defense
* Improvements to defaults
Default list will properly make sure to stay within individual maximums
If a player is asked for a choice that isn't actually a choice because each choice's min and max are equal, instead the default response is immediately returned. This helps with situations like moving a counter off of Goldberry when she only has one counter on her.
* -1/-1 Counter test
* Fixed issue with -1/-1 counters
* Adjusted dialog to properly enforce constraints
* Add (failing) unit test on Pyrrhic Revival
* fix PyrrhicRevival
reworked ReturnFromGraveyardToBattlefieldWithCounterTargetEffect to support having multiple cards in its targetPointer.
added test for Persist (the card from mh2).
* refactor & cleanup
* add myself as author, the effect was remade.
* adjust clash effect
* Make clash not a singleton
* Add unit test for Clash effect
* fix test (skip init shuffling)
* Fix CLASHED event flag logic and add to unit test
* Additional test and comments
* comments in GameEvent
* param name typo
* Fix Ring-bearer choosing & add some GUI + logs
* use a ring svg in a separate gold panel
* use a fontawesome svg
* add a couple null checks, group icon with commander
* rework rinbearer logic according to review
* fix typo in game log
* small fixes
* Fix#9419 bug with EntersBattlefieldThisOrAnotherTriggeredAbility
* Add test for Radagast as well as Risen Reef and Caldaia Guardian
* Minor associated cleanup for EntersBattlefieldAllTriggeredAbility
* New PermanentEnteringBattlefieldManaValue
* [CSP] Fix the RevealTargetFromHandCost not allowing payment when no valid card to reveal
* cleanup text of cost
* fix martyr of bones text
* fix test using a Martyr of Sands
* Refactor OnLeaveReturnExiledAbility
to accommodate hand zone as well as battlefield
* Cleanup Brain Maggot and Kitesail Freebooter
* Refactor to include delayed trigger with main effect
* minor cleanup
* merge fix
* further adjustments
* Cleanup Valki, God of Lies
* fix test choices
Did not test the final code for Stone of Erech, when a filter was added to the ReplacementEffect :(
Added an unit test, just to be safer this time around.
