DerEchtePilz

Maybe I am just misunderstanding something but I don't really know why there is a `PlayerPreTeleportEvent` added. I have a plugin which makes use of the `PlayerTeleportEvent` and in order...

> The issue that this resolves is mentioned at the top of this PR: https://github.com/PaperMC/Paper/issues/10168 I mean, nevertheless, the name `PlayerPreTeleportEvent` is misleading then since the current `PlayerTeleportEvent` is called...

> I think it's perfectly fine personally, PreTeleportEvent implies that is is called before the teleport event. It's not my PR but if you don't like the name then maybe...

I am literally fine with any name. If it ends up being named like it currently is - so be it. I thought about the names I suggested the minute...

I really like this concept but I am unsure if the class should be named `PaperServerInfo`. Why not name it `ServerInfo`?

> Idk, PaperServerInfo sounds better for me, but if someone else agrees then i can change that to ServerInfo Well, I think for Paper it's fine either way but I...

> > I'd say this better to be abstracted as an interface with getPickupRadius() method (also maybe getItemStack()?), and make related Entity interfaces extend it (I guess it's just Item...

I've been digging around in PlugManX's source and it should be totally their fault. They assume that every command is added by a plugin as far as my quick analysis...

@TheBlackEntity Whether it's your command wrapper class or not, the plugin causing this issue definitely is your plugin since reloading using the built-in /reload command works without issues (well, if...

@TheBlackEntity For what it's worth, I did experiment a little yesterday and could only confirm that it is your plugin that causes the issue. I've also cloned your repo and...