CHANGELOG

CHANGELOG.md

@@ -1,5 +1,6 @@
 - Core: Admin can change passwords without using own password again
 - Email: Set references header when forwarding mail
+- Newsletters: fix php compatibility error when sending newsletters
 
 28-10-2024: 6.8.80
 - Core: bugfix editing individual field set in entity

www/go/base/cron/EmailReminders.php

@@ -81,11 +81,6 @@ public function run(CronJob $cronJob){
 			);
 
 			while ($reminderModel = $remindersStmt->fetch()) {
-//					$relatedModel = $reminderModel->getRelatedModel();
-
-//					var_dump($relatedModel->name);
-
-//					$modelName = $relatedModel ? $relatedModel->localizedName : \GO::t("Unknown");
 				$subject = \GO::t("Reminder").': '.$reminderModel->name;
 
 				$time = !empty($reminderModel->vtime) ? $reminderModel->vtime : $reminderModel->time;
@@ -95,8 +90,6 @@ public function run(CronJob $cronJob){
 				$body = \GO::t("Time").': '.date($userModel->completeDateFormat.' '.$userModel->time_format,$time)."\n";
 				$body .= \GO::t("Name").': '.str_replace('<br />',',',$reminderModel->name)."\n";
 
-//					date_default_timezone_set(\GO::user()->timezone);
-
 				$message = \GO\Base\Mail\Message::newInstance($subject, $body);
 				$message->addFrom(\GO::config()->noreply_email,\GO::config()->title);
 				$message->addTo(new Address($userModel->email,$userModel->name));

www/go/core/mail/PHPMailer.php

@@ -431,7 +431,7 @@ public function getSentMIMEMessage()
 	 *
 	 * @return bool
 	 */
-	protected function smtpSend($header, $body)
+	protected function smtpSend($header, $body): bool
 	{
 		$header = static::stripTrailingWSP($header) . static::$LE . static::$LE;
 		$bad_rcpt = [];