Alpixel\Bundle\CMSBundle\Listener\SitemapListener

Complexity

Total Complexity

11

Size/Duplication

Total Lines	58
Duplicated Lines	10.34 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	2
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
D	populateSitemap()	6	46	10

<?php

namespace Alpixel\Bundle\CMSBundle\Listener;

use Alpixel\Bundle\SEOBundle\Event\SitemapPopulateEvent;
use Alpixel\Bundle\SEOBundle\Service\SitemapListenerInterface;
use Alpixel\Bundle\SEOBundle\Sitemap\Url\UrlConcrete;
use Doctrine\Bundle\DoctrineBundle\Registry;
use Symfony\Component\Routing\RouterInterface;

class SitemapListener implements SitemapListenerInterface
{
    protected $doctrine;
    private $router;

    public function __construct(Registry $doctrine, RouterInterface $router)
    {
        $this->doctrine = $doctrine;
        $this->router = $router;
    }

    public function populateSitemap(SitemapPopulateEvent $event)
    {
        $section = $event->getSection();
        if (is_null($section) || $section == 'cms') {
            $entities = [];
            $entityManager = $this->doctrine->getManager();
            $meta = $entityManager->getMetadataFactory()->getAllMetadata();

            foreach ($meta as $m) {

This code seems to be duplicated across your project.

                $relations = $m->getAssociationMappings();
                if (array_key_exists('node', $relations) && $relations['node']['targetEntity'] == 'Alpixel\Bundle\CMSBundle\Entity\Node') {
                    $entities[] = $m;
                }
            }

            $pages = [];
            foreach ($entities as $entity) {
                $objects = $entityManager
                            ->getRepository($entity->getName())
                            ->findAll();
                foreach ($objects as $object) {
                    if ($object->getNode()->getPublished() === true) {
                        $pages[] = $object;
                    }
                }
            }

            foreach ($pages as $cms) {
                $url = $this->router->generate('front_cms', [
                        'slug' => $cms->getNode()->getSlug(),
                    ],
                    true

true is of type boolean, but the function expects a integer.

                );

                $event->getGenerator()->addUrl(
                    new UrlConcrete(
                        $url,
                        new \DateTime(),
                        UrlConcrete::CHANGEFREQ_MONTHLY,
                        .5
                    ),
                    'cms'
                );
            }
        }
    }
}