SWP\Bundle\MultiTenancyBundle\Doctrine\ORM\TenantRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	65
Duplicated Lines	41.54 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	findOneBySubdomainAndDomain()	15	15	1
A	findOneByDomain()	12	12	1
A	findOneByCode()	0	11	1
A	findAvailableTenants()	0	9	1

<?php

/*
 * This file is part of the Superdesk Web Publisher MultiTenancy Bundle.
 *
 * Copyright 2015 Sourcefabric z.u. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2015 Sourcefabric z.ú
 * @license http://www.superdesk.org/license
 */

namespace SWP\Bundle\MultiTenancyBundle\Doctrine\ORM;

use SWP\Bundle\StorageBundle\Doctrine\ORM\EntityRepository;
use SWP\Component\MultiTenancy\Repository\TenantRepositoryInterface;

/**
 * Repository interface for tenants.
 */
class TenantRepository extends EntityRepository implements TenantRepositoryInterface

There is one abstract method findOneBySubdomain in this class; you could implement it, or declare this class as abstract.

{
    /**
     * {@inheritdoc}
     */
    public function findOneBySubdomainAndDomain($subdomain, $domain)

This method seems to be duplicated in your project.

    {
        return $this
            ->createQueryBuilder('t')
            ->select('t', 'o')
            ->leftJoin('t.organization', 'o')
            ->where('t.subdomain = :subdomain')
            ->andWhere('t.domainName = :domainName')
            ->setParameters([
                'subdomain' => $subdomain,
                'domainName' => $domain,
            ])
            ->getQuery()
            ->getOneOrNullResult();
    }

    /**
     * {@inheritdoc}
     */
    public function findOneByDomain($domain)

This method seems to be duplicated in your project.

    {
        return $this
            ->createQueryBuilder('t')
            ->select('t', 'o')
            ->leftJoin('t.organization', 'o')
            ->where('t.domainName = :domainName')
            ->andWhere('t.subdomain IS NULL')
            ->setParameter('domainName', $domain)
            ->getQuery()
            ->getOneOrNullResult();
    }

    /**
     * {@inheritdoc}
     */
    public function findOneByCode($code)
    {
        return $this
            ->createQueryBuilder('t')
            ->select('t', 'o')
            ->leftJoin('t.organization', 'o')
            ->where('t.code = :code')
            ->setParameter('code', $code)
            ->getQuery()
            ->getOneOrNullResult();
    }

    /**
     * {@inheritdoc}
     */
    public function findAvailableTenants()
    {
        return $this
            ->createQueryBuilder('t')
            ->select('t', 'o')
            ->leftJoin('t.organization', 'o')
            ->getQuery()
            ->getArrayResult();
    }
}