PiedWeb\CMSBundle\Controller\PageController

Complexity

Total Complexity

23

Size/Duplication

Total Lines	103
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	checkIfUriIsCanonical()	0	13	3
B	feed()	0	25	7
B	show()	0	40	10
A	preview()	0	15	3

<?php

namespace PiedWeb\CMSBundle\Controller;

use PiedWeb\CMSBundle\Entity\PageInterface as Page;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Translation\TranslatorInterface;

class PageController extends AbstractController
{
    protected $translator;

    public function show(
        ?string $slug,
        Request $request,
        TranslatorInterface $translator,
        ParameterBagInterface $params
    ) {
        $slug = (null === $slug || '' === $slug) ? 'homepage' : rtrim(strtolower($slug), '/');
        $page = $this->getDoctrine()
            ->getRepository($this->container->getParameter('app.entity_page'))

The method getParameter() does not exist on Psr\Container\ContainerInterface. It seems like you code against a sub-type of Psr\Container\ContainerInterface such as Symfony\Component\DependencyInjection\ContainerInterface.

            ->findOneBySlug($slug);

The method findOneBySlug() does not exist on Doctrine\Persistence\ObjectRepository. Did you maybe mean findOneBy()?

        // Check if page exist
        if (null === $page) {
            throw $this->createNotFoundException();
        }

        if (null !== $page->getLocale()) { // avoid bc break, todo remove it
            $request->setLocale($page->getLocale());
        }

        // Check if page is public
        if ($page->getCreatedAt() > new \DateTimeImmutable() && !$this->isGranted('ROLE_ADMIN')) {
            throw $this->createNotFoundException();
        }

        // SEO redirection if we are not on the good URI (for exemple /fr/tagada instead of /tagada)
        $redirect = $this->checkIfUriIsCanonical($request, $page);
        if (false !== $redirect) {
            return $this->redirect($redirect[0], $redirect[1]);
        }

        // Maybe the page is a redirection
        if (false !== $page->getRedirection()) {
            return $this->redirect($page->getRedirection(), $page->getRedirectionCode());
        }

        // method_exists($this->container->getParameter('app.entity_page'), 'getTemplate') &&
        $template = null !== $page->getTemplate() ? $page->getTemplate() : $params->get('app.default_page_template');

        return $this->render($template, ['page' => $page]);
    }

    protected function checkIfUriIsCanonical($request, $page)
    {
        $real = $request->getRequestUri();

        $expected = 'homepage' == $page->getSlug() ?
            $this->get('piedweb.page_canonical')->generatePathForHomepage() :
            $this->get('piedweb.page_canonical')->generatePathForPage($page->getRealSlug());

        if ($real != $expected) {
            return [$request->getBasePath().$expected, 301];
        }

        return false;
    }

    public function preview(
        ?string $slug,
        Request $request
    ) {
        $pageEntity = $this->container->getParameter('app.entity_page');

        $page = (null === $slug || '' === $slug) ?
            new $pageEntity()
            : $this->getDoctrine()
            ->getRepository($pageEntity)
            ->findOneBySlug(rtrim(strtolower($slug), '/'));

        $plainText = \PiedWeb\CMSBundle\Twig\AppExtension::convertMarkdownImage($request->request->get('plaintext'));

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

    public function feed(
        ?string $slug,
        Request $request,

The parameter $request is not used and could be removed.

        TranslatorInterface $translator,

The parameter $translator is not used and could be removed.

        ParameterBagInterface $params

The parameter $params is not used and could be removed.

    ) {
        $slug = (null === $slug || '' === $slug) ? 'homepage' : rtrim(strtolower($slug), '/');
        $page = $this->getDoctrine()
            ->getRepository($this->container->getParameter('app.entity_page'))
            ->findOneBySlug($slug);

        // Check if page exist
        if (null === $page || $page->getChildrenPages()->count() < 1) {
            throw $this->createNotFoundException();
        }

        // Check if page is public
        if ($page->getCreatedAt() > new \DateTimeImmutable() && !$this->isGranted('ROLE_ADMIN')) {
            throw $this->createNotFoundException();
        }

        $response = new Response();
        $response->headers->set('Content-Type', 'text/xml');

        return $this->render('@PiedWebCMS/page/rss.xml.twig', ['page' => $page], $response);
    }
}