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 logging caused by wind charges #701

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

quintindunn
Copy link

Fixes #696 wind charges still logging blocks as broken when the entity is a "breeze_wind_charge".
Fixes blocks that are actually broken by wind charges not being logged.

@quintindunn
Copy link
Author

Still other interactions not implemented, as seen here.

  • Non-iron doors are flipped.
  • Fence gates are flipped.
  • Buttons are pressed.
  • Levers are flipped.
  • Bells are rung and swung.
  • Lit candles (both standalone and on cake) are extinguished.

Which aren't implemented, while I'm not sure if all of these would be considered intended to be logged, I do think that some of these would make sense to be logged.

@HaTInAnd
Copy link

Good fox

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.

Wind Charge causes fake explosion logs in CoreProtect
2 participants