Slick\Mail\Message

Complexity

Total Complexity

19

Size/Duplication

Total Lines	263
Duplicated Lines	14.83 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes	7
Bugs	0	Features	3

16 Methods

Rating	Name	Duplication	Size	Complexity
A	isValid()	0	6	1
A	getHeaders()	0	7	2
A	addHeader()	0	7	1
A	setFrom()	9	9	1
A	getFromAddressList()	0	6	1
A	setTo()	0	7	1
A	getToAddressList()	0	6	1
A	addTo()	9	9	1
A	addCc()	7	7	1
A	addBcc()	7	7	1
A	addReplyTo()	7	7	1
A	setSubject()	0	6	1
A	getSubject()	0	5	1
A	setBody()	0	6	1
A	getBodyText()	0	8	2
A	getHeader()	0	9	2

<?php

/**
 * This file is part of slick/mail package
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Slick\Mail;

use Slick\Mail\Header\AddressList;
use Slick\Mail\Header\AddressListInterface;
use Slick\Mail\Header\GenericHeader;
use Slick\Mail\Header\HeaderInterface;

/**
 * Wraps an e-mail message
 *
 * @package Slick\Mail
 * @author  Filipe Silva <silvam.filipe@gmail.com>
 */
class Message implements MessageInterface
{

    /**
     * @var HeaderInterface[]
     */
    protected $headers;

    /**
     * @var MessageBodyInterface
     */
    protected $body;

    /**
     * Is the message valid?
     *
     * If we don't have any From addresses, we're invalid, according
     * to RFC2822.
     *
     * @return bool
     */
    public function isValid()
    {
        /** @var AddressList $from */
        $from = $this->getHeader('From', new AddressList('From'));
        return !$from->isEmpty();
    }

    /**
     * Access headers collection
     *
     * Lazy-loads if not already attached.
     *
     * @return HeaderInterface[]
     */
    public function getHeaders()
    {
        if (null == $this->headers) {
            $this->headers['Date'] = New GenericHeader('Date', date('r'));
        }
        return $this->headers;
    }

    /**
     * Adds an header with provided value
     *
     * If key already exists it will be overridden
     *
     * @param string $name
     * @param HeaderInterface $header
     *
     * @return MessageInterface
     */
    public function addHeader($name, HeaderInterface $header)
    {
        $headers = $this->getHeaders();
        $headers[$name] = $header;
        $this->headers = $headers;

It seems like $headers of type array<string,object<Slick\Mail\Header\HeaderInterface>> is incompatible with the declared type array<integer,object<Slick\Mail\Header\HeaderInterface>> of property $headers.

        return $this;
    }

    /**
     * Set (overwrite) From addresses
     *
     * @param  string $emailAddress
     * @param  string|null $name
     *
     * @return MessageInterface
     */
    public function setFrom($emailAddress, $name = null)

This method seems to be duplicated in your project.

    {
        $address = new AddressList\Address($emailAddress, $name);

It seems like $name defined by parameter $name on line 92 can also be of type string; however, Slick\Mail\Header\AddressList\Address::__construct() does only seem to accept null, maybe add an additional type check?

        $from = (new AddressList('From'))->add($address);
        $this->addHeader('From', $from);
        $sender = new GenericHeader('Sender', (string) $address);
        $this->addHeader('Sender', $sender);
        return $this;
    }

    /**
     * Access the address list of the From header.
     *
     * @return string
     */
    public function getFromAddressList()
    {
        /** @var AddressList $from */
        $from = $this->getHeader('From', new AddressList('From'));
        return trim(str_replace('From:', '', $from->toString()));
    }

    /**
     * Overwrite the address list in the To recipients
     *
     * @param  string $email
     * @param  string|null $name
     *
     * @return MessageInterface
     */
    public function setTo($email, $name = null)
    {
        $toHeader = (new AddressList('To'))
            ->add(new AddressList\Address($email, $name));

It seems like $name defined by parameter $name on line 122 can also be of type string; however, Slick\Mail\Header\AddressList\Address::__construct() does only seem to accept null, maybe add an additional type check?

        $this->addHeader('To', $toHeader);
        return $this;
    }

    /**
     * Access the address list of the To header.
     *
     * @return string
     */
    public function getToAddressList()
    {
        /** @var AddressList $from */
        $from = $this->getHeader('To', new AddressList('To'));
        return trim(str_replace('To:', '', $from->toString()));
    }

    /**
     * Add one or more addresses to the To recipients
     *
     * Appends to the list.
     *
     * @param  string $email
     * @param  null|string $name
     *
     * @return MessageInterface
     */
    public function addTo($email, $name = null)

This method seems to be duplicated in your project.

    {
        $address = new AddressList\Address($email, $name);

It seems like $name defined by parameter $name on line 152 can also be of type string; however, Slick\Mail\Header\AddressList\Address::__construct() does only seem to accept null, maybe add an additional type check?

        /** @var AddressListInterface $toHeader */
        $toHeader = $this->getHeader('To', new AddressList('To'));
        $toHeader->add($address);
        $this->addHeader('To', $toHeader);
        return $this;
    }

    /**
     * Add a "Cc" address
     *
     * @param  string $email
     * @param  null|string $name
     *
     * @return MessageInterface
     */
    public function addCc($email, $name = null)

This method seems to be duplicated in your project.

    {
        /** @var AddressList $header */
        $header = $this->getHeader('Cc', new AddressList('Cc'));
        $header->add(new AddressList\Address($email, $name));

It seems like $name defined by parameter $name on line 170 can also be of type string; however, Slick\Mail\Header\AddressList\Address::__construct() does only seem to accept null, maybe add an additional type check?

        return $this;
    }

    /**
     * Add a "Bcc" address
     *
     * @param  string $email
     * @param  null|string $name
     *
     * @return MessageInterface
     */
    public function addBcc($email, $name = null)

This method seems to be duplicated in your project.

    {
        /** @var AddressList $header */
        $header = $this->getHeader('Bcc', new AddressList('Bcc'));
        $header->add(new AddressList\Address($email, $name));

It seems like $name defined by parameter $name on line 186 can also be of type string; however, Slick\Mail\Header\AddressList\Address::__construct() does only seem to accept null, maybe add an additional type check?

        return $this;
    }

    /**
     * Add one or more addresses to the Reply-To recipients
     *
     * Appends to the list.
     *
     * @param  string $email
     * @param  null|string $name
     *
     * @return MessageInterface
     */
    public function addReplyTo($email, $name = null)

This method seems to be duplicated in your project.

    {
        /** @var AddressList $header */
        $header = $this->getHeader('Reply-To', new AddressList('Reply-To'));
        $header->add(new AddressList\Address($email, $name));

It seems like $name defined by parameter $name on line 204 can also be of type string; however, Slick\Mail\Header\AddressList\Address::__construct() does only seem to accept null, maybe add an additional type check?

        return $this;
    }

    /**
     * Set the message subject header value
     *
     * @param  string $subject
     *
     * @return MessageInterface
     */
    public function setSubject($subject)
    {
        $subject = new GenericHeader('Subject', $subject);
        $this->addHeader('Subject', $subject);
        return $this;
    }

    /**
     * Get the message subject header value
     *
     * @return null|string
     */
    public function getSubject()
    {
        $subject = $this->getHeader('Subject');
        return $subject->getValue();
    }

    /**
     * Set the message body
     *
     * @param  MessageBodyInterface $body
     *
     * @return MessageInterface
     */
    public function setBody(MessageBodyInterface $body)
    {
        $body->setMailMessage($this);
        $this->body = $body;
        return $this;
    }

    /**
     * Get the string-serialized message body text
     *
     * @return string
     */
    public function getBodyText()
    {
        $text = '';
        if (null != $this->body) {
            $text = $this->body->getBodyText();
        }
        return $text;
    }

    /**
     * Gets the header with provided name.
     *
     * If no header was set with that name, the default value will be
     * assign to a new header and returned.
     *
     * @param string $name
     * @param mixed $default
     *
     * @return HeaderInterface
     */
    protected function getHeader($name, $default = '')
    {
        if (array_key_exists($name, $this->getHeaders())) {
            return $this->headers[$name];
        }

        $this->headers[$name] = $default;
        return $default;
    }
}