Merge branch 'dump_state_with_owner' into 'master'

Fix dumping state with cards with non-matching owner/controllers.

See merge request core-developers/forge!2342
This commit is contained in:
Myrd
2019-12-20 17:56:37 +00:00

View File

@@ -242,7 +242,7 @@ public abstract class GameState {
if (card instanceof DetachedCardEffect) { if (card instanceof DetachedCardEffect) {
continue; continue;
} }
addCard(zone, card.getOwner() == ai ? aiCardTexts : humanCardTexts, card); addCard(zone, card.getController() == ai ? aiCardTexts : humanCardTexts, card);
} }
} }
} }
@@ -269,6 +269,10 @@ public abstract class GameState {
} }
if (zoneType == ZoneType.Battlefield) { if (zoneType == ZoneType.Battlefield) {
if (c.getOwner() != c.getController()) {
// TODO: Handle more than 2-player games.
newText.append("|Owner:" + (c.getOwner().isAI() ? "AI" : "Human"));
}
if (c.isTapped()) { if (c.isTapped()) {
newText.append("|Tapped"); newText.append("|Tapped");
} }