[Koha-bugs] [Bug 22417] Add a task queue

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

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

--- Comment #155 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Kyle M Hall from comment #148)
> This looks really great! There are some rough edges that will need polished,
> but that doesn't necessarily need to happen right now on this bug.
> 
> A few notes:
> 
> * Why are koha_worker.pl and new_koha_job.pl in root dir?
> 
> * Why does new_koha_job.pl use Koha::BackgroundJob::BatchUpdateBiblio but
> not Koha::BackgroundJob::BatchUpdateAuthority? Is this needed

As Marcel said, those 2 scripts are for testing only. They won't be pushed.

> * use Scalar::Util qw( weaken ); not used
> 
> I think at this point, the focus could be on cleaning up the ui and not
> refactoring the internals until they need to be refactored.
> 
> The most glaring issue is this message presented to the user:
> The job has been enqueued! It will be processed as soon as possible (FIXME -
> well, it could depend on a config?)

Fixed.

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