Command Hooks

[SamStudio8]

No description provided.

[reality]

This will be implemented as part of #85

[reality]

So we can achieve something like this relatively simply with a couple of lines in the command module, like so:

dbot.hooks[commandName].each(function(hook) {
    hook();
}.bind(this));

However, the question is - how do we pass relevant information to the hook function? For example, one of our initial hooks is going to be for fixStats upon the calling of ~mergeusers or ~setaliasparent. How do we pass the oldUser value from these commands to the fixStats hook function?

[SamStudio8]

A parameter object maybe?

On Thu, Jan 10, 2013 at 7:00 AM, Luke Slater notifications@github.comwrote:

So we can achieve something like this relatively simply with a couple of
lines in the command module, like so:

dbot.hooks[commandName].each(function(hook) {
hook(); }.bind(this));

However, the question is - how do we pass relevant information to the hook
function? For example, one of our initial hooks is going to be for fixStats
upon the calling of ~mergeusers or ~setaliasparent. How do we pass the
oldUser value from these commands to the hook function?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/86#issuecomment-12082987.

[reality]

You mean pass the params which are passed to the command? That's a little messy as then the hooks will have to do their own parsing of the input.

[reality]

Going to do this by having commands open to hooks return data at the end for use.

[reality]

Completed with previous commit.