ByTIC\GoogleRecaptcha\Config\Traits\CanInitFromConfigTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	canInitFromConfig()	0	21	6
A	initFromConfig()	0	5	1
A	getConfigVar()	0	4	1
A	fromConfig()	0	5	1

<?php

namespace ByTIC\GoogleRecaptcha\Config\Traits;

use ByTIC\GoogleRecaptcha\Config\Config;
use Nip\Container\Container;

/**
 * Trait CanInitFromConfigTrait
 * @package ByTIC\GoogleRecaptcha\Config\Traits
 */
trait CanInitFromConfigTrait
{
    protected static $configRoot = null;

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

    /**
     * @return bool
     */
    public static function canInitFromConfig(): bool
    {
        if (!function_exists('config')) {
            return false;
        }

        $container = function_exists('app') ? app() : Container::getInstance();
        if (!$container->has('config')) {
            return false;
        }

        $config = config();
        if ($config->has('recaptcha')) {
            static::$configRoot = 'recaptcha';
            return true;
        }
        if ($config->has('services.recaptcha')) {
            static::$configRoot = 'services.recaptcha';
            return true;
        }
        return false;
    }

    protected function initFromConfig()
    {
        $this->setEnabled(static::getConfigVar('enabled'));

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

        $this->setSiteKey(static::getConfigVar('site_key'));

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

        $this->setSecretKey(static::getConfigVar('secret_key'));

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

    }

    /**
     * @param string $value
     * @param null $default

Are you sure the doc-type for parameter $default is correct as it would always require null to be passed?

     * @return mixed|null
     */
    protected static function getConfigVar($value, $default = null)
    {
        $config = config();
        return $config->get(static::$configRoot.'.'.$value, $default);
    }
}