PiedWeb\CMSBundle\Controller\PageScannerController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	scanAction()	0	18	3

<?php

namespace PiedWeb\CMSBundle\Controller;

use PiedWeb\CMSBundle\Service\PageScannerService;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;

class PageScannerController extends AbstractController
{
    public function __construct(PageScannerService $scanner)
    {
        $this->scanner = $scanner;

The property scanner does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    public function scanAction()
    {
        $pages = $this->getDoctrine()
            ->getRepository($this->container->getParameter('app.entity_page'))
            ->findAll();

        $errors = [];

        foreach ($pages as $page) {
            // todo import scanner via setScanner + services.yaml
            $scan = $this->scanner->scan($page);
            if (true !== $scan) {
                $errors = array_merge($errors, $scan);
            }
        }

        return $this->render('@PiedWebCMS/admin/scanView.html.twig', [
            'errors' => $errors,
        ]);
    }
}