ByTIC\GoogleRecaptcha\Forms\FomHasRecaptchaTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	processValidationRecaptcha()	0	6	2
A	initRecaptcha()	0	4	1

<?php

namespace ByTIC\GoogleRecaptcha\Forms;

use ByTIC\GoogleRecaptcha\Utility\GoogleRecaptcha;

/**
 * Trait FomHasRecaptchaTrait
 * @package ByTIC\GoogleRecaptcha\Forms
 */
trait FomHasRecaptchaTrait
{
    protected function initRecaptcha()
    {
        $this->addHidden('recaptchaResponse', 'recaptchaResponse', false);

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

        $this->getElement('recaptchaResponse')->setId('recaptchaResponse');

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

    }

    protected function processValidationRecaptcha()
    {
        $resp = GoogleRecaptcha::getManager()->verify($_POST['recaptchaResponse']);
        if (!$resp->isSuccess()) {
            $errors = $resp->getErrorCodes();

The assignment to $errors is dead and can be removed.

            $this->addError("Eroare validare email. Va rugam sa mai incercati!");

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

        }
    }
}