Shieldon\Firewall\Kernel\CaptchaTrait::captchaResponse()

Complexity

Conditions	4
Paths	5

Size

Total Lines	18
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function captchaResponse(): bool
    {
        foreach ($this->captcha as $captcha) {
            
            if (!$captcha->response()) {
                return false;
            }
        }

        /**
         * $sessionLimit @ SessionTrait
         * sessionHandler() @ SessionTrait
         */
        if (!empty($this->sessionLimit['count'])) {
            $this->result = $this->sessionHandler(self::RESPONSE_ALLOW);

The property result does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

The constant Shieldon\Firewall\Kernel\CaptchaTrait::RESPONSE_ALLOW was not found. Maybe you did not declare it correctly or list all dependencies?

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

        }

        return true;
    }