Skip to content

Empty catwalk railings fixed#251

Open
LIUKRAST wants to merge 2 commits intotalrey:1.21-neofrom
LIUKRAST:1.21-neo
Open

Empty catwalk railings fixed#251
LIUKRAST wants to merge 2 commits intotalrey:1.21-neofrom
LIUKRAST:1.21-neo

Conversation

@LIUKRAST
Copy link

This pull request fixes a kind of annoying bug I've encountered in the mod:
When sneak+wrench a single side catwalk railing the block is updated to [north=false,south=false,west=false,east=false] instead of breaking the block. This causes the block to not be empty, and it becomes impossible for a player to place a block there.

Only solution was placing a block attached to the incriminated block. This will trigger neighbour update and the block gets removed.

This is now solved with a simple if check: when sneak+wrench, before any check, if the block only has one single side, the interaction gets handled by super (IWrenched). It makes sense that it doesnt execute rest of the logic because theres no need to check which side of the railing we are checking to remove it, as there's only one available.

Hope this commit can improve the mod quality ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant