Don't pass the callback to the handler

[dmihal]

The publish handler was being passed the callback function as an argument. This can cause issues with publish functions that set default values for the last parameter.

For example:

Meteor.publish('files', function(limit=10) {
...

const collector = new PublicationCollector();
collector.collect('files', function(collections) {
...

Currently, the limit value will be set to the callback function, instead of 10. Moving these lines until after the args.pop() method is called removes the callback from the argument list.

[johanbrook]

Good catch, thanks! 👏 Are there any tests we can add for this before merging?

[dmihal]

Test added!

[johanbrook]

Thanks!

[johanbrook]

Published as 1.0.5 🎉