diff --git a/src/main/java/forge/card/CardFace.java b/src/main/java/forge/card/CardFace.java index 00c76c579a2..f254b3e1eb6 100644 --- a/src/main/java/forge/card/CardFace.java +++ b/src/main/java/forge/card/CardFace.java @@ -32,7 +32,7 @@ final class CardFace implements ICardCharacteristics { private int iToughness = -1; private String power = null; private String toughness = null; - private int initialLoyalty = 0; + private int initialLoyalty = -1; private final List keywords = new ArrayList(); diff --git a/src/main/java/forge/gui/deckeditor/tables/SColumnUtil.java b/src/main/java/forge/gui/deckeditor/tables/SColumnUtil.java index 7816bb0b7f9..bcc83d6531e 100644 --- a/src/main/java/forge/gui/deckeditor/tables/SColumnUtil.java +++ b/src/main/java/forge/gui/deckeditor/tables/SColumnUtil.java @@ -324,19 +324,18 @@ public final class SColumnUtil { return i instanceof CardPrinted ? ((CardPrinted) i).getRules().getColor() : ColorSet.getNullColor(); } - private static Integer toPower(final InventoryItem i) { - Integer result = -1; + private static int toPower(final InventoryItem i) { + int result = -1; if (i instanceof CardPrinted) { result = ((CardPrinted) i).getRules().getIntPower(); - if (result == null) { - result = Integer.valueOf(((CardPrinted) i).getRules().getInitialLoyalty()); - if (result == null) { result = -1; } + if (result == -1) { + result = ((CardPrinted) i).getRules().getInitialLoyalty(); } } return result; } - private static Integer toToughness(final InventoryItem i) { + private static int toToughness(final InventoryItem i) { return i instanceof CardPrinted ? ((CardPrinted) i).getRules().getIntToughness() : -1; } @@ -383,10 +382,6 @@ public final class SColumnUtil { return ranking; } - private static String toRankingStr(final InventoryItem i) { - return String.valueOf(toRankingCmp(i)); - } - //========== /** Lamda sort fnQtyCompare. */ @@ -476,7 +471,7 @@ public final class SColumnUtil { private static final Function, Comparable> FN_POWER_COMPARE = new Function, Comparable>() { @Override public Comparable apply(final Entry from) { - return SColumnUtil.toPower(from.getKey()); + return Integer.valueOf(SColumnUtil.toPower(from.getKey())); } }; @@ -484,7 +479,7 @@ public final class SColumnUtil { private static final Function, Object> FN_POWER_GET = new Function, Object>() { @Override public Object apply(final Entry from) { - return SColumnUtil.toPower(from.getKey()); + return Integer.valueOf(SColumnUtil.toPower(from.getKey())); } }; @@ -492,7 +487,7 @@ public final class SColumnUtil { private static final Function, Comparable> FN_TOUGHNESS_COMPARE = new Function, Comparable>() { @Override public Comparable apply(final Entry from) { - return SColumnUtil.toToughness(from.getKey()); + return Integer.valueOf(SColumnUtil.toToughness(from.getKey())); } }; @@ -500,7 +495,7 @@ public final class SColumnUtil { private static final Function, Object> FN_TOUGHNESS_GET = new Function, Object>() { @Override public Object apply(final Entry from) { - return SColumnUtil.toToughness(from.getKey()); + return Integer.valueOf(SColumnUtil.toToughness(from.getKey())); } }; @@ -580,7 +575,7 @@ public final class SColumnUtil { private static final Function, Object> FN_RANKING_GET = new Function, Object>() { @Override public Object apply(final Entry from) { - return SColumnUtil.toRankingStr(from.getKey()); + return String.valueOf(SColumnUtil.toRankingCmp(from.getKey())); } }; }