tkanstantsin\fileupload\formatter\adapter\PsImageOptimizer

Complexity

Total Complexity

5

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getOptimizer()	0	3	1
A	getTempDir()	0	3	1
A	init()	0	5	2
A	exec()	0	10	1

<?php
declare(strict_types=1);

namespace tkanstantsin\fileupload\formatter\adapter;

use ImageOptimizer\Optimizer;
use ImageOptimizer\OptimizerFactory;
use tkanstantsin\fileupload\model\BaseObject;
use tkanstantsin\fileupload\model\IFile;

/**
 * Class ImageOptimizer
 * Optimize images using `ps/image-optimizer`
 * @see https://github.com/psliwa/image-optimizer
 */
class PsImageOptimizer extends BaseObject implements IFormatAdapter
{
    public const DEFAULT_CONFIG = [
        'ignore_errors'                     => true,
        'execute_only_first_jpeg_optimizer' => false,
        'execute_only_first_png_optimizer'  => false,
        'jpegoptim_options'                 => ['-m90', '--strip-all', '--all-progressive'],
        'jpegtran_options'                  => ['-optimize', '-progressive', '-perfect'],
    ];

    /**
     * @var string|null
     */
    public $tempDir;

    /**
     * @var array
     */
    public $optimizerConfig = self::DEFAULT_CONFIG;

    /**
     * @inheritdoc
     */
    public function init(): void
    {
        parent::init();
        if (!class_exists(OptimizerFactory::class)) {
            trigger_error(sprintf('%s class not found', OptimizerFactory::class));
        }
    }


    /**
     * Applies filters or something to content and return it
     *
     * @param IFile $file
     * @param       $content
     *
     * @return mixed
     * @throws \ImageOptimizer\Exception\Exception
     */
    public function exec(IFile $file, $content)
    {
        $tmpPath = tempnam($this->getTempDir(), '');
        file_put_contents($tmpPath, $content);
        $this->getOptimizer()->optimize($tmpPath);
        $content = file_get_contents($tmpPath);

        @unlink($tmpPath);

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

        return $content;
    }

    /**
     * @return string
     */
    protected function getTempDir(): string
    {
        return (string) ($this->tempDir ?? sys_get_temp_dir());
    }

    /**
     * @param string $name
     *
     * @return Optimizer
     * @throws \ImageOptimizer\Exception\Exception
     */
    protected function getOptimizer($name = OptimizerFactory::OPTIMIZER_SMART): Optimizer
    {
        return (new OptimizerFactory($this->optimizerConfig))->get($name);
    }
}