check for news expiration in cronjob

MatrixPostNewsCronjob.php

@@ -77,6 +77,7 @@ class MatrixPostNewsCronjob extends CronJob
             print_r($news);
         }
         foreach (StudipNews::findMany($news) as $one) {
+            if ($one->date + $one->expire >= time()) {
                 foreach ($one->news_ranges as $range) {
                     $success = false;
                     if ($range->course && MatrixRoom::hasRoom($range->course->id)) {
@@ -95,7 +96,7 @@ class MatrixPostNewsCronjob extends CronJob
                             if ($parameters['verbose']) {
                                 echo sprintf(
                                     "Posted news entry $1%s to Matrix room $2%s in course $3%s.\n",
-                                $news->id, $room->matrix_room_id, $room->range_id
+                                    $one->id, $room->matrix_room_id, $room->range_id
                                 );
                             }
 
@@ -106,6 +107,12 @@ class MatrixPostNewsCronjob extends CronJob
                         }
                     }
                 }
+            } else {
+                DBManager::get()->execute(
+                    "DELETE FROM `matrix_upcoming_news` WHERE `news_id` = :id",
+                    ['id' => $one->id]
+                );
+            }
         }
 
         // Finally: cleanup news entries that are already expired and need not be considered anymore.

plugin.manifest

 pluginname=Matrix-Chat
 pluginclassname=MatrixPlugin
 origin=data-quest
-version=1.4.3
+version=1.4.4
 screenshot=assets/images/matrix_logo.png
 description=Matrix chat for Stud.IP courses
 studipMinVersion=4.5