Fix logic for Henzie

forge-core/src/main/java/forge/card/mana/ManaCost.java

@@ -42,7 +42,7 @@ public final class ManaCost implements Comparable<ManaCost>, Iterable<ManaCostSh
 
     private List<ManaCostShard> shards;
     private final int genericCost;
-    private boolean hasNoCost = true; // lands cost
+    private final boolean hasNoCost; // lands cost
     private String stringValue; // precalculated for toString;
 
     private Float compareWeight = null;
@@ -93,14 +93,14 @@ private void sealClass(List<ManaCostShard> shards0) {
     public ManaCost(final IParserManaCost parser) {
         final List<ManaCostShard> shardsTemp = Lists.newArrayList();
         while (parser.hasNext()) {
-            this.hasNoCost = false;
             final ManaCostShard shard = parser.next();
             if (shard != null && shard != ManaCostShard.GENERIC) {
                 shardsTemp.add(shard);
             } // null is OK - that was generic mana
         }
-        this.genericCost = parser.getTotalGenericCost(); // collect generic mana
-        // here
+        int generic = parser.getTotalGenericCost(); // collect generic mana here
+        this.hasNoCost = generic == -1;
+        this.genericCost = generic == -1 ? 0 : generic;
         sealClass(shardsTemp);
     }
 

forge-core/src/main/java/forge/card/mana/ManaCostParser.java

@@ -1,6 +1,6 @@
 package forge.card.mana;
 
-import org.apache.commons.lang3.StringUtils;
+import com.google.common.primitives.Ints;
 
 
 /**
@@ -54,7 +54,7 @@ public final int getTotalGenericCost() {
      */
     @Override
     public final boolean hasNext() {
-        return this.nextToken < this.cost.length && !this.cost[this.nextToken].equals("-1");
+        return this.nextToken < this.cost.length;
     }
 
     /*
@@ -65,8 +65,10 @@ public final boolean hasNext() {
     @Override
     public final ManaCostShard next() {
         final String unparsed = this.cost[this.nextToken++];
-        if (StringUtils.isNumeric(unparsed)) {
-            this.genericCost += Integer.parseInt(unparsed);
+        // consider negation sign
+        Integer i = Ints.tryParse(unparsed);
+        if (i != null) {
+            this.genericCost += i;
             return null;
         }
 

forge-game/src/main/java/forge/game/mana/ManaCostBeingPaid.java

@@ -97,7 +97,10 @@ public boolean hasNext() {
         @Override
         public int getTotalGenericCost() {
             ShardCount c = unpaidShards.get(ManaCostShard.GENERIC);
-            return c == null ? 0 : c.totalCount;
+            if (c == null) {
+                return unpaidShards.isEmpty() ? -1 : 0;
+            }
+            return c.totalCount;
         }
     }