Divergence\Models\Media\PDF

Complexity

Total Complexity

12

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	analyzeFile()	0	13	2
A	getImage()	0	10	3
A	__classLoaded()	0	8	1
A	getValue()	0	22	6

<?php
namespace Divergence\Models\Media;

use Exception;

class PDF extends Media
{
    // configurables
    public static $extractPageCommand = 'convert \'%1$s[%2$u]\' JPEG:- 2>/dev/null'; // 1=pdf path, 2=page
    public static $extractPageIndex = 0;


    // magic methods
    public static function __classLoaded()
    {
        $className = get_called_class();

        Media::$mimeHandlers['application/pdf'] = $className;
        Media::$mimeHandlers['application/postscript'] = $className;
        Media::$mimeHandlers['image/svg+xml'] = $className;
        parent::__classLoaded();

The method __classLoaded() does not exist on Divergence\Models\Media\Media. Maybe you want to declare this class abstract?

    }


    public function getValue($name)
    {
        switch ($name) {
            case 'ThumbnailMIMEType':
                return 'image/png';

            case 'Extension':

                switch ($this->MIMEType) {
                    case 'application/pdf':
                        return 'pdf';
                    case 'application/postscript':
                        return 'eps';
                    case 'image/svg+xml':
                        return 'svg';
                    default:
                        throw new Exception('Unable to find document extension for mime-type: '.$this->MIMEType);
                }

                // no break
            default:
                return parent::getValue($name);
        }
    }


    // public methods
    public function getImage($sourceFile = null)
    {
        if (!isset($sourceFile)) {
            $sourceFile = $this->FilesystemPath ? $this->FilesystemPath : $this->BlankPath;

The property BlankPath does not exist on Divergence\Models\Media\PDF. Since you implemented __get, consider adding a @property annotation.

The property FilesystemPath does not exist on Divergence\Models\Media\PDF. Since you implemented __get, consider adding a @property annotation.

        }

        $cmd = sprintf(static::$extractPageCommand, $sourceFile, static::$extractPageIndex);
        $fileImage = imagecreatefromstring(shell_exec($cmd));

        return $fileImage;
    }

    // static methods
    public static function analyzeFile($filename, $mediaInfo = [])
    {
        $cmd = sprintf(static::$extractPageCommand, $filename, static::$extractPageIndex);
        $pageIm = @imagecreatefromstring(shell_exec($cmd));

        if (!$pageIm) {

$pageIm is of type resource, thus it always evaluated to false.

            throw new Exception('Unable to convert PDF, ensure that imagemagick is installed on the server');
        }

        $mediaInfo['width'] = imagesx($pageIm);
        $mediaInfo['height'] = imagesy($pageIm);

        return $mediaInfo;
    }
}