[1.12.2] All entities using one renderer

[nathodood]

Hi all! I have been having some trouble recently with my entities rendering. In the method I have where the entities render (com.nathodood.miscmod.util.registry.Entities.registerRenders) I have four separate statements declaring a separate render handler for each of my entities. However, all my entities use only the renderer (com.nathodood.miscmod.entity.render.RenderTenTNTPrimed) referred to in the first of the statements even though I specifically say in the other render statements to use the other renders, which is of course what I don't want to happen. As a side note, in the getEntityTexture method in each renderer class I have it returning null because it is never called in the first place (I had to get rid of this.bindEntityTexture in all of the doRender methods because that rendered a super zoomed-in version of what I wanted to be rendered). Here's my github repo for you to browse to help me find possible issues. I added comments only in places that seemed relevant to my issues. If you have any further questions regarding anything I will do my best to answer them as best as possible. Thanks!

Edited April 9, 2019 by nathodood

[nathodood]

3 hours ago, diesieben07 said:

Various notes:

• Code style, issue 1, 3.
• Problematic code, issue 10 (here), 13.
• No, you can't put this here. ModelRegistryEvent is implicitly a client-only class. It can only be referenced from client only classes, such as your client proxy.
• No. This completely defeats the point of a client proxy.
• Why are you doing this string-concatenation-dance? Construct a ResourceLocation and use the EntityEntryBuilder#id method that takes one. Also you are giving all your entities the same ID. An ID must be unique.

Okay I have no clue how I missed me giving them all the same ID. What a trivial thing to do lol. Thank you though! I fixed my issue. Now do I mark this solved or do you do that?

Edited April 10, 2019 by nathodood