Test-Account666 / PlugManX

Plugin manager for Bukkit servers.
https://www.spigotmc.org/resources/plugman-improved.88135/
MIT License
113 stars 44 forks source link

Add event system in api #49

Open 404Setup opened 1 month ago

404Setup commented 1 month ago

Haven't done some testing on events yet, I'll come back to that later.

I'm sorry some parts are very crude, please give me feedback so I can improve them.

Available for both Bukkit and Bungeecord.

Test-Account666 commented 1 month ago

Hmmm, is there a reason why the switch from maven to gradle is necessary?

404Setup commented 1 month ago

Hmmm, is there a reason why the switch from maven to gradle is necessary?

OK, it's rollback

404Setup commented 1 month ago

The real reason is that maven suddenly failed to build indexes correctly on idea, which confused me a bit, so I changed it to gradle.

Maybe it's a bug in idea.

404Setup commented 1 month ago

bukkit events not compiling and not working have been fixed, and there is now a spigot example.

Example Plugin Source

Example Plugin Jar

PlugManX-Event Jar

image

404Setup commented 1 month ago

bukkit events not compiling and not working have been fixed, and there is now a spigot example.

This example has been updated to the latest commit.

image

Example plugin source | Example plugin jar file | PlugManX Event jar file

Test-Account666 commented 1 month ago

So... I finally came around to looking at the code.

I'd like to have proper bulk-operation support with the events.

Not like the PreDisablePluginEvent having a bool. That doesn't feel right

Test-Account666 commented 1 month ago

I also think the user should also be informed about what happened, if an event has been cancelled

404Setup commented 1 month ago

Let me think about what to do