mambax7/extgallery

Method image_type_to_mime_type has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function image_type_to_mime_type($type)
{
    switch ($type) {
        case IMAGETYPE_GIF:
            return 'image/gif';

Avoid too many return statements within this method.

Open

            return 'image/bmp';

Avoid too many return statements within this method.

Open

            return 'image/tiff';

Avoid too many return statements within this method.

Open

            return 'image/xbm';

Avoid too many return statements within this method.

Open

            return 'image/jp2';

Avoid too many return statements within this method.

Open

            return 'application/x-shockwave-flash';

Avoid too many return statements within this method.

Open

            return 'application/octect-stream';

Avoid too many return statements within this method.

Open

            return 'image/iff';

Avoid too many return statements within this method.

Open

            return 'image/vnd.wap.wbmp';

The function image_type_to_mime_type() has a Cyclomatic Complexity of 17. The configured cyclomatic complexity threshold is 10.

Open

function image_type_to_mime_type($type)
{
    switch ($type) {
        case IMAGETYPE_GIF:
            return 'image/gif';

• 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;
            }
        }
    }
}

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 15 and the first side effect is on line 8.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check