Skript

Skript

743k Downloads

Skript ... overcomplication

Vulcano771 opened this issue ยท 2 comments

commented

Skript/Server Version

https://github.com/SkriptLang/Skript/blob/master/src/main/java/ch/njol/skript/command/Commands.java#L107

Why is that done with reflection when there exists: `Bukkit.getCommandMap()`?

Bug Description

NotAbug

Expected Behavior

NotAbug

Steps to Reproduce

NotAbug

Errors or Screenshots

NotAbug

Other

NotAbug

Agreement

  • I have read the guidelines above and affirm I am following them with this report.
commented

Reflection in some capacity is still necessary for Spigot support

commented

Because it's legacy code from when it was necessary. Please make an properly formatted suggestion or PR for this; this isn't a useful issue in its current form.

edit: like blue says below, spigot still doesn't provide getCommandMap so we're stuck with it as long as we support Spigot.