Skip to content

Fix crash on version 1.20#77

Open
jbazant wants to merge 1 commit intoKovuTheHusky:mainfrom
jbazant:main
Open

Fix crash on version 1.20#77
jbazant wants to merge 1 commit intoKovuTheHusky:mainfrom
jbazant:main

Conversation

@jbazant
Copy link

@jbazant jbazant commented Nov 6, 2023

Fixes #75.

Proposed Changes

  • fixed access to unitialized BIOMES item
  • Small refactoring of relevant code

@AmonBune AmonBune mentioned this pull request Jan 25, 2024
@RedstoneFuture
Copy link

RedstoneFuture commented Feb 25, 2024

The PR works well on my server. Thanks!

Tested with PaperMC 1.20.4 (build 409) and Dynmap v. 3.7-beta-4-935

grafik

@Pezmc
Copy link

Pezmc commented Aug 22, 2024

For those less technically minded, I've built a release jar which includes this change (thanks go to @jbazant) and some version upgrades, and can be downloaded at https://github.com/Pezmc/dynmap-structures/releases/tag/2.6.2

@UplandJacob
Copy link

For those less technically minded, I've built a release jar which includes this change (thanks go to @jbazant) and some version upgrades, and can be downloaded at https://github.com/Pezmc/dynmap-structures/releases/tag/2.6.2

Amazing! Still works on 1.21.1.

@NODeeJay
Copy link

Yep, but sadly no on 1.21.4 anymore.

1 similar comment
@NODeeJay
Copy link

Yep, but sadly no on 1.21.4 anymore.

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.

Not loading in 1.20

5 participants