Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paper Plugin Implementation #51

Merged
merged 5 commits into from Mar 7, 2023
Merged

Paper Plugin Implementation #51

merged 5 commits into from Mar 7, 2023

Conversation

WolfyScript
Copy link
Owner

This PR adds the paper-plugin.yml to specify a separate entry point for the plugin, and to fix load order issues, where CustomCrafting bootstrap happened before WolfyUtils, even with the depend options.

Some changes were necessary (including breaking changes!):

  • WolfyCoreBukkit must no longer be used and you can no longer get an instance for it! Use WolfyUtilCore instead!
  • Casting WolfyUtilCore to WolfyCoreBukkit may cause crashes on Paper servers!
  • Commands are no longer defined in the paper-plugin.yml nor in the plugin.yml. They are directly registered into the internal command map using reflections.

Problems:
* The Bukkit registration of commands doesn't seem to work with the new `paper-plugin.yml`
* The new `paper-plugin.yml` is required to fix loading issues.
* Because this is not supported by paper!
* That is basically everything, but might be more separated in the future.
* Register Commands dynamically, so they work with the `paper-plugin.yml` in place.
* Removed WolfyCoreBukkit.getInstance, as that would interfere with WolfyCorePaper. (Use WolfyUtilCore.getInstance instead!)
* Replaced WolfyCoreBukkit usages with WolfyUtilCore
@WolfyScript WolfyScript added the enhancement New feature or request label Mar 7, 2023
@WolfyScript WolfyScript merged commit 270056e into master Mar 7, 2023
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.

None yet

1 participant