Draco18s

PmsEnabled would be better named togglePMs

Ah for fuck's sake. That's your StandRecipeHandler.class, not your recipe's entity class. You want to reference the field, just like any other property of your recipe. StandRecipeHandler.sacrifice.isInstance(...)

clazz.isInstance(obj)

Ignore it. You aren't writing a method override in a subclass, so you don't need it.

You can't just rewrite existing methods or add methods to existing classes. And creating a custom mob won't really solve your problem either. What you should do is subscribe to the LivingUpdateEvent and use the zombie code to set other mobs on fire.

The class of the entity you are interested in. e.g. EntityLivingBase.class

You want a class reference. If you want to insure that it is an EntityLivingBase, your "add this recipe" method accepts Class<? extends EntityLivingBase>

PlentifulMiscAPI.addStandRecipe(new ItemStack, blah blah blah..., chicken); "Chicken?" Chicken? Where is this variable defined? All you did is declare it. Why are you (trying to) pass a reference to a specific entity? If you want a recipe you need to pass something generic. Like, say, a class. PlentifulMiscAPI.addStandRecipe(new ItemStack, blah blah blah..., EntityChicken.class); Then you can check if the specific entity is the same type.

Screenshot is a screenshot of you writing your post, not code.

You're still using a wonky for-loop with an iterator.

Also, why do you need an internal version of the method anyway? Especially one that is public (i.e. externally visible)?

You can't modify a list while you are iterating over it. If you want to remove an item from the list safely, use it.remove()

or for(BlockPos p : it) { }

escapemc_ore = new escapemc_ore(...) is the block of your block. You need to find where you call new ItemBlock(escapemc_ore) and add .setRegistryName(...) to that.

You didn't set a registry name for the Item form of your Block. You must do this.

Because it's old. Its not needed any more. The new way is better. And no, that change won't fix your problem, but it's still better.

GameRegistry.register(new ItemBlock(escapemc_ore), escapemc_ore.getRegistryName()); As you've already set the registry name, you don't need to pass it. GameRegistry.register(new ItemBlock(escapemc_ore)); is sufficient.

Don't implement ITileEntityProvider, just override the hasTileEntity and getTileEntity methods that are already present in the Block class. Also, why the fuck are you doing this: tile.writeToNBT(tag); int color = tag.getInteger("color"); return color + 10000; You have this available to you: public int getColor() { return this.color; }

I think that that is unfortunately due to Minecraft's approach to transparent rendering passes. Ice and water do that all the time.

Yes, we know. Diesieben07 has pointed out the problems with your current code. You need to fix those.

I'd add Capabilities in there between Block with Entity and Custom Crafting table. Which necessitates the Packets.

Still abstract, still has a bunch of methods that are tacked on without apparent purpose, still didn't answer diesieben07's question.

You see this bit in your blockstate file? "model": "cube_all" What do you suppose that does and what you might do to change it such that your model is used instead?

I had to logout/login at that point, as the client was unable to keep traveling farther. It was busy trying to fall.

He is not checking equality. Ah, you are correct. But yes, still nonsense.