AppBundle\Controller\FieldNotes\FieldNotesController

Complexity

Total Complexity

13

Size/Duplication

Total Lines	155
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	indexAction()	0	65	7
B	deleteAction()	0	25	2
B	deleteMultipleAction()	0	33	4

<?php

namespace AppBundle\Controller\FieldNotes;

use AppBundle\Controller\AbstractController;
use AppBundle\Form\UploadFieldNotesType;
use AppBundle\Util\DateUtil;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Component\HttpFoundation\File\UploadedFile;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;

/**
 * Class FieldNotesController
 *
 * @package AppBundle\Controller\FieldNotes

Unexpected tag type @package in doc block

 */
class FieldNotesController extends AbstractController
{
    /**
     * Index action for field-notes.
     *
     * @param Request $request

Request => \Symfony\Component\HttpFoundation\Request

     *
     * @return Response

Response => \Symfony\Component\HttpFoundation\Response

     *
     * @Route("/field-notes/", name="field-notes")
     */
    public function indexAction(Request $request)
    {
        $this->denyAccessUnlessGranted('ROLE_USER');
        $user = $this->getUser();

        $fieldNoteService = $this->get('app.service.field_note');
        $dataProvider = $this->get('app.dataprovider.upload_field_note');

        $repository = $this->getDoctrine()->getRepository('AppBundle:FieldNote');
        $fieldNotes = $repository->findBy([
            'user' => $user->getId()
        ], [
            'date' => 'ASC',
            'id' => 'ASC'
        ]);

        $form = $this->createForm(UploadFieldNotesType::class, $dataProvider->getData($user->getId()));
        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid()) {
            /**
             * @var UploadedFile $file

UploadedFile => \Symfony\Component\HttpFoundation\File\UploadedFile

             */
            $file = $form->getData()[UploadFieldNotesType::FIELD_FILE];

            try {
                $ignoreDate = null;

                if (!empty($form->getData()[UploadFieldNotesType::FIELD_IGNORE])) {
                    $ignoreDate = DateUtil::dateTimeFromMySqlFormat(
                        $form->getData()[UploadFieldNotesType::FIELD_IGNORE_DATE]
                    );
                }

                $fieldNoteService->importFromFile($file->getRealPath(), $user->getId(), $ignoreDate);
            } catch (\Exception $e) {
                $this->addErrorMessage($e->getMessage());

                return $this->redirectToRoute('field-notes');
            }

            if ($fieldNoteService->hasErrors()) {
                foreach ($fieldNoteService->getErrors() as $error) {
                    $this->addErrorMessage($error);
                }

                return $this->redirectToRoute('field-notes');
            }

            $this->addSuccessMessage(
                $this->get('translator')->trans('field_notes.upload.success')
            );

            return $this->redirectToRoute('field-notes');
        }

        $this->setMenu(MNU_MYPROFILE_FIELD_NOTES);
        $this->setTitle($this->get('translator')->trans('field_notes.field_notes'));

        return $this->render('field-notes/index.html.twig', [
            'user' => $user,
            'form' => $form->createView(),
            'fieldNotes' => $fieldNotes,
        ]);
    }

    /**
     * Action to delete one field-note.
     *
     * @param int $id
     *
     * @return RedirectResponse

RedirectResponse => \Symfony\Component\HttpFoundation\RedirectResponse

     *
     * @Route("/field-notes/delete/{id}", name="field-notes.delete")
     */
    public function deleteAction($id)
    {
        $this->denyAccessUnlessGranted('ROLE_USER');
        $user = $this->getUser();

        $repository = $this->getDoctrine()->getRepository('AppBundle:FieldNote');
        $fieldNote = $repository->findOneBy([
            'user' => $user->getId(),
            'id' => $id
        ]);

        if (!$fieldNote) {
            return $this->redirectToRoute('field-notes');
        }

        $em = $this->getDoctrine()->getManager();
        $em->remove($fieldNote);
        $em->flush();

        $this->addSuccessMessage(
            $this->get('translator')->trans('field_notes.success.deleted')
        );

        return $this->redirectToRoute('field-notes');
    }

    /**
     * Action to delete multiple field-notes.
     *
     * @param Request $request

Request => \Symfony\Component\HttpFoundation\Request

     *
     * @return RedirectResponse

RedirectResponse => \Symfony\Component\HttpFoundation\RedirectResponse

     *
     * @Route("/field-notes/delete-multiple/", name="field-notes.delete-multiple")
     */
    public function deleteMultipleAction(Request $request)
    {
        $this->denyAccessUnlessGranted('ROLE_USER');
        $user = $this->getUser();

        $selectedFieldNotes = $request->get('selected-field-notes');
        if (!is_array($selectedFieldNotes)) {
            return $this->redirectToRoute('field-notes');
        }

        $repository = $this->getDoctrine()->getRepository('AppBundle:FieldNote');
        $em = $this->getDoctrine()->getManager();

        foreach ($selectedFieldNotes as $fieldNoteId) {
            $fieldNote = $repository->findOneBy([

Are you sure the assignment to $fieldNote is correct as $repository->findOneBy(array('user' => $user->getId(), 'id' => $fieldNoteId)) (which targets Doctrine\Common\Persistence\ObjectRepository::findOneBy()) seems to always return null.

                'user' => $user->getId(),
                'id' => $fieldNoteId
            ]);

            if (!$fieldNote) {
                continue;
            }
            $em->remove($fieldNote);
        }

        $em->flush();

        $this->addSuccessMessage(
            $this->get('translator')->trans('field_notes.success.deleted_multiple')
        );

        return $this->redirectToRoute('field-notes');
    }
}