Draco18s

You don't need ItemBase either. You can call this from your ModelRegistryEvent just fine: ModelLoader.setCustomModelResourceLocation(item, 0, new ModelResourceLocation(item.getRegistryName(),"normal")); See above

You're current registering the item with a "" variant. You should use " inventory" Also IHasModel is stupid. All items need models and none of the information necessary to register a model is private. ItemBase is an antipattern:

This is what happens when you render opaque things with transparency. There's a way to specify dual layer models, but I can't think of the search term to look for at the moment.

enum != Enumeration https://stackoverflow.com/a/19445327/1663383 Don't know about what you're trying to do, though.

Yeah. Gotta be an array, but lists are easier to work with. Its why my utils class got so convoluted so quickly.

Take a look at this: https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/hardlib/util/LootUtils.java

Your "code so far" is an autogenerated implementation. You've done fuckall so far. You might want to actually write some code.

That part requires Java code. Look at: the furnace, pistons, hoppers, droppers, and logs.

/me sighs at the heavy use of the arrow antipattern.

Something is null.

I am pretty sure what he's trying to do is make edits to vanilla and bypass Forge: Which is 100% absolutely positively against the law, EULA, and the TOS.

Probably all of these things:

Its a protected field. You need to call SetAccessible(true).

No.

ActualState In either case, I do not recommend TEs for logs. Use multiple blocks like Vanilla does.

You don't need this. IHasModel is stupid. All items need models and all the information needed to register a model is public. Your log is useless, there are no errors. Not sure why the cart won't pass.

Nine Log Types * 3 axis of orientation = 27 possible state variants. Metadata is strictly limited to 16 possible state variants.

In English, when do you want the TE to be deleted? Think about it.

Your blockstatecontainer only allows for 2 variants. If we look at your enum class, there are 9 total variants. This can't fit in 4 bits of metadata along with 2 bits used for log axis. Which we see: https://github.com/TheSlayerMC/Journey-1.12/blob/master/main/java/net/slayer/api/block/BlockModLog.java#L55 https://github.com/TheSlayerMC/Journey-1.12/blob/master/main/java/net/slayer/api/block/BlockModLog.java#L96 BTW, your API has references to your non-API. This means your API cannot be packaged separately. https://github.com/TheSlayerMC/Journey-1.12/blob/master/main/java/net/slayer/api/block/BlockModLog.java#L7-L15 Also, IHasModel is stupid. All items need models and none of the information necessary to register a model is protected or private.

Yes that. But that implementation won't do what you need.

You don't need BlockBase. Using inheritance as a means for code-reuse is an antipattern.

Register a different model for the item.

Considering that OBJs don't have GUIs you're going to need to clarify what you are trying to do.

It is extremely and highly unlikely that that will even be possible.