ChestSortEvent

[mortal heron]

könnten sie nicht das ChestSortEvent benutzen um das Menü-item auf "unsortable" zu setzen oder direkt den slot?
link zum plugin: https://github.com/mfnalex/ChestSort/blob/master/HOW_TO_USE_API.md#custom-chestsort-event

couldn't you use the ChestSortEvent to set the menu item to "unsortable" or the slot directly?

link to plugin: https://github.com/mfnalex/ChestSort/blob/master/HOW_TO_USE_API.md#custom-chestsort-event

[cold kettle]

I wonder what you are asking for, help with development? asking for DeluxeMenu to add a feature?

[mortal heron]

I want the plugin to add the event because I can then duplicate the item

couldn't you use the ChestSortEvent to set the menu item to "unsortable" or the slot directly?

[mortal heron]

Did you understand it like that?

[cold kettle]

Afaik ChestSort plugin should not allow sorting in custom inventaries

Imagine that the plugin alloe duplicating items in any gui created by another plugin

Thats probably ChestSort issue

Without that plugin you can duplicate items in menus?

If not, then probably ChestSort checks are wrong

Try asking them

Otherwise wait for a HelpChat dev to reply

[mortal heron]

No, the point is that I can increase the menu item in the inventory. If you sort it and rejoin it, you have it twice

[slender willow]

This won't really by our issue to fix

but what do you mean by I can increase the menu item in the inventory?

[mortal heron]

The sorting plugin changes the slot where the item should be, but the developer said that I should ask if you can do it so that it can't be moved

[slender willow]

the chest sorting plugin should be doing that...

It should not be letting you sort items inside a custom deluxemenus menu

[dapper sand]

Yes 🙂

[slender willow]

deluxemenus shouldn't have to build any implementation for it, nor would we

[dapper sand]

It should only work in vanilla GUIs

@rotund torrent hi

[slender willow]

in fact

the developer says to contact them

https://i.imgur.com/NHHnsPU.jpeg

so it's weird they told you to come to us

[slender willow]

😄

nice, makes it easier

[dapper sand]

https://github.com/mfnalex/ChestSort/blob/51fc5a05cbbadae0fcff6724328764124900d6ac/src/main/java/de/jeff_media/chestsort/listeners/ChestSortListener.java#L804 manual check for certain plugins' GUIs 😵

[slender willow]

https://github.com/mfnalex/ChestSort/tree/51fc5a05cbbadae0fcff6724328764124900d6ac/src/main/java/de/jeff_media/chestsort/hooks
all the hooks

[dapper sand]

I would test smth but I'm busy rn: see if "eventory.getHolder() instanceof Container" is enough to make it not sort inventories made by plugins

[rotund torrent]

hi, I don't remember what this is about tbh. is it about only setting one specific item or slot unsortable or the whole inventory?

[rotund torrent]

yep, it's a total mess, it was my first plugin ever

[rotund torrent]

that's why one can just use a regex for inventoryholders. I don't really know what this thread here is about in the first place

[cold kettle]

Hello mfnalex, i know i mean there must be a way to not sort that, imagine if a sort plugin allows that in all menus, not good, probably something needed to be done

[rotund torrent]

Yes, so the question is how could ChestSort detect that an item is supposed to be "unmovable" in the inventory? Is there any API for that?

I couldn't find any API information on the spigotmc page, nor in the wiki.

Alternatively, as already suggested in the first post of this thread, DeluxeMenus could simply listen to ChestSortEvent and check if any of the items is a DeluxeMenu item, and whether any of those items is supposed to be unmovable, and then prevent it from being sorted

[dapper sand]

I would suggest to ignore any inventories that don't have a vanilla InventoryHolder. DM uses its own holder https://github.com/HelpChat/DeluxeMenus/blob/40fa1be845fe95511a8a0f3571f360f031d1f838/src/main/java/com/extendedclip/deluxemenus/menu/MenuHolder.java#L23

[rotund torrent]

from what I understand, this issue here is about an item in the normal player inventory

not an open GUI

it's about preventing the item that is used to open the GUI from being moved

[slender willow]

oh

deluxemenus doesn't even provide that functionality in the first place (the menu item in player inventory)

so that's from another plugin

[slender willow]

@mortal heron

[mortal heron]

what exactly

[slender willow]

your issue does not lie within deluxemenus

it's another plugin

so you would need to talk to that developer, or see if mfnalex could add support for whatever plugin you're using to add the menu item to the player's inventory

[rotund torrent]

oh lol. Alright, thanks for the info!