Merge branch '6.0' into 6.1

symbiote · Sep 17, 2023 · c131b3e · c131b3e
2 parents e507c51 + 1d59b00
commit c131b3e
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/Extensions/WorkflowEmbargoExpiryExtension.php b/src/Extensions/WorkflowEmbargoExpiryExtension.php
@@ -176,7 +176,7 @@ public function updateCMSFields(FieldList $fields)
     public function clearPublishJob()
     {
         $job = $this->owner->PublishJob();
-        if ($job && $job->exists()) {
+        if ($job && $job->exists() && !$job->JobFinished) {
             $job->delete();
         }
         $this->owner->PublishJobID = 0;
@@ -189,7 +189,7 @@ public function clearUnPublishJob()
     {
         // Cancel any in-progress unpublish job
         $job = $this->owner->UnPublishJob();
-        if ($job && $job->exists()) {
+        if ($job && $job->exists() && !$job->JobFinished) {
             $job->delete();
         }
         $this->owner->UnPublishJobID = 0;
@@ -293,6 +293,10 @@ public function onBeforeWrite()
         if (!$this->owner->ID) {
             return;
         }
+
+        if ($this->owner->hasMethod('isPublishJobRunning') && $this->owner->isPublishJobRunning()) {
+            return;
+        }
 
         // Check requested dates of publish / unpublish, and whether the page should have already been unpublished
         $now = DBDatetime::now()->getTimestamp();