MailInABoxChangePasswordDriver::ChangePassword() - Code Metrics - Inspection of "Merge pull request #1491 from mariogrip/mailinabox..." - RainLoop/rainloop-webmail - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 59c197...ef601c )

by Rain

created 2018-02-20 23:04 UTC

MailInABoxChangePasswordDriver::ChangePassword() C

↳ Parent: MailInABoxChangePasswordDriver

Complexity

Conditions	10
Paths	9

Size

Total Lines	58
Code Lines	43

Duplication

Lines	4
Ratio	6.9 %

Importance

Changes

Metric	Value
cc	10
eloc	43
nc	9
nop	3
dl	4
loc	58
rs	6.6515
c	0
b	0
f	0

How to fix Long Method Complexity

<?php
/*
 * Mail-in-a-box Password Change Plugin
 *
 * Based on VirtualminChangePasswordDriver
 *
 * Author: Marius Gripsgard
 */
class MailInABoxChangePasswordDriver implements \RainLoop\Providers\ChangePassword\ChangePasswordInterface
namespace YourVendor;

class YourClass { }
{
	/**
	 * @var string
	 */
	private $sAllowedEmails = '';
	/**
	 * @var string
	 */
	private $sHost = '';
	/**
	 * @var string
	 */
	private $sAdminUser = '';
	/**
	 * @var string
	 */
	private $sAdminPassword = '';
	/**
	 * @var \MailSo\Log\Logger
	 */
	private $oLogger = null;
	/**
	 * @param string $sHost
	 * @param string $sAdminUser
	 * @param string $sAdminPassword
	 *
	 * @return \MailInABoxChangePasswordDriver
	 */
	public function SetConfig($sHost, $sAdminUser, $sAdminPassword)
	{
		$this->sHost = $sHost;
		$this->sAdminUser = $sAdminUser;
		$this->sAdminPassword = $sAdminPassword;
		return $this;
	}
	/**
	 * @param string $sAllowedEmails
	 *
	 * @return \MailInABoxChangePasswordDriver
	 */
	public function SetAllowedEmails($sAllowedEmails)
	{
		$this->sAllowedEmails = $sAllowedEmails;
		return $this;
	}
	/**
	 * @param \MailSo\Log\Logger $oLogger
	 *
	 * @return \MailInABoxChangePasswordDriver
	 */
	public function SetLogger($oLogger)
	{
		if ($oLogger instanceof \MailSo\Log\Logger)
if ($x instanceof DoesNotExist) {
    // Do something.
}
		{
			$this->oLogger = $oLogger;
		}
		return $this;
	}
	/**
	 * @param string $sDesc
	 * @param int $iType = \MailSo\Log\Enumerations\Type::INFO
	 *
	 * @return \MailInABoxChangePasswordDriver
	 */
	public function WriteLog($sDesc, $iType = \MailSo\Log\Enumerations\Type::INFO)
	{
		if ($this->oLogger)
		{
			$this->oLogger->Write($sDesc, $iType);
		}
		return $this;
	}
	/**
	 * @param \RainLoop\Model\Account $oAccount
	 *
	 * @return bool
	 */
	public function PasswordChangePossibility($oAccount)
	{
		return $oAccount && $oAccount->Email() &&
			\RainLoop\Plugins\Helper::ValidateWildcardValues($oAccount->Email(), $this->sAllowedEmails);
	}
	/**
	 * @param \RainLoop\Model\Account $oAccount
	 * @param string $sPrevPassword
	 * @param string $sNewPassword
	 *
	 * @return bool
	 */
	public function ChangePassword(\RainLoop\Account $oAccount, $sPrevPassword, $sNewPassword)
	{
		$this->WriteLog('Mail-in-a-box: Try to change password for '.$oAccount->Email());
		$bResult = false;
		if (!empty($this->sHost) && !empty($this->sAdminUser) && !empty($this->sAdminPassword) && $oAccount)
		{
			$this->WriteLog('Mail-in-a-box:[Check] Required Fields Present');
			$sEmail = \trim(\strtolower($oAccount->Email()));
			$sHost = \rtrim(\trim($this->sHost), '/');
			$sUrl = $sHost.'/admin/mail/users/password';

			$sAdminUser = $this->sAdminUser;
			$sAdminPassword = $this->sAdminPassword;
			$iCode = 0;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
			$aPost = array(
				'email'		=> $sEmail,
				'password'		=> $sNewPassword,
			);
			$aOptions = array(
				CURLOPT_URL => $sUrl,
				CURLOPT_HEADER => false,
				CURLOPT_FAILONERROR => true,
				CURLOPT_SSL_VERIFYPEER => false,
				CURLOPT_RETURNTRANSFER => true,
				CURLOPT_POST => true,
				CURLOPT_POSTFIELDS => \http_build_query($aPost, '', '&'),
				CURLOPT_TIMEOUT => 20,
				CURLOPT_SSL_VERIFYHOST => false,
				CURLOPT_USERPWD => $sAdminUser.':'.$sAdminPassword,
				CURLOPT_HTTPAUTH => CURLAUTH_BASIC
			);
			$oCurl = \curl_init();
			\curl_setopt_array($oCurl, $aOptions);
			$this->WriteLog('Mail-in-a-box: Send post request: '.$sUrl);
			$mResult = \curl_exec($oCurl);
			$iCode = (int) \curl_getinfo($oCurl, CURLINFO_HTTP_CODE);
			$sContentType = (string) \curl_getinfo($oCurl, CURLINFO_CONTENT_TYPE);
			$this->WriteLog('Mail-in-a-box: Post request result: (Status: '.$iCode.', ContentType: '.$sContentType.')');
			if (false === $mResult || 200 !== $iCode)

			{
				$this->WriteLog('Mail-in-a-box: Error: '.\curl_error($oCurl), \MailSo\Log\Enumerations\Type::WARNING);
			}
			if (\is_resource($oCurl))
			{
				\curl_close($oCurl);
			}
			if (false !== $mResult && 200 === $iCode)
			{
				$this->WriteLog('Mail-in-a-box: Password Change Status: Success');
				$bResult = true;
			}
			else
			{
				$this->WriteLog('Mail-in-a-box[Error]: Empty Response: Code: '.$iCode);
			}
		}
		return $bResult;
	}
}


Push — master ( 59c197...ef601c )

MailInABoxChangePasswordDriver::ChangePassword() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Missing dependencies

2. Missing use statement

1		<?php
2		/*
3		* Mail-in-a-box Password Change Plugin
4		*
5		* Based on VirtualminChangePasswordDriver
6		*
7		* Author: Marius Gripsgard
8		*/
9		class MailInABoxChangePasswordDriver implements \RainLoop\Providers\ChangePassword\ChangePasswordInterface
		0 ignored issues – show Coding Style Compatibility introduced 2018-02-20 23:24 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
10		{
11		/**
12		* @var string
13		*/
14		private $sAllowedEmails = '';
15		/**
16		* @var string
17		*/
18		private $sHost = '';
19		/**
20		* @var string
21		*/
22		private $sAdminUser = '';
23		/**
24		* @var string
25		*/
26		private $sAdminPassword = '';
27		/**
28		* @var \MailSo\Log\Logger
29		*/
30		private $oLogger = null;
31		/**
32		* @param string $sHost
33		* @param string $sAdminUser
34		* @param string $sAdminPassword
35		*
36		* @return \MailInABoxChangePasswordDriver
37		*/
38		public function SetConfig($sHost, $sAdminUser, $sAdminPassword)
39		{
40		$this->sHost = $sHost;
41		$this->sAdminUser = $sAdminUser;
42		$this->sAdminPassword = $sAdminPassword;
43		return $this;
44		}
45		/**
46		* @param string $sAllowedEmails
47		*
48		* @return \MailInABoxChangePasswordDriver
49		*/
50		public function SetAllowedEmails($sAllowedEmails)
51		{
52		$this->sAllowedEmails = $sAllowedEmails;
53		return $this;
54		}
55		/**
56		* @param \MailSo\Log\Logger $oLogger
57		*
58		* @return \MailInABoxChangePasswordDriver
59		*/
60		public function SetLogger($oLogger)
61		{
62		if ($oLogger instanceof \MailSo\Log\Logger)
		0 ignored issues – show Bug introduced 2018-02-20 23:24 UTC by Report Bug Copy Issue Report The class `MailSo\Log\Logger` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
63		{
64		$this->oLogger = $oLogger;
65		}
66		return $this;
67		}
68		/**
69		* @param string $sDesc
70		* @param int $iType = \MailSo\Log\Enumerations\Type::INFO
71		*
72		* @return \MailInABoxChangePasswordDriver
73		*/
74		public function WriteLog($sDesc, $iType = \MailSo\Log\Enumerations\Type::INFO)
75		{
76		if ($this->oLogger)
77		{
78		$this->oLogger->Write($sDesc, $iType);
79		}
80		return $this;
81		}
82		/**
83		* @param \RainLoop\Model\Account $oAccount
84		*
85		* @return bool
86		*/
87		public function PasswordChangePossibility($oAccount)
88		{
89		return $oAccount && $oAccount->Email() &&
90		\RainLoop\Plugins\Helper::ValidateWildcardValues($oAccount->Email(), $this->sAllowedEmails);
91		}
92		/**
93		* @param \RainLoop\Model\Account $oAccount
94		* @param string $sPrevPassword
95		* @param string $sNewPassword
96		*
97		* @return bool
98		*/
99		public function ChangePassword(\RainLoop\Account $oAccount, $sPrevPassword, $sNewPassword)
100		{
101		$this->WriteLog('Mail-in-a-box: Try to change password for '.$oAccount->Email());
102		$bResult = false;
103		if (!empty($this->sHost) && !empty($this->sAdminUser) && !empty($this->sAdminPassword) && $oAccount)
104		{
105		$this->WriteLog('Mail-in-a-box:[Check] Required Fields Present');
106		$sEmail = \trim(\strtolower($oAccount->Email()));
107		$sHost = \rtrim(\trim($this->sHost), '/');
108		$sUrl = $sHost.'/admin/mail/users/password';
109
110		$sAdminUser = $this->sAdminUser;
111		$sAdminPassword = $this->sAdminPassword;
112		$iCode = 0;
		0 ignored issues – show Unused Code introduced 2018-02-20 23:24 UTC by Report Bug Copy Issue Report `$iCode` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
113		$aPost = array(
114		'email' => $sEmail,
115		'password' => $sNewPassword,
116		);
117		$aOptions = array(
118		CURLOPT_URL => $sUrl,
119		CURLOPT_HEADER => false,
120		CURLOPT_FAILONERROR => true,
121		CURLOPT_SSL_VERIFYPEER => false,
122		CURLOPT_RETURNTRANSFER => true,
123		CURLOPT_POST => true,
124		CURLOPT_POSTFIELDS => \http_build_query($aPost, '', '&'),
125		CURLOPT_TIMEOUT => 20,
126		CURLOPT_SSL_VERIFYHOST => false,
127		CURLOPT_USERPWD => $sAdminUser.':'.$sAdminPassword,
128		CURLOPT_HTTPAUTH => CURLAUTH_BASIC
129		);
130		$oCurl = \curl_init();
131		\curl_setopt_array($oCurl, $aOptions);
132		$this->WriteLog('Mail-in-a-box: Send post request: '.$sUrl);
133		$mResult = \curl_exec($oCurl);
134		$iCode = (int) \curl_getinfo($oCurl, CURLINFO_HTTP_CODE);
135		$sContentType = (string) \curl_getinfo($oCurl, CURLINFO_CONTENT_TYPE);
136		$this->WriteLog('Mail-in-a-box: Post request result: (Status: '.$iCode.', ContentType: '.$sContentType.')');
137	View Code Duplication	if (false === $mResult \|\| 200 !== $iCode)
		0 ignored issues – show Duplication introduced 2018-02-20 23:24 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
138		{
139		$this->WriteLog('Mail-in-a-box: Error: '.\curl_error($oCurl), \MailSo\Log\Enumerations\Type::WARNING);
140		}
141		if (\is_resource($oCurl))
142		{
143		\curl_close($oCurl);
144		}
145		if (false !== $mResult && 200 === $iCode)
146		{
147		$this->WriteLog('Mail-in-a-box: Password Change Status: Success');
148		$bResult = true;
149		}
150		else
151		{
152		$this->WriteLog('Mail-in-a-box[Error]: Empty Response: Code: '.$iCode);
153		}
154		}
155		return $bResult;
156		}
157		}
158

RainLoop / rainloop-webmail

Push — master ( 59c197...ef601c )

MailInABoxChangePasswordDriver::ChangePassword() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like