AppBundle\Controller\Admin\TagController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

21.43%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	roleAction()	0	10	1
B	editRoleAction()	0	32	3

<?php

namespace AppBundle\Controller\Admin;

use AppBundle\Entity\Tag;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\HttpFoundation\Request;

/**
 * Class AdminController
 * @package AppBundle\Controller\Admin
 * @Route("/admin")
 */
class TagController extends Controller
{
    /**
     * @Method("GET")
     * @Route("/tags", name="tagsAdmin")
     * @Template("AppBundle:admin:tags.html.twig")
     *
     * @return Response
     */
    public function roleAction()
    {
        $em = $this->getDoctrine()->getManager();
        $tags = $em->getRepository('AppBundle:Tag')
            ->findAll();

        return [
            'tags'  => $tags,
        ];
    }

    /**
     * @param Request $request
     * @Route("/tag/new", name="tagNew")
     * @Method({"GET", "POST"})
     * @Template("AppBundle:admin/form:tag.html.twig")
     *
     * @return Response
     */
    public function editRoleAction(Request $request)
    {
        $em = $this->getDoctrine()->getManager();

        $tag = new Tag();
        $title = 'Create new tag';

        $form = $this->createFormBuilder($tag)

It seems like you code against a specific sub-type and not the parent class Symfony\Component\Form\FormConfigBuilder as the method add() does only exist in the following sub-classes of Symfony\Component\Form\FormConfigBuilder: Symfony\Component\Form\FormBuilder. Maybe you want to instanceof check for one of these explicitly?

            ->setAction($this->generateUrl('tagNew'))
            ->setMethod('POST')
            ->add('name', TextType::class, array(
                    'attr' => array('placeholder' => '* Tag name'),
                )
            )
            ->add('save', SubmitType::class, array('label' => 'Save'))
            ->getForm();

        if ($request->getMethod() == 'POST') {
            $form->handleRequest($request);
            if ($form->isValid()) {
                $em->persist($tag);
                $em->flush();

                return $this->redirectToRoute('tagsAdmin');
            }
        }

        return [
            'title' => $title,
            'form'  => $form->createView(),
        ];
    }
}