lauchmelder/BloodMagic
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Swallow the NPE caused when other mods add recipes with null inputs

Browse source 
Also swallows the NPE caused by players adding a dead and outdated mod to their instance.

Read as: Screw you BiblioWoods Forestry.
This commit is contained in:
Nick 2015-10-26 16:35:53 -07:00
parent 6e54f83ebd
commit b38db9ae7a
2 changed files with 25 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/main/java/WayofTime/alchemicalWizardry/common/compress/StorageBlockCraftingManager.java
View file

@ -3,6 +3,7 @@ package WayofTime.alchemicalWizardry.common.compress;
import java.util.LinkedList; import java.util.LinkedList;
import java.util.List; import java.util.List;
import WayofTime.alchemicalWizardry.AlchemicalWizardry;
import net.minecraft.entity.player.EntityPlayer; import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.Container; import net.minecraft.inventory.Container;
import net.minecraft.inventory.InventoryCrafting; import net.minecraft.inventory.InventoryCrafting;
@ -26,7 +27,7 @@ public class StorageBlockCraftingManager
{ {
this.recipes = new StorageBlockCraftingRecipeAssimilator().getPackingRecipes(); this.recipes = new StorageBlockCraftingRecipeAssimilator().getPackingRecipes();
System.out.println("Total number of compression recipes: " + this.recipes.size()); AlchemicalWizardry.logger.info("Total number of compression recipes: " + this.recipes.size());
// List<IRecipe> tempRecipeList = new LinkedList(); // List<IRecipe> tempRecipeList = new LinkedList();
// //
// World world = DimensionManager.getWorld(0); // World world = DimensionManager.getWorld(0);

21
src/main/java/WayofTime/alchemicalWizardry/common/compress/StorageBlockCraftingRecipeAssimilator.java
View file

@ -21,6 +21,11 @@ import net.minecraftforge.oredict.ShapedOreRecipe;
import net.minecraftforge.oredict.ShapelessOreRecipe; import net.minecraftforge.oredict.ShapelessOreRecipe;
public class StorageBlockCraftingRecipeAssimilator { public class StorageBlockCraftingRecipeAssimilator {
public static String[] problemMods = {
"BiblioWoodsForestry"
};
public List<IRecipe> getPackingRecipes() { public List<IRecipe> getPackingRecipes() {
// grab all recipes potentially suitable for packing or unpacking // grab all recipes potentially suitable for packing or unpacking
@ -220,12 +225,26 @@ public class StorageBlockCraftingRecipeAssimilator {
} }
private boolean areInputsIdentical(ItemStack a, ItemStack b) { private boolean areInputsIdentical(ItemStack a, ItemStack b) {
if (a.getItem() != b.getItem()) return false;
try {
if (a.getItem() != b.getItem())
return false;
int dmgA = a.getItemDamage(); int dmgA = a.getItemDamage();
int dmgB = b.getItemDamage(); int dmgB = b.getItemDamage();
return dmgA == dmgB || dmgA == OreDictionary.WILDCARD_VALUE || dmgB == OreDictionary.WILDCARD_VALUE; return dmgA == dmgB || dmgA == OreDictionary.WILDCARD_VALUE || dmgB == OreDictionary.WILDCARD_VALUE;
} catch (NullPointerException e) {
AlchemicalWizardry.logger.error("A mod in this instance has registered an item with a null input. Known problem mods are:");
String err = "";
for (String problem : problemMods)
err += (err.length() > 0 ? ", " : "") + problem;
AlchemicalWizardry.logger.error(err);
return false;
}
} }
private static class PackingRecipe { private static class PackingRecipe {