Victoire\Bundle\BusinessEntityBundle\EventSubscriber\BusinessEntitySubscriber

Complexity

Total Complexity

31

Size/Duplication

Total Lines	259
Duplicated Lines	15.44 %

Coupling/Cohesion

Components	1
Dependencies	10

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
C	updateBusinessPages()	0	48	9
A	__construct()	0	10	1
A	getSubscribedEvents()	0	8	1
A	postUpdate()	0	20	3
A	postPersist()	0	4	1
C	updateViewReference()	40	58	9
B	preRemove()	0	56	7

<?php

namespace Victoire\Bundle\BusinessEntityBundle\EventSubscriber;

use Doctrine\Common\EventSubscriber;
use Doctrine\ORM\EntityManager;
use Doctrine\ORM\Event\LifecycleEventArgs;
use Doctrine\ORM\UnitOfWork;
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use Victoire\Bundle\BusinessEntityBundle\Entity\BusinessEntity;
use Victoire\Bundle\BusinessEntityBundle\Helper\BusinessEntityHelper;
use Victoire\Bundle\BusinessPageBundle\Builder\BusinessPageBuilder;
use Victoire\Bundle\BusinessPageBundle\Entity\BusinessPage;
use Victoire\Bundle\BusinessPageBundle\Entity\BusinessTemplate;
use Victoire\Bundle\BusinessPageBundle\Helper\BusinessPageHelper;
use Victoire\Bundle\BusinessPageBundle\Repository\BusinessPageRepository;
use Victoire\Bundle\ViewReferenceBundle\Event\ViewReferenceEvent;
use Victoire\Bundle\ViewReferenceBundle\ViewReferenceEvents;

class BusinessEntitySubscriber implements EventSubscriber
{
    protected $businessPageBuilder;
    protected $dispatcher;

    /**
     * @param BusinessPageBuilder      $businessPageBuilder
     * @param BusinessEntityHelper     $businessEntityHelper
     * @param BusinessPageHelper       $businessPageHelper
     * @param EventDispatcherInterface $dispatcher
     */
    public function __construct(BusinessPageBuilder          $businessPageBuilder,
                                BusinessEntityHelper         $businessEntityHelper,
                                BusinessPageHelper           $businessPageHelper,
                                EventDispatcherInterface     $dispatcher
    ) {
        $this->businessPageBuilder = $businessPageBuilder;
        $this->businessEntityHelper = $businessEntityHelper;

The property businessEntityHelper does not exist. Did you maybe forget to declare it?

        $this->businessPageHelper = $businessPageHelper;

The property businessPageHelper does not exist. Did you maybe forget to declare it?

        $this->dispatcher = $dispatcher;
    }

    /**
     * bind to LoadClassMetadata method.
     *
     * @return string[] The subscribed events
     */
    public function getSubscribedEvents()
    {
        return [
            'postUpdate',
            'postPersist',
            'preRemove',
        ];
    }

    /**
     * @param LifecycleEventArgs $eventArgs
     */
    public function postUpdate(LifecycleEventArgs $eventArgs)
    {
        /** @var EntityManager $entityManager */
        $entityManager = $eventArgs->getEntityManager();
        /** @var UnitOfWork $uow */
        $uow = $entityManager->getUnitOfWork();

        foreach ($uow->getScheduledEntityInsertions() as $entity) {
            $businessEntity = $this->businessEntityHelper->findByEntityInstance($entity);
            if ($businessEntity) {
                $this->updateBusinessPages(
                    $entity,
                    $businessEntity,
                    $entityManager,
                    $uow->getScheduledEntityDeletions()
                );
            }
        }
        $this->updateViewReference($eventArgs);
    }

    /**
     * @param LifecycleEventArgs $eventArgs
     */
    public function postPersist(LifecycleEventArgs $eventArgs)
    {
        $this->updateViewReference($eventArgs);
    }

    /**
     * get BusinessTemplate concerned by this entity (if so)
     * then get BusinessPages
     * for each BusinessPage, update its slug according to the new slug (if so).
     *
     * @param $entity
     * @param BusinessEntity $businessEntity
     * @param EntityManager  $entityManager
     * @param array          $deletions
     *
     * @throws \Exception
     *
     * @internal param LifecycleEventArgs $eventArgs
     */
    public function updateBusinessPages($entity, BusinessEntity $businessEntity, EntityManager $entityManager, $deletions)
    {
        $businessTemplates = $entityManager->getRepository('VictoireBusinessPageBundle:BusinessTemplate')->findPagePatternByBusinessEntity($businessEntity);
        foreach ($businessTemplates as $businessTemplate) {
            if ($this->businessPageHelper->isEntityAllowed($businessTemplate, $entity, $entityManager)) {
                /** @var BusinessPageRepository $bepRepo */
                $bepRepo = $entityManager->getRepository('VictoireBusinessPageBundle:BusinessPage');
                $virtualBusinessPage = $this->businessPageBuilder->generateEntityPageFromTemplate(
                    $businessTemplate,
                    $entity,
                    $entityManager
                );
                // Get the BusinessPage if exists for the given entity
                /** @var BusinessPage $businessPage */
                $businessPage = $bepRepo->findPageByBusinessEntityAndPattern(
                    $businessTemplate,
                    $entity,
                    $businessEntity
                );
                // If there is diff between persisted BEP and computed, persist the change
                $scheduledForRemove = false;
                foreach ($deletions as $deletion) {
                    if (get_class($deletion) == get_class($businessPage)
                        && $deletion->getId() === $businessPage->getId()
                    ) {
                        $scheduledForRemove = true;
                    }
                }

                if ($businessPage && !$scheduledForRemove) {
                    $oldSlug = $businessPage->getSlug();
                    $newSlug = $entity->getSlug();
                    $staticUrl = $businessPage->getStaticUrl();

                    if ($staticUrl) {
                        $staticUrl = preg_replace('/'.$oldSlug.'/', $newSlug, $staticUrl);
                        $businessPage->setStaticUrl($staticUrl);

It seems like $staticUrl defined by preg_replace('/' . $oldSlug . '/', $newSlug, $staticUrl) on line 137 can also be of type array<integer,string>; however, Victoire\Bundle\BusinessPageBundle\Entity\BusinessPage::setStaticUrl() does only seem to accept string, maybe add an additional type check?

                    }

                    $businessPage->setName($virtualBusinessPage->getName());
                    $businessPage->setSlug($virtualBusinessPage->getSlug());

                    $entityManager->persist($businessPage);
                    $entityManager->flush();
                }
            }
        }
    }

    /**
     * This method throw an event if needed for a view related to a businessEntity.
     *
     * @param LifecycleEventArgs $eventArgs
     *
     * @throws \Exception
     */
    private function updateViewReference(LifecycleEventArgs $eventArgs)
    {
        $entity = $eventArgs->getEntity();
        //if it's a businessEntity we need to rebuild virtuals (BPs are rebuild in businessEntitySubscriber)
        if ($businessEntity = $this->businessEntityHelper->findByEntityInstance($entity)) {
            $em = $eventArgs->getEntityManager();
            //find all BT that can represent the businessEntity
            $businessTemplates = $em->getRepository('VictoireBusinessPageBundle:BusinessTemplate')->findPagePatternByBusinessEntity($businessEntity);
            foreach ($businessTemplates as $businessTemplate) {

This code seems to be duplicated across your project.

                if ($page = $em->getRepository(
                    'Victoire\Bundle\BusinessPageBundle\Entity\BusinessPage'
                )->findPageByBusinessEntityAndPattern($businessTemplate, $entity, $businessEntity)
                ) {
                    //if it's a BP we update the BP
                    $this->businessPageBuilder->updatePageParametersByEntity($page, $entity);
                } else {
                    $page = $this->businessPageBuilder->generateEntityPageFromTemplate(
                        $businessTemplate,
                        $entity,
                        $em
                    );
                }
                if ($this->businessPageHelper->isEntityAllowed($businessTemplate, $entity, $em)) {
                    //update the reference
                    $event = new ViewReferenceEvent($page);
                    $this->dispatcher->dispatch(ViewReferenceEvents::UPDATE_VIEW_REFERENCE, $event);
                }
            }
        }
        //if it a businessTemplate we have to rebuild virtuals or update BP
        if ($entity instanceof BusinessTemplate) {
            $em = $eventArgs->getEntityManager();
            $businessEntityId = $entity->getBusinessEntityId();
            $businessEntity = $this->businessEntityHelper->findById($businessEntityId);
            //find all entities
            $entities = $this->businessPageHelper->getEntitiesAllowed($entity, $em);
            foreach ($entities as $be) {

This code seems to be duplicated across your project.

                if ($this->businessPageHelper->isEntityAllowed($entity, $be, $em)) {
                    if ($page = $em->getRepository(
                        'Victoire\Bundle\BusinessPageBundle\Entity\BusinessPage'
                    )->findPageByBusinessEntityAndPattern($entity, $be, $businessEntity)
                    ) {
                        //rebuild page if its a BP
                        $this->businessPageBuilder->updatePageParametersByEntity($page, $be);
                    } else {
                        $page = $this->businessPageBuilder->generateEntityPageFromTemplate(
                            $entity,
                            $be,
                            $em
                        );
                    }
                    // update reference
                    $event = new ViewReferenceEvent($page);
                    $this->dispatcher->dispatch(ViewReferenceEvents::UPDATE_VIEW_REFERENCE, $event);
                }
            }
        }
    }

    /**
     * @param LifecycleEventArgs $eventArgs
     *
     * @throws \Exception
     */
    public function preRemove(LifecycleEventArgs $eventArgs)
    {
        $entity = $eventArgs->getEntity();

        //if we remove a BP we need to remplace by a VBP ref
        if ($entity instanceof BusinessPage) {
            //remove BP ref
            $event = new ViewReferenceEvent($entity);
            $this->dispatcher->dispatch(ViewReferenceEvents::REMOVE_VIEW_REFERENCE, $event);
            $em = $eventArgs->getEntityManager();
            $businessTemplate = $entity->getTemplate();
            $page = $this->businessPageBuilder->generateEntityPageFromTemplate(
                $businessTemplate,

$businessTemplate is of type string, but the function expects a object<Victoire\Bundle\BusinessPageBundle\Entity\BusinessTemplate>.

                $entity->getBusinessEntity(),
                $em
            );
            //create VBP ref
            //TODO :: dont rebuild if businessEntity or businessTemplate doesn't exist
            $event = new ViewReferenceEvent($page);
            $this->dispatcher->dispatch(ViewReferenceEvents::UPDATE_VIEW_REFERENCE, $event);
        }

        //if it's a businessEntity, we need to remove all BP and VBP ref
        if ($businessEntity = $this->businessEntityHelper->findByEntityInstance($entity)) {
            $em = $eventArgs->getEntityManager();
            $businessTemplates = $em->getRepository('VictoireBusinessPageBundle:BusinessTemplate')->findPagePatternByBusinessEntity($businessEntity);
            foreach ($businessTemplates as $businessTemplate) {
                if ($page = $em->getRepository('Victoire\Bundle\BusinessPageBundle\Entity\BusinessPage')->findPageByBusinessEntityAndPattern($businessTemplate, $entity, $businessEntity)) {
                    $event = new ViewReferenceEvent($page);
                    $this->dispatcher->dispatch(ViewReferenceEvents::REMOVE_VIEW_REFERENCE, $event);
                } else {
                    $page = $this->businessPageBuilder->generateEntityPageFromTemplate(
                        $businessTemplate,
                        $entity,
                        $em
                    );
                    $event = new ViewReferenceEvent($page);
                    $this->dispatcher->dispatch(ViewReferenceEvents::REMOVE_VIEW_REFERENCE, $event);
                }
            }
        }
        //if we remove a businessTemplate remove all VBT ref (BP cascade remove)
        if ($entity instanceof BusinessTemplate) {
            $em = $eventArgs->getEntityManager();
            $entities = $this->businessPageHelper->getEntitiesAllowed($entity, $em);
            foreach ($entities as $be) {
                $page = $this->businessPageBuilder->generateEntityPageFromTemplate(
                    $entity,
                    $be,
                    $em
                );
                $event = new ViewReferenceEvent($page);
                $this->dispatcher->dispatch(ViewReferenceEvents::REMOVE_VIEW_REFERENCE, $event);
            }
        }
    }
}