Sf4\Api\RequestHandler\Traits\SitesTrait

Complexity

Total Complexity

11

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	addSite()	0	11	6
A	getSites()	0	3	1
A	setSites()	0	3	1
A	addSites()	0	8	3

<?php
/**
 * Created by PhpStorm.
 * User: siim
 * Date: 18.02.19
 * Time: 9:06
 */

namespace Sf4\Api\RequestHandler\Traits;

use Sf4\Api\DependencyInjection\Configuration;

trait SitesTrait
{
    /** @var array|null $sites */
    protected $sites = [];

    /**
     * @return array|null
     */
    public function getSites(): ?array
    {
        return $this->sites;
    }

    /**
     * @param array|null $sites
     */
    public function setSites(?array $sites): void
    {
        $this->sites = $sites;
    }

    /**
     * @param array|null $sites
     */
    public function addSites(?array $sites)
    {
        if (!$sites) {

The expression $sites of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return ;
        }

        foreach ($sites as $site) {
            $this->addSite($site);
        }
    }

    /**
     * @param array $site
     */
    public function addSite(array $site)
    {
        if (isset($site[Configuration::SITES_SITE]) &&
            isset($site[Configuration::SITES_URL]) &&
            isset($site[Configuration::SITES_TOKEN])) {
            foreach ($this->sites as $thisSite) {
                if ($thisSite[Configuration::SITES_SITE] === $site[Configuration::SITES_SITE]) {
                    return ;
                }
            }
            $this->sites[] = $site;
        }
    }
}