Loevgaard\PakkelabelsBundle\Controller\LabelController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	55
Duplicated Lines	20 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	indexAction()	11	11	1
A	showAction()	0	6	1
A	imageAction()	0	9	1

<?php

namespace Loevgaard\PakkelabelsBundle\Controller;

use Loevgaard\PakkelabelsBundle\Entity\Label;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\BinaryFileResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;

/**
 * @Route("/label")
 */
class LabelController extends Controller
{
    /**
     * @Method("GET")
     * @Route("", name="loevgaard_pakkelabels_label_index")
     *
     * @param Request $request
     *
     * @return Response
     */
    public function indexAction(Request $request)

This method seems to be duplicated in your project.

    {
        $repos = $this->get('loevgaard_pakkelabels.label_repository');

        /** @var Label[] $countryMappings */
        $labels = $repos->findAllWithPaging($request->query->getInt('page', 1));

        return $this->render('@LoevgaardPakkelabels/label/index.html.twig', [
            'labels' => $labels,
        ]);
    }

    /**
     * @Method("GET")
     * @Route("/{id}/show", name="loevgaard_pakkelabels_label_show")
     *
     * @param Label $label
     *
     * @return Response
     */
    public function showAction(Label $label): Response
    {
        return $this->render('@LoevgaardPakkelabels/label/show.html.twig', [
            'label' => $label,
        ]);
    }

    /**
     * @Method("GET")
     * @Route("/{id}/image", name="loevgaard_pakkelabels_label_image")
     *
     * @param Label $label
     *
     * @return BinaryFileResponse
     */
    public function imageAction(Label $label): BinaryFileResponse
    {
        $labelFactory = $this->get('loevgaard_pakkelabels.label_file_factory');

        // @todo remember to catch exception, maybe create a new exception class
        $labelFile = $labelFactory->create($label, true);

        return new BinaryFileResponse($labelFile);
    }
}