[1.12.1] NBT Reading Problem

[MineDen]

1 minute ago, diesieben07 said:

You can do that. But you must also have a no-argument constructor and you must be aware that only the no-argument constructor will be used when reading your TileEntity from disk.

I'm now using zero-argument constructor, but energy again restores to 0.

UPD: I created zero-argument constructor and it deleted! What?

[MineDen]

2 minutes ago, diesieben07 said:

In the code you posted the energy is never changed.

 

What indeed, please clarify what "deleted".

Sorry, this is problem with GitHub. 

[MineDen]

16 minutes ago, diesieben07 said:

You can do that. But you must also have a no-argument constructor and you must be aware that only the no-argument constructor will be used when reading your TileEntity from disk.

I recrated zero-argument constructor and uploaded new code to GitHub.

[MineDen]

18 hours ago, diesieben07 said:

Ok, great. Now, what's your issue?

getEnergy() always returns  10000.

/blockdata <x> <y> <z> {} shows {Energy: 10000}.

[MineDen]

1 minute ago, diesieben07 said:

 

Once again, not sure what else you expected, given that:

 

I can record video of what changes in my world.

[MineDen]

Just now, diesieben07 said:

I don't want video. I want code that shows where you change the energy. You have shown no such code, as such I can only assume that the energy value never changes.

Energy don't change, but after world restart it always maximum. I think, that this is storage creating or update() method problem.

[MineDen]

9 minutes ago, diesieben07 said:

This is broken. If setEnergy is called with 0 as it's argument, it will set the energy to capacity.

I fix it. But now it sets to 0.

[MineDen]

After i typed /blockdata ... ... ... {Energy:5000}, energy changed to 5000 properly. When I reloaded world, energy saved and readed properly! GUI shows 5034 (this is previous value). But /blockdata shows 5000. I need to send energy level to client?

[Draco18s]

12 minutes ago, MineDen said:

After i typed /blockdata ... ... ... {Energy:5000}, energy changed to 5000 properly. When I reloaded world, energy saved and readed properly! GUI shows 5034 (this is previous value). But /blockdata shows 5000. I need to send energy level to client?

Any time the TE data changes and you want to update the client, call these methods:
https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/farming/entities/TileEntityTanner.java#L95-L98

And have these methods overridden

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/farming/entities/TileEntityTanner.java#L174-L189

[MineDen]

2 hours ago, Draco18s said:

Any time the TE data changes and you want to update the client, call these methods:
https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/farming/entities/TileEntityTanner.java#L95-L98

And have these methods overridden

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/farming/entities/TileEntityTanner.java#L174-L189

In 1.12.1, method this.getState() not exist. What method i need to use?

[Kokkie]

47 minutes ago, MineDen said:

In 1.12.1, method this.getState() not exist. What method i need to use?

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/farming/entities/TileEntityTanner.java#L105-L107

[MineDen]

RETEST: NBT don't saves automatically. Only /blockdata changes NBT, and after world reload, NBT entered in /blockdata restored.

[loordgek]

https://github.com/MineDen/m1/blob/master/src/main/java/com/mineden/mtest/tileentity/AutoTEEnergy.java#L56-L57

On 30-10-2017 at 3:38 PM, diesieben07 said:

Why are you adding energy every tick on the client?

nbt is only saved on the server

Edited October 31, 2017 by loordgek

[MineDen]

37 minutes ago, loordgek said:

https://github.com/MineDen/m1/blob/master/src/main/java/com/mineden/mtest/tileentity/AutoTEEnergy.java#L56-L57

nbt is only saved on the server

I'm adding energy on server side.

[loordgek]

nope

 

this is the javadoc for isremote

 

Quote

/**
 * True if the world is a 'slave' client; changes will not be saved or propagated from this world. For example,
 * server worlds have this set to false, client worlds have this set to true.
 */

 

[MineDen]

1 minute ago, diesieben07 said:

For the hundredth time: WHERE?

 

Here.

 

2 minutes ago, loordgek said:

nope

 

this is the javadoc for isremote

 

 

Lol. Sorry, I'm read only 0.2% of this comments.

[loordgek]

cant you see the quote ??

[loordgek]

when you check for world.isremote you check for client

you need !world.isremote

[MineDen]

5 minutes ago, loordgek said:

when you check for world.isremote you check for client

you need !world.isremote

I understand it.

[loordgek]

what is the problem now

[MineDen]

3 minutes ago, loordgek said:

what is the problem now

Thanks, Draco18s, diesieben07, Kokkie, loordgek! All works properly!