OC\Accounts\Hooks

Complexity

Total Complexity

14

Size/Duplication

Total Lines	71
Duplicated Lines	16.9 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
C	changeUserHook()	12	32	11
A	getAccountManager()	0	9	2

<?php
/**
 * @copyright Copyright (c) 2016 Bjoern Schiessle <bjoern@schiessle.org>
 *
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * 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
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */


namespace OC\Accounts;

use OCP\ILogger;
use OCP\IUser;

class Hooks {

	/** @var  AccountManager */
	private $accountManager = null;

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

	/**
	 * Hooks constructor.
	 *
	 * @param ILogger $logger
	 */
	public function __construct(ILogger $logger) {
		$this->logger = $logger;
	}

	/**
	 * update accounts table if email address or display name was changed from outside
	 *
	 * @param array $params
	 */
	public function changeUserHook($params) {

		$accountManager = $this->getAccountManager();

		/** @var IUser $user */
		$user = isset($params['user']) ? $params['user'] : null;
		$feature = isset($params['feature']) ? $params['feature'] : null;
		$newValue = isset($params['value']) ? $params['value'] : null;

		if (is_null($user) || is_null($feature) || is_null($newValue)) {
			$this->logger->warning('Missing expected parameters in change user hook');
			return;
		}

		$accountData = $accountManager->getUser($user);

		switch ($feature) {
			case 'eMailAddress':

This code seems to be duplicated across your project.

				if ($accountData[AccountManager::PROPERTY_EMAIL]['value'] !== $newValue) {
					$accountData[AccountManager::PROPERTY_EMAIL]['value'] = $newValue;
					$accountManager->updateUser($user, $accountData);
				}
				break;
			case 'displayName':

This code seems to be duplicated across your project.

				if ($accountData[AccountManager::PROPERTY_DISPLAYNAME]['value'] !== $newValue) {
					$accountData[AccountManager::PROPERTY_DISPLAYNAME]['value'] = $newValue;
					$accountManager->updateUser($user, $accountData);
				}
				break;
		}

	}

	/**
	 * return instance of accountManager
	 *
	 * @return AccountManager
	 */
	protected function getAccountManager() {
		if (is_null($this->accountManager)) {
			$this->accountManager = new AccountManager(
				\OC::$server->getDatabaseConnection(),
				\OC::$server->getEventDispatcher()
			);
		}
		return $this->accountManager;
	}

}