Fix wrong calculation of the remaining notifications in the daily notification email

BurntimeX · BurntimeX · commit 75658932cdda · 2024-11-04T10:59:30.000+01:00
diff --git a/wcfsetup/install/files/lib/system/cronjob/DailyMailNotificationCronjob.class.php b/wcfsetup/install/files/lib/system/cronjob/DailyMailNotificationCronjob.class.php
@@ -259,6 +259,9 @@ public function execute(Cronjob $cronjob)
                 $maximumNotificationCount++;
             }
             $remainingNotificationCount = $notificationCount - $maximumNotificationCount;
+            if ($remainingNotificationCount < 0) {
+                $remainingNotificationCount = 0;
+            }
             $notifications = \array_slice($notifications, 0, $maximumNotificationCount);
 
             $html = new RecipientAwareTextMimePart(

Original file line number	Diff line number	Diff line change
`@@ -259,6 +259,9 @@ public function execute(Cronjob $cronjob)`
`259`	`259`	`$maximumNotificationCount++;`
`260`	`260`	`}`
`261`	`261`	`$remainingNotificationCount = $notificationCount - $maximumNotificationCount;`
	`262`	`+ if ($remainingNotificationCount < 0) {`
	`263`	`+ $remainingNotificationCount = 0;`
	`264`	`+ }`
`262`	`265`	`$notifications = \array_slice($notifications, 0, $maximumNotificationCount);`
`263`	`266`
`264`	`267`	`$html = new RecipientAwareTextMimePart(`