github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-11-24 19:19:56 +00:00
Code Issues Projects Releases Packages Wiki Activity

[DMC] Fix Historian's Boon. Remove unused and incorrectly-implemented EventType.ABILITY_TRIGGERED

Browse source
This commit is contained in:
Alex W. Jackson 2022-11-05 04:12:35 -04:00
parent 59d99e5ba2
commit a9cf4154e5
3 changed files with 6 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
Mage.Sets/src/mage/cards/h/HistoriansBoon.java
View file

@ -1,6 +1,5 @@
package mage.cards.h;
import mage.abilities.Ability;
import mage.abilities.TriggeredAbilityImpl;
import mage.abilities.common.EntersBattlefieldThisOrAnotherTriggeredAbility;
import mage.abilities.common.SagaAbility;
@ -19,6 +18,7 @@ import mage.game.events.GameEvent;
import mage.game.permanent.Permanent;
import mage.game.permanent.token.AngelVigilanceToken;
import mage.game.permanent.token.SoldierToken;
import mage.game.stack.StackObject;
import mage.util.CardUtil;
import java.util.UUID;
@ -74,13 +74,15 @@ class HistoriansBoonTriggeredAbility extends TriggeredAbilityImpl {
@Override
public boolean checkEventType(GameEvent event, Game game) {
return event.getType() == GameEvent.EventType.ABILITY_TRIGGERED;
return event.getType() == GameEvent.EventType.TRIGGERED_ABILITY;
}
@Override
public boolean checkTrigger(GameEvent event, Game game) {
StackObject stackObject = game.getStack().getStackObject(event.getTargetId());
Permanent permanent = game.getPermanent(event.getSourceId());
if (permanent == null
if (stackObject == null
|| permanent == null
|| !permanent.isControlledBy(getControllerId())
|| !permanent.hasSubtype(SubType.SAGA, game)) {
return false;
@ -90,8 +92,7 @@ class HistoriansBoonTriggeredAbility extends TriggeredAbilityImpl {
.map(SagaAbility::getMaxChapter)
.mapToInt(SagaChapter::getNumber)
.sum();
Ability ability = game.getAbility(event.getTargetId(), event.getSourceId()).orElse(null);
return SagaAbility.isFinalAbility(ability, maxChapter);
return SagaAbility.isFinalAbility(stackObject.getStackAbility(), maxChapter);
}
@Override

1
Mage/src/main/java/mage/game/events/GameEvent.java
View file

@ -201,7 +201,6 @@ public class GameEvent implements Serializable {
playerId player that tries to use this ability
*/
TRIGGERED_ABILITY,
ABILITY_TRIGGERED,
RESOLVING_ABILITY,
/* COPY_STACKOBJECT
targetId id of the spell/ability to copy

6
Mage/src/main/java/mage/players/PlayerImpl.java
View file

@ -1524,12 +1524,6 @@ public abstract class PlayerImpl implements Player, Serializable {
}
}
restoreState(bookmark, triggeredAbility.getRule(), game); // why restore is needed here? (to remove the triggered ability from the stack because of no possible targets)
GameEvent event = new GameEvent(
GameEvent.EventType.ABILITY_TRIGGERED,
ability.getId(), ability, ability.getControllerId()
);
game.getState().setValue(event.getId().toString(), ability.getTriggerEvent());
game.fireEvent(event);
return false;
}