Aimeos\MW\Mail\Zend2

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__clone()	0	4	1
A	createMessage()	0	3	1
A	send()	0	3	1
A	__construct()	0	3	1

<?php

/**
 * @license LGPLv3, http://opensource.org/licenses/LGPL-3.0
 * @copyright Aimeos (aimeos.org), 2014-2018
 * @package MW
 * @subpackage Mail
 */


namespace Aimeos\MW\Mail;


/**
 * Zend implementation for creating and sending e-mails.
 *
 * @package MW
 * @subpackage Mail
 */
class Zend2 implements \Aimeos\MW\Mail\Iface
{
	private $transport;


	/**
	 * Initializes the instance of the class.
	 *
	 * @param \Zend\Mail\Transport\TransportInterface $transport Mail transport object
	 */
	public function __construct( \Zend\Mail\Transport\TransportInterface $transport )
	{
		$this->transport = $transport;
	}


	/**
	 * Creates a new e-mail message object.
	 *
	 * @param string $charset Default charset of the message
	 * @return \Aimeos\MW\Mail\Message\Iface E-mail message object
	 */
	public function createMessage( $charset = 'UTF-8' )
	{
		return new \Aimeos\MW\Mail\Message\Zend2( new \Zend\Mail\Message(), $charset );
	}


	/**
	 * Sends the e-mail message to the mail server.
	 *
	 * @param \Aimeos\MW\Mail\Message\Iface $message E-mail message object
	 */
	public function send( \Aimeos\MW\Mail\Message\Iface $message )
	{
		$this->transport->send( $message->getObject() );

The method getObject() does not exist on Aimeos\MW\Mail\Message\Iface. It seems like you code against a sub-type of Aimeos\MW\Mail\Message\Iface such as Aimeos\MW\Mail\Message\Zend2.

	}


	/**
	 * Clones the internal objects.
	 */
	public function __clone()
	{
		$this->object = clone $this->object;

The property object does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

		$this->transport = clone $this->transport;
	}
}