Function GD2ICOstring has a Cognitive Complexity of 27 (exceeds 5 allowed). Consider refactoring.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up

Method GD2ICOstring has 74 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

The method GD2ICOstring() has an NPath complexity of 728. The configured NPath complexity threshold is 200.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method GD2ICOstring() has a Cyclomatic Complexity of 14. The configured cyclomatic complexity threshold is 10.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

Avoid using undefined variables such as '$ImageHeights' which will lead to PHP notices.

Open

            for ($y = $ImageHeights[$key] - 1; $y >= 0; $y--) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icXOR' which will lead to PHP notices.

Open

            $icXOR[$key] = '';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icAND' which will lead to PHP notices.

Open

            $icAND[$key] = '';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$totalcolors' which will lead to PHP notices.

Open

            $totalcolors[$key]  = imagecolorstotal($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icXOR' which will lead to PHP notices.

Open

                        $icXOR[$key] .= chr($b) . chr($g) . chr($r) . chr($a);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);      // LONG   biWidth;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] = '';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$bpp' which will lead to PHP notices.

Open

                    if ($bpp[$key] == 32) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icXOR' which will lead to PHP notices.

Open

                        $icXOR[$key] .= chr($b) . chr($g) . chr($r);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icANDmask' which will lead to PHP notices.

Open

                    $icANDmask[$key][$y] .= '0';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icAND' which will lead to PHP notices.

Open

                    $icAND[$key] .= chr(bindec(str_pad(substr($scanlinemaskbits, $i, 8), 8, '0', STR_PAD_LEFT)));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= "\x28\x00\x00\x00";                              // DWORD  biSize;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icANDmask' which will lead to PHP notices.

Open

                        @$icANDmask[$key][$y] .= '0';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$bpp' which will lead to PHP notices.

Open

            $bpp[$key]          = imageistruecolor($gd_image) ? 32 : 24;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageWidths' which will lead to PHP notices.

Open

                for ($x = 0; $x < $ImageWidths[$key]; $x++) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageHeights' which will lead to PHP notices.

Open

            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icANDmask' which will lead to PHP notices.

Open

                while (strlen($icANDmask[$key][$y]) % 32) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageWidths' which will lead to PHP notices.

Open

            $ImageWidths[$key]  = imagesx($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$bpp' which will lead to PHP notices.

Open

                    } elseif ($bpp[$key] == 24) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icANDmask' which will lead to PHP notices.

Open

                        @$icANDmask[$key][$y] .= '1';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icANDmask' which will lead to PHP notices.

Open

            foreach ($icANDmask[$key] as $y => $scanlinemaskbits) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageWidths' which will lead to PHP notices.

Open

            $biSizeImage = $ImageWidths[$key] * $ImageHeights[$key] * ($bpp[$key] / 8);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageHeights' which will lead to PHP notices.

Open

            $biSizeImage = $ImageWidths[$key] * $ImageHeights[$key] * ($bpp[$key] / 8);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$bpp' which will lead to PHP notices.

Open

            $biSizeImage = $ImageWidths[$key] * $ImageHeights[$key] * ($bpp[$key] / 8);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageWidths' which will lead to PHP notices.

Open

            $icondata .= chr($ImageWidths[$key]);                     // bWidth;          // Width, in pixels, of the image

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageHeights' which will lead to PHP notices.

Open

            $icondata .= chr($ImageHeights[$key]);                    // bHeight;         // Height, in pixels, of the image

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$bpp' which will lead to PHP notices.

Open

            $icondata .= chr($bpp[$key]) . "\x00";                      // wBitCount;       // Bits per pixel

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                              // LONG   biXPelsPerMeter;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                              // DWORD  biCompression;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= "\x01\x00";                                      // WORD   biPlanes;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                              // DWORD  biClrImportant;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageWidths' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);      // LONG   biWidth;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$ImageHeights' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                              // LONG   biYPelsPerMeter;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icAND' which will lead to PHP notices.

Open

            $dwImageOffset += strlen($icAND[$key]);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$totalcolors' which will lead to PHP notices.

Open

            $icondata .= chr($totalcolors[$key]);                     // bColorCount;     // Number of colors in image (0 if >=8bpp)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icXOR' which will lead to PHP notices.

Open

            $dwBytesInRes = 40 + strlen($icXOR[$key]) + strlen($icAND[$key]);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icXOR' which will lead to PHP notices.

Open

            $dwImageOffset += strlen($icXOR[$key]);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$bpp' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= chr($bpp[$key]) . "\x00";                          // wBitCount;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icXOR' which will lead to PHP notices.

Open

            $icondata .= $icXOR[$key];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                              // DWORD  biClrUsed;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= chr($bpp[$key]) . "\x00";                          // wBitCount;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4);            // DWORD  biSizeImage;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $dwImageOffset += strlen($BitmapInfoHeader[$key]);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icAND' which will lead to PHP notices.

Open

            $dwBytesInRes = 40 + strlen($icXOR[$key]) + strlen($icAND[$key]);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$BitmapInfoHeader' which will lead to PHP notices.

Open

            $icondata .= $BitmapInfoHeader[$key];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$icAND' which will lead to PHP notices.

Open

            $icondata .= $icAND[$key];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Remove error control operator '@' on line 51.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Remove error control operator '@' on line 53.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Avoid using static access to class 'phpthumb_functions' in method 'GD2ICOstring'.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);      // LONG   biWidth;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method GD2ICOstring uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                    } else {
                        @$icANDmask[$key][$y] .= '0';
                    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'phpthumb_functions' in method 'GD2ICOstring'.

Open

                    $argb = phpthumb_functions::GetPixelColor($gd_image, $x, $y);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'phpthumb_functions' in method 'GD2ICOstring'.

Open

            $icondata      .= phpthumb_functions::LittleEndian2String($dwImageOffset, 4);      // dwImageOffset;   // Where in the file is this image?

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'phpthumb_functions' in method 'GD2ICOstring'.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'phpthumb_functions' in method 'GD2ICOstring'.

Open

            $icondata     .= phpthumb_functions::LittleEndian2String($dwBytesInRes, 4);       // dwBytesInRes;    // How many bytes in this resource?

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'phpthumb_functions' in method 'GD2ICOstring'.

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4);            // DWORD  biSizeImage;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'phpthumb_functions' in method 'GD2ICOstring'.

Open

        $icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2);  // idCount;      // How many images?

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class phpthumb_ico

• Exclude checks
  • Disable engine
  • Disable check

The class phpthumb_ico is not named in CamelCase.

Open

class phpthumb_ico
{

    // removed for XOOPS
    //function phpthumb_ico() {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class class_name {
}

Avoid variables with short names like $g. Configured minimum length is 3.

Open

                    $g    = $argb['green'];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The parameter $gd_image_array is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething($user_name) {
    }
}

Avoid variables with short names like $a. Configured minimum length is 3.

Open

                    $a    = round(255 * ((127 - $argb['alpha']) / 127));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $r. Configured minimum length is 3.

Open

                    $r    = $argb['red'];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $b. Configured minimum length is 3.

Open

                    $b    = $argb['blue'];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Line exceeds 120 characters; contains 132 characters

Open

            $icondata .= chr($totalcolors[$key]);                     // bColorCount;     // Number of colors in image (0 if >=8bpp)

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 150 characters

Open

            $icondata      .= phpthumb_functions::LittleEndian2String($dwImageOffset, 4);      // dwImageOffset;   // Where in the file is this image?

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 123 characters

Open

            $icondata .= chr($ImageWidths[$key]);                     // bWidth;          // Width, in pixels, of the image

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);      // LONG   biWidth;

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 127 characters

Open

            $BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 149 characters

Open

            $icondata     .= phpthumb_functions::LittleEndian2String($dwBytesInRes, 4);       // dwBytesInRes;    // How many bytes in this resource?

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

            $icondata .= chr($ImageHeights[$key]);                    // bHeight;         // Height, in pixels, of the image

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

        $icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2);  // idCount;      // How many images?

• Exclude checks
  • Disable engine
  • Disable check

Class name "phpthumb_ico" is not in camel caps format

Open

class phpthumb_ico

• Exclude checks
  • Disable engine
  • Disable check

The variable $gd_image_array is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageHeights is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image_array is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image_array is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageWidths is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageWidths is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageHeights is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageHeights is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image_array is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image_array is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image_array is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageWidths is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageHeights is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageWidths is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $gd_image is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageHeights is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $BitmapInfoHeader is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ImageWidths is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The method GD2ICOstring is not named in camelCase.

Open

    public function GD2ICOstring(&$gd_image_array)
    {
        foreach ($gd_image_array as $key => $gd_image) {
            $ImageWidths[$key]  = imagesx($gd_image);
            $ImageHeights[$key] = imagesy($gd_image);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}