Component/Filesystem: contribute cron jobs

Author: nhaagen

components/ILIAS/Filesystem/Filesystem.php

@@ -36,5 +36,12 @@ public function init(
             new \ilFileSystemSetupAgent(
                 $pull[\ILIAS\Refinery\Factory::class]
             );
+
+        $contribute[\ILIAS\Cron\CronJob::class] = static fn() =>
+            new \ilFileSystemCleanTempDirCron(
+                self::class,
+                $use[\ILIAS\Language\Language::class],
+                $use[\ILIAS\Logging\LoggerFactory::class]
+            );
     }
 }

components/ILIAS/Filesystem/classes/class.ilFileSystemCleanTempDirCron.php

@@ -28,29 +28,19 @@
 class ilFileSystemCleanTempDirCron extends ilCronJob
 {
     protected \ILIAS\Filesystem\Filesystem $filesystem;
-
-    protected ilLanguage $language;
-
     protected ilLogger $logger;
 
-    /**
-     * @inheritDoc
-     */
-    public function __construct()
+    public function init(): void
     {
+        $this->logger = $this->logger_factory->getRootLogger();
+
         /**
          * @var $DIC Container
          */
         global $DIC;
-        if ($DIC->offsetExists('lng')) {
-            $this->language = $DIC['lng'];
-        }
         if ($DIC->offsetExists('filesystem')) {
             $this->filesystem = $DIC->filesystem()->temp();
         }
-        if ($DIC->offsetExists('ilLoggerFactory')) {
-            $this->logger = $DIC->logger()->root();
-        }
     }
 
     private function initDependencies(): void
@@ -64,12 +54,12 @@ public function getId(): string
 
     public function getTitle(): string
     {
-        return $this->language->txt('file_system_clean_temp_dir_cron');
+        return $this->lng->txt('file_system_clean_temp_dir_cron');
     }
 
     public function getDescription(): string
     {
-        return $this->language->txt("file_system_clean_temp_dir_cron_info");
+        return $this->lng->txt("file_system_clean_temp_dir_cron_info");
     }
 
     public function hasAutoActivation(): bool
@@ -128,7 +118,7 @@ public function run(): ilCronJobResult
         $folders = $this->filesystem->finder()->in([""]);
         $folders = $folders->directories();
         $folders = $folders->date($date);
-        $folders = $folders->sort(fn (
+        $folders = $folders->sort(fn(
             Metadata $a,
             Metadata $b
         ): int => strlen($a->getPath()) - strlen($b->getPath()));

components/ILIAS/Filesystem/service.xml

@@ -3,7 +3,4 @@
 		id="filesys">
 	<baseclasses>
 	</baseclasses>
-	<crons>
-		<cron id="file_system_clean_temp_dir" class="ilFileSystemCleanTempDirCron" />
-	</crons>
 </service>