ShadMOrdre wrote:When granting a user multiple privs in one shot, ie, /grant UserName fast, fly, home, this mod fails. It attempts to grant fly and home privs for "fast" length of time.
Since the time param is optional, this will always fail. So my recommendation is that you use some identifier to specify indefinite, instead of relying on the absense of the time param. And yes, since you override builtin for this, you might want to make sure that privs checks are done, and enforced, on the user granting said privs. You might want to consider figuring out a way to add to builtin instead of override, or as suggested above, send a PR to have this added to builtin.
Sounds like a definite issue, have you tried entering the priv string without spaces? I'm pretty sure I tested with multiples but I never used spaces.
I will definitely investigate, if this works with the default commands it should work with this mod too.
ShadMOrdre wrote:
EDIT: On second thought, maybe just a different chatcommand, say, /grant_temp, so that you don't have to override builtin. This should be considered if PR to add this to builtin is denied.
I don't personally like having multiple commands for this, but I guess if I can't get it to work exactly like default with or without the time I'll have to consider it.
Above all thank you for submitting this issue.