Skip to content

Add support for Folia, dropped compability non PaperMC servers#31

Open
KosyakMakc wants to merge 2 commits intoMark-225:mainfrom
KosyakMakc:main
Open

Add support for Folia, dropped compability non PaperMC servers#31
KosyakMakc wants to merge 2 commits intoMark-225:mainfrom
KosyakMakc:main

Conversation

@KosyakMakc
Copy link

Add support for Folia, dropped compability non PaperMC servers

For save compability this PR can be merged for special branch, if you created

@Mark-225
Copy link
Owner

Mark-225 commented Sep 9, 2024

I'll think about if/how I want to support Foilia in this project. Mainly, I want this to be able to run on as many versions BlueMap and the other plugins run on as possible. And that does include Spigot.
Maybe keeping this as a fork is the best approach for now but I might merge it into a separate branch in the future.

In any case thanks for creating this fork, more compatibility is always appreciated! :D

@sidboy55555
Copy link

@KosyakMakc Your fork works with Folia, well done!
Only there is one issue, when a region with grief prevention is made, a java.lang.UnsupportedOperationException: null error is generated and the region is not shown on the bluemap. Only after a server restart the region is actually visible.

Grief prevention fork which is used on folia: https://github.com/ssquadteam/GriefPrevention-Folia

Would be nice to see folia support being added to the origin plugin soon :)

@KosyakMakc
Copy link
Author

@sidboy55555 can you provide full stack trace of error?

@CFPlusPlus
Copy link

@KosyakMakc
The fork seems to work fine with Folia 1.21.5, but if you create or modify a claim, an error is displayed in the console (see Pastebin). The change is only visible after a server restart.

Grief prevention fork: https://github.com/ssquadteam/GriefPrevention-Folia
Pastebin: https://pastebin.com/U5yk0xaz

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.

4 participants