IEEP update client [Solved]

[UberAffe]

Here is a link to my code on github.

 

Messagedrafting gets called by a keybinding and alters the players IEEP on the server. MessageIEEPchanged is getting called in messagedrafting to update the client.

 

This setup is not updating the client. If I use the return of Messagedrafting the client freezes indefinitely.

Any advice is appreciated.

 

Solved:

-----------------------

The solution is more of a trouble shooting list so if you have a similar problem take a look at the steps we took here.

[diesieben07]

Is MessageIEEPChanged getting received on the client?

[UberAffe]

I didn't specifically check but I registered it as side.client and changed ctx.serverhandler to minecraft.getminecraft

[diesieben07]

Well, please do check.

[UberAffe]

Finally home again, and yes it is getting received. I guess that means there is something wrong with my save and load. Can you look over how I am passing the nbt information to make sure I didn't mess that up.

 

Message code

DraftingManager

ExtendedProperties save and load are at the bottom.

[diesieben07]

The problem is in your MessageDraftSelection handler.

 

This code runs one tick later from the main thread. So at the point where you access message.response here it is not set yet.

The solution is to not send the response using FMLs response mechanism, but manually.

[UberAffe]

Can I send the message from inside that runnable with:

CommonProxy.network.sendTo(new MessageIEEPChanged(DraftingManager.PerformDrafting(player)), (EntityPlayerMP) player);

 

or should I send the message from inside PerformDrafting?

[diesieben07]

That would be fine.

[UberAffe]

So I have been following this tutorial on IEEP.

 

But my get seems to be returning something other than what I registered.

I say this because all values that were set in the instructor are now set to 0.

 

IEEP class

Event Hooks

Class that uses the get

[diesieben07]

Use a debugger to find out where they get set to 0. This is basically impossible to debug by just staring at the code.

[UberAffe]

I was hoping it was just some glaring thing I had missed. Oh well.

[UberAffe]

Ok, so I stepped through things and I am getting this error

java.util.concurrent.ExecutionException: java.lang.IllegalArgumentException: Empty string not allowed

 

When I try to step over this

-> nbtExtendedProperties.setString("drafting_selection", draftingSelection);
    for(String luxin : luxinLevel.keySet())

 

If I look at the contents of LuxinLevel before stepping over I see this

{Blue=0.0, Green=0.0}

Which is what I expect to be there.

 

This is the declaration/ instantiation

private HashMap<String, Float> luxinLevel = new HashMap<String, Float>();

 

any ideas as to why this is reporting an empty string?

 

 

 

Edit:

------------------------------------

I'm dumb ... I was so fixated on the for loop that i didn't even notice that it was a string on the line before. I even included it in the code here ... had to instantiate that and once I did everything started working beautifully.

 

Thanks for the help!