Phauthentic\Email\Mailer\PhpMailMailer

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	send()	0	8	2
A	mail()	0	3	1

<?php
declare(strict_types=1);
/**
 * Copyright (c) Phauthentic (https://github.com/Phauthentic)
 *
 * Licensed under The MIT License
 * For full copyright and license information, please see the LICENSE.txt
 * Redistributions of files must retain the above copyright notice.
 *
 * @copyright     Copyright (c) Phauthentic (https://github.com/Phauthentic)
 * @link          https://github.com/Phauthentic
 * @license       https://opensource.org/licenses/mit-license.php MIT License
 */
namespace Phauthentic\Email\Mailer;

use Phauthentic\Email\EmailInterface;

/**
 * A very simple mailer using phps mail() function
 *
 * Be aware that this mailer in it's current form will send ONLY plain text mail!
 */
class PhpMailMailer implements MailerInterface
{
    /**
     * @inheritDoc
     */
    public function send(EmailInterface $email): bool
    {
        $receiver = [];

The assignment to $receiver is dead and can be removed.

        foreach ($email->getReceivers() as $receiver) {
            $receivers[] = (string)$receiver;
        }

        return $this->mail(implode(' ,', $receivers), $email->getSubject(), (string)$email->getTextContent());

The variable $receivers seems to be defined by a foreach iteration on line 31. Are you sure the iterator is never empty, otherwise this variable is not defined?

    }

    /**
     * Wrapper around mail()
     *
     * @link http://php.net/manual/en/function.mail.php
     * @param string $to Receiver
     * @param string $subject Subject
     * @param string $message
     * @param string $headers
     * @param string $paramters
     * @return bool
     */
    public function mail(string $to, string $subject, string $message, $headers = '', string $parameters = ''): bool
    {
        return mail($to, $subject, $message, $headers, $parameters);
    }
}