bytic / services-google-recaptcha

src/Manager/Traits/HasConfigTrait.php

<?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?

        }
    }
}