App\Repository\SimilarRepository

Complexity

Total Complexity

9

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	isModuleEnabled()	0	6	1
A	findSimilarProperties()	0	21	6
A	findByArea()	0	15	2

<?php

declare(strict_types=1);

namespace App\Repository;

use App\Entity\Property;

final class SimilarRepository extends PropertyRepository
{
    public const NUM_ITEMS = 6;

    public function findSimilarProperties(Property $property): array
    {
        if (!$this->isModuleEnabled()) {
            return [];
        }

        if ($property->getNeighborhood()) {
            // Find in a small area
            $result = $this->findByArea($property, 'neighborhood');

            if (empty($result) && $property->getDistrict()) {
                // Find in a larger area
                $result = $this->findByArea($property);
            }

            return $result;
        } elseif ($property->getDistrict()) {
            return $this->findByArea($property);
        }

        return [];
    }

    private function findByArea(Property $property, string $area = 'district'): array
    {
        $qb = $this->createQueryBuilder('p')
            ->where("p.state = 'published'")
            ->andWhere('p.id != '.(int) ($property->getId()))
            ->andWhere('p.deal_type = '.(int) ($property->getDealType()->getId()))
            ->andWhere('p.category = '.(int) ($property->getCategory()->getId()));

        if ('neighborhood' === $area) {
            $qb->andWhere('p.neighborhood = '.(int) ($property->getNeighborhood()->getId()));
        } else {
            $qb->andWhere('p.district = '.(int) ($property->getDistrict()->getId()));
        }

        return $qb->getQuery()->setMaxResults(self::NUM_ITEMS)->getResult();

The expression return $qb->getQuery()->setMaxResults(self::NUM_ITEMS)->getResult() could return the type integer which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

    }

    private function isModuleEnabled(): bool
    {
        $repository = $this->getEntityManager()->getRepository('App:Settings');
        $state = $repository->findOneBy(['setting_name' => 'show_similar_properties']);

        return '1' === $state->getSettingValue();
    }
}