_default armor texture resource

[jeffryfisher]

Forge put a tantalizing getArmorResource() method into RenderBiped. By honoring the register armor prefix, I had the default producing the exact string I needed for my armor model resource... except that the domain (modid:) was missing from the front. I couldn't for the life of me figure out how to inject modid (or grab the s1 string and concatenate).

 

I gave up and overrode Item.getArmorTexture to produce the entire domain:path string myself. It works, so I am happy with the result, but... It would be so much more elegant if there were a simple way to inject my modid somewhere that Forge's method would be able to finish the job that it starts so nicely.

 

Maybe modid could be passed along to addNewArmourRendererPrefix(), which could append it to a parallel armorDomain[] array pre-seeded with "minecraft" entries for the vanilla armor levels. Then domains would be available when s1 (_default) is formatted.

 

Modders adhering to the <modid>:textures/models/armor/<prefix>_layer_<layer>.png format would be done without overriding getArmorTexture(). Not only that, but modders would be led into creating dynamic armor indices rather than all trying to claim 5 (a collision waiting to happen if/when mods are shared and combined).

 

[LexManos]

The intention is that you override the function.

So.. No, it's working as intended.

The other system is introduced by FML as a compatibility thing for ModLoader.

As mod loder is now dead, we may end up removing it in favor of telling people to override this function.