Open zedphi opened 3 days ago
In my opinion, this is a wont-fix, at least for now. We actively did not expose any API for this yet as it is known to be rather broken on spigot and hence paper given the fact the ServerPlayer instance is re-used and is initialized early on the server side for the respective events.
Messing with internals like this is unsupported for a reason and, until someone finds the time to correctly implement this (which would be a rather large effort from my first glance) I don't think it is worth considering this a bug on our issue tracker. At best a feature request in our discussion section for proper API support during the configuration stage.
So more input on my opinion would be great tho, maybe this is an easy fix I am simply not seeing.
I think that this is generally just hard fork territory. We could maybe forcefully re-init stuff and stray further away from vanilla handling here, but that just sounds like an investment into code debt, which is better off spent elsewhere, such as fixing the actual underlying cause of this once we hard fork
Stack trace
Server Log containing the errors: https://paste.gg/p/anonymous/fb749d8c53cb422ea335f69a2ef75fac
Plugin and Datapack List
Actions to reproduce (if known)
/to-config
as a playerstartConfig
orunconfig
commandPaper version
Other
Here's the full testplugin code: