It is currently 12 May 2025, 16:10
   
Text Size

[fix] Sagas don't trigger their abilities at the right time

Moderators: BAgate, drool66, Aswan jaguar, gmzombie, stassy, CCGHQ Admins

[fix] Sagas don't trigger their abilities at the right time

Postby Aswan jaguar » 06 Mar 2021, 11:53

Describe the Bug:
Sagas don't trigger chapter abilities at the right time.
When a Saga enters the battlefield and gets a lore counter the chapter 1 ability doesn't trigger. It will trigger next time it will get a counter so all chapters go one turn later and as a result the 3 chapter never triggers as the card has been sacrificed previous turn.

Relevant Rules:
As a Saga enters the battlefield, its controller puts a lore counter on it. As your precombat main phase begins (immediately after your draw step), you put another lore counter on each Saga you control. Putting a lore counter on a Saga in either of these ways doesn't use the stack. Each symbol on the left of a Saga's text box represents a chapter ability.
A chapter ability is a triggered ability that triggers when a lore counter that is put on the Saga causes the number of lore counters on the Saga to become equal to the ability's chapter number. Chapter abilities are put onto the stack and may be responded to.

Which card did behave improperly?
saga mechanism

Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
March 2020 - Allegiances and Alliances, duel

What exactly should be the correct behavior/interaction?


Are any other cards possibly affected by this bug?
-
Last edited by drool66 on 10 Mar 2021, 03:53, edited 3 times in total.
Reason: Fixed
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8129
Joined: 13 May 2010, 12:17
Has thanked: 748 times
Been thanked: 477 times

Re: Sagas don't trigger chapter abilities at the right time

Postby drool66 » 07 Mar 2021, 04:33

Confirmed. It's because of cec0735 where I moved XTRIGGER_REPLACE_COUNTERS to the beginning of add_or_remove_counters_impl(). I'm pretty sure this is the only thing that looks for it, so I'll try to figure out if it's better to move the trigger or change how sagas work.

[EDIT] Fixed by changing how Sagas work in ad60aa7. Nothing now looks for XTRIGGER_REPLACE_COUNTERS
User avatar
drool66
Programmer
 
Posts: 1185
Joined: 25 Nov 2010, 22:38
Has thanked: 187 times
Been thanked: 280 times


Return to Archived Reports

Who is online

Users browsing this forum: No registered users and 36 guests


Who is online

In total there are 36 users online :: 0 registered, 0 hidden and 36 guests (based on users active over the past 10 minutes)
Most users ever online was 4143 on 23 Jan 2024, 08:21

Users browsing this forum: No registered users and 36 guests

Login Form