Skip to content
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

Folia Support #1315

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Folia Support #1315

wants to merge 9 commits into from

Conversation

Anon8281
Copy link

@me4502
Copy link
Member

me4502 commented Jun 25, 2023

This PR renames the plugin and heavily alters formatting away from the style guide. Including a random dependency is also not something that should be done

I also question if this is enough to make it folia compatible in the first place. Most likely any folia compatibility for CraftBook would need to be done in CraftBook 5, as it’ll almost definitely entirely break support for spigot

@Anon8281
Copy link
Author

i changed the name of plugin in settings from 'craftbook' to 'CraftBook' because of java.lang.IllegalStateException: Module entity with name: CraftBook.main should be available

According to my tests, it's enough to make folia compatible.

@Clercq
Copy link

Clercq commented Jun 26, 2023

I also question if this is enough to make it folia compatible in the first place. Most likely any folia compatibility for CraftBook would need to be done in CraftBook 5, as it’ll almost definitely entirely break support for spigot

We've managed to achieve support of Java 8(and old versions of spigot) by using this library. It builds the library using the tricky way, by including(not shaded) some mocked java 8 paper/folia API.

Basically it means, that the plugin will support folia as well as paper, spigot, and old versions of them that using java 8.

Of course, it's maybe not enough to get the stable support of folia, however, it could be a nice start that will not broke any spigot/paper integrations and the same time will achieve folia support.

We can include the code from library just in your project, hovewer, I'm not sure that it's a good thing because it will cause some issues with old spigot/paper

@me4502
Copy link
Member

me4502 commented Jun 26, 2023

Renaming the plugin and modifying large amounts of the formatting are not acceptable to do in a PR, we have a style guide and renaming the plugin is an extremely breaking change.

As for the library, it's not something that I'm really comfortable including here. CraftBook has existed for a very long time, and external libraries have almost always ended up causing large issues. If PaperLib provides support for this that maintains Spigot compatibility we can add it to CraftBook 3, otherwise Folia support won't come outside of CraftBook 5.

Either way though, until WorldEdit supports Folia this is a non-starter, as WorldEdit is a hard dependency.

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.

3 participants