Skip to content
Snippets Groups Projects
Commit 18a735a6 authored by Stefan Galinski's avatar Stefan Galinski :video_game:
Browse files

[BUGFIX] Fix more issues with the mail queue

parent 80a80984
No related branches found
No related tags found
No related merge requests found
...@@ -151,11 +151,11 @@ class MailTemplateService { ...@@ -151,11 +151,11 @@ class MailTemplateService {
} }
} }
if (sizeof($this->bccAddresses) > 0) { if (count($this->bccAddresses) > 0) {
$this->mailMessage->setBcc($this->bccAddresses); $this->mailMessage->setBcc($this->bccAddresses);
} }
if (sizeof($this->ccAddresses) > 0) { if (count($this->ccAddresses) > 0) {
$this->mailMessage->setCc($this->ccAddresses); $this->mailMessage->setCc($this->ccAddresses);
} }
} }
...@@ -214,6 +214,7 @@ class MailTemplateService { ...@@ -214,6 +214,7 @@ class MailTemplateService {
// no language found and no default template // no language found and no default template
$this->setLanguage('en'); $this->setLanguage('en');
$this->sendEmail(); $this->sendEmail();
return TRUE;
} }
} }
...@@ -233,13 +234,12 @@ class MailTemplateService { ...@@ -233,13 +234,12 @@ class MailTemplateService {
if (!isset($defaultTemplateContent)) { if (!isset($defaultTemplateContent)) {
$emailView->setTemplateSource($template->getContent()); $emailView->setTemplateSource($template->getContent());
$this->mailMessage->setSubject($template->getSubject()); $subject = $template->getSubject();
} else { } else {
$emailView->setTemplateSource($defaultTemplateContent); $emailView->setTemplateSource($defaultTemplateContent);
$this->mailMessage->setSubject( $subject = self::$registerArray[$this->extensionKey][$this->templateName]['subjects'][$this->language];
self::$registerArray[$this->extensionKey][$this->templateName]['subjects'][$this->language]
);
} }
$this->mailMessage->setSubject($subject);
$emailView->assignMultiple($this->markers); $emailView->assignMultiple($this->markers);
$emailBody = $emailView->render(); $emailBody = $emailView->render();
...@@ -252,7 +252,7 @@ class MailTemplateService { ...@@ -252,7 +252,7 @@ class MailTemplateService {
$this->mailMessage->setBody($emailBody, 'text/html'); $this->mailMessage->setBody($emailBody, 'text/html');
$this->mailMessage->send(); $this->mailMessage->send();
} else { } else {
$this->addMailToMailQueue($emailBody, $this->priority); $this->addMailToMailQueue($subject, $emailBody, $this->priority);
} }
return TRUE; return TRUE;
...@@ -261,17 +261,18 @@ class MailTemplateService { ...@@ -261,17 +261,18 @@ class MailTemplateService {
/** /**
* Adds a new mail to the mail queue. * Adds a new mail to the mail queue.
* *
* @param string $subject
* @param string $emailBody * @param string $emailBody
* @param int $priority * @param int $priority
*/ */
private function addMailToMailQueue($emailBody, $priority) { private function addMailToMailQueue($subject, $emailBody, $priority) {
/** @var ObjectManager $objectManager */ /** @var ObjectManager $objectManager */
$objectManager = GeneralUtility::makeInstance(ObjectManager::class); $objectManager = GeneralUtility::makeInstance(ObjectManager::class);
$mail = $objectManager->get(Mail::class); $mail = $objectManager->get(Mail::class);
$mail->setFromAddress($this->fromAddress); $mail->setFromAddress($this->fromAddress);
$mail->setToAddress($this->toAddresses); $mail->setToAddress($this->toAddresses);
$mail->setMailSubject($this->subject); $mail->setMailSubject($subject);
$mail->setMailBody($emailBody); $mail->setMailBody($emailBody);
$mail->setPriority($priority); $mail->setPriority($priority);
$mail->setBccAddresses(implode(',', $this->bccAddresses)); $mail->setBccAddresses(implode(',', $this->bccAddresses));
......
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