Bug 31245: Do not call report for job detail view if job not started

We should not generate the report if the job hasn't started yet

Test plan:
Stop the koha-worker processes, modify a new item using the batch mod
tool, then see the detail page of the job

Without this patch an error is generated:
Can't use an undefined value as an ARRAY reference at /kohadevbox/koha/Koha/BackgroundJob/BatchUpdateItem.pm line 175

Signed-off-by: Andrew <andrewfh@dubcolib.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

diff --git a/admin/background_jobs.pl b/admin/background_jobs.pl
index 8a27c00..a4b7b90 100755 (executable)
--- a/admin/background_jobs.pl
+++ b/admin/background_jobs.pl
@@ -52,8 +52,10 @@ if ( $op eq 'view' ) {
         }
         else {
             $template->param( job => $job, );
-            my $report = $job->additional_report() || {};
-            $template->param( %$report );
+            if ( $job->status ne 'new' ) {
+                my $report = $job->additional_report() || {};
+                $template->param( %$report );
+            }
         }
     } else {
         $op = 'list';

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/background_jobs.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/background_jobs.tt
index 4da24fe..ab1cd65 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/background_jobs.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/background_jobs.tt
@@ -137,10 +137,10 @@
                 [% job.ended_on | $KohaDates with_hours = 1 %]
             </li>
             <li><label for="job_data">Report: </label>
-                [% PROCESS 'report' %]
+                [% IF job.status != 'new' %][% PROCESS 'report' %][% END %]
             </li>
             <li><label for="job_data">Detailed messages: </label>
-                [% PROCESS 'detail' %]
+                [% IF job.status != 'new' %][% PROCESS 'detail' %][% END %]
             </li>
         </ol>
     </fieldset>