RunOpenCode\Backup\Processor\GzipArchiveProcessor

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	process()	0	36	4

<?php
/*
 * This file is part of the Backup package, an RunOpenCode project.
 *
 * (c) 2015 RunOpenCode
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * This project is fork of "kbond/php-backup", for full credits info, please
 * view CREDITS file that was distributed with this source code.
 */
namespace RunOpenCode\Backup\Processor;

use RunOpenCode\Backup\Backup\File;
use RunOpenCode\Backup\Contract\EventDispatcherAwareInterface;
use RunOpenCode\Backup\Contract\FileInterface;
use RunOpenCode\Backup\Contract\LoggerAwareInterface;
use RunOpenCode\Backup\Contract\ProcessorInterface;
use RunOpenCode\Backup\Event\BackupEvent;
use RunOpenCode\Backup\Event\EventDispatcherAwareTrait;
use RunOpenCode\Backup\Exception\ProcessorException;
use RunOpenCode\Backup\Log\LoggerAwareTrait;
use Symfony\Component\Process\ProcessBuilder;

/**
 * Class GzipArchiveProcessor
 *
 * Gzip archive processor combines all backup file into single gz compressed archive.
 *
 * @package RunOpenCode\Backup\Processor
 */
class GzipArchiveProcessor implements ProcessorInterface, EventDispatcherAwareInterface, LoggerAwareInterface
{

    use EventDispatcherAwareTrait;
    use LoggerAwareTrait;

    /**
     * @var string
     */
    protected $filename;

    /**
     * @var string
     */
    protected $flags;

    public function __construct($flags = '-czvf', $filename = 'archive.tar.gz')
    {
        $this->filename = $filename;
        $this->flags = $flags;
    }

    /**
     * {@inheritdoc}
     */
    public function process(array $files)
    {
        $tmpFile = tempnam(sys_get_temp_dir(), 'gzip-archive-processor');

        $processBuilder = new ProcessBuilder();

        $processBuilder->add('tar');

        if (!is_null($this->flags)) {
            $processBuilder->add($this->flags);
        }

        $processBuilder->add($tmpFile);

        /**
         * @var FileInterface $backup
         */
        foreach ($files as $backup) {
            $processBuilder->add($backup->getPath());
        }

        $process = $processBuilder->getProcess();

        $process->run();

        if (!$process->isSuccessful()) {
            $this->getLogger()->error('Unable to create gzip archive.');
            throw new ProcessorException();
        }

        $this->getEventDispatcher()->addListener(BackupEvent::TERMINATE, function() use ($tmpFile) {
            @unlink($tmpFile);

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

        });

        return array(File::fromLocal($tmpFile, dirname($tmpFile), $this->filename));
    }
}