From 1568189c1bab77425a992fc3d3e483559cdcf6e0 Mon Sep 17 00:00:00 2001 From: Oleg Agafonov Date: Wed, 9 Apr 2025 01:22:33 +0400 Subject: [PATCH] other: fixed typos in omen related code (#13501); --- .../plugins/card/dl/sources/ScryfallApiCard.java | 6 +++--- .../test/java/mage/verify/VerifyCardDataTest.java | 12 ++++++++++-- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/Mage.Client/src/main/java/org/mage/plugins/card/dl/sources/ScryfallApiCard.java b/Mage.Client/src/main/java/org/mage/plugins/card/dl/sources/ScryfallApiCard.java index b864f74a21d..4ea496665d0 100644 --- a/Mage.Client/src/main/java/org/mage/plugins/card/dl/sources/ScryfallApiCard.java +++ b/Mage.Client/src/main/java/org/mage/plugins/card/dl/sources/ScryfallApiCard.java @@ -54,7 +54,7 @@ public class ScryfallApiCard { this.card_faces.forEach(ScryfallApiCardFace::prepareCompatibleData); } - // workaround for adventure card name fix: + // workaround for adventure/omen card name fix: // - scryfall: Ondu Knotmaster // Throw a Line // - xmage: Ondu Knotmaster if (this.layout.equals("adventure")) { @@ -100,12 +100,12 @@ public class ScryfallApiCard { } this.name = this.card_faces.get(0).name; } else if (this.card_faces.get(0).layout.equals("adventure")) { - // adventure card + // adventure/omen card // Bloomvine Regent // Claim Territory // https://scryfall.com/card/tdm/381/bloomvine-regent-claim-territory-bloomvine-regent this.name = this.card_faces.get(0).name; if (this.card_faces.get(0).name.equals(this.card_faces.get(1).name)) { - throw new IllegalArgumentException("Scryfall: unsupported data type, adventure's reversible_card must have diff names in faces " + throw new IllegalArgumentException("Scryfall: unsupported data type, adventure/omen's reversible_card must have diff names in faces " + this.set + " - " + this.collector_number + " - " + this.name); } } else if (this.card_faces.get(0).layout.equals("token")) { diff --git a/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java b/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java index 72b6d6dabad..0bd47644157 100644 --- a/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java +++ b/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java @@ -2560,12 +2560,20 @@ public class VerifyCardDataTest { refRules[i]; } } - + if (ref.subtypes.contains("Omen")) { + for (int i = 0; i < refRules.length; i++) { + refRules[i] = "Omen " + + ref.types.get(0) + " - " + + ref.faceName + ' ' + + ref.manaCost + " - " + + refRules[i]; + } + } String[] cardRules = card .getRules() .stream() - .filter(s -> !(card instanceof CardWithSpellOption) || !s.startsWith("Adventure ") || !s.startsWith("Omen ")) + .filter(s -> !(card instanceof CardWithSpellOption) || !(s.startsWith("Adventure ") || s.startsWith("Omen "))) .collect(Collectors.joining("\n")) .replace("
", "\n") .replace("
", "\n")