-
Notifications
You must be signed in to change notification settings - Fork 594
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MH3 card scripts #4754
MH3 card scripts #4754
Conversation
Update paths_of_tuinvale.txt
Editing out
@dracontes I invited you to be a contributor so you can claim the cards you're working on in the projects |
Much obliged for the invitation :) Claiming done. |
SVar:TrigToken:DB$ Token | TokenAmount$ 1 | TokenScript$ w_2_1_cat_warrior | TokenOwner$ You | ||
T:Mode$ ChangesZoneAll | TriggerZones$ Battlefield | ValidCards$ Cat.Other+YouCtrl | Origin$ Battlefield | Destination$ Graveyard | Execute$ TrigExile | OptionalDecider$ You | TriggerDescription$ Whenever one or more other Cats you control die, you may exile NICKNAME, then return him to the battlefield transformed under his owner's control. | ||
SVar:TrigExile:DB$ ChangeZone | Origin$ Battlefield | Destination$ Exile | RememberChanged$ True | SubAbility$ DBReturn | ||
SVar:DBReturn:DB$ ChangeZone | Defined$ Remembered | Origin$ Exile | Destination$ Battlefield | Transformed$ True | ForgetOtherRemembered$ True | SubAbility$ DBCleanup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since you're using ForgetOtherRemembered the Cleanup does become redundant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed ForgetOtherRemembered
then as it ostensibly reads as less widely applicable. The card script works fine at least.
A:AB$ PutCounterAll | Cost$ AddCounter<2/LOYALTY> | ValidCards$ Cat.YouCtrl | CounterType$ P1P1 | CounterNum$ 1 | Planeswalker$ True | SpellDescription$ Put a +1/+1 counter on each Cat you control. | ||
A:AB$ Token | Cost$ AddCounter<0/LOYALTY> | TokenAmount$ 1 | TokenScript$ w_2_1_cat_warrior | TokenOwner$ You | RememberTokens$ True | SubAbility$ DBImmediateTrig1 | Planeswalker$ True | SpellDescription$ Create a 2/1 white Cat Warrior creature token. When you do, if you control a red permanent other than CARDNAME, he deals damage equal to the number of creatures you control to any target. | ||
SVar:DBImmediateTrig1:DB$ ImmediateTrigger | ConditionDefined$ Remembered | ConditionPresent$ Card | Execute$ ImmediateTrig2 | SubAbility$ DBCleanup | TriggerDescription$ When you do,... | ||
SVar:ImmediateTrig2:DB$ ImmediateTrigger | ConditionPresent$ Permanent.Red+YouCtrl+Other | Execute$ TrigDamage | TriggerDescription$ When you do, if you control a red permanent other than CARDNAME, he deals damage equal to the number of creatures you control to any target. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this would give you two reflexive triggers
you should only use one, ConditionPresent2$ exists so you can have both
and the DealDamage needs to check for it too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much obliged for the pointer :) Done and tested. Unfortunately this pull request even split is one big mess so I'll be starting this one over on a fresh branch. My apologies for that.
Turns out to be too much of a mess for me to fix so closing it and starting over. |
(Split from #4746 )
Tested card scripts
ImmediateTrig
on the second loyalty ability aren't in any way elegant but these were the only thing I could think of to satisfy both the "When you do [create a token this way]" and the "if you control a red permanent other than Ajani, Nacatl Avenger" conditions. Probably not the smartest thing to draw attention to the former as an intermediate step. I'm all ears for better options.