-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Rework PlotHider for 1.18+ support #96
Rework PlotHider for 1.18+ support #96
Conversation
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.
Which mc version range is this change proposed compatible with?
I don't know enough about the internals of PlotHider to submit a qualified review, but testing it on 1.19 works as expected.
src/main/java/com/plotsquared/plothider/storage/BlockStorage.java
Outdated
Show resolved
Hide resolved
src/main/java/com/plotsquared/plothider/storage/palette/APalettedContainer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/plotsquared/plothider/storage/palette/APalettedContainer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/plotsquared/plothider/storage/palette/APalettedContainer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/plotsquared/plothider/storage/palette/APalettedContainer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/plotsquared/plothider/storage/palette/PaddedPalettedContainer.java
Show resolved
Hide resolved
Due to java version and PS requirements, PH should only work on 1.16-1.19 (recent PS crash on a 1.15.2 server). |
I see, if we enforce Java 17 here too, means we can ignore anything pre 1.16.5. |
src/main/java/com/plotsquared/plothider/storage/palette/LegacyPalettedContainer.java
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.
Works great, thanks a bunch for your contribution.
(While this is a breaking change, let's not kid ourself, there are no consumers of this plugin but IntellectualSites.)
If nobody else is up for a review, I'll go ahead and add it to this weeks release. cc @IntellectualSites/plotsquared-team |
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 good, just a minor style-wise nitpick
src/main/java/com/plotsquared/plothider/storage/palette/APalettedContainer.java
Outdated
Show resolved
Hide resolved
Improve support for older versions, especially for 1.16+ and implied chunk format changes
0eb4fcc
to
f6d2ce0
Compare
Overview
Fixes #34
Fixes #42
Fixes #75
Description
Improve support for older versions, especially for 1.16+ and implied chunk format changes (https://minecraft.fandom.com/wiki/Talk:Chunk_format#1.16_Chunk_format_changes), by completely revamping block storage/modification system.
All the changes are based on the official Minecraft protocol documentation (https://wiki.vg/Protocol#Chunk_Data_And_Update_Light) and have been tested on 1.16.5/1.17.1/1.18.2/1.19 Paper servers.
The current method to fetch the data packet is not optimal and just for test purposes. I will address this new Wrapper (dmulloy2/ProtocolLib#1592) as soon as it is merged into ProtocolLib v5.
For now then, this PR is in a draft state to let you review the other points.
Overall, we are ready to support newer versions :D !
Submitter Checklist
@since TODO
.