-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add Structure void #6156
base: minor-next
Are you sure you want to change the base?
Add Structure void #6156
Conversation
a545a54
to
3b37327
Compare
src/data/bedrock/block/convert/BlockStateToObjectDeserializer.php
Outdated
Show resolved
Hide resolved
src/data/bedrock/block/convert/BlockObjectToStateSerializer.php
Outdated
Show resolved
Hide resolved
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 PR looks mostly fine. Since we don't have any structure blocks, the difference between structure void and structure air is not important for now, though I would like to know the differences in the future.
There are some minor nits that need to be addressed.
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.
Looks OK to me
e92c6d7
to
71dc036
Compare
src/block/StructureVoid.php
Outdated
$w->enum($this->type); | ||
} | ||
|
||
public function getType() : StructureVoidType{ |
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.
getType()
is not generally used because it's too ambiguous
getStructureVoidType()
would be better
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.
Should setType()
also be renamed to setStructureVoidType()
?
Thank you very much @pandaa-be :) |
Introduction
This pull request is extracted from #6045. It adds the structure void block, along with it's air variant.
Relevant issues
Changes
This is a non-disruptive addition of blocks.
Follow-up
The card linked above should be updated to done.
Tests
/give <player> structure_void