TMPEffects icon indicating copy to clipboard operation
TMPEffects copied to clipboard

Can't set string parameter

Open S0meGUiaivar opened this issue 10 months ago • 2 comments

Hi! Your project is awesome! I love it. I don't seem to understand, but the command tag is not recognized if I set required to true.

Image

Image

The same issue with debug command tag. But if I set required to false and set a name, the tag is recognized but only prints the name if I type it like this:

Image

Image

In any case, it does not print anything except the name or aliases.

S0meGUiaivar avatar Apr 09 '25 04:04 S0meGUiaivar

Hi! I went ahead and quickly repro'd this, which led me to discovering a good few issues with autoparameters right now. I wont have time to fix this right now, so Ill list them here for now:

  • string AutoParameter validation is missing the negation on its condition when its a required parameter (which leads to your first issue)
  • Non-required parameters with "" alias are sort of inherently broken, since the parameter dictionary always will contain the key "", which's value will also be "" when not set, leading to it failing the parameter parsing for any type but string
  • With strings, its setting the parameter to the key and not the value

Dont know when exactly most of these cropped up, I tested auto parameters quite a bit and never ran into any of these back then.

Until I fixed this string auto parameters are basically broken, sorry :/

Luca3317 avatar Apr 09 '25 08:04 Luca3317

Don't worry. Take time as much as you'd like. I can wait.

S0meGUiaivar avatar Apr 09 '25 09:04 S0meGUiaivar