Shieldon\Firewall\Firewall\Captcha\ItemImageCaptcha::get()

Complexity

Conditions	4
Paths	4

Size

Total Lines	22
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public static function get(array $setting): CaptchaInterface
    {
        $type = $setting['config']['type'] ?? 'alnum';
        $length = $setting['config']['length'] ?? 8;

        switch ($type) {
            case 'numeric':
                $imageCaptchaConfig['pool'] = '0123456789';

$imageCaptchaConfig was never initialized. Although not strictly required by PHP, it is generally a good practice to add $imageCaptchaConfig = array(); before regardless.

                break;

            case 'alpha':
                $imageCaptchaConfig['pool'] = '0123456789abcdefghijklmnopqrstuvwxyz';
                break;

            case 'alnum':
            default:
                $imageCaptchaConfig['pool'] = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
        }

        $imageCaptchaConfig['word_length'] = $length;

        return new ImageCaptcha($imageCaptchaConfig);
    }
}