Sync blockstate

[matthew123]

I am making a pipe mod. The blocks also use a block entity. I added boolean properties for north, south, east, west, up, down and wrote to them on the setPlaced method. In my blockstate json, I use a multipart model to apply the pieces of the pipe to each side. 

It does not seem to be working. How can I synchronize the new blockstate values so that my pipe gets rendered properly?

[matthew123]

1 hour ago, diesieben07 said:

Show your code.

I am using BlockState.setValue on the overriden setPlacedBy method. The values are getting set, i added some logging there. I am using BlockStateProperties.NORTH, SOUTH, ... .

I also made sure they are added in the createBlockStateDefinition method and registerDefaultState in the constructor.

this is the blockstate:

{
    "multipart": [
        {
            "apply": { "model": "cables:/block/cables/without_connections" }
        },
        {
            "when": { "north": true },
            "apply": { "model": "cables:/block/cables/connection", "uvlock": true }
        },
        {
            "when": { "west": true },
            "apply": { "model": "cables:/block/cables/connection", "y": 90, "uvlock": true }
        },
        {
            "when": { "south": true },
            "apply": { "model": "cables:/block/cables/connection", "y": 180, "uvlock": true }
        },
        {
            "when": { "east": true },
            "apply": { "model": "cables:/block/cables/connection", "y": 270, "uvlock": true }
        }
    ]
}

 

[matthew123]

1 minute ago, diesieben07 said:

Please show this code.

blockState.setValue(SOUTH, true)

this is all there is in the setPlacedBy overridden method.

[matthew123]

8 minutes ago, diesieben07 said:

That achieves nothing. BlockState is immutable. You should override getStateForPlacement instead and return the modified BlockState.

That sucks. I have some logic on the tile entity that will be used to get those values. Do I really have to move that logic out of the tile entity? I assume the tile entity is placed after that method.

[matthew123]

2 minutes ago, diesieben07 said:

I am not sure how that is at all relevant?

I.e i have to move that logic out of the tile entity because the tile entity has not been placed yet when getStateForPlacement is being called.

[matthew123]

29 minutes ago, diesieben07 said:

Why are you duplicating data? Either store the info in the BlockState or in the TileEntity. Don't do both.

the logic I am referring to gets the Directions where other pipes that can be connected are. To set the right values in the block state, I would have to move that logic out of the tile entity so i can use it from the block. Preferably, I should move it to the block class. Is that correct?