Skip to content

Fixes multiblock rendering issues on world load#4302

Merged
jurrejelle merged 1 commit intoGregTechCEu:re/1.21-with-lovefrom
Saereth:re/1.21-with-love
Dec 11, 2025
Merged

Fixes multiblock rendering issues on world load#4302
jurrejelle merged 1 commit intoGregTechCEu:re/1.21-with-lovefrom
Saereth:re/1.21-with-love

Conversation

@Saereth
Copy link

@Saereth Saereth commented Dec 11, 2025

What

Addresses an issue where multiblocks sometimes fail to render correctly after initial chunk loading due to the block entity not being fully initialized when the chunk is first rendered.

Implementation Details

Forces a model data refresh on the client side when a block entity finishes loading to ensure correct rendering of multiblocks, especially in cases where the chunk was rendered before the block entity was fully available. Also, it avoids returning null model render types, which could cause the block to be invisible, by instead returning the render types of the default model.

Outcome

Fixes #4276

Fixes GregTechCEu#4276
Addresses an issue where multiblocks sometimes fail to render correctly after initial chunk loading due to the block entity not being fully initialized when the chunk is first rendered.

Forces a model data refresh on the client side when a block entity finishes loading to ensure correct rendering of multiblocks, especially in cases where the chunk was rendered before the block entity was fully available. Also, it avoids returning null model render types, which could cause the block to be invisible, by instead returning the render types of the default model.
@Saereth Saereth requested a review from a team as a code owner December 11, 2025 14:35
@github-actions github-actions bot added the 1.21 label Dec 11, 2025
@jurrejelle jurrejelle added ignore changelog PR should not be added to the changelog. type: bugfix General bug fixes Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Dec 11, 2025
Copy link
Contributor

@jurrejelle jurrejelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks and congratz on first contribution :D

@jurrejelle jurrejelle merged commit afa0c28 into GregTechCEu:re/1.21-with-love Dec 11, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.21 ignore changelog PR should not be added to the changelog. Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants