videos with missing config_id should stay failed.

tgloeggl · tgloeggl · commit 878d08350278 · 2025-02-07T10:45:36.000+01:00
diff --git a/cronjobs/opencast_worker.php b/cronjobs/opencast_worker.php
@@ -59,7 +59,9 @@ public function execute($last_result, $parameters = array())
 
                     if (empty($video->config_id)) {
                         echo 'No config_id for video '. $video->id. "\n";
-                        $task->delete();
+                        $task->state = 'failed';
+                        $task->data = json_encode(['error' => 'No config_id for video '. $video->id]);
+                        $task->store();
                         continue;
                     }
 

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,9 @@ public function execute($last_result, $parameters = array())`
`59`	`59`
`60`	`60`	`if (empty($video->config_id)) {`
`61`	`61`	`echo 'No config_id for video '. $video->id. "\n";`
`62`		`- $task->delete();`
	`62`	`+ $task->state = 'failed';`
	`63`	`+ $task->data = json_encode(['error' => 'No config_id for video '. $video->id]);`
	`64`	`+ $task->store();`
`63`	`65`	`continue;`
`64`	`66`	`}`
`65`	`67`