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

David Cook <[email protected]> changed:

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

--- Comment #22 from David Cook <[email protected]> ---
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.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to