CedricZiel\ShariffBundle\Model\ShariffConfig

Complexity

Total Complexity

27

Size/Duplication

Total Lines	220
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

16 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	17	6
A	getCacheDir()	0	4	1
A	setCacheDir()	0	4	1
A	getCacheTtl()	0	4	1
A	setCacheTtl()	0	4	1
A	getDomain()	0	4	1
A	setDomain()	0	4	1
A	getForceProtocol()	0	4	1
A	setForceProtocol()	0	4	1
A	getServices()	0	4	1
A	setServices()	0	4	1
A	getAdapter()	0	4	1
A	setAdapter()	0	4	1
A	getAdapterOptions()	0	4	1
A	setAdapterOptions()	0	4	1
C	toArray()	0	32	7

<?php

namespace CedricZiel\ShariffBundle\Model;

/**
 * Configures the shariff backend
 *
 * @package CedricZiel\ShariffBundle
 */
class ShariffConfig
{
    /**
     * @var string
     */
    protected $domain;

    /**
     * @var string
     */
    protected $forceProtocol;

    /**
     * @var string
     */
    protected $cacheDir;

    /**
     * @var int
     */
    protected $cacheTtl;

    /**
     * @var array
     */
    protected $services;

    /**
     * @var array
     */
    protected $serviceConfig = [];

    /**
     * @var string
     */
    protected $adapter;

    /**
     * @var array
     */
    protected $adapterOptions;

    /**
     * @var array
     */
    protected $client;

    /**
     * @param string  $domain
     * @param boolean $forceProtocol
     * @param array   $services
     * @param string  $cache
     * @param string  $client
     */
    public function __construct($domain, $forceProtocol, $services, $cache, $client)
    {
        $this->domain = $domain;
        $this->forceProtocol = $forceProtocol;

The property $forceProtocol was declared of type string, but $forceProtocol is of type boolean. Maybe add a type cast?

        $this->services = array_keys($services);
        $this->cacheTtl = $cache['ttl'];

The property $cacheTtl was declared of type integer, but $cache['ttl'] is of type string. Maybe add a type cast?

        $this->cacheDir = isset($cache['cacheDir']) ? $cache['cacheDir'] : null;
        $this->adapter = isset($cache['adapter']) ? $cache['adapter'] : null;
        $this->adapterOptions = isset($cache['adapterOptions']) ? $cache['adapterOptions'] : null;

It seems like isset($cache['adapterOptions']) ? $cache['adapterOptions'] : null of type string or null is incompatible with the declared type array of property $adapterOptions.

        foreach ($services as $name => $serviceConfig) {
            if (null !== $serviceConfig) {
                $this->serviceConfig[$name] = $serviceConfig;
            }
        }
        $this->client = $client;

It seems like $client of type string is incompatible with the declared type array of property $client.

    }

    /**
     * @return string
     */
    public function getCacheDir()
    {
        return $this->cacheDir;
    }

    /**
     * @param string $cacheDir
     */
    public function setCacheDir($cacheDir)
    {
        $this->cacheDir = $cacheDir;
    }

    /**
     * @return int
     */
    public function getCacheTtl()
    {
        return $this->cacheTtl;
    }

    /**
     * @param int $cacheTtl
     */
    public function setCacheTtl($cacheTtl)
    {
        $this->cacheTtl = $cacheTtl;
    }

    /**
     * @return string
     */
    public function getDomain()
    {
        return $this->domain;
    }

    /**
     * @param string $domain
     */
    public function setDomain($domain)
    {
        $this->domain = $domain;
    }

    /**
     * @return string
     */
    public function getForceProtocol()
    {
        return $this->forceProtocol;
    }

    /**
     * @param string $forceProtocol
     */
    public function setForceProtocol($forceProtocol)
    {
        $this->forceProtocol = $forceProtocol;
    }

    /**
     * @return array
     */
    public function getServices()
    {
        return $this->services;
    }

    /**
     * @param array $services
     */
    public function setServices($services)
    {
        $this->services = $services;
    }

    /**
     * @return string
     */
    public function getAdapter()
    {
        return $this->adapter;
    }

    /**
     * @param string $adapter
     */
    public function setAdapter($adapter)
    {
        $this->adapter = $adapter;
    }

    /**
     * @return array
     */
    public function getAdapterOptions()
    {
        return $this->adapterOptions;
    }

    /**
     * @param array $adapterOptions
     */
    public function setAdapterOptions($adapterOptions)
    {
        $this->adapterOptions = $adapterOptions;
    }

    /**
     * @return array
     */
    public function toArray()
    {
        $result = [];
        $result['domains'][] = $this->domain;
        $result['force_protocol'] = $this->forceProtocol;
        $result['services'] = $this->services;
        $result = array_merge($result, $this->serviceConfig);

        $result['services'] = array_filter(
            $result['services'],
            function ($v) {
                return 'Twitter' != $v && 'Mail' != $v;
            }
        );

        $result['cache'] = ['ttl' => $this->cacheTtl];
        if (null !== $this->cacheDir) {
            $result['cache']['cacheDir'] = $this->cacheDir;
        }
        if (null !== $this->adapter) {
            $result['cache']['adapter'] = $this->adapter;
        }
        if (null !== $this->adapterOptions) {
            $result['cache']['adapterOptions'] = $this->adapterOptions;
        }

        if (null !== $this->client && count($this->client)) {
            $result['client'] = $this->client;
        }

        return $result;
    }
}