pass args with dm open command

[FakEEE14]

added arguments to /dm open command so you can pass arguments for that player
like this:
/dm open punish_menu target -p:me arg1 arg2 arg3 ...
or
/dm open punish_menu target arg1 arg2 arg3 -p:me ...

[FakEEE14]

Hello @BlitzOffline , as you said i break down each change into its own PR, this one need to be accepted because default value is on top of this

[FakEEE14]

hello @BlitzOffline , what you think ?

[FakEEE14]

hi @BlitzOffline , any news ?

[BlitzOffline]

Hi. I will take a look at this by the end of the week.

[FakEEE14]

what you think about this @BlitzOffline

[BlitzOffline]

Hi. There's still a problem with this. the command can't tell the difference between a <viewer> name and the first [args] value. It just always assumes the parameter is the <viewer> parameter.

So this breaks /dm open <menu> myArg. This used to just open the menu for yourself. Now it tries to treat myArg as a player name, fails to find them online, and errors out.

Also, if an arg happens to start with -p:, it gets mistaken for the placeholder flag instead of being passed as an arg.

My suggestion would be that we add a clear marker before [args] starts, like -args:. Once the parser sees that, it stops guessing and just takes everything after it as args. This keeps the old commands working exactly as before:

/dm open <menu> [-args: <args>]
/dm open <menu> -p:<target> [-args: <args>]
/dm open <menu> <viewer> [-args: <args>]
/dm open <menu> <viewer> -p:<target> [-args: <args>]

[FakEEE14]

dats a bettr idea. im on it

[FakEEE14]

this should do it @BlitzOffline , what you think