github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-11-25 03:00:11 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix searching for split card halves returning full card in incorrect instances. (#9534)

Browse source
This commit is contained in:
Alex Vasile 2022-09-18 03:41:35 -04:00 committed by GitHub
parent 8f164b1efe
commit 7d57831ed0
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 55 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
Mage.Tests/src/test/java/org/mage/test/utils/CardRepositoryTest.java
View file

@ -128,9 +128,35 @@ public class CardRepositoryTest {
assertFindCard("dizzYIng sWOop");
}
/**
* Reported bug: https://github.com/magefree/mage/issues/9533
*
* Each half of a split card displays the combined information of both halves in the deck editor.
*
* `findCards`'s `returnSplitCardHalf` parameter should handle this issue
*/
@Test
public void splitCardInfoIsntDoubled() {
// Consecrate // Consume
// {1}{W/B} // {2}{W}{B}
List<CardInfo> fullCard1 = CardRepository.instance.findCards("Consecrate", 1, false);
Assert.assertTrue(fullCard1.get(0).isSplitCard());
Assert.assertEquals("Consecrate // Consume", fullCard1.get(0).getName());
List<CardInfo> fullCard2 = CardRepository.instance.findCards("Consume", 1, false);
Assert.assertTrue(fullCard2.get(0).isSplitCard());
Assert.assertEquals("Consecrate // Consume", fullCard2.get(0).getName());
List<CardInfo> splitHalfCardLeft = CardRepository.instance.findCards("Consecrate", 1, true);
Assert.assertTrue(splitHalfCardLeft.get(0).isSplitCardHalf());
Assert.assertEquals("Consecrate", splitHalfCardLeft.get(0).getName());
List<CardInfo> splitHalfCardRight = CardRepository.instance.findCards("Consume", 1, true);
Assert.assertTrue(splitHalfCardRight.get(0).isSplitCardHalf());
Assert.assertEquals("Consume", splitHalfCardRight.get(0).getName());
}
/**
* Checks if the card with name cardName can be found when searched for
* using the case sensitive approach.
* using the case-sensitive approach.
*
* @param cardName The name of the card to search by.
*/

4
Mage/src/main/java/mage/cards/mock/MockSplitCard.java
View file

@ -49,13 +49,13 @@ public class MockSplitCard extends SplitCard {
this.addAbility(textAbilityFromString(ruleText));
}
CardInfo leftHalf = CardRepository.instance.findCardWPreferredSet(getLeftHalfName(card), card.getSetCode());
CardInfo leftHalf = CardRepository.instance.findCardWPreferredSet(getLeftHalfName(card), card.getSetCode(), true);
if (leftHalf != null) {
this.leftHalfCard = new MockSplitCardHalf(leftHalf);
((SplitCardHalf) this.leftHalfCard).setParentCard(this);
}
CardInfo rightHalf = CardRepository.instance.findCardWPreferredSet(getRightHalfName(card), card.getSetCode());
CardInfo rightHalf = CardRepository.instance.findCardWPreferredSet(getRightHalfName(card), card.getSetCode(), true);
if (rightHalf != null) {
this.rightHalfCard = new MockSplitCardHalf(rightHalf);
((SplitCardHalf) this.rightHalfCard).setParentCard(this);

29
Mage/src/main/java/mage/cards/repository/CardRepository.java
View file

@ -391,12 +391,15 @@ public enum CardRepository {
*
* @param name name of the card, or side of the card, to find
* @param expansion the set name from which to find the card
* @param returnSplitCardHalf whether to return a half of a split card or the corresponding full card.
* Want this `false` when user is searching by either names in a split card so that
* the full card can be found by either name.
* @return
*/
public CardInfo findCardWPreferredSet(String name, String expansion) {
public CardInfo findCardWPreferredSet(String name, String expansion, boolean returnSplitCardHalf) {
List<CardInfo> cards;
cards = findCards(name);
cards = findCards(name, 0, returnSplitCardHalf);
if (!cards.isEmpty()) {
for (CardInfo cardinfo : cards) {
@ -408,6 +411,10 @@ public enum CardRepository {
return findPreferredCoreExpansionCard(name);
}
public CardInfo findCardWPreferredSet(String name, String expansion) {
return findCardWPreferredSet(name, expansion, false);
}
public List<CardInfo> findCards(String name) {
return findCards(name, 0);
}
@ -426,10 +433,14 @@ public enum CardRepository {
*
* @param name the name of the card to search for
* @param limitByMaxAmount return max amount of different cards (if 0 then return card from all sets)
* @return A list of the reprints of the card if it was found (up to limitByMaxAmount number), or
* an empty list if the card was not found.
* @param returnSplitCardHalf whether to return a half of a split card or the corresponding full card.
* Want this `false` when user is searching by either names in a split card so that
* the full card can be found by either name.
* Want this `true` when the client is searching for info on both halves to display it.
* @return a list of the reprints of the card if it was found (up to limitByMaxAmount number),
* or an empty list if the card was not found.
*/
public List<CardInfo> findCards(String name, long limitByMaxAmount) {
public List<CardInfo> findCards(String name, long limitByMaxAmount, boolean returnSplitCardHalf) {
List<CardInfo> results;
QueryBuilder<CardInfo, Object> queryBuilder = cardDao.queryBuilder();
if (limitByMaxAmount > 0) {
@ -467,8 +478,8 @@ public enum CardRepository {
// Check that a full card was found and not a SplitCardHalf
// Can be caused by searching for "Fire" instead of "Fire // Ice"
CardInfo firstCardInfo = results.get(0);
if (firstCardInfo.isSplitCardHalf()) {
// Find the main card by it's setCode and CardNumber
if (firstCardInfo.isSplitCardHalf() && !returnSplitCardHalf) {
// Find the main card by its setCode and CardNumber
queryBuilder.where()
.eq("setCode", new SelectArg(firstCardInfo.setCode)).and()
.eq("cardNumber", new SelectArg(firstCardInfo.cardNumber));
@ -498,6 +509,10 @@ public enum CardRepository {
return Collections.emptyList();
}
public List<CardInfo> findCards(String name, long limitByMaxAmount) {
return findCards(name, limitByMaxAmount, false);
}
public List<CardInfo> findCardsByClass(String canonicalClassName) {
try {
QueryBuilder<CardInfo, Object> queryBuilder = cardDao.queryBuilder();