-
-
Notifications
You must be signed in to change notification settings - Fork 368
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
ExprExplodedBlocks - new #2511
ExprExplodedBlocks - new #2511
Conversation
src/main/java/ch/njol/skript/expressions/ExprExplodedBlocks.java
Outdated
Show resolved
Hide resolved
- Split up examples onto separate lines
Huh, I've done this myself yesterday, I just didn't push it |
src/main/java/ch/njol/skript/expressions/ExprExplodedBlocks.java
Outdated
Show resolved
Hide resolved
src/main/java/ch/njol/skript/expressions/ExprExplodedBlocks.java
Outdated
Show resolved
Hide resolved
- Remove "destroyed" from pattern to cut down confusion with break events
There should be a way to prevent explosion of specific blocks SkQuery does it like loop exploded blocks:
if .. :
remove loop-block from exploded blocks should also support item types like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The expression itself is fine right now but it can be improved, you may consider the following:
- Implementing ItemType changers to the expression as Blueyescat suggested.
- Implementing the iterator method since the main usage of the expression is looping.
- Support the EntityExplodeEvent, even if we don't support this event right now, being future-proof isn't bad.
We can improve the expression later. This is much better than nothing. |
Description
Add an expression to retrieve the exploded blocks from an explosion event
Target Minecraft Versions: any
Requirements: none
Related Issues: #2509