agenda icon indicating copy to clipboard operation
agenda copied to clipboard

Fixed this={} in processDbResult

Open Solpatium opened this issue 6 years ago • 2 comments
trafficstars

this in processDbResult in save-job.js file is always {}, but it should be bound to agenda instance. Because of this a job is never run immediately, which is not a big issue, but it raises an error, for example, after Babel transpilation, because this is undefined.

Fixing it breaks the tests, because

processJobs.call(this, job);

might actually run and it doesn't take _jobsToLock into consideration. This is why a check for this was added.

Solpatium avatar Feb 05 '19 13:02 Solpatium

Could this be reviewed , need this fix

Beretta1979 avatar Jul 04 '19 10:07 Beretta1979

rebased onto latest master

mkaput avatar Sep 24 '19 07:09 mkaput