Component/Forum: contribute cron jobs

Author: nhaagen

components/ILIAS/Forum/Forum.php

@@ -38,7 +38,15 @@ public function init(
             );
         $contribute[Component\Resource\PublicAsset::class] = fn() =>
             new Component\Resource\ComponentJS($this, "autosave_forum.js");
+
         $contribute[Component\Resource\PublicAsset::class] = fn() =>
-        new Component\Resource\ComponentCSS($this, "forum_table.css");
+            new Component\Resource\ComponentCSS($this, "forum_table.css");
+
+        $contribute[\ILIAS\Cron\CronJob::class] = static fn() =>
+            new \ilForumCronNotification(
+                'components\\' . self::class,
+                $use[\ILIAS\Language\Language::class],
+                true
+            );
     }
 }

components/ILIAS/Forum/classes/class.ilForumCronNotification.php

@@ -41,7 +41,6 @@ class ilForumCronNotification extends ilCronJob
     /** @var array<int, ilObjCourse|ilObjGroup|null> */
     private static array $container_by_frm_ref_id = [];
 
-    private readonly ilLanguage $lng;
     private readonly ilSetting $settings;
     private ilLogger $logger;
     private ilTree $tree;
@@ -51,22 +50,26 @@ class ilForumCronNotification extends ilCronJob
     private readonly \ILIAS\Refinery\Factory $refinery;
     private readonly ilCronManager $cronManager;
 
+
     public function __construct(
-        ilDBInterface $database = null,
-        ilForumNotificationCache $notificationCache = null,
-        ilLanguage $lng = null,
-        ilSetting $settings = null,
-        \ILIAS\Refinery\Factory $refinery = null,
-        ilCronManager $cronManager = null
+        string $component,
+        \ILIAS\Language\Language $lng,
+        bool $registration = false
     ) {
-        global $DIC;
+        parent::__construct($component, $lng);
+        if (!$registration) {
+            $this->additionalConstruct();
+        }
+    }
 
-        $this->settings = $settings ?? new ilSetting('frma');
-        $this->lng = $lng ?? $DIC->language();
-        $this->ilDB = $database ?? $DIC->database();
-        $this->notificationCache = $notificationCache ?? new ilForumNotificationCache();
-        $this->refinery = $refinery ?? $DIC->refinery();
-        $this->cronManager = $cronManager ?? $DIC->cron()->manager();
+    private function additionalConstruct()
+    {
+        global $DIC;
+        $this->settings = new ilSetting('frma');
+        $this->ilDB = $DIC->database();
+        $this->notificationCache = new ilForumNotificationCache();
+        $this->refinery = $DIC->refinery();
+        $this->cronManager = $DIC->cron()->manager();
     }
 
     public function getId(): string

components/ILIAS/Forum/module.xml

@@ -32,9 +32,6 @@
 		<event type="raise" id="mergedThreads" />
 		<event type="raise" id="movedThreads" />
 	</events>
-	<crons>		
-		<cron id="frm_notification" class="ilForumCronNotification" />
-	</crons>
 	<logging />
 	<gsproviders>
 		<gsprovider purpose="tools" class_name="ilForumGlobalScreenToolsProvider" />