ByTIC\GoogleRecaptcha\Manager\Traits\HasClientTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setClient()	0	3	1
A	initClient()	0	4	1
A	getClient()	0	6	2

<?php

namespace ByTIC\GoogleRecaptcha\Manager\Traits;

use ReCaptcha\ReCaptcha;

/**
 * Trait HasClientTrait
 * @package ByTIC\GoogleRecaptcha\Manager\Traits
 */
trait HasClientTrait
{
    /**
     * @var null|ReCaptcha
     */
    protected $client = null;

    /**
     * @return ReCaptcha|null
     */
    public function getClient(): ReCaptcha
    {
        if ($this->client === null) {
            $this->initClient();
        }
        return $this->client;
    }

    /**
     * @param ReCaptcha|null $client
     */
    public function setClient(?ReCaptcha $client): void
    {
        $this->client = $client;
    }

    protected function initClient()
    {
        $client = new \ReCaptcha\ReCaptcha($this->getSecretKey());

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

        $this->setClient($client);
    }
}