XoopsModules\Tdmcreate\LogoGenerator::createLogo()

Complexity

Conditions	7
Paths	6

Size

Total Lines	45
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php namespace XoopsModules\Tdmcreate;
    public static function createLogo($logoIcon, $moduleName)
    {
        if (!extension_loaded('gd')) {
            return false;
        }
        $requiredFunctions = ['imagecreatefrompng', 'imagefttext', 'imagecopy', 'imagepng', 'imagedestroy', 'imagecolorallocate'];
        foreach ($requiredFunctions as $func) {
            if (!function_exists($func)) {
                return false;
            }
        }

        $dirname = 'tdmcreate';
        $iconFileName = XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32/' . basename($logoIcon);

        //$dirFonts = TDMC_PATH . "/assets/fonts";
        //$dirLogos = TDMC_PATH . "/assets/images/logos";
        $dirFonts = XOOPS_ROOT_PATH . '/modules/' . $dirname . '/assets/fonts';
        $dirLogos = XOOPS_ROOT_PATH . '/modules/' . $dirname . '/assets/images/logos';

        if (!file_exists($imageBase = $dirLogos . '/empty.png') ||
            !file_exists($font = $dirFonts . '/VeraBd.ttf') ||
            !file_exists($iconFile = $iconFileName)) {
            return false;
        }

        $imageModule = imagecreatefrompng($imageBase);
        $imageIcon = imagecreatefrompng($iconFile);

        // Write text
        $textColor = imagecolorallocate($imageModule, 0, 0, 0);

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

        $spaceToBorder = (92 - mb_strlen($moduleName) * 7.5) / 2;
        imagefttext($imageModule, 8.5, 0, $spaceToBorder, 45, $textColor, $font, $moduleName, []);

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

$spaceToBorder of type double is incompatible with the type integer expected by parameter $x of imagefttext().

        imagecopy($imageModule, $imageIcon, 29, 2, 0, 0, 32, 32);

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

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

        //$targetImage = TDMC_UPLOAD_IMGMOD_URL . "/" . $moduleName . "_logo.png";
        $targetImage = '/uploads/' . $dirname . '/images/modules/' . $moduleName . '_logo.png';

        imagepng($imageModule, XOOPS_ROOT_PATH . $targetImage);

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

        imagedestroy($imageModule);

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

        imagedestroy($imageIcon);

        return XOOPS_URL . $targetImage;
    }