PiedWeb\CMSBundle\Extension\PageScanner\PageScannerController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	scanAction()	0	24	4

<?php

namespace PiedWeb\CMSBundle\Extension\PageScanner;

use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\DependencyInjection\Container;
use Symfony\Component\DependencyInjection\ParameterBag\ContainerBag;
use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;

class PageScannerController extends AbstractController
{
    /**
     * @var PageScannerService
     */
    protected $scanner;

    /**
     * @var Container
     */
    protected $container;

    /**
     * @var ContainerBag
     */
    protected $params;

    public function __construct(PageScannerService $scanner, ParameterBagInterface $params)
    {
        $this->scanner = $scanner;
        $this->params = $params;

$params is of type Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface, but the property $params was declared to be of type Symfony\Component\DependencyInjection\ParameterBag\ContainerBag. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    public function scanAction()
    {
        $pages = $this->getDoctrine()
            ->getRepository($this->params->get('pwc.entity_page'))
            ->findAll();

        $errors = [];
        $errorNbr = 0;

        foreach ($pages as $page) {
            // todo import scanner via setScanner + services.yaml
            $scan = $this->scanner->scan($page);
            if (true !== $scan) {
                $errors[$page->getId()] = $scan;
                $errorNbr = $errorNbr + \count($errors[$page->getId()]);
            }

            if ($errorNbr > 100) {
                break;
            }
        }

        return $this->render('@pwcPageScanner/results.html.twig', [
            'errorsByPages' => $errors,
        ]);
    }
}