480936 - Implement /delay N somecommand

Status: RESOLVED FIXED

Description

[:Gijs (he/him)]

Spun off from bug 476933.

I don't really think it would be as useful to have a type of special marker in the autoperform list that would delay the rest of the autoperform. Instead, I think implementing an actual command that delays execution for the <rest> of its arguments by N seconds would be easier and more versatile (could also easily do an alias for timed bans, for instance).

Presumably this would work approximately like this:

cmd.delay.params = <delay> <rest>

function cmdDelay(e)
{
    setTimeout(dispatch, e.delay * 1000, e.rest, e, e.isInteractive, undefined);
}

(setting all the params because otherwise the timeout difference will be filled in there, potentially breaking things - could work around that with an anonymous function as well, if desired)

Comment 1

[Glen Mailer]

For those wanting this functionality in current builds, the following alias should suffice.

/alias delay eval var obj = this\; setTimeout(function() { obj.dispatch("$(2+)") },$(1) * 1000)

Comment 2

[James Ross]

I think /delayed would be a better name; /delay sounds like a blocking sleep, where as this would execute the command asynchronously.

Comment 3

[:Gijs (he/him)]

Created attachment 410595 [details] [diff] [review]
Patch

File it, fix it, as they say...

Comment 4

[:Gijs (he/him)]

http://hg.mozilla.org/chatzilla/rev/d9cb5dc50dd0