[Koha-bugs] [Bug 31245] Job detail view for batch mod explode if job not started

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

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

Andrew Fuerste-Henry <andrewfh at dubcolib.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrewfh at dubcolib.org

--- Comment #9 from Andrew Fuerste-Henry <andrewfh at dubcolib.org> ---
Without these patches, I'm getting this same error on master when trying to
view a job that's currently underway (status='started'). I'm on a sandbox, so I
don't have a way to stop the koha-workers for testing. Here's how I've been
testing:

1 - write a report, SELECT itemnumber FROM items. With the default dev data,
that's 961 items.
2 - use your report to send all of those to batch mod. The page will take a
while to load
3 - while your batch mod page is loading, open background_jobs.pl in another
tab and have it ready to go
4 - modify something in all 961 records -- add a public note or something --
once you submit this job it will take about 45 seconds to complete
5 - BEFORE THE JOB IS DONE - reload background_jobs.pl, see your job is listed
and shows status=started. try to view details of your job, get an error
6 - once the job has finished, click View again and see your job details
successfully

With these patches, I do not get an error. When I try to see the details for a
job that's underway, the details page loads fine. It shows the status (started)
and progress (122/961) correctly. However, it says "No items modified." This is
inaccurate. If we can't load and list the items that have already been
successfully modified, I'd rather the message say something like "This job is
not yet complete."

All that said, these patches do what they say and correct a messy error, so I'm
signing off. Maybe that message can get changed in a followup.

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