pkp/pkp-lib#9678 removed schedule task table and dao

pkp · Jul 1, 2024 · 66a7802 · 66a7802
1 parent 559a406
commit 66a7802
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 13 deletions.
diff --git a/classes/plugins/PubObjectsExportPlugin.php b/classes/plugins/PubObjectsExportPlugin.php
@@ -594,12 +594,7 @@ public function registerSchedules(?Scheduler $scheduler = null): void
             ->addSchedule(new DOAJInfoSender())
             ->everyMinute()
             ->name(DOAJInfoSender::class)
-            ->withoutOverlapping()
-            ->then(function () {
-                /** @var \PKP\scheduledTask\ScheduledTaskDAO $scheduledTaskDao */
-                $scheduledTaskDao = DAORegistry::getDAO('ScheduledTaskDAO');
-                $scheduledTaskDao->updateLastRunTime(DOAJInfoSender::class);
-            });
+            ->withoutOverlapping();
     }
 
     /**

diff --git a/classes/scheduler/Scheduler.php b/classes/scheduler/Scheduler.php
@@ -16,24 +16,21 @@ public function registerSchedules(): void
             ->call(fn () => (new SubscriptionExpiryReminder())->execute())
             ->daily()
             ->name(SubscriptionExpiryReminder::class)
-            ->withoutOverlapping()
-            ->then(fn () => $this->scheduledTaskDao->updateLastRunTime(SubscriptionExpiryReminder::class));
+            ->withoutOverlapping();
 
         $this
             ->schedule
             ->call(fn () => (new UsageStatsLoader([]))->execute())
             ->daily()
             ->name(UsageStatsLoader::class)
-            ->withoutOverlapping()
-            ->then(fn () => $this->scheduledTaskDao->updateLastRunTime(UsageStatsLoader::class));
+            ->withoutOverlapping();
 
         $this
             ->schedule
             ->call(fn () => (new OpenAccessNotification())->execute())
             ->hourly()
             ->name(OpenAccessNotification::class)
-            ->withoutOverlapping()
-            ->then(fn () => $this->scheduledTaskDao->updateLastRunTime(OpenAccessNotification::class));
+            ->withoutOverlapping();
 
         parent::registerSchedules();
     }

diff --git a/dbscripts/xml/install.xml b/dbscripts/xml/install.xml
@@ -33,7 +33,6 @@
 	<migration class="PKP\migration\install\InvitationsMigration" />
 	<migration class="PKP\migration\install\ReviewFormsMigration" />
 	<migration class="PKP\migration\install\SubmissionFilesMigration" />
-	<migration class="PKP\migration\install\ScheduledTasksMigration" />
 	<migration class="PKP\migration\install\LibraryFilesMigration" />
 	<migration class="PKP\migration\install\ReviewsMigration" />
 	<migration class="PKP\migration\install\TemporaryFilesMigration" />

diff --git a/dbscripts/xml/upgrade.xml b/dbscripts/xml/upgrade.xml
@@ -125,6 +125,7 @@
 	<upgrade minversion="3.1.0.0" maxversion="3.4.9.9">
 		<migration class="PKP\migration\upgrade\v3_5_0\PreflightCheckMigration" fallback="3.4.9.9" />
 		<migration class="PKP\migration\upgrade\v3_5_0\I9895_AddAppKeyToConfigFile"/>
+		<migration class="PKP\migration\upgrade\v3_5_0\I9678_RemoveScheduledTasksTable"/>
 		<migration class="PKP\migration\upgrade\v3_5_0\InstallEmailTemplates"/>
 		<migration class="PKP\migration\upgrade\v3_5_0\I9197_MigrateAccessKeys"/>
 		<migration class="PKP\migration\upgrade\v3_5_0\I9253_SiteAnnouncements"/>