Victoire\Bundle\BusinessEntityBundle\Helper\BusinessEntityHelper

Complexity

Total Complexity

12

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	findByEntityInstance()	0	11	4
A	findByEntityClassname()	0	4	1
B	getAvailableForWidget()	0	22	6

<?php

namespace Victoire\Bundle\BusinessEntityBundle\Helper;

use Doctrine\ORM\EntityRepository;
use Victoire\Bundle\BusinessEntityBundle\Entity\BusinessEntity;
use Victoire\Bundle\BusinessEntityBundle\Entity\BusinessEntityRepository;
use Victoire\Bundle\BusinessEntityBundle\Entity\BusinessProperty;
use Victoire\Bundle\BusinessEntityBundle\Reader\BusinessEntityCacheReader;
use Victoire\Bundle\CoreBundle\Cache\Builder\CacheBuilder;
use Victoire\Bundle\ORMBusinessEntityBundle\Entity\ORMBusinessEntityRepository;

/**
 * The BusinessEntityHelper.
 *
 * ref: victoire_core.helper.business_entity_helper
 */
class BusinessEntityHelper
{
    /**
     * @var BusinessEntityRepository
     */
    private $businessEntityRepository;
    /**
     * @var ORMBusinessEntityRepository
     */
    private $ormBusinessEntityRepository;
    /**
     * @var BusinessEntityCacheReader
     */
    private $cacheReader;

    /**
     * Constructor.
     *
     * @param EntityRepository|BusinessEntityRepository $businessEntityRepository
     * @param EntityRepository                          $ormBusinessEntityRepository
     * @param BusinessEntityCacheReader                 $cacheReader
     *
     * @internal param BusinessEntityCacheReader $reader
     * @internal param CacheBuilder $builder
     */
    public function __construct(EntityRepository $businessEntityRepository, EntityRepository $ormBusinessEntityRepository, BusinessEntityCacheReader $cacheReader)
    {
        $this->ormBusinessEntityRepository = $ormBusinessEntityRepository;

$ormBusinessEntityRepository is of type object<Doctrine\ORM\EntityRepository>, but the property $ormBusinessEntityRepository was declared to be of type object<Victoire\Bundle\ORMBusinessEntityBundle\Entity\ORMBusinessEntityRepository>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->businessEntityRepository = $businessEntityRepository;

$businessEntityRepository is of type object<Doctrine\ORM\EntityRepository>, but the property $businessEntityRepository was declared to be of type object<Victoire\Bundle\BusinessEntityBundle\Entity\BusinessEntityRepository>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->cacheReader = $cacheReader;
    }

    /**
     * Get a business entity.q.
     *
     * @param mixed $entity
     *
     * @return BusinessEntity
     */
    public function findByEntityInstance($entity)
    {
        $businessEntity = null;
        $class = new \ReflectionClass($entity);
        while (!$businessEntity && $class && $class->name !== null) {
            $businessEntity = $this->ormBusinessEntityRepository->findOneBy(['class' => $class->name]);
            $class = $class->getParentClass();
        }

        return $businessEntity;
    }

    /**
     * Get a business entity by classname.
     *
     * @param string $classname
     *
     * @return BusinessEntity
     */
    public function findByEntityClassname($classname)
    {
        return $this->ormBusinessEntityRepository->findOneBy(['class' => $classname]);
    }

    public function getAvailableForWidget($widgetName)
    {
        $classes = $this->businessEntityRepository->getByAvailableWidgets($widgetName);

        $receiverProperties = $this->cacheReader->getReceiverProperties($widgetName);
        foreach ($classes as $businessEntity) {
            $businessProperties = $businessEntity->getBusinessProperties();
            foreach ($receiverProperties as $receiverType => $receiverProperty) {
                $businessEntity->setDisable(true);
                if (count($businessProperties) > 0) {
                    /** @var BusinessProperty $businessProperty */
                    foreach ($businessProperties as $businessProperty) {
                        if (in_array($receiverType, $businessProperty->getTypes())) {
                            $businessEntity->setDisable(false);
                        }
                    }
                }
            }
        }

        return $classes;
    }
}