DERHANSEN\SfEventMgt\Service\EmailService::sendEmailMessage()

Complexity

Conditions	11
Paths	37

Size

Total Lines	46
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function sendEmailMessage(
        string $sender,
        string $recipient,
        string $subject,
        string $body,
        ?string $name = null,
        array $attachments = [],
        ?string $replyTo = null
    ): bool {
        if ($subject === '' || !GeneralUtility::validEmail($sender) || !GeneralUtility::validEmail($recipient)) {
            return false;
        }

        $useFluidEmail = (bool)(GeneralUtility::makeInstance(ExtensionConfiguration::class)
            ->get('sf_event_mgt', 'useFluidEmail'));

        $email = $this->getEmailObject($useFluidEmail);
        $email->from(new Address($sender, $name));

It seems like $name can also be of type null; however, parameter $name of Symfony\Component\Mime\Address::__construct() does only seem to accept string, maybe add an additional type check?

        $email->to($recipient);
        $email->subject($subject);
        $email->html($body);
        if ($replyTo !== null && $replyTo !== '' && GeneralUtility::validEmail($replyTo)) {
            $email->replyTo(new Address($replyTo));
        }
        foreach ($attachments as $attachment) {
            if (file_exists($attachment)) {
                $email->attachFromPath($attachment);
            }
        }

        if ($email instanceof FluidEmail) {
            $email->format('html');
            $email->setTemplate('EventManagement');
            $email->assignMultiple([
                'title' => $subject,
                'body' => $body,
            ]);
        }

        $mailer = GeneralUtility::makeInstance(MailerInterface::class);

        try {
            $mailer->send($email);
            return $mailer->getSentMessage() !== null;
        } catch (TransportExceptionInterface $e) {
            return false;
        }