csphere\core\mail\Driver_Sendmail

Complexity

Total Complexity

3

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	prepare()	0	19	1
A	send()	0	17	1

<?php

/**
 * Provides mail functionality using the php mail function
 *
 * PHP Version 5
 *
 * @category  Core
 * @package   Mail
 * @author    Hans-Joachim Piepereit <contact@csphere.eu>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

namespace csphere\core\mail;

/**
 * Provides mail functionality using the php mail function
 *
 * @category  Core
 * @package   Mail
 * @author    Hans-Joachim Piepereit <contact@csphere.eu>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

class Driver_Sendmail extends Base
{
    /**
     * Creates the mail handler object
     *
     * @param array $config Configuration details as an array
     *
     * @return \csphere\core\mail\Driver_Sendmail
     **/

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

There must be no blank lines after the function comment

    public function __construct(array $config)
    {
        parent::__construct($config);

        // Change sendmail setting
        ini_set('sendmail_from', $this->config['from']);
    }

    /**
     * Prepares a mail by setting its content
     *
     * @param string $subject Subject to use in email
     * @param string $message Message to use in email
     * @param string $type    Defaults to text/plain
     *
     * @return boolean
     **/

There must be no blank lines after the function comment

    public function prepare($subject, $message, $type = 'text/plain')
    {
        // Run initial preparations
        parent::prepare($subject, $message, $type);

        // Set header data
        $headers = ['MIME-Version: 1.0',
                    'Content-Type: ' . $type . '; charset=UTF-8',
                    'Content-Transfer-Encoding: base64',
                    'X-Mailer: cSphere',
                    'From: ' . $this->config['from']];

        $this->headers = array_merge($this->headers, $headers);

        // Create encoded subject and content strings
        $this->encode();

        return true;
    }

    /**
     * Sends the prepared content to a given email
     *
     * @param string  $email Email target
     * @param boolean $clear Defaults to true which keeps mail data
     *
     * @return boolean
     **/

There must be no blank lines after the function comment

    public function send($email, $clear = true)
    {
        // Send mail
        $headers = implode($this->config['eol'], $this->headers);

        $send = mail(
            $email, $this->content['subject_b64'],
            $this->content['message_b64'], $headers
        );

        // Log mail and handle clear status
        $this->log($email, $send);

        $this->clear($clear);

        return $send;
    }
}