JhUser\Controller\RoleController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
B	setRoleAction()	0	26	4

<?php

namespace JhUser\Controller;

use JhUser\Entity\HierarchicalRole;
use JhUser\Entity\Permission;
use Zend\Mvc\Controller\AbstractActionController;
use Zend\Console\Request as ConsoleRequest;
use Doctrine\Common\Persistence\ObjectManager;
use JhUser\Repository\UserRepositoryInterface;
use JhUser\Repository\RoleRepositoryInterface;

/**
 * Class RoleController
 * @package JhUser\Controller
 * @author Aydin Hassan <aydin@hotmail.co.uk>
 */
class RoleController extends AbstractActionController
{

    /**
     * @var \ObjectManager
     */
    protected $objectManager;

    /**
     * @var \JhUser\Repository\UserRepositoryInterface
     */
    protected $userRepository;

    /**
     * @var \JhUser\Repository\RoleRepositoryInterface
     */
    protected $roleRepository;

    /**
     * @param ObjectManager $objectManager
     * @param UserRepositoryInterface $userRepository
     * @param RoleRepositoryInterface $roleRepository
     */
    public function __construct(
        ObjectManager $objectManager,
        UserRepositoryInterface $userRepository,
        RoleRepositoryInterface $roleRepository
    ) {
        $this->objectManager    = $objectManager;

It seems like $objectManager of type object<Doctrine\Common\Persistence\ObjectManager> is incompatible with the declared type object<ObjectManager> of property $objectManager.

        $this->userRepository   = $userRepository;
        $this->roleRepository   = $roleRepository;
    }

    /**
     * Set a Users role,
     * Removes all existing roles
     *
     * @throws \RuntimeException
     */
    public function setRoleAction()
    {
        $request = $this->getRequest();

        $email  = $request->getParam('userEmail');
        $roleId = $request->getParam('role');

        $user   = $this->userRepository->findOneByEmail($email);

        if (!$user) {
            throw new \RuntimeException(sprintf('User with email: "%s" could not be found', $email));
        }

        $newRole = $this->roleRepository->findByName($roleId);

        if (!$newRole) {
            throw new \RuntimeException(sprintf('Role: "%s" could not be found', $roleId));
        }

        foreach ($user->getRoles() as $role) {
            $user->removeRole($role);
        }

        $user->addRole($newRole);
        $this->objectManager->flush();
    }
}