https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22417
--- Comment #155 from Jonathan Druart <[email protected]> --- (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. _______________________________________________ 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/
