qrimage

Complexity

Total Complexity

9

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	jpg()	0	12	2
A	png()	0	18	3
A	image()	0	38	4

<?php
/*
 * PHP QR Code encoder
 *
 * Image output of code using GD2
 *
 * PHP QR Code is distributed under LGPL 3
 * Copyright (C) 2010 Dominik Dzienia <deltalab at poczta dot fm>
 *
 * This library is free software; you can redistribute it and/or
 * modify it under the terms of the GNU Lesser General Public
 * License as published by the Free Software Foundation; either
 * version 3 of the License, or any later version.
 *
 * This library is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
 * Lesser General Public License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public
 * License along with this library; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
 */

    define('QR_IMAGE', true);

    class qrimage
    {
        //----------------------------------------------------------------------
        public static function png($frame, $filename, $pixelPerPoint, $outerFrame, $saveandprint, $back_color, $fore_color)
        {
            $image = self::image($frame, $pixelPerPoint, $outerFrame, $back_color, $fore_color);

            if ($filename === false) {
                header('Content-type: image/png');
                imagepng($image);
            } else {
                if ($saveandprint === true) {
                    imagepng($image, $filename);
                    header('Content-type: image/png');
                    imagepng($image);
                } else {
                    @imagepng($image, $filename);

It seems like you do not handle an error condition for imagepng(). This can introduce security issues, and is generally not recommended.

                }
            }

            imagedestroy($image);
        }

        //----------------------------------------------------------------------
        public static function jpg($frame, $filename = false, $pixelPerPoint = 8, $outerFrame = 4, $q = 85)
        {
            $image = self::image($frame, $pixelPerPoint, $outerFrame);

            if ($filename === false) {
                header('Content-type: image/jpeg');
                imagejpeg($image, null, $q);
            } else {
                imagejpeg($image, $filename, $q);
            }

            imagedestroy($image);
        }

        //----------------------------------------------------------------------
        private static function image($frame, $pixelPerPoint = 4, $outerFrame = 4, $back_color = 0xFFFFFF, $fore_color = 0x000000)
        {
            $h = count($frame);
            $w = strlen($frame[0]);

            $imgW = $w + 2 * $outerFrame;
            $imgH = $h + 2 * $outerFrame;

            $base_image = imagecreate($imgW, $imgH);

            // convert a hexadecimal color code into decimal eps format (green = 0 1 0, blue = 0 0 1, ...)
            $r1 = round((($fore_color & 0xFF0000) >> 16), 5);
            $b1 = round((($fore_color & 0x00FF00) >> 8), 5);
            $g1 = round(($fore_color & 0x0000FF), 5);

            // convert a hexadecimal color code into decimal eps format (green = 0 1 0, blue = 0 0 1, ...)
            $r2 = round((($back_color & 0xFF0000) >> 16), 5);
            $b2 = round((($back_color & 0x00FF00) >> 8), 5);
            $g2 = round(($back_color & 0x0000FF), 5);

            $col[0] = imagecolorallocate($base_image, $r2, $b2, $g2);

$b2 of type double is incompatible with the type integer expected by parameter $green of imagecolorallocate().

$r2 of type double is incompatible with the type integer expected by parameter $red of imagecolorallocate().

$g2 of type double is incompatible with the type integer expected by parameter $blue of imagecolorallocate().

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

            $col[1] = imagecolorallocate($base_image, $r1, $b1, $g1);

            imagefill($base_image, 0, 0, $col[0]);

            for ($y = 0; $y < $h; $y++) {
                for ($x = 0; $x < $w; $x++) {
                    if ($frame[$y][$x] == '1') {
                        imagesetpixel($base_image, $x + $outerFrame, $y + $outerFrame, $col[1]);
                    }
                }
            }

            $target_image = imagecreate($imgW * $pixelPerPoint, $imgH * $pixelPerPoint);
            imagecopyresized($target_image, $base_image, 0, 0, 0, 0, $imgW * $pixelPerPoint, $imgH * $pixelPerPoint, $imgW, $imgH);
            imagedestroy($base_image);

            return $target_image;
        }
    }