Draco18s

And change the block type to not be a full, opaque cube so it doesn't create holes in the world.

but good practice

You divide out the amount you want to transfer before you figure out the number of directions a transfer is valid for. Not your problem, but worth mentioning. Rather than doing getMaterialsAround() (which hardcodes every side...and stores the results in a class property!? Why!?) and then iterating over all the sides...just iterate over the sides and call world.getTileEntity inside the loop.

in an EntityJoinWorldEvent event handler.

stack.shrink()

Quote of the day right here. You need to add a new task to the zombie's AI task list. It's public.

That sounds more like a populator or a decorator, the way vanilla does clay deposits in water or dirt and gravel underground.

Which line is line 34?

Its likely just a vertex color operation that changes the color. Vertex color is super cheap. It's used for particle systems with hundreds of thousands of particles and it runs just fine, one or a dozen items, easy peasy.

Have you tried looking at the vanilla blocks that do this?

You changed the physical height of the block, but you didn't increase it's texture UV size. You, in fact, decreased it: "uv": [ 0, 12, 16, 16 ] This says "this face has UV coordinates from (0,12) to (16,16)." Or 16x4.

Don't post your code as attachments. Its much easier for us if you post it on Pastebin (or similar) or as a clonable Github repository. That said: http://mcforge.readthedocs.io/en/latest/models/using/#item-models

Fair enough.

Thanks Lex, I wasn't sure what the target version was (i.e. if Minecraft had moved up to J9 while I wasn't looking), but I knew it wasn't 10.

Hmm. "The IHasModel is not necessary. All items require models that must be registered and the registration can be done without any non-public information from the Item instance. The same goes for blocks: not all blocks have items but the ones that do, follow the same rule. IHasModel was created and perpetuated through a series of uninformed tutorials and has become an example of Cargo Cult Programming." May be worth noting that ModelRegistryEvent is client-side-only as a Problematic Code separately. "The ModelRegistryEvent is in the Forge client namespace. This means that it may not be present on the dedicated server. As such, it should only be used inside a Client Proxy class to avoid issues."

Generics are amazing. For example, one of my mods has this method: private <T extends Comparable<T>, V extends T,U extends Comparable<U>, W extends U> void addArbitraryOre(String orename, int numFlowers, int clusterSize, int threshold, OreFlowerDictator dictator, IBlockState flower, IBlockState desertFlower, @Nullable IProperty<T> flowerProp, @Nullable V flowerValue, @Nullable IProperty<U> desertProp, @Nullable W desertValue) { //... } T & V and U & W are what they are because I need to interact with an IBlockState object and its property without referencing a specific property (as I am using four related, but incompatible, enum variants). In theory I could get rid of U & W (and declare two <T> parameters), but there's no reason to.

Also problematic code #7. Additionally, IHasModel is stupid. ALL items need models. The IHasModel interface does fuckall but make you write the same line of code over and over again when you could instead do this: @SubscribeEvent public static void onModelRegister(ModelRegistryEvent event) { for(Item item : ModItems.ITEMS) { Main.proxy.registerItemRenderer(item, 0, "inventory"); } } And I'll also point out that the ModelRegistryEvent is client side only and so this event should already be in your Client Proxy, which means you can do this: @SubscribeEvent public static void onModelRegister(ModelRegistryEvent event) { for(Item item : ModItems.ITEMS) { ModelLoader.setCustomModelResourceLocation(item, 0, new ModelResourceLocation(item.getRegistryName(), "inventory") } } Hey @diesieben07 can we add the stupidness of IHasModel to the list of Code Style issues? I've probably said "this is dumb" like 14 times now.

Its not an array. It's a generic.

Obstacles in dungeons, allowing all players to see when one person holds the item is a reasonable thing. If a group goes in together showing the blocks to everyone is a Quality of Life improvement IMO.

Run, yes, compile, no. You want the Java 9 JDK (but the Java 10 RTE is fine).

They can't, but that mob looks pretty easy to rebuild as a model in one of them. Looks like the model could be done in *counts* 13 cuboids. Once for the central body and then 4 for each of the 3 axises (they'd be long, so rather than 24 spikes, you'd make 12 rods).

It's TerraFirmaCraft, the Minecraft mod that gives no fucks about being compatible with anyone or anything. And also:

Posting a picture of what it looks like would probably be more helpful. In any case, I suggest using a tool like Blockbench or MrCrayfish. There are a few others as well.

That is not the code. That is the error.