intelligently_scale_image()

Complexity

Conditions	8
Paths	64

Size

Total Lines	58
Code Lines	38

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function intelligently_scale_image($sourcefile, $fw, $fh) {
    $gd_info = gd_info();
    // libgd version numbers seem to be always three numbers
    preg_match('/(\d).(\d).(\d)/', $gd_info['GD Version'], $match);
    $newGD = ($match[1]>=2);

    list($ow, $oh, $from_type) = getimagesize($sourcefile);
    switch($from_type) {
    case 1: // GIF
        $srcImage = imageCreateFromGif($sourcefile);

Calls to inbuilt PHP functions must be lowercase; expected "imagecreatefromgif" but found "imageCreateFromGif"

        break;
    case 2: // JPG
        $srcImage = imageCreateFromJpeg($sourcefile);

Calls to inbuilt PHP functions must be lowercase; expected "imagecreatefromjpeg" but found "imageCreateFromJpeg"

        break;
    case 3: // PNG
        $srcImage = imageCreateFromPng($sourcefile);

Calls to inbuilt PHP functions must be lowercase; expected "imagecreatefrompng" but found "imageCreateFromPng"

        break;
    }

    $tempw = $fw;
    $temph = number_format((($oh*$fw)/$ow), 0);

    if($temph < $fh) {
        $tempw = number_format((($ow*$fh)/$oh), 0);
        $temph = $fh;
    }

    if ($newGD){
        $tempImage = imageCreateTrueColor($tempw, $temph);

Calls to inbuilt PHP functions must be lowercase; expected "imagecreatetruecolor" but found "imageCreateTrueColor"

        // Seems not to work:
        // imageAntiAlias($tempImage, true);
        imagecopyresampled($tempImage, $srcImage, 0, 0, 0, 0, $tempw, $temph, $ow, $oh);

The variable $srcImage does not seem to be defined for all execution paths leading up to this point.

    } else {
        $tempImage = imageCreate($tempw, $temph);

Calls to inbuilt PHP functions must be lowercase; expected "imagecreate" but found "imageCreate"

        imagecopyresized($tempImage, $srcImage, 0, 0, 0, 0, $tempw, $temph, $ow, $oh);
    }

Functions must not contain multiple empty lines in a row; found 2 empty lines

    // Calculate offsets
    if($temph < $fh) {
        $offsety = number_format(($temph/2)-($fh/2), 0);
        $offsetx = 0;
    } else {
        $offsety = 0;
        $offsetx = number_format(($tempw/2)-($fw/2), 0);
    }

    if ($newGD){
        $destImage = imageCreateTrueColor($fw, $fh);

Calls to inbuilt PHP functions must be lowercase; expected "imagecreatetruecolor" but found "imageCreateTrueColor"

        // Seems not to work:
        // imageAntiAlias($tempImage, true);
        imagecopyresampled($destImage, $tempImage, 0, 0, $offsetx, $offsety, $fw, $fh, $fw, $fh);

It seems like $offsetx can also be of type string; however, parameter $src_x of imagecopyresampled() does only seem to accept integer, maybe add an additional type check?

It seems like $offsety can also be of type string; however, parameter $src_y of imagecopyresampled() does only seem to accept integer, maybe add an additional type check?

    } else {
        $destImage = imageCreate($fw, $fh);

Calls to inbuilt PHP functions must be lowercase; expected "imagecreate" but found "imageCreate"

        imagecopyresized($destImage, $tempImage, 0, 0, $offsetx, $offsety, $fw, $fh, $fw, $fh);

It seems like $offsety can also be of type string; however, parameter $src_y of imagecopyresized() does only seem to accept integer, maybe add an additional type check?

It seems like $offsetx can also be of type string; however, parameter $src_x of imagecopyresized() does only seem to accept integer, maybe add an additional type check?

    }

    return $destImage; //imageJpeg($destImage, $destfile, $jpegquality);
}