cronjobs: remove scheduling once and priority, fixes #4078

Closes #4078

Merge request studip/studip!2922

2 files changed, 20 insertions, 6 deletions

diff --git a/db/migrations/1.231_add_files_search_index.php b/db/migrations/1.231_add_files_search_index.php
index 18a22ae..5ac455a 100644
--- a/db/migrations/1.231_add_files_search_index.php
+++ b/db/migrations/1.231_add_files_search_index.php
@@ -181,12 +181,7 @@ class AddFilesSearchIndex extends Migration
 
     private function installCronjob()
     {
-        $scheduler = CronjobScheduler::getInstance();
-
         require_once 'lib/classes/FilesSearch/Cronjob.php';
-        $task = new  \FilesSearch\Cronjob();
-        $taskId = $scheduler->registerTask($task);
-        $scheduler->scheduleOnce($taskId, strtotime('+1 minute'))->activate();
-        $scheduler->schedulePeriodic($taskId, 55, 0)->activate();
+        \FilesSearch\Cronjob::register()->schedule(55, 0)->activate(true);
     }
 }
diff --git a/db/migrations/6.0.1_remove_cronjobs_scheduling_once_and_priority.php b/db/migrations/6.0.1_remove_cronjobs_scheduling_once_and_priority.php
new file mode 100644
index 0000000..d3d95b0
--- /dev/null
+++ b/db/migrations/6.0.1_remove_cronjobs_scheduling_once_and_priority.php
@@ -0,0 +1,19 @@
+<?php
+ return new class extends Migration
+ {
+     protected function up()
+     {
+         $query = "ALTER TABLE `cronjobs_schedules`
+                   DROP COLUMN `priority`,
+                   DROP COLUMN `type`";
+         DBManager::get()->exec($query);
+     }
+
+     protected function down()
+     {
+         $query = "ALTER TABLE `cronjobs_schedules`
+                   ADD COLUMN `priority` ENUM('low', 'normal', 'high') COLLATE `latin1_bin` DEFAULT NULL AFTER `parameters`,
+                   ADD COLUMN `type` ENUM('periodic', 'once') COLLATE `latin1_bin` DEFAULT NULL AFTER `priority`";
+         DBManager::get()->exec($query);
+     }
+ };