Merge branch 'simfix' into 'master'

Simulation: fix wrong ActivatingPlayer

See merge request core-developers/forge!6420
This commit is contained in:
Michael Kamensky
2022-03-21 03:49:17 +00:00
5 changed files with 74 additions and 58 deletions

View File

@@ -26,7 +26,6 @@ import java.util.List;
import java.util.Map;
import java.util.Set;
import forge.game.event.GameEventDayTimeChanged;
import org.apache.commons.lang3.tuple.Pair;
import com.google.common.base.Predicate;
@@ -55,6 +54,7 @@ import forge.game.card.CardView;
import forge.game.card.CounterType;
import forge.game.combat.Combat;
import forge.game.event.Event;
import forge.game.event.GameEventDayTimeChanged;
import forge.game.event.GameEventGameOutcome;
import forge.game.phase.Phase;
import forge.game.phase.PhaseHandler;

View File

@@ -434,7 +434,8 @@ public abstract class SpellAbility extends CardTraitBase implements ISpellAbilit
public boolean setActivatingPlayer(final Player player, final boolean lki) {
// trickle down activating player
boolean updated = false;
if (player == null || !player.equals(activatingPlayer)) {
// don't use equals because player might be from simulation
if (player == null || player != activatingPlayer) {
activatingPlayer = player;
updated = true;
}