WebinoMailLib\Factory

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	13	1
A	resolveTransportOptions()	0	19	4

<?php

namespace WebinoMailLib;

use WebinoConfigLib\Feature\FileMailer;
use Zend\Mail\Transport\Factory as TransportFactory;

/**
 * Class Factory
 */
final class Factory
{
    /**
     * Create a mailer
     *
     * @param array|\Traversable $options
     * @return MailerInterface
     */
    public function create($options = null)
    {

        $transport = TransportFactory::create($this->resolveTransportOptions($options));

It seems like $this->resolveTransportOptions($options) targeting WebinoMailLib\Factory::resolveTransportOptions() can also be of type object<Traversable>; however, Zend\Mail\Transport\Factory::create() does only seem to accept array, maybe add an additional type check?

        $mailer = new Mailer;

        $mailer->setTransport($transport);

        // TODO config message

        return $mailer;
    }

    /**
     * @param array|\Traversable $options
     * @return array|\Traversable
     */
    private function resolveTransportOptions($options = null)
    {
        // TODO optional
        $defaultTransport = FileMailer::class;

        // TODO constants
        if (null !== $options && isset($options['transports'])) {
            /** @var \WebinoAppLib\Application\Config $transports */
            $transports = $options['transports'];

            if (!empty($transports[$defaultTransport])) {
                return $transports[$defaultTransport];
            } else {
                return current($transports->toArray());
            }
        }

        return [];
    }
}