mfnalex / ChestSort

Best sorting plugin for Minecraft / Spigot 1.14+
https://www.chestsort.de
GNU General Public License v3.0
91 stars 34 forks source link

trouble with CMI /ec and/or opening shulker-boxes directly in inventory #67

Closed Bibithom closed 4 years ago

Bibithom commented 4 years ago

Used ChestSort version 8.7 (latest)

Used Spigot version papermc#318

Other plugin used CMI (latest) 8.6.26.5 ( and many others but this happened with a feature of CMI) https://www.spigotmc.org/resources/cmi-270-commands-insane-kits-portals-essentials-economy-mysql-sqlite-much-more.3742/

Describe the bug When shulker boxes open and others shulkerboxes in inventory, there is a dupplication bug. GUI of shulker is open, try to sort inventory with, something newly added inside, will result a dupe of what was inserted in shulker and closing GUI. Maybe it's the order of sorting event and closing gui cause shulker was moved into the inventory ?

To Reproduce have 3 or more shulker in inventory, open shulker with shift + right-clicking, add 3 object and sort inventory with some hotkey. Shulker need to be moved into inventory to close GUI 'cause of sorting.

Expected behavior keeping shulker-boxes with proper inventory.

Additional context in addition with a bug in CMI, need contact author.

mfnalex commented 4 years ago

Unfortunately, CMI is a premium plugin and I don't wanna buy it. I will have to ask the author if he can provide me the .jar file for free, however if he does not agree, there is nothing I can do.

mfnalex commented 4 years ago

I just saw that I already messaged the author of CMI on May 16th and he didn't reply yet, seems like he isn't interested in getting this fixed. Maybe you should also report the bug at CMI and let me know what answer you got.

Bibithom commented 4 years ago

Yes i try to do that, 'cause shulker-boxes disappearing is because of them it happens when chestsort was off. So thank's for you help, maybe backpack fix will help us with dupplication it seems to be same kind of thing.

Thanks for your help !

mfnalex commented 4 years ago

You‘re welcome, please let me know if you get any information from the CMI author

mfnalex commented 4 years ago

You could also ask the author of CMI to have CMI softdepend on ChestSort, which is now available in a public maven repo. CMI could then listen on the ChestSortEvent event and cancel that whenever a CMI inventory is about to be sorted.

More information: https://github.com/JEFF-Media-GbR/Spigot-ChestSort/blob/master/HOW_TO_USE_API.md

BTW: Did you encounter any of the mentioned problems again?

mfnalex commented 4 years ago

This seems to be a bug in PaperMC in general: https://github.com/PaperMC/Paper/issues/3470