[Issue]: Natural blocks having different metas leads to disconnected CTMs

[DarkBumus]

Description

There is a bug that involves vertical texture mode that doesn't display them right reliably:

Firstly, when I place down sugar cane or grow it naturally/with bone meal, it looks as it's supposed to, making use of the vertical connection.
However, there are some (naturally generated, usually, but also appeared in my little farm that had existed before I installed RPMCP) sugar canes that look weird, making use of only the "uppermost" texture. Even stranger, if you place additional sugar canes ONTO the weird-looking ones, they continue to look weird.

Edit: On rare occasions they also become bugged when I place them down. I have... no idea...

Edit2: If this actually affects the vertical connection mode, it might also affect horizontal connection, so it's probably best to check the implementation for both? 🤔

The pack in question:

Connected_Sugarcane.zip

[basdxz]

This is intended behavior, as the default "connection" require matching both the block id and meta values.

It also occurs with MCPatcher Forge:

Sugar canes use their meta values to determine their current growth state, which means there is no grantee that the top and bottom block will have the same meta value. Hence, the connection doesn't happen.

Using connect=tile can be used to work around this limitation, by having the texture instead match the texture used on the other block.

Is this a good enough solution, or do we want to have some list of blocks where meta is always "disregarded"?

[Houstonruss]

matchBlocks= minecraft:reeds
method=vertical
metadata=0 1 2 3 4 5
tiles= 0 1 2 3

Just shotgun it. and have tile 3 be a full block tall.

[Houstonruss]

Adding a feature to ignore metadata could work, but it'd really only be of use for natural blocks like leaves which use metadata for determining if leaves are natural or player placed.

[basdxz]

I think I'm going to leave it as-is for now and keep this issue open.

Having better metadata handling for "natural" blocks would probably save a fair bit of headache in the long run.

[DarkBumus]

matchBlocks= minecraft:reeds method=vertical metadata=0 1 2 3 4 5 tiles= 0 1 2 3

Just shotgun it. and have tile 3 be a full block tall.

Btw, this hasn't worked. But cheers. I'd really like for tile 3 to not be a full block tall :)))