[Solved][1.7.2]Fluid Container Registry Null Pointer Exception

[Taji34]

So I have created a liquid and I'm creating a bucket for it, however, the fluid container registry line is throwing a null pointer exception.

 

Troncraft.java: https://gist.github.com/anonymous/c6006d69a773edea63e5

Crash Report: https://gist.github.com/anonymous/fc4f35a84df9bab4d088

 

This line in particular is the one having the issue:

FluidContainerRegistry.registerFluidContainer(FluidRegistry.getFluidStack("liquidEnergy", FluidContainerRegistry.BUCKET_VOLUME), new ItemStack(liquidEnergyBucket), new ItemStack(Items.bucket));

 

When I comment this line out, everything works fine, bucket functions normally. I figured this line was there to reqister how many mB it is and some other properties, so I need it correct? Every parameter appears to me to not be null, but obviously that's not the case, could someone help me figure this out? Again, it's not affecting the buckets functionality, I just figured this line was important since it was in the tutorial.

[larsgerrits]

Try putting a breakpoint at that line, so you can see the null value so that you can search deeper as why that's null, and how to solve it.

[sequituri]

You cannot call getFluidStack() on an unregistered fluid. You must call FluidRegistry#registerFluid(Fluid f) before that, or you will get a null.

[Taji34]

You cannot call getFluidStack() on an unregistered fluid. You must call FluidRegistry#registerFluid(Fluid f) before that, or you will get a null.

 

But I did register a fluid. This is the second line of my preint:

FluidRegistry.registerFluid(liquidEnergy);

Which I do before I call the getFluidStack. Does the name parameter need to be the localized or unlocalized name of my fluid?

 

EDIT: I did some experimenting and figured it out! I needed to do liquidEnergy.getName() for the string name parameter.

[sequituri]

No, the real issue is you need to register (and retrieve) your fluid with an all lowercase name, as that is how the key is stored.

[Taji34]

I mean, is doing it the way I have right now going to cause problems in the future?

[sequituri]

I mean, is doing it the way I have right now going to cause problems in the future?

 

Not knowing when to use the proper case is always a source of problems in programming.

[Taji34]

Ok, the case issue makes sense, staying consistent and what not with standards. However, is there anything wrong with using fluid.getName() instead of "fluidname"?

[sequituri]

Ok, the case issue makes sense, staying consistent and what not with standards. However, is there anything wrong with using fluid.getName() instead of "fluidname"?

Nothing wrong with doing it that way. Whatever works.