Armor Model does not Initialize.

[InternalCow]

Error Log: https://pastebin.com/PjKwnPvg

ModItems: https://pastebin.com/ZfeJvZyz

 

The error log says on line 51 of ModItems there is an error. I have no idea what is wrong, however.  I think it has to do something with when the mod registers the armor models.  This applies to lines 51-54.  I used MCJty's tutorials to learn modding, but he has not provided an armor tutorial, so any help would be much appreciated :)

[Cadiboo]

Non broken links:

https://pastebin.com/PjKwnPvg 

https://pastebin.com/ZfeJvZyz

 

I don’t see any problems with your code other than Mod Ids need to be lowercase. Also you should be able to just add @ObjectHolder to your ModItems class & name your items the same names as the variables. Also, you haven’t shown where your method is being called from, and please use a GitHub repository for posting code. Pastebin & GitHubGist are fine for logs but they don’t really work for code.

Edited September 30, 2018 by Cadiboo

[Synoot]

Github

(I also work on the mod)

Edited September 30, 2018 by Synoot

[Cadiboo]

1) Code Style#2 - A common proxy doesn’t make sense. Proxies hold code that runs on both sides BUT DOES DIFFERENT THINGS. If the code does the same thing on both sides (common code) it shouldn’t be in a proxy. Put it literally anywhere else

2) Please reorganise your packages into the way vanilla does it

3) 

  On 9/30/2018 at 7:18 AM, Cadiboo said:

Mod Ids need to be lowercase

Expand  

You mod Id is “Boundless”.

4) 

  On 9/30/2018 at 7:18 AM, Cadiboo said:

just add @ObjectHolder to your ModItems class & name your items the same names as the variables.

Expand  

 

[Synoot]

1) Yeah, makes sense I don't know why he did it like that

2) ¯\_(ツ)_/¯

3) Our modid is "boundless"

4) For example

@GameRegistry.ObjectHolder("boundless:leather_sword")
public static LeatherSword leather_sword;

 

 

Edited September 30, 2018 by Synoot

[Cadiboo]

  On 9/30/2018 at 7:53 AM, Synoot said:

1) Yeah, makes sense I don't know why he did it like that

2) ¯\_(ツ)_/¯

3) Our modid is "boundless"

4) For example

@GameRegistry.ObjectHolder("boundless:leather_sword")
public static LeatherSword leather_sword;

 

 

Expand  

Uh...

[Synoot]

Ah, that's what you meant. Don't ask me how it worked before, I have no clue

[Cadiboo]

  On 9/30/2018 at 8:09 AM, Cadiboo said:

Uh...

Expand  

Try doing MOD_ID+":"+"name" next time

[InternalCow]

Note, that I said in my original post, that I used McJty's tutorials to learn modding, this is generally how he did it in those tutorials.

[Draco18s]

  On 9/30/2018 at 3:44 PM, InternalCow said:

Note, that I said in my original post, that I used McJty's tutorials to learn modding, this is generally how he did it in those tutorials.

Expand  

Just because that's how they did it does not mean it is correct. It simply doesn't crash.

[InternalCow]

  On 9/30/2018 at 3:46 PM, Draco18s said:

Just because that's how they did it does not mean it is correct. It simply doesn't crash.

Expand  

Yes, but the thing is, he doesn't have any armor tutorials etc.  I had to use other tutorials and bend it to the way I learned (McJty).  Also, it being not correct, the only issue I have is with the Armor, everything else works.  I even got ore generation to work.

Edited September 30, 2018 by InternalCow
Fixed things.

[Draco18s]

  On 9/30/2018 at 3:47 PM, InternalCow said:

everything else works.

Expand  

for now.

That said, the "code style" type issues are ones that are just bad practice, not that they cause the game to break. In some cases it makes things more complicated than they need to be (IHasModel) and in some cases simplifies something that shouldn't be (Proxies).

[InternalCow]

  On 9/30/2018 at 3:50 PM, Draco18s said:

for now.

That said, the "code style" type issues are ones that are just bad practice, not that they cause the game to break. In some cases it makes things more complicated than they need to be (IHasModel) and in some cases simplifies something that shouldn't be (Proxies).

Expand  

I understand that IHasModel would work, possibly easier.  I don't want to restart because I already have this all done.  So, instead of making me restart my mod, you, or someone else could help me find an answer. And if there is none, i'll restart it.

Edited September 30, 2018 by InternalCow
Fixed Typo

[Animefan8888]

  On 9/30/2018 at 3:54 PM, InternalCow said:

someone else could help me find an answer.

Expand  

I believe if you are using @ObjectHolder the fields need to be public static final and equal to null. And the string passed into the annotations needs to match the registry name of the object. Also it seems that it is erroring on your Lunarium Sword because the field is null.

 

 

[InternalCow]

  On 9/30/2018 at 4:53 PM, Animefan8888 said:

I believe if you are using @ObjectHolder the fields need to be public static final and equal to null. And the string passed into the annotations needs to match the registry name of the object. Also it seems that it is erroring on your Lunarium Sword because the field is null.

 

 

Expand  

Can you show me some example code, what do you mean by make it public static?  (I know what it means, but where would I put it) . EDIT: I did all that you said, and I end up getting the same errors at lines 51-54.  I think it has to do with when it registers the models.

Edited September 30, 2018 by InternalCow
Fixed things.

[Animefan8888]

  On 9/30/2018 at 4:58 PM, InternalCow said:

I think it has to do with when it registers the models.

Expand  

Of course it does the error is saying that your lunarium_sword field is null. Are you actually registering an instance of LunariumSword?

[InternalCow]

  On 9/30/2018 at 6:02 PM, Animefan8888 said:

Of course it does the error is saying that your lunarium_sword field is null. Are you actually registering an instance of LunariumSword?

Expand  

Hmm, okay let me check, I thought the error was with the armor, however.

[Animefan8888]

  On 9/30/2018 at 6:04 PM, InternalCow said:

Hmm, okay let me check, I thought the error was with the armor, however.

Expand  

Sorry I was off a line. It seems like your lunarium_helmet is null. Are you sure you registered an instance that has that registry name.

[InternalCow]

  On 9/30/2018 at 6:07 PM, Animefan8888 said:

Sorry I was off a line. It seems like your lunarium_helmet is null. Are you sure you registered an instance that has that registry name.

Expand  

I've tested, and if I remove the lunarium_helmet, the same happens to the chestplate etc.  So it has to be something with all of them.

[Animefan8888]

@InternalCow You never register an instance of LunariumHelmet

[InternalCow]

  On 9/30/2018 at 6:09 PM, Animefan8888 said:

@InternalCow You never register an instance of LunariumHelmet

Expand  

Yeah, just realized that, fixing it now!

[InternalCow]

Okay, your fix fixed it.  I must've of accidentally deleted the helmet in the process! Thanks!

[Animefan8888]

  On 9/30/2018 at 6:08 PM, InternalCow said:

I've tested, and if I remove the lunarium_helmet, the same happens to the chestplate etc.  So it has to be something with all of them.

Expand  

Removed.

Edited September 30, 2018 by Animefan8888
Not needed.

[InternalCow]

  On 9/30/2018 at 6:15 PM, Animefan8888 said:

Could you also post the debug.log, there may be some errors printed out in there.

Expand  

I fixed it. (It was the helmet, read my comment above yours)

Edited September 30, 2018 by InternalCow