Skip to content

Commit

Permalink
Fix bad references for snapshots
Browse files Browse the repository at this point in the history
  • Loading branch information
tehdiplomat committed Aug 13, 2024
1 parent 67b18b2 commit 6886992
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
8 changes: 4 additions & 4 deletions forge-game/src/main/java/forge/game/GameSnapshot.java
Original file line number Diff line number Diff line change
Expand Up @@ -362,16 +362,16 @@ public void copyGameState(Game fromGame, Game toGame) {
if (fromCard.getCloneOrigin() != null) {
newCard.setCloneOrigin(toGame.findById(fromCard.getCloneOrigin().getId()));
}
if (newCard.getHaunting() != null) {
if (fromCard.getHaunting() != null) {
newCard.setHaunting(toGame.findById(fromCard.getHaunting().getId()));
}
if (newCard.getEffectSource() != null) {
if (fromCard.getEffectSource() != null) {
newCard.setEffectSource(toGame.findById(fromCard.getEffectSource().getId()));
}
if (newCard.isPaired()) {
if (fromCard.isPaired()) {
newCard.setPairedWith(toGame.findById(fromCard.getPairedWith().getId()));
}
if (newCard.getCopiedPermanent() != null) {
if (fromCard.getCopiedPermanent() != null) {
newCard.setCopiedPermanent(toGame.findById(fromCard.getCopiedPermanent().getId()));
}
// TODO: Verify that the above relationships are preserved bi-directionally or not.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ public final Card copyCard(boolean assignNewId, Player owner) {

// need to copy this values for the tokens
out.setTokenSpawningAbility(copyFrom.getTokenSpawningAbility());
out.setCopiedPermanent(copyFrom.getCopiedPermanent());
} else {
out = assignNewId ? getCard(copyFrom.getPaperCard(), owner, toGame)
: getCard(copyFrom.getPaperCard(), owner, copyFrom.getId(), toGame);
Expand Down

0 comments on commit 6886992

Please sign in to comment.