Skip to content

convert authorization exceptions#2938

Merged
me4502 merged 5 commits intoEngineHub:version/7.4.xfrom
SamuraiWorf:fix/convert-auth-exceptions
Apr 7, 2026
Merged

convert authorization exceptions#2938
me4502 merged 5 commits intoEngineHub:version/7.4.xfrom
SamuraiWorf:fix/convert-auth-exceptions

Conversation

@SamuraiWorf
Copy link
Copy Markdown
Contributor

Pretty niche issue, but caused some more serious exceptions.

-> 1. Only have worldedit.undo.self permission. 2. Run //undo anything

@SamuraiWorf SamuraiWorf requested a review from a team as a code owner April 6, 2026 17:45
@octylFractal
Copy link
Copy Markdown
Member

What happens currently? The existing WorldEditException handler should already be mapping this properly.

Maybe there's some null issues? I think we should just pass the proper Component in at call sites for now, and maybe deprecate having null for Component in WorldEditException in a separate PR.

@SamuraiWorf
Copy link
Copy Markdown
Contributor Author

Yeah exactly, its the null issues. Switched to fixing it at the call sites.

@me4502 me4502 merged commit 51ec8a8 into EngineHub:version/7.4.x Apr 7, 2026
2 checks passed
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