[Koha-bugs] [Bug 32393] background job worker explodes if JSON is incorrect

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32393

David Cook <dcook at prosentient.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #22 from David Cook <dcook at prosentient.com.au> ---
So I think we should just wrap try{} around the code for getting $job.

If we catch an error retrieving $job, then we log a warning. 

--

If we want to fail a job based on a fatal error in $job->process(), then we
need to put a try/catch in the "process_job" function. 

It does need to be separate after all. I think we could rename this bug to be
"Add exception handling to background jobs worker".

(Note that this doesn't stop individual jobs from doing their own exception
handling within the "process" function. This would just provide default
exception handling.)

-- 
You are receiving this mail because:
You are watching all bug changes.