ByTIC\GoogleRecaptcha\Manager\Traits\HasConfigTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setConfig()	0	4	1
A	fromConfig()	0	5	1
A	populateFromConfig()	0	12	3

<?php

namespace ByTIC\GoogleRecaptcha\Manager\Traits;

use ByTIC\GoogleRecaptcha\Config\Config;

/**
 * Trait HasConfigTrait
 * @package ByTIC\GoogleRecaptcha\Manager\Traits
 */
trait HasConfigTrait
{
    /**
     * @var Config
     */
    protected $config;

    /**
     * @param Config $config
     * @return static
     */
    public static function fromConfig($config)
    {
        $agent = new static();
        $agent->setConfig($config);
        return $agent;
    }

    /**
     * @param Config $config
     */
    public function setConfig($config)
    {
        $this->config = $config;
        $this->populateFromConfig();
    }

    protected function populateFromConfig()
    {
        $this->setEnabled($this->config->isEnabled());

It seems like setEnabled() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $site = $this->config->getSiteKey();
        if ($site) {
            $this->setSiteKey($site);

It seems like setSiteKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        $key = $this->config->getSecretKey();
        if ($key) {
            $this->setSecretKey($key);

It seems like setSecretKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }
}