mrcnpdlk/grandstream-xmlapp

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

Open

    private function resizeImage(int $width = 100, int $height = null)
    {
        if (!file_exists($this->sImgPath) || !is_readable($this->sImgPath)) {
            throw new Exception('Image not exists');
        }

Missing class import via use statement (line '141', column '47').

Open

        $imagick->setImageBackgroundColor(new \ImagickPixel('white'));

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

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '151', column '23').

Open

        $canvas = new \Imagick();

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

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '152', column '48').

Open

        $canvas->newImage($width, $height, new \ImagickPixel('white'));

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

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '123', column '24').

Open

        $imagick = new \Imagick(realpath($this->sImgPath));

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

function make() {
    return new \stdClass();
}

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

Open

        } else {//za wysoki
            $imgHeight = $height;
            $imgWidth  = intval($height * $origFactor);
        }

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

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