Enjoys\Forms\Captcha\Defaults\Defaults::createImage()

Complexity

Conditions	5
Paths	9

Size

Total Lines	53
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    private function createImage($code, $width = 150, $height = 50)
    {
        // Создаем пустое изображение
        $img = \imagecreatetruecolor($width, $height);

        $background_color = [\mt_rand(200, 255), \mt_rand(200, 255), \mt_rand(200, 255)];
        // Заливаем фон белым цветом
        $background = \imagecolorallocate($img, $background_color[0], $background_color[1], $background_color[2]);

It seems like $img can also be of type false; however, parameter $image of imagecolorallocate() does only seem to accept resource, maybe add an additional type check?

        \imagefill($img, 0, 0, $background);

It seems like $img can also be of type false; however, parameter $image of imagefill() does only seem to accept resource, maybe add an additional type check?

        // Накладываем защитный код
        $x = 0;
        $letters = \str_split($code);
        $figures = [50, 70, 90, 110, 130, 150, 170, 190, 210];

        foreach ($letters as $letter) {
            //Ориентир
            $h = 1;
            //Рисуем
            $color = \imagecolorallocatealpha(
                    $img,

It seems like $img can also be of type false; however, parameter $image of imagecolorallocatealpha() does only seem to accept resource, maybe add an additional type check?

                    $figures[\rand(0, \count($figures) - 1)],
                    $figures[\rand(0, \count($figures) - 1)],
                    $figures[\rand(0, \count($figures) - 1)],
                    rand(10, 30)
            );


            // Формируем координаты для вывода символа
            if (empty($x)) {
                $x = (int) ($width * 0.08);
            } else {
                $x = (int) ($x + ($width * 0.8) / \count($letters) + \rand(0, (int) ($width * 0.01)));
            }

            if ($h == rand(1, 2)) {
                $y = (int) ((($height * 1) / 4) + \rand(0, (int) ($height * 0.1)));
            } else {
                $y = (int) ((($height * 1) / 4) - \rand(0, (int) ($height * 0.1)));
            }


            // Изменяем регистр символа
            if ($h == \rand(0, 1)) {
                $letter = \strtoupper($letter);
            }
            // Выводим символ на изображение
            \imagestring($img, 6, $x, $y, $letter, $color);

It seems like $img can also be of type false; however, parameter $image of imagestring() does only seem to accept resource, maybe add an additional type check?

            $x++;
        }

        return $img;
    }