swap jobType and fn before validating id in job.get

Automattic / kue

Kue is a priority job queue backed by redis, built for node.js.

http://automattic.github.io/kue

MIT License

9.45k stars 862 forks source link

Open ojhaujjwal opened 6 years ago

ojhaujjwal commented 6 years ago

I mistakenly passed undefined as id and my 2nd param was function and I was getting: TypeError: fn is not a function1

Instead, It should have shown invalid id param.

ojhaujjwal commented 5 years ago

any comments on this?