Skip to content
Snippets Groups Projects
Commit 3ee3a60b authored by Torsten Oppermann's avatar Torsten Oppermann
Browse files

[TASK] Fixing a bug with multiple to addresses

parent d76d4595
No related branches found
No related tags found
No related merge requests found
...@@ -369,9 +369,9 @@ class MailTemplateService { ...@@ -369,9 +369,9 @@ class MailTemplateService {
); );
if ($this->ignoreMailQueue && !$isTemplateBlacklisted) { if ($this->ignoreMailQueue && !$isTemplateBlacklisted) {
$this->mailMessage->setBody($emailBody, 'text/html'); $this->mailMessage->setBody($emailBody, 'text/html');
$plaintextService = GeneralUtility::makeInstance(PlaintextService::class); $plaintextService = GeneralUtility::makeInstance(PlaintextService::class);
$plainTextBody = $plaintextService->makePlain($emailBody); $plainTextBody = $plaintextService->makePlain($emailBody);
$this->mailMessage->addPart($plainTextBody, 'text/plain'); $this->mailMessage->addPart($plainTextBody, 'text/plain');
$this->mailMessage->send(); $this->mailMessage->send();
$dateTime = new DateTime(); $dateTime = new DateTime();
$currentTimestamp = $dateTime->getTimestamp(); $currentTimestamp = $dateTime->getTimestamp();
...@@ -463,7 +463,12 @@ class MailTemplateService { ...@@ -463,7 +463,12 @@ class MailTemplateService {
$plaintextService = GeneralUtility::makeInstance(PlaintextService::class); $plaintextService = GeneralUtility::makeInstance(PlaintextService::class);
$plaintextBody = $plaintextService->makePlain($mailToSend->getMailBody()); $plaintextBody = $plaintextService->makePlain($mailToSend->getMailBody());
$this->mailMessage->addPart($plaintextBody, 'text/plain'); $this->mailMessage->addPart($plaintextBody, 'text/plain');
$this->mailMessage->setTo(trim($mailToSend->getToAddress())); $toAddresses = trim($mailToSend->getToAddress());
$addressesArray = GeneralUtility::trimExplode(',', $toAddresses, TRUE);
if (\count($addressesArray) > 1) {
$toAddresses = $addressesArray;
}
$this->mailMessage->setTo($toAddresses);
$this->mailMessage->setFrom($mailToSend->getFromAddress(), $mailToSend->getFromName()); $this->mailMessage->setFrom($mailToSend->getFromAddress(), $mailToSend->getFromName());
$this->mailMessage->setSubject($mailToSend->getMailSubject()); $this->mailMessage->setSubject($mailToSend->getMailSubject());
...@@ -491,9 +496,9 @@ class MailTemplateService { ...@@ -491,9 +496,9 @@ class MailTemplateService {
} }
} }
$dateTime = new DateTime(); $dateTime = new DateTime();
if ((int)$mailToSend->getSendingTime() === 0) { if ((int) $mailToSend->getSendingTime() === 0) {
$mailToSend->setSendingTime($dateTime->getTimestamp()); $mailToSend->setSendingTime($dateTime->getTimestamp());
} }
$mailToSend->setLastSendingTime($dateTime->getTimestamp()); $mailToSend->setLastSendingTime($dateTime->getTimestamp());
$this->mailMessage->send(); $this->mailMessage->send();
$mailRepository->update($mailToSend); $mailRepository->update($mailToSend);
...@@ -505,9 +510,14 @@ class MailTemplateService { ...@@ -505,9 +510,14 @@ class MailTemplateService {
* @return MailTemplateService * @return MailTemplateService
*/ */
public function setToAddresses($toAddresses) { public function setToAddresses($toAddresses) {
$toAddresses = preg_replace('~\x{00a0}~siu', ' ', $toAddresses); $toAddresses = trim(preg_replace('~\x{00a0}~siu', ' ', $toAddresses));
$this->toAddresses = trim($toAddresses); $this->toAddresses = $toAddresses;
$this->mailMessage->setTo(trim($toAddresses));
$addressesArray = GeneralUtility::trimExplode(',', $toAddresses, TRUE);
if (\count($addressesArray) > 1) {
$toAddresses = $addressesArray;
}
$this->mailMessage->setTo($toAddresses);
return $this; return $this;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment