AppBundle\Controller\Api\FormRequestController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	81
Duplicated Lines	9.88 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	listAction()	0	14	2
B	addAction()	8	48	5

<?php

namespace AppBundle\Controller\Api;

use AppBundle\Entity\FormRequest;
use Mcfedr\JsonFormBundle\Controller\JsonController;
use AppBundle\Exception\JsonHttpException;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Routing\Annotation\Route;
use Symfony\Component\Validator\ConstraintViolation;

/**
 * @Route("/form-request", name="form_requests")
 */
class FormRequestController extends JsonController
{
    /**
     * @Route("/{status}", requirements={"status": "pending|past"})
     * @Method("GET")
     *
     * @return JsonResponse
     */
    public function listAction(Request $request, $status)
    {
        $requestForms = $this->get('knp_paginator')
            ->paginate(
                $this->getDoctrine()->getManager()->getRepository(FormRequest::class)->findBy([
                    'user' => $this->getUser(),
                    'status' => $status == 'pending' ? 'pending' : ['approved', 'rejected'],
                ]),
                $request->query->getInt('page', 1),
                10
            );

        return $this->json(['requestForms' => $requestForms]);
    }

    /**
     * @Route("/{type}", requirements={"type"= "sick-day|personal-day|overnight-guest"})
     * @Method("POST")
     *
     * @param $type
     * @param Request $request
     *
     * @return JsonResponse
     */
    public function addAction($type, Request $request)
    {
        if (!$request->getContent()) {
            throw new JsonHttpException(404, 'Request body is empty');
        }
        $em = $this->getDoctrine()->getManager();

        $formRequest = new FormRequest();
        $formRequest
            ->setType(str_replace('-', ' ', $type))
            ->setUser($this->getUser());

$this->getUser() is of type null|object, but the function expects a object<AppBundle\Entity\User>.

        $formRequest = $this->get('serializer')->deserialize(
            $request->getContent(),
            FormRequest::class,
            'json',
            ['object_to_populate' => $formRequest]
        );

        if (is_null($formRequest->getDate())) {
            throw new JsonHttpException(404, 'Date is required');
        }

        $errors = $this->get('validator')->validate($formRequest);
        if ($errors->count()) {

This code seems to be duplicated across your project.

            $outErrors = [];
            /** @var ConstraintViolation $error */
            foreach ($errors as $error) {
                $outErrors[$error->getPropertyPath()] = $error->getMessage();
            }
            throw new JsonHttpException(400, 'Bad Request', $outErrors);
        }
        $em->persist($formRequest);
        $em->flush();

        $this->get('app.email_notification')->sendNotification(
            $formRequest->getUser()->getEmail(),
            'Form request',
            'Hello, '.$formRequest->getUser()->getFirstName().'. Your form request was created.'
        );

        return $this->json([
            'success' => [
                'code' => 200,
                'message' => 'Request form created.',
            ],
        ], 200);
    }
}