Should pass this i.s.o. callback to callback.apply about rubberduck

[daffl]

It should be null really because usually the callback context is not defined. Pre-bound methods (and arrow functions) should still have the correct context anyway.

[Download]

It should be this, as the wrapper function will receive the same this as the original function would have. If the original function would have received null, this in the wrapper will be null. (I'm assuming that the wrapper should not interfere with the original functionality, just add to it)

If the wrapper does not pass this, then bind on the wrapped function will not work as expected.

[Download]

Ah yes I missed that you already saved this as context. Thanks for the fix!

[daffl]

👍 Thanks for bringing it up. Definitely was a good fix.