https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30943
--- Comment #12 from Tomás Cohen Arazi <tomasco...@gmail.com> ---
(In reply to Marcel de Rooy from comment #11)
> Please read the history on both bugs.
On bug 32305, they discuss the Koha::BackgroundJob->process method, making it
not initiate a background job unless its status is 'new()', with different
approaches and in discussion, as you noticed.
This bug is only taking care of making already existing helpers be used to
reduce the amount of unnecessary code, like this:
- my $job_progress = 0;
- $self->started_on(dt_from_string)->progress($job_progress)
- ->status('started')->store;
+ $self->start;
the place in which they might have an intersection, is that ->start will raise
an exception if the status is not new().
We don't have a 'retry' mechanism on failure and we don't have a proper way to
deal with stale/wrong messages in rabbit. But I'm pretty sure we'd prefer to
explode earlier if a job is asked to be started when it shouldn't.
I'd say, as per comment 4, that we could rename start() => mark_started(). But
that's out of the scope of this bug.
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/
