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 issues with modded tile entities and lights #3838

Merged
merged 2 commits into from
Apr 13, 2023

Conversation

Yeregorix
Copy link
Member

Forge adds specific methods to check whether a block has a tile entity or is emitting light. We need to use these methods instead of the vanilla ones to properly support modded blocks. Previously this was done correctly in API 7, but got lost in the update process.

This fixes issues with Create's gearbox and gearshit (#3835) and probably issues for many other mods as well.

Copy link
Member

@gabizou gabizou left a comment

Choose a reason for hiding this comment

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

LGTM

@Inscrutable
Copy link
Member

This fixes all the mechanical issues I had with Create vs SpongeForge. Tested on a server with forge-1.16.5-36.2.39. Nice work!
The Placement Assist for cogs is still wacky, so that's gotta be down to something else.

@Yeregorix
Copy link
Member Author

I just amended the last commit because a forgot two final keywords.

Copy link
Contributor

@ImMorpheus ImMorpheus left a comment

Choose a reason for hiding this comment

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

LGTM! I fixed the issue with api-10 in another commit

@Yeregorix Yeregorix merged commit 3ec7db1 into SpongePowered:api-8 Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants