Swift_Transport_FailoverTransport

Complexity

Total Complexity

15

Size/Duplication

Total Lines	112
Duplicated Lines	29.46 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

78.38%

Importance

Changes	5
Bugs	1	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	ping()	0	14	4
C	send()	33	33	7
A	_getNextTransport()	0	8	2
A	_killCurrentTransport()	0	5	1

<?php

/*
 * This file is part of SwiftMailer.
 * (c) 2004-2009 Chris Corbyn
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

/**
 * Contains a list of redundant Transports so when one fails, the next is used.
 *
 * @author Chris Corbyn
 */
class Swift_Transport_FailoverTransport extends Swift_Transport_LoadBalancedTransport

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * Registered transport currently used.
     *
     * @var Swift_Transport
     */
    private $_currentTransport;

    // needed as __construct is called from elsewhere explicitly
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Check if this Transport mechanism is alive.
     *
     * If a Transport mechanism session is no longer functional, the method
     * returns FALSE. It is the responsibility of the developer to handle this
     * case and restart the Transport mechanism manually.
     *
     * @example
     *
     *   if (!$transport->ping()) {
     *      $transport->stop();
     *      $transport->start();
     *   }
     *
     * The Transport mechanism will be started, if it is not already.
     *
     * It is undefined if the Transport mechanism attempts to restart as long as
     * the return value reflects whether the mechanism is now functional.
     *
     * @return bool TRUE if the transport is alive
     */
    public function ping()
    {
        $maxTransports = count($this->transports);

The property transports does not seem to exist. Did you mean _deadTransports?

        for ($i = 0; $i < $maxTransports
            && $transport = $this->getNextTransport(); ++$i) {

The method getNextTransport() does not exist on Swift_Transport_FailoverTransport. Did you maybe mean _getNextTransport()?

            if ($transport->ping()) {
                return true;
            } else {
                $this->killCurrentTransport();

The method killCurrentTransport() does not exist on Swift_Transport_FailoverTransport. Did you maybe mean _killCurrentTransport()?

            }
        }

        return count($this->transports) > 0;

The property transports does not seem to exist. Did you mean _deadTransports?

    }

    /**
     * Send the given Message.
     *
     * Recipient/sender data will be retrieved from the Message API.
     * The return value is the number of recipients who were accepted for delivery.
     *
     * @param Swift_Mime_Message $message
     * @param string[]           $failedRecipients An array of failures by-reference
     *
     * @return int
     * @throws Swift_TransportException
     */
    public function send(Swift_Mime_Message $message, &$failedRecipients = null)

This method seems to be duplicated in your project.

    {
        $maxTransports = count($this->_transports);
        $sent = 0;
        $this->_lastUsedTransport = null;

        for ($i = 0; $i < $maxTransports
                     && $transport = $this->_getNextTransport(); ++$i) {
            try {
                if (!$transport->isStarted()) {
                    $transport->start();
                }

                $sent = $transport->send($message, $failedRecipients);
                if ($sent) {
                    $this->_lastUsedTransport = $transport;

                    return $sent;
                }

            } catch (Swift_TransportException $e) {
                $this->_killCurrentTransport();
            }
        }

        if (count($this->_transports) === 0) {
            throw new Swift_TransportException(
                'All Transports in FailoverTransport failed, or no Transports available'
            );
        }

        return $sent;
    }

    protected function _getNextTransport()
    {
        if (!isset($this->_currentTransport)) {
            $this->_currentTransport = parent::_getNextTransport();
        }

        return $this->_currentTransport;
    }

    protected function _killCurrentTransport()
    {
        $this->_currentTransport = null;
        parent::_killCurrentTransport();
    }
}