External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-24 20:41:58 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

refactor: fixed miss deep copy of AlternativeSourceCosts in PlayerImpl (related to #12420)

Browse source
This commit is contained in:
Oleg Agafonov 2024-06-07 22:45:39 +04:00
parent 273770246c
commit 37dbdb4b4b
1 changed files with 3 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -158,7 +158,7 @@ public abstract class PlayerImpl implements Player, Serializable {
protected boolean drawsOnOpponentsTurn = false;
protected FilterPermanent sacrificeCostFilter;
protected final List<AlternativeSourceCosts> alternativeSourceCosts = new ArrayList<>();
protected List<AlternativeSourceCosts> alternativeSourceCosts = new ArrayList<>();
// TODO: rework turn controller to use single list (see other todos)
//protected Stack<UUID> allTurnControllers = new Stack<>();
@ -264,7 +264,7 @@ public abstract class PlayerImpl implements Player, Serializable {
this.payLifeCostLevel = player.payLifeCostLevel;
this.sacrificeCostFilter = player.sacrificeCostFilter;
this.alternativeSourceCosts.addAll(player.alternativeSourceCosts);
this.alternativeSourceCosts = CardUtil.deepCopyObject(player.alternativeSourceCosts);
this.storedBookmark = player.storedBookmark;
this.topCardRevealed = player.topCardRevealed;
@ -365,8 +365,7 @@ public abstract class PlayerImpl implements Player, Serializable {
this.canPlayCardsFromGraveyard = player.canPlayCardsFromGraveyard();
this.canPlotFromTopOfLibrary = player.canPlotFromTopOfLibrary();
this.drawsOnOpponentsTurn = player.isDrawsOnOpponentsTurn();
this.alternativeSourceCosts.clear();
this.alternativeSourceCosts.addAll(player.getAlternativeSourceCosts());
this.alternativeSourceCosts = CardUtil.deepCopyObject(player.getAlternativeSourceCosts());
this.topCardRevealed = player.isTopCardRevealed();