Netgen\Bundle\InformationCollectionBundle\Mailer\Mailer

Complexity

Total Complexity

7

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace Netgen\InformationCollection\Core\Mailer;

use Netgen\InformationCollection\API\Exception\EmailNotSentException;
use Netgen\InformationCollection\API\Value\DataTransfer\EmailContent;
use Netgen\InformationCollection\API\Mailer\MailerInterface;

class Mailer implements MailerInterface
{
    /**
     * @var \Swift_Mailer
     */
    protected $internalMailer;

    /**
     * Mailer constructor.
     *
     * @param \Swift_Mailer $internalMailer
     */
    public function __construct(\Swift_Mailer $internalMailer)
    {
        $this->internalMailer = $internalMailer;
    }

    /**
     * {@inheritdoc}
     */
    public function createAndSendMessage(EmailContent $data): void
    {
        $message = new \Swift_Message();

        try {
            $message->setTo($data->getRecipient());
        } catch (\Swift_RfcComplianceException $e) {
            throw new EmailNotSentException('recipient', $e->getMessage());
        }

        try {
            $message->setFrom($data->getSender());
        } catch (\Swift_RfcComplianceException $e) {
            throw new EmailNotSentException('sender', $e->getMessage());
        }

        $message->setSubject($data->getSubject());
        $message->setBody($data->getBody(), 'text/html');

        if ($data->hasAttachments()) {
            foreach ($data->getAttachments() as $attachment) {

The expression $data->getAttachments() of type array<integer,object<eZ\Publish\Core\FieldType\BinaryFile\Value>>|null is not guaranteed to be traversable. How about adding an additional type check?

                $message->attach(
                    \Swift_Attachment::fromPath($attachment->inputUri, $attachment->mimeType)
                        ->setFilename($attachment->fileName)
                );
            }
        }

        if (!$this->internalMailer->send($message)) {
            throw new EmailNotSentException('send', 'invalid mailer configuration?');
        }
    }
}