LEClient\LEClient::__construct()

Complexity

Conditions	28
Paths	349

Size

Total Lines	95
Code Lines	53

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	34
CRAP Score	28

Importance

Changes

0

<?php
	public function __construct($email, $acmeURL = LEClient::LE_PRODUCTION, $log = LEClient::LOG_OFF, $certificateKeys = 'keys/', $accountKeys = '__account/')
	{
		$this->log = $log;

		if (is_bool($acmeURL))
		{
			if ($acmeURL === true) $this->baseURL = LEClient::LE_STAGING;

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

			elseif ($acmeURL === false) $this->baseURL = LEClient::LE_PRODUCTION;

The condition $acmeURL === false is always true.

		}
		elseif (is_string($acmeURL))

The condition is_string($acmeURL) is always true.

		{
			$this->baseURL = $acmeURL;
		}
		else throw LEClientException::InvalidArgumentException('acmeURL must be set to string or bool (legacy).');

		if (is_array($certificateKeys) && is_string($accountKeys)) throw LEClientException::InvalidArgumentException('When certificateKeys is array, accountKeys must be array too.');
		elseif (is_array($accountKeys) && is_string($certificateKeys)) throw LEClientException::InvalidArgumentException('When accountKeys is array, certificateKeys must be array too.');

		if (is_string($certificateKeys))
		{
			$certificateKeysDir = $certificateKeys;

			if(!file_exists($certificateKeys))
			{
				mkdir($certificateKeys, 0755, true);
				LEFunctions::createhtaccess($certificateKeys);
			}

			$this->certificateKeys = array(
				"public_key" => $certificateKeys.'/public.pem',
				"private_key" => $certificateKeys.'/private.pem',
				"certificate" => $certificateKeys.'/certificate.crt',
				"fullchain_certificate" => $certificateKeys.'/fullchain.crt',
				"order" => $certificateKeys.'/order'
			);
		}
		elseif (is_array($certificateKeys))

The condition is_array($certificateKeys) is always true.

		{
			if (!isset($certificateKeys['certificate']) && !isset($certificateKeys['fullchain_certificate'])) throw LEClientException::InvalidArgumentException('certificateKeys[certificate] or certificateKeys[fullchain_certificate] file path must be set.');
			if (!isset($certificateKeys['private_key'])) throw LEClientException::InvalidArgumentException('certificateKeys[private_key] file path must be set.');
			if (!isset($certificateKeys['order'])) $certificateKeys['order'] = dirname($certificateKeys['private_key']).'/order';
			if (!isset($certificateKeys['public_key'])) $certificateKeys['public_key'] = dirname($certificateKeys['private_key']).'/public.pem';

			foreach ($certificateKeys as $param => $file) {
				$parentDir = dirname($file);
				if (!is_dir($parentDir)) throw LEClientException::InvalidDirectoryException($parentDir);
			}

			$this->certificateKeys = $certificateKeys;
		}
		else
		{
			throw LEClientException::InvalidArgumentException('certificateKeys must be string or array.');
		}

		if (is_string($accountKeys))
		{
			$accountKeys = $certificateKeysDir.'/'.$accountKeys;

The variable $certificateKeysDir does not seem to be defined for all execution paths leading up to this point.

			if(!file_exists($accountKeys))
			{
				mkdir($accountKeys, 0755, true);
				LEFunctions::createhtaccess($accountKeys);
			}

			$this->accountKeys = array(
				"private_key" => $accountKeys.'/private.pem',
				"public_key" => $accountKeys.'/public.pem'
			);
		}
		elseif (is_array($accountKeys))

The condition is_array($accountKeys) is always true.

		{
			if (!isset($accountKeys['private_key'])) throw LEClientException::InvalidArgumentException('accountKeys[private_key] file path must be set.');
			if (!isset($accountKeys['public_key'])) throw LEClientException::InvalidArgumentException('accountKeys[public_key] file path must be set.');

			foreach ($accountKeys as $param => $file) {
				$parentDir = dirname($file);
				if (!is_dir($parentDir)) throw LEClientException::InvalidDirectoryException($parentDir);
			}

			$this->accountKeys = $accountKeys;
		}
		else
		{
			throw LEClientException::InvalidArgumentException('accountKeys must be string or array.');
		}

		$this->connector = new LEConnector($this->log, $this->baseURL, $this->accountKeys);
		$this->account = new LEAccount($this->connector, $this->log, $email, $this->accountKeys);
		
		if($this->log instanceof \Psr\Log\LoggerInterface) 

$this->log is never a sub-type of Psr\Log\LoggerInterface.

		{
			$this->log->info('LEClient finished constructing');
		}
		elseif($this->log >= LEClient::LOG_STATUS) LEFunctions::log('LEClient finished constructing', 'function LEClient __construct');

'LEClient finished constructing' of type string is incompatible with the type object expected by parameter $data of LEClient\LEFunctions::log().

	}