[SOLVED] Rendering pipes #2

[TSEngineer]

Why?

And when I place conduits somewhere else:

 

Edited October 10, 2018 by TSEngineer

[TSEngineer]

Source, I think is needed: https://github.com/TSEngineer/ExtraExpansion

[Cadiboo]

Seems to me as though you've got your horizontal EnumFacing values wrong. What exactly are you trying to do i.e. what are you trying to transfer? In your code I see vein and a custom energy capability and a whole bunch of classes with abbreviations in their name who's purpose is not immediately obvious.

Also don't use IHasModel (or any variant - ItemM)

Don't us static initialisers, use the @ObjectHolder annotation

[TSEngineer]

They're just rendering wrong. See:
src/main/resources/assets/extraexpansion/blockstates/conduit.json
src/main/resources/assets/extraexpansion/models/conduit_(u,d,n,s,w,e).json
src/main/java/ru/tse/extraexpansion/tech/conduit/BConduit.java

[TSEngineer]

If one conduit changed, it's connections will be applied in every conduit in the world, but NOT always, something strange is happening

Edited October 9, 2018 by TSEngineer
Additional info

[Animefan8888]

2 hours ago, TSEngineer said:

If one conduit changed, it's connections will be applied in every conduit in the world, but NOT always, something strange is happening

Did you push all of your code to github or just your jsons? Cause we also need to see the Block class too.

[V0idWa1k3r]

You can't do this. Blocks are singletons meaning that these values will apply to all blocks in the world when you change it. Resolve the needed connections in your getActualState method. Or migrate the variables responsible to the connections to a TileEntity.

[TSEngineer]

Now tile entity is not saving u, d, n, s, w, e. https://github.com/TSEngineer/ExtraExpansion

[Animefan8888]

20 minutes ago, TSEngineer said:

Now tile entity is not saving u, d, n, s, w, e. https://github.com/TSEngineer/ExtraExpansion

No, it's totally saving them. You just aren't updating the client. You need to override getUpdatePacket, handleUpdatePacket, and two other methods I cant think of right now.

[TSEngineer]

Will markDirty() also help?

[Animefan8888]

1 minute ago, TSEngineer said:

Will markDirty() also help?

That just tells the server that something has changed and it needs to be saved, it may query sending the packet based on if it has been marked dirty or not.

[TSEngineer]

So I need to make an update packet and send it to client, then handle it on client, right?

[Animefan8888]

2 minutes ago, TSEngineer said:

So I need to make an update packet and send it to client, then handle it on client, right?

You can, but you dont have to, vanilla has one. And the methods I told you to override are used by them.

[TSEngineer]

Ok, where do I override handleUpdatePacket then?

[Animefan8888]

1 minute ago, TSEngineer said:

Ok, where do I override handleUpdatePacket then?

Should be in the TileEntity however the name may be slightly different but there are two other methods but I cant remember the names so you should look in the TileEntity class.

[TSEngineer]

Maybe handleUpdateTag and getUpdateTag?

Edited October 9, 2018 by TSEngineer

[Animefan8888]

4 minutes ago, TSEngineer said:

Maybe handleUpdateTag and getUpdateTag?

Those sound about right.

[TSEngineer]

@Override
public void handleUpdateTag(NBTTagCompound tag) {
    u = tag.getBoolean("u");
    d = tag.getBoolean("d");
    n = tag.getBoolean("n");
    s = tag.getBoolean("s");
    w = tag.getBoolean("w");
    e = tag.getBoolean("e");
}

@Override
public NBTTagCompound getUpdateTag() {
    NBTTagCompound nbt = new NBTTagCompound();
    nbt.setBoolean("u", u);
    nbt.setBoolean("d", d);
    nbt.setBoolean("n", n);
    nbt.setBoolean("s", s);
    nbt.setBoolean("w", w);
    nbt.setBoolean("e", e);
    return nbt;
}

 

[TSEngineer]

But it's not working

[Cadiboo]

What’s the error?

you should also probably read https://mcforge.readthedocs.io/en/latest/tileentities/tileentity/#synchronizing-the-data-to-the-client

[TSEngineer]

Where do I put notifyBlockUpdate?

[TSEngineer]

Oh, thanks! It's solved.