OpauthMemberLoginFormExtension

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	forgotPassword()	0	20	4

<?php
class OpauthMemberLoginFormExtension extends Extension {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

	/**
	 * @config
	 * @var boolean
	 */
	private static $allow_password_reset = true;

The property $allow_password_reset is not used and could be removed.

	/**
	 * Deny password resets
	 * 
	 * @param Member $member
	 * @return boolean
	 */
	public function forgotPassword($member) {
		if(Config::inst()->get('OpauthMemberLoginFormExtension', 'allow_password_reset')) {
			return null;
		}

		$identity = OpauthIdentity::get()->find('MemberID', $member->ID);
		if(!$member->Password && $identity) {
			$this->owner->sessionMessage(
				_t(
					'OpauthMemberLoginFormExtension.NoResetPassword',
					'Can\'t reset password for accounts registered through {provider}',
					array('provider' => $identity->Provider)

array('provider' => $identity->Provider) is of type array<string,?,{"provider":"?"}>, but the function expects a string.

				),
				'bad'
			);
			return false;
		} else {
			return null;
		}
	}

}