OCA\Mail\Service\AutoConfig\AutoConfig

Complexity

Total Complexity

19

Size/Duplication

Total Lines	167
Duplicated Lines	9.58 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
C	createAutoDetected()	16	80	16
A	detectImapAndSmtp()	0	10	2
A	__construct()	0	16	1

<?php

/**
 * @author Bernhard Scheirle <bernhard+git@scheirle.de>
 * @author Christoph Wurst <christoph@winzerhof-wurst.at>
 *
 * Mail
 *
 * This code is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License, version 3,
 * as published by the Free Software Foundation.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License, version 3,
 * along with this program.  If not, see <http://www.gnu.org/licenses/>
 *
 */
namespace OCA\Mail\Service\AutoConfig;

use Horde_Mail_Transport_Smtphorde;
use OCP\Security\ICrypto;
use OCA\Mail\Account;
use OCA\Mail\Db\MailAccount;
use OCA\Mail\Service\Logger;

class AutoConfig {

	/** @var Logger */
	private $logger;

	/** @var string */
	private $userId;

	/** @var ICrypto */
	private $crypto;

	/** @var IspDb */
	private $ispDb;

	/** @var MxRecord */
	private $mxRecord;

	/** @var ImapConnectivityTester */
	private $imapConnectivityTester;

	/** @var ImapServerDetector */
	private $imapServerDetector;

	/** @var ImapConnector */
	private $imapConnector;

	/** @var SmtpConnectivityTester */
	private $smtpConnectivityTester;

	/** @var SmtpServerDetector */
	private $smtpServerDetector;

	/**
	 * 
	 * @param Logger $logger
	 * @param string $UserId
	 * @param IspDb $ispDb
	 * @param MxRecord $mxRecord
	 * @param ImapConnectivityTester $imapTester
	 * @param ImapServerDetector $imapDetector
	 * @param SmtpConnectivityTester $smtpTester
	 * @param SmtpServerDetector $smtpDetector
	 * @param ImapConnector $imapConnector
	 * @param ICrypto $crypto
	 */
	public function __construct(Logger $logger, $UserId,
		IspDb $ispDb, MxRecord $mxRecord,
		ImapConnectivityTester $imapTester, ImapServerDetector $imapDetector,
		SmtpConnectivityTester $smtpTester, SmtpServerDetector $smtpDetector,
		ImapConnector $imapConnector, ICrypto $crypto) {
		$this->logger = $logger;
		$this->userId = $UserId;
		$this->crypto = $crypto;
		$this->ispDb = $ispDb;
		$this->mxRecord = $mxRecord;
		$this->imapConnectivityTester = $imapTester;
		$this->imapServerDetector = $imapDetector;
		$this->imapConnector = $imapConnector;
		$this->smtpConnectivityTester = $smtpTester;
		$this->smtpServerDetector = $smtpDetector;
	}

	/**
	 * @param string $email
	 * @param string $password
	 * @param string $name
	 * @return null|MailAccount
	 */
	public function createAutoDetected($email, $password, $name) {

		// splitting the email address into user and host part
		// TODO: use horde libs for email address parsing
		list(, $host) = explode("@", $email);

		$ispdb = $this->ispDb->query($host);
		if (!empty($ispdb)) {
			$account = null;
			if (isset($ispdb['imap'])) {
				foreach ($ispdb['imap'] as $imap) {
					$host = $imap['hostname'];
					$port = $imap['port'];
					$encryptionProtocol = null;
					if ($imap['socketType'] === 'SSL') {
						$encryptionProtocol = 'ssl';
					}
					if ($imap['socketType'] === 'STARTTLS') {
						$encryptionProtocol = 'tls';
					}
					if ($imap['username'] === '%EMAILADDRESS%') {

This code seems to be duplicated across your project.

						$user = $email;
					} elseif ($imap['username'] === '%EMAILLOCALPART%') {
						list($user, ) = explode("@", $email);
					} else {
						$this->logger->info("Unknown username variable: " . $imap['username']);
						return null;
					}
					try {
						$account = $this->imapConnector->connect($email, $password, $name, $host,
							$port, $encryptionProtocol, $user);
						break;
					} catch (\Horde_Imap_Client_Exception $e) {

The class Horde_Imap_Client_Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

						$error = $e->getMessage();
						$this->logger->info("Test-Account-Failed: $this->userId, $host, $port, $user, $encryptionProtocol -> $error");
					}
				}
			}
			if (!is_null($account)) {
				foreach ($ispdb['smtp'] as $smtp) {
					try {
						if ($smtp['username'] === '%EMAILADDRESS%') {

This code seems to be duplicated across your project.

							$user = $email;
						} elseif ($smtp['username'] === '%EMAILLOCALPART%') {
							list($user, ) = explode("@", $email);
						} else {
							$this->logger->info("Unknown username variable: " . $smtp['username']);
							return null;
						}

						$account->setOutboundHost($smtp['hostname']);
						$account->setOutboundPort($smtp['port']);
						$password = $this->crypto->encrypt($password);
						$account->setOutboundPassword($password);
						$account->setOutboundUser($user);
						$account->setOutboundSslMode(strtolower($smtp['socketType']));

						$a = new Account($account);
						$smtp = $a->createTransport();
						if ($smtp instanceof Horde_Mail_Transport_Smtphorde) {

The class Horde_Mail_Transport_Smtphorde does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json?

							$smtp->getSMTPObject();
						}

						break;
					} catch (\PEAR_Exception $ex) {

The class PEAR_Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

						$error = $ex->getMessage();
						$this->logger->info("Test-Account-Failed(smtp): $error");
					}
				}
				return $account;
			}
		}

		$account = $this->detectImapAndSmtp($email, $password, $name);
		if (!is_null($account)) {
			return $account;
		}

		return null;
	}

	/**
	 * @param $email
	 * @param $password
	 * @param $name
	 * @return null|MailAccount
	 */
	private function detectImapAndSmtp($email, $password, $name) {
		$account = $this->imapServerDetector->detect($email, $password, $name);
		if (is_null($account)) {
			return null;
		}

		$this->smtpServerDetector->detect($account, $email, $password);

		return $account;
	}

}