Return the original callback from commands' __call__

[Mahi]

Before if you did this:

@ClientCommand('test')
@SayCommand('test')
def test(*args):
    print('test() was called')

It would raise an error because _BaseCommand.__call__() returns the command instance itself, as opposed to the original test function. This would cause SayCommand to attempt to register the ClientCommand instance as a callback, which would fail.

@Ayuto already hotfixed this here, but I believe that's just a workaround and doesn't solve the actual issue: the decorator is returned instead of the decorated function. Also, his workaround still doesn't allow one to call the test() function after decorating it as a command. The following code wouldn't work because test is now a ClientCommand instance:

@ClientCommand('test')
def test(*args):
    print('test() was called')

test()

Returning the command instance was necessary with the old AutoUnload class which operated on global variables based on their name, but this update greatly improved the mechanism and returning the object is no longer required.

[satoon101]

This looks correct, but I would like to test it when I get home tonight before pulling. Thank you!

[Ayuto]

The same actually applies to all other decorators (e.g. the Event decorator).

[Mahi]

Yeah good point, I didn't have too much time to go through all the decorators.

[satoon101]

Thanks again. I will update the other decorators as needed today.