GUIs with multiple registered commands don't reload with /dm reload | HelpChat | Page 1

polar solar May 13, 2023, 2:42 PM

#

title :-(

upbeat mountain May 13, 2023, 2:43 PM

#

that's normal

#

you need to restart

#

it's also explained on the wiki

polar solar May 13, 2023, 2:47 PM

#

it updates for the first command in the open-commands: list

#

but not for any of the others

#

GUIs with multiple registered commands don't reload with /dm reload

#

renamed bc we figured that out just now

upbeat mountain May 13, 2023, 2:48 PM

#

again, /dm reload won't do anything

#

you have to restart the server

full pewter May 13, 2023, 2:49 PM

#

- jobslist
- joblist
- jobinfo
- jobsinfo```

doing `/dm reload` applies the changes for jobslist, but not for the other 3

upbeat mountain May 13, 2023, 2:49 PM

#

#

https://wiki.helpch.at/clips-plugins/deluxemenus/options-and-configurations/gui

GUI

All GUI menu related options & configurations

full pewter May 13, 2023, 2:50 PM

#

this page is about adding a registered command, which isn't what our issue is

upbeat mountain May 13, 2023, 2:50 PM

#

#

that's what you said though

full pewter May 13, 2023, 2:51 PM

#

full pewter ```open_command: - jobslist - joblist - jobinfo - jobsinfo``` doing `/dm reload...

all the registered commands work, and can be opened, but only the first one in the list gets changes applied with /dm reload

upbeat mountain May 13, 2023, 2:52 PM

#

what changes aren't applied?

full pewter May 13, 2023, 2:52 PM

#

for example, this is the change i just made the the config file, to cross out the £25,000 and add £50,000 on the end - this is what happens when i do /jobslist (the first command in the open_command section)

#

but it doesn't update in any of the other commands there (this is doing /joblist and /jobinfo)

upbeat mountain May 13, 2023, 2:53 PM

#

ah

#

what's your DeluxeMenus version?

full pewter May 13, 2023, 2:53 PM

#

sorry for the confusion

upbeat mountain May 13, 2023, 2:58 PM

#

I don't know what the issue could be

#

make sure you're on the latest version

full pewter May 13, 2023, 3:02 PM

#

just tested it on the latest dev build (160) and it isn't working there

upbeat mountain May 13, 2023, 3:02 PM

#

yeah, got the same issue

#

I guess the devs would need to look at it

rapid monolith May 13, 2023, 4:12 PM

#

Can yall break down the issue so I can look into it?

upbeat mountain May 13, 2023, 4:29 PM

#

have a menu with 2 or more open_command and register_command: true
restart the server so commands are loaded
change something in the menu (title for example)
/dm reload
use the first open_command, menu updated correctly
use the another open_command, menu didn't update

rapid monolith May 13, 2023, 4:37 PM

#

Interesting and does this happen only in dev build 160?

upbeat mountain May 13, 2023, 4:38 PM

#

I tried with the latest release (1.13.7) and they tried with the latest dev build

rapid monolith May 13, 2023, 4:41 PM

#

and MC versions

upbeat mountain May 13, 2023, 4:42 PM

#

I was on 1.19.4

rapid monolith May 13, 2023, 4:50 PM

#

So only with /dm reload but with restart the server it updates it

upbeat mountain May 13, 2023, 4:50 PM

#

yeah

rapid monolith May 13, 2023, 4:51 PM

#

super interesting bug lmao

#

the plugin has a mind of its own

gray jewel May 13, 2023, 4:57 PM

#

We've known for this issue for a while. It happens with any deluxemenus version on spigot 1.13 and higher. It is related to the way commands are registered at runtime. Haven't found a solution yet

rapid monolith May 13, 2023, 4:58 PM

#

oh

upbeat mountain May 13, 2023, 6:45 PM

#

gray jewel We've known for this issue for a while. It happens with any deluxemenus version ...

could you maybe register all commands as individual commands instead of aliases to the first one? if that's even the issue

gray jewel May 14, 2023, 12:32 PM

#

upbeat mountain could you maybe register all commands as individual commands instead of aliases ...

maybe? Idk either way it will require some refactoring

#

as right now each Menu is also a Command

#GUIs with multiple registered commands don't reload with /dm reload