Prevent storage of null ingredients.

BenWoodworth · May 9, 2016 · db3f080 · db3f080
1 parent fe4a6cc
commit db3f080
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -6,7 +6,7 @@
 
     <groupId>co.kepler.fastcraftplus</groupId>
     <artifactId>fastcraftplus</artifactId>
-    <version>0.20.2</version>
+    <version>0.20.3</version>
 
     <build>
         <plugins>

diff --git a/src/main/java/co/kepler/fastcraftplus/recipes/FastRecipe.java b/src/main/java/co/kepler/fastcraftplus/recipes/FastRecipe.java
@@ -66,7 +66,9 @@ public final ItemStack[] getMatrix(int multiplier) {
      * @return Returns the ingredients required to craft this recipe.
      */
     public final Map<Ingredient, Integer> getIngredients() {
-        return new HashMap<>(getIngredientsInternal());
+        HashMap<Ingredient, Integer> result = new HashMap<>(getIngredientsInternal());
+        result.remove(null);
+        return result;
     }
 
     /**
@@ -114,7 +116,6 @@ public Set<ItemStack> getByproducts() {
         // Count the number of buckets to be returned
         int buckets = 0;
         for (Ingredient i : getIngredients().keySet()) {
-            if (i == null) continue;
             switch (i.getMaterial()) {
             case LAVA_BUCKET:
             case MILK_BUCKET: