OCA\Circles\Events\UserEvents

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	onUserCreated()	0	3	1
A	onUserDeleted()	0	4	1

<?php


namespace OCA\Circles\Events;


use OCA\Circles\Service\MembersService;
use OCA\Circles\Service\MiscService;

class UserEvents {

	/** @var MembersService */
	private $membersService;

	/** @var MiscService */
	private $miscService;

	public function __construct(MembersService $membersService, MiscService $miscService) {
		$this->membersService = $membersService;
		$this->miscService = $miscService;
	}

	/**
	 * @param array $params
	 */
	public function onUserCreated(array $params) {

The parameter $params is not used and could be removed.

		//$userId = $params['uid'];

67% of this comment could be valid code. Did you maybe forget this after debugging?

	}


	/**
	 * @param array $params
	 */
	public function onUserDeleted(array $params) {
		$userId = $params['uid'];
		$this->membersService->removeUser($userId);
	}

}