KonstantinKuklin / AsseticStaticGzipBundle

Command/DumpCommand.php

<?php

namespace Symfony\Bundle\AsseticBundle\Command;

use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class GzipProperties for wrapping variables to file_put_contents
 *
 * @author KonstantinKuklin <konstantin.kuklin@gmail.com>
 */
class GzipProperties
{
    /**
     * Is compression enabled
     *
     * @var bool
     */
    public static $use = false;

    /**
     * Level of compression
     *
     * @var int
     */
    public static $level = 9;

    /**
     * Output flow interface
     *
     * @var OutputInterface
     */
    public static $output = null;
}

/**
 * Override standard function file_put_contents for
 * Symfony\Bundle\AsseticBundle\Command namespace
 *
 * @param string $filePath Assetic file path
 * @param string $content  Assetic file content
 */
function file_put_contents($filePath, $content)
{
    // save assetic file
    @\file_put_contents($filePath, $content);

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

    // check gzipping
    if (GzipProperties::$use) {

        // check gzip extension
        if (!function_exists("gzencode")) {
            throw new \RuntimeException('Unable to find Zlib library');
        }

        $filePath .= '.gz';
        GzipProperties::$output->writeln(
            sprintf(
                '<comment>%s</comment> <info>[gzipped file+]</info> %s',
                date('H:i:s'),
                $filePath
            )
        );

        if (false === @\file_put_contents($filePath, gzencode($content, GzipProperties::$level))) {
            throw new \RuntimeException('Unable to write file ' . $filePath);
        }
    }
}

namespace KonstantinKuklin\AsseticStaticGzipBundle\Command;

use Symfony\Bundle\AsseticBundle\Command\DumpCommand as BaseCommand;

USE declarations must go after the first namespace declaration

use Symfony\Bundle\AsseticBundle\Command\GzipProperties;

USE declarations must go after the first namespace declaration

use Symfony\Component\Console\Input\InputInterface;

USE declarations must go after the first namespace declaration

use Symfony\Component\Console\Output\OutputInterface;

USE declarations must go after the first namespace declaration

/**
 * Override assetic bundle command for gzip purposes.
 *
 * @author Konstantin Kuklin <konstantin.kuklin@gmail.com>
 */
class DumpCommand extends BaseCommand

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * {@inheritdoc}
     */
    protected function initialize(InputInterface $input, OutputInterface $output)
    {
        // save properties
        GzipProperties::$use = $this->getContainer()->getParameter('assetic_static_gzip.use');
        GzipProperties::$level = $this->getContainer()->getParameter('assetic_static_gzip.level');
        GzipProperties::$output = $output;

        parent::initialize($input, $output);
    }
}