Skip to content

Conversation

@Jsinco
Copy link

@Jsinco Jsinco commented Feb 3, 2026

WorldGuard doesn't officially support Folia yet but there are a few PRs in progress and forks available

@pop4959
Copy link
Owner

pop4959 commented Feb 5, 2026

Was this tested? I had a look at the code, but I have reasons to doubt that it works fully out of the box on Folia without changes.

@pop4959 pop4959 added the enhancement New feature or request label Feb 5, 2026
@Jsinco
Copy link
Author

Jsinco commented Feb 5, 2026

I tested both commands and the WorldGuard flag. They all appear to work fine :)

[11:40:46 INFO]: Jsinco issued server command: /rg flags -w "world" -p 8 test
[11:40:47 INFO]: [WorldGuard] Region data changes made in 'world' have been background saved
[11:40:49 INFO]: Jsinco issued server command: /rg flags -w "world" -p 7 test
[11:40:51 INFO]: Jsinco issued server command: /rg flags -w "world" -p 6 test
[11:40:53 INFO]: Jsinco issued server command: /rg flag -w "world" -h 6 test bolt-protections ALLOW
[11:41:06 INFO]: Jsinco issued server command: /boltworldguard:purgeregion
[11:41:08 INFO]: Jsinco issued server command: /boltworldguard:purgeregion test
[11:41:17 INFO]: [WorldGuard] Region data changes made in 'world' have been background saved
[11:41:29 INFO]: Jsinco issued server command: /boltworldguard:purgeregion test
[11:41:35 INFO]: Jsinco issued server command: /rg flag -w "world" -h 6 test bolt-protections DENY
[11:41:47 INFO]: [WorldGuard] Region data changes made in 'world' have been background saved
[11:42:00 INFO]: Jsinco issued server command: /boltworldguard:protectregion test

@pop4959
Copy link
Owner

pop4959 commented Feb 5, 2026

It might work fine in a test, doesn't necessarily mean it's safe.

I see world access being done here without any scheduling or checks. E.g.

final Block block = bukkitWorld.getBlockAt(x, y, z);

It's likely that this would work if you're lucky, but might randomly throw errors in other places. Etc for any other accesses.

@Jsinco
Copy link
Author

Jsinco commented Feb 7, 2026

I believe this should be okay now, but I may be wrong. I'll test with a bigger/global region and see if I can get any more errors

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants