WZRD\Mail\SwiftMailer

Complexity

Total Complexity

8

Size/Duplication

Total Lines	73
Duplicated Lines	35.62 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
C	send()	26	47	7

<?php

namespace WZRD\Mail;

use Swift_Attachment;
use Swift_EmbeddedFile;
use Swift_Mailer;
use Swift_Message;
use WZRD\Contracts\Mail\Mailer;
use WZRD\Contracts\Mail\Message as MessageContract;

class SwiftMailer implements Mailer
{
    /**
     * Swift.
     *
     * @var Swift_Mailer
     */
    private $swift;

    /**
     * Construct with SwiftMailer instance.
     *
     * @param Swift_Mailer $swift
     */
    public function __construct(Swift_Mailer $swift)
    {
        $this->swift = $swift;
    }

    /**
     * Send a mail.
     *
     * @param WZRD\Contracts\Mail\Message $message
     * @param array                       $options
     */
    public function send(MessageContract $message, $options = [])
    {
        // Initialisation d'un nouveau message
        $swift_message = Swift_Message::newInstance();

        // Compose
        $swift_message->setFrom($message->getFrom());

$message->getFrom() is of type array, but the function expects a string.

        $swift_message->setTo($message->getTo());
        $swift_message->setCc($message->getCc());
        $swift_message->setBcc($message->getBcc());
        $swift_message->setSubject($message->getSubject());
        $swift_message->addPart($message->getText(), 'text/plain');
        $swift_message->addPart($message->getHtml(), 'text/html');

        // Attachments
        foreach ($message->getAttachments() as $attachment) {

This code seems to be duplicated across your project.

            list($file, $options) = $attachment;

            $content_type = empty($options['content-type']) ? null : $options['content-type'];

            $attachment = Swift_Attachment::fromPath($file, $content_type);

            if (!empty($options['filename'])) {
                $attachment->setFilename($options['filename']);
            }

            $swift_message->attach($attachment);
        }

        // Inline attachments
        foreach ($message->getInlines() as $inline) {

This code seems to be duplicated across your project.

            list($file, $options) = $inline;

            $content_type = empty($options['content-type']) ? null : $options['content-type'];

            $attachment = Swift_EmbeddedFile::fromPath($file, $content_type);

The call to Swift_EmbeddedFile::fromPath() has too many arguments starting with $content_type.

            if (!empty($options['filename'])) {
                $attachment->setFilename($options['filename']);
            }

            $swift_message->embed($attachment);
        }

        // Send !
        $this->swift->send($swift_message);
    }
}