From 3f79fcd5aa5efed7883045e3ce467ead79727131 Mon Sep 17 00:00:00 2001
From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Date: Mon, 5 Sep 2022 10:33:01 +0000
Subject: [PATCH] Bug 27421: (QA follow-up) Sub finish should respect failed
 status too

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 Koha/BackgroundJob.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Koha/BackgroundJob.pm b/Koha/BackgroundJob.pm
index f5840fff02..e1274ba144 100644
--- a/Koha/BackgroundJob.pm
+++ b/Koha/BackgroundJob.pm
@@ -246,7 +246,7 @@ Makes the job record as finished. If the job status is I<cancelled>, it is kept.
 sub finish {
     my ( $self, $data ) = @_;
 
-    $self->status('finished') unless $self->status eq 'cancelled';
+    $self->status('finished') unless $self->status eq 'cancelled' or $self->status eq 'failed';
 
     return $self->set(
         {
-- 
2.20.1