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

Can overwrite existing player-placed light blocks #7

Closed
sydist opened this issue Jun 30, 2023 · 4 comments
Closed

Can overwrite existing player-placed light blocks #7

sydist opened this issue Jun 30, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@sydist
Copy link

sydist commented Jun 30, 2023

Describe the bug

Title says it all really, if you're playing a Minecraft map or something that has player-placed light blocks by the map builders, this will overwrite those and can remove them.

Steps to reproduce

No response

Expected behavior

Light blocks placed by players should not be touched.

Additional context

No response

@sydist sydist added the bug Something isn't working label Jun 30, 2023
@Tschipcraft
Copy link
Owner

Thanks for the bug report!

@Tschipcraft Tschipcraft self-assigned this Aug 12, 2023
@sydist
Copy link
Author

sydist commented Aug 13, 2023

May I ask how this was fixed?

@Tschipcraft
Copy link
Owner

Sure. I just added a condition in the update functions in internal/place_light/ to abort the whole process if a light block exists without a marker entity of Dynamic Lights at the current position.
This may not work well if the placed light block has a low light level, since dynamic lights no longer work at that position, but otherwise I would have had to rewrite the no_space_resolve function or add additional data to the marker entities to reset the player-placed light block to its original light level after the dynamic light has moved, which would have added unnecessary overhead for normal use.

@sydist
Copy link
Author

sydist commented Aug 13, 2023

Oh I see, that's pretty cool!
Thanks for clarifying

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants