Skip to content

Update to 26.3 - #168

Open
granny wants to merge 10 commits into
v3from
chore/26.3
Open

granny wants to merge 10 commits into
v3from
chore/26.3

Conversation

@granny

@granny granny commented Sep 22, 2026 •

Copy link
Copy Markdown
Owner

Currently compiles. Haven't fully tested to see if it works properly. Some features may or may not throw errors, run at your own risk.

  • add missing blocks
  • test for issues

Download the pl3xmap jar for this pull request: pl3xmap-168.zip

@granny granny added enhancement New feature or request build-pr-jar Enables a workflow to build Pl3xMap jars on the pull request. labels Sep 22, 2026
@benct

benct commented Sep 24, 2026 •

Copy link
Copy Markdown

Pl3xMap-26.3-SNAPSHOT.jar tested on a 26.3 vanilla fabric server, rendering failed with the following errors:
pl3xmap-errors.log

@Dalarwin

Copy link
Copy Markdown

Hello
latest.log
got some errors too

@DaMarine

DaMarine commented Sep 30, 2026 •

Copy link
Copy Markdown

Download the pl3xmap jar for this pull request: pl3xmap-168.zip

@granny From some testing, this PR jar is erroring without some more changes needed from BlueMap's upstream BlockStateDeserializer.java‎, notably:

    public BlockState read(NBTReader reader) throws IOException {
+        if (reader.peek() == TagType.STRING)
+            return new BlockState(Key.parse(reader.nextString()));
               ...
-                case "Name" -> id = reader.nextString();
-                case "Properties" -> {              
+                case "id", "Name", "" -> id = reader.nextString();
+                case "properties", "Properties" -> {

something minimal like this is enough to get Pl3xMap 26.3 working:

+  import de.bluecolored.bluenbt.TagType;
    ...
    public BlockState read(NBTReader reader) throws IOException {
+        if (reader.peek() == TagType.STRING) {
+            return new BlockState(Pl3xMap.api().getBlockRegistry().get(reader.nextString()));
               ...
-                case "Name" -> id = reader.nextString();
-                case "Properties" -> {              
+                case "id", "Name", "" -> id = reader.nextString();
+                case "properties", "Properties" -> {
        }

though you may ofc wish to absorb some other changes BlueMap also made for default blockstates in BlueMap-Minecraft/BlueMap@76ba188

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-pr-jar Enables a workflow to build Pl3xMap jars on the pull request. enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants