ControleOnline\Controller\GetActionByPeopleAction

Complexity

Total Complexity

7

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	36	4
A	getMenuByPeople()	0	29	2
A	__construct()	0	4	1

<?php

namespace ControleOnline\Controller;

use ControleOnline\Entity\People;

The type ControleOnline\Entity\People was not found. Maybe you did not declare it correctly or list all dependencies?

use Symfony\Component\HttpFoundation\Request;

The type Symfony\Component\HttpFoundation\Request was not found. Maybe you did not declare it correctly or list all dependencies?

use Doctrine\ORM\EntityManagerInterface;

The type Doctrine\ORM\EntityManagerInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Exception;
use Symfony\Component\HttpFoundation\JsonResponse;

The type Symfony\Component\HttpFoundation\JsonResponse was not found. Maybe you did not declare it correctly or list all dependencies?

use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface as Security;

The type Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface was not found. Maybe you did not declare it correctly or list all dependencies?

class GetActionByPeopleAction
{
  private $manager = null;
  private $security = null;

  public function __construct(Security $security, EntityManagerInterface $entityManager)
  {
    $this->manager = $entityManager;
    $this->security = $security;
  }

  public function __invoke(Request $request): JsonResponse
  {
    try {
      $menu = [];

The assignment to $menu is dead and can be removed.

      $company = $request->query->get('myCompany', null);

      if ($company === null) {
        throw new Exception("Company not found", 404);
      }

      $myCompany = $this->manager->getRepository(People::class)->find($company);

      if ($myCompany === null) {
        throw new Exception("Company not found", 404);
      }

      $currentUser = $this->security->getToken()->getUser();
      $userPeople = $currentUser->getPeople();

      $menu = $this->getMenuByPeople($userPeople, $myCompany);

      return new JsonResponse([
        'response' => [
          'data' => $menu,
          'count' => 1,
          'error' => '',
          'success' => true,
        ],
      ]);
    } catch (\Exception $e) {
      return new JsonResponse([
        'response' => [
          'data' => [],
          'count' => 0,
          'error' => $e->getMessage(),
          'success' => false,
        ],
      ]);
    }
  }

  private function getMenuByPeople(People $userPeople, People $myCompany)
  {
    $return = [];
    $connection = $this->manager->getConnection();

    $sql = 'SELECT action.*, routes.route
                FROM action 
                INNER JOIN routes ON routes.id = action.route_id
                INNER JOIN action_role ON action_role.action_id = action.id
                INNER JOIN role ON role.id = action_role.role_id
                INNER JOIN people_role ON people_role.role_id = role.id
                WHERE people_role.company_id = :myCompany 
                AND people_role.people_id = :userPeople 
                AND routes.route = :route
                GROUP BY action.id';

    $params = [
      'myCompany' => $myCompany->getId(),
      'userPeople' => $userPeople->getId(),
      'route' => $this->route,

The property route does not exist on ControleOnline\Controller\GetActionByPeopleAction. Did you maybe forget to declare it?

    ];

    $result = $connection->executeQuery($sql, $params)->fetchAllAssociative();

    foreach ($result as $action) {
      $return['routes'][trim($action['route'])]['actions'][$action['id']] = trim($action['action']);
    }

    return $return;
  }
}