EbonJaeger / PerWorldInventory

Spigot multi-world inventory management
GNU General Public License v3.0
21 stars 17 forks source link

Potential compatibility issue with minigame plugins #292

Open blablubbabc opened 6 years ago

blablubbabc commented 6 years ago

Consider this situation:

Observed problem:

Unfortunely I have no good idea for fixing this: Somehow PWI would need to disable after all other plugins / enabled before every other plugin, so that it sees the player's latest inventory when shutting down and saving. Unfortunely there is no option inside bukkit to load before all other plugins (besides maybe using load: STARTUP ...).

Currently this means that every minigame plugin out there (and every plugin doing similar things during shutdown) will have to add PWI in their soft-dependencies, to ensure that PWI gets loaded first / disabled later.

It might make sense to add some notice about this potential compatibility issue somewhere in the plugin's description for the time being. And maybe find a way to solve this on PWI's side instead someday, or get the spigot devs to add something to the plugin.yml for this situation.

ScuroK commented 6 years ago

i tested it and i can confirm it. huge mess :(

EbonJaeger commented 6 years ago

Minigame plugins and inventory separation plugins have always been a giant mess, sadly, for exactly this reason. And as far as I can tell, there is no real solution to the problem, except having your minigames in the same world and players just get sent to a lobby in that same world as well. :/

ScuroK commented 6 years ago

i think you are wrong. my players WERE in the same world when they joined the minigame but still lost the inventory on server restart. And back in the days we didnt have this problem (using the same plugins but other versions)

bc-hill commented 6 years ago

@Gnat008 This will not be improved in your rewrite?