LoginCidadao\SupportBundle\Controller\PersonSupportController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	indexAction()	0	33	5
A	validateSupportTicketId()	0	10	3
A	viewAction()	0	17	1
A	getPersonGrid()	0	12	1

<?php
/**
 * This file is part of the login-cidadao project or it's bundles.
 *
 * (c) Guilherme Donato <guilhermednt on github>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace LoginCidadao\SupportBundle\Controller;

use Doctrine\ORM\NonUniqueResultException;
use Doctrine\ORM\QueryBuilder;
use LoginCidadao\APIBundle\Security\Audit\ActionLogger;
use LoginCidadao\CoreBundle\Entity\PersonRepository;
use LoginCidadao\CoreBundle\Entity\SentEmail;
use LoginCidadao\CoreBundle\Helper\GridHelper;
use LoginCidadao\CoreBundle\Model\PersonInterface;
use LoginCidadao\SupportBundle\Form\PersonSearchFormType;
use LoginCidadao\SupportBundle\Model\PersonSearchRequest;
use LoginCidadao\SupportBundle\Service\SupportHandler;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Security;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;

/**
 * Class PersonSupportController
 * @package LoginCidadao\SupportBundle\Controller
 *
 * @Security("has_role('ROLE_SUPPORT_AGENT')")
 * @codeCoverageIgnore
 */
class PersonSupportController extends Controller
{
    /**
     * @Route("/support/search", name="lc_support_person_search")
     */
    public function indexAction(Request $request)
    {
        $gridView = null;
        $search = new PersonSearchRequest();

        $search->smartSearch = $request->get('search', null);

        $form = $this->createForm(PersonSearchFormType::class, $search);
        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid()) {
            /** @var PersonRepository $repo */
            $repo = $this->getDoctrine()->getRepository('LoginCidadaoCoreBundle:Person');
            $query = $repo->getSmartSearchQuery($search->smartSearch);
            try {
                $person = $query->getQuery()->getOneOrNullResult();

                if ($person instanceof PersonInterface) {
                    return $this->redirectToRoute('lc_support_person_view', [
                        'id' => $person->getId(),
                        'ticket' => $search->supportTicket,
                    ]);
                }
            } catch (NonUniqueResultException $e) {
                $grid = $this->getPersonGrid($query, $form);
                $gridView = $grid->createView($request);
            }
        }

        return $this->render('LoginCidadaoSupportBundle:PersonSupport:index.html.twig', [
            'form' => $form->createView(),
            'grid' => $gridView,
            'search' => $search,
        ]);
    }

    /**
     * @Route("/support/person/{id}", name="lc_support_person_view")
     */
    public function viewAction(Request $request, $id)
    {
        $supportRequest = $this->validateSupportTicketId($request->get('ticket'));

        /** @var SupportHandler $supportHandler */
        $supportHandler = $this->get(SupportHandler::class);

        $person = $supportHandler->getSupportPerson($id);

        /** @var ActionLogger $actionLogger */
        $actionLogger = $this->get('lc.action_logger');
        $actionLogger->registerProfileView($request, $person, $this->getUser(), [$this, 'viewAction']);

It seems like $this->getUser() can also be of type null; however, parameter $viewer of LoginCidadao\APIBundle\Security\Audit\ActionLogger::registerProfileView() does only seem to accept LoginCidadao\CoreBundle\Model\IdentifiablePersonInterface, maybe add an additional type check?

        return $this->render('LoginCidadaoSupportBundle:PersonSupport:view.html.twig', [
            'person' => $person,
            'supportRequest' => $supportRequest,
            'dataValidation' => $supportHandler->getValidationMap($person),
        ]);
    }

    private function validateSupportTicketId(string $ticket = null): SentEmail
    {
        /** @var SupportHandler $supportHandler */
        $supportHandler = $this->get(SupportHandler::class);
        $sentEmail = $ticket ? $supportHandler->getInitialMessage($ticket) : null;
        if (!$sentEmail instanceof SentEmail) {
            throw $this->createNotFoundException("Invalid Support Ticket ID");
        }

        return $sentEmail;
    }

    private function getPersonGrid(QueryBuilder $query, $form): GridHelper
    {
        $grid = new GridHelper();
        $grid->setId('person-grid');
        $grid->setPerPage(5);
        $grid->setMaxResult(5);
        $grid->setInfiniteGrid(true);
        $grid->setRoute('lc_support_person_search');
        $grid->setRouteParams([$form->getName()]);
        $grid->setQueryBuilder($query);

The function LoginCidadao\CoreBundle\Helper\GridHelper::setQueryBuilder() has been deprecated: since version 1.1.0

        return $grid;
    }
}