SmtpTransport::getSwiftTransportInstance() - Code Metrics - Inspection of "Fix code style" - 2amigos/mailer-library - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#21)

unknown

created 2024-01-20 08:07 UTC

SmtpTransport::getSwiftTransportInstance() A

↳ Parent: SmtpTransport

Complexity

Conditions	3
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	11
ccs	8
cts	8
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	6
nc	2
nop	0
crap	3

<?php

namespace Da\Mailer\Transport;

use Symfony\Component\Mailer\Transport\Dsn;
use Symfony\Component\Mailer\Transport\Smtp\EsmtpTransport;
use Symfony\Component\Mailer\Transport\Smtp\EsmtpTransportFactory;

class SmtpTransport implements TransportInterface
{
    /**
     * @var EsmtpTransport
     */
    private $instance;
/**
     * @var string the mail server host name or ip
     */
    private $host;
/**
     * @var int the mail server port
     */
    private $port;
/**
     * @var array the extra options for the Smtp transport -ie username, password, encryption, authMode
     */
    private $options = [];
/**
     * SmtpTransport constructor.
     *
     * @param string $host the mail server name or ip address
     * @param int $port the mail server port
     * @param array $options the extra options
     */
    public function __construct($host = 'localhost', $port = 25, $options = [])
    {
        $this->host = $host;
        $this->port = $port;
        $this->options = $options;
    }

    /**
     * @return EsmtpTransport
     */
    public function getInstance(): EsmtpTransport
    {
        if ($this->instance === null) {
            $user = $this->options['username'] ?? null;
            $password = $this->options['password'] ?? null;
            $this->instance = (new EsmtpTransportFactory())->create(new Dsn('smtp', $this->host, $user, $password, $this->port, $this->options));
        }

        return $this->instance;
    }

    private function getScheme()

    {
        return $this->options['tls']
            ? 'smtps'
            : 'smtp';
    }
}


1		<?php
2
3		namespace Da\Mailer\Transport;
4
5		use Symfony\Component\Mailer\Transport\Dsn;
6		use Symfony\Component\Mailer\Transport\Smtp\EsmtpTransport;
7		use Symfony\Component\Mailer\Transport\Smtp\EsmtpTransportFactory;
8
9		class SmtpTransport implements TransportInterface
10		{
11		/**
12		* @var EsmtpTransport
13		*/
14		private $instance;
15		/**
16		* @var string the mail server host name or ip
17		*/
18		private $host;
19		/**
20		* @var int the mail server port
21		*/
22		private $port;
23		/**
24		* @var array the extra options for the Smtp transport -ie username, password, encryption, authMode
25		*/
26		private $options = [];
27		/**
28		* SmtpTransport constructor.
29		*
30		* @param string $host the mail server name or ip address
31		* @param int $port the mail server port
32	2	* @param array $options the extra options
33		*/
34	2	public function __construct($host = 'localhost', $port = 25, $options = [])
35	2	{
36	2	$this->host = $host;
37	2	$this->port = $port;
38		$this->options = $options;
39		}
40
41		/**
42		* @return EsmtpTransport
43		*/
44	1	public function getInstance(): EsmtpTransport
45		{
46	1	if ($this->instance === null) {
47	1	$user = $this->options['username'] ?? null;
48	1	$password = $this->options['password'] ?? null;
49	1	$this->instance = (new EsmtpTransportFactory())->create(new Dsn('smtp', $this->host, $user, $password, $this->port, $this->options));
50	1	}
51	1
52		return $this->instance;
53	1	}
54
55		private function getScheme()
		0 ignored issues – show Unused Code introduced 2024-01-20 08:33 UTC by Report Bug Copy Issue Report The method `getScheme()` is not used, and could be removed. This check looks for private methods that have been defined, but are not used inside the class. Loading history...
56		{
57		return $this->options['tls']
58		? 'smtps'
59		: 'smtp';
60		}
61		}
62

2amigos / mailer-library

Pull Request — master (#21)

SmtpTransport::getSwiftTransportInstance() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like