Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix collapsible brick slab blocks #733

Merged
merged 2 commits into from
Jan 25, 2025

Conversation

Egogorka
Copy link
Contributor

What is the new behavior?

This PR fixes collapsible bricks. Bricks are supposed to be sturdier than raw/smooth rock blocks. But even if bricks are not collapsible, brick slab blocks are for some reason. And because they can be used for decoration, it can bring a lot of pain.

Outcome

Brick slab blocks don't collapse

Potential Compatibility Issues

Constant global.TFC_ROCK_SLAB_BLOCK_TYPES and tags tfg:rock_slabs, tfg:rock_stairs, tfg:rock_walls got changed contents. But search showed no other usage for them besides in the changed file in the code section for collapsible stuff, so it must be okay.

and added TFC_BRICK_SLAB_BLOCK_TYPES, although naming may be bad, because not all bricks are included there

Signed-off-by: Egogorka <[email protected]>
@Egogorka Egogorka changed the title Fix collapsible bricks Fix collapsible brick slabs Jan 19, 2025
@Egogorka Egogorka changed the title Fix collapsible brick slabs Fix collapsible brick slab blocks Jan 19, 2025
@AirRice
Copy link
Contributor

AirRice commented Jan 19, 2025

It might be worth noting that this modifies recipes such as tfg:create/shaped/schematic_table, minecraft:grindstone, railways/track_create_andesite_narrow_from_steel etc. as they used the tfc:rock_slabs, tfg:rock_walls tags for definition.

I think the change is still appropriate, and the recipes changed make sense too to me (you wouldn't really be using stone brick slabs for these purposes), but it should be pointed out.

@Xikaro Xikaro added the 1. Type: Bug Inconsistencies or issues which will cause a problem for users. label Jan 21, 2025
@Xikaro Xikaro added this to the 0.7.19 milestone Jan 21, 2025
@Exzept1on Exzept1on merged commit 8e6e131 into TerraFirmaGreg-Team:main Jan 25, 2025
@Egogorka Egogorka deleted the fix-collapsible-bricks branch January 25, 2025 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Type: Bug Inconsistencies or issues which will cause a problem for users.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants