ControleOnline\Controller\GetActionByPeopleAction

Complexity

Total Complexity

7

Size/Duplication

Total Lines	125
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	45	4
A	getMenuByPeople()	0	40	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\Security;

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

class GetActionByPeopleAction
{
  /**
   * Entity Manager
   *
   * @var EntityManagerInterface
   */
  private $manager = null;

  /**
   * Request
   *
   * @var Request
   */
  private $request  = null;

The private property $request is not used, and could be removed.

  /**
   * Security
   *
   * @var Security
   */
  private $security = null;

  /**
   * @var \ControleOnline\Repository\MenuRepository

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

   */
  private $repository = null;

The private property $repository is not used, and could be removed.

  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->getUser();
      /**
       * @var People
       */
      $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();

    // build query

    $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 = [];

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

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

    // execute query

    $statement = $connection->prepare($sql);
    $statement->execute($params);

    $result = $statement->fetchAll();

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

    return $return;
  }
}