Zenify / DoctrineMigrations

src/EventSubscriber/SetConsoleOutputEventSubscriber.php

<?php

/**
 * This file is part of Zenify
 * Copyright (c) 2014 Tomas Votruba (http://tomasvotruba.cz)
 */

namespace Zenify\DoctrineMigrations\EventSubscriber;

use Doctrine\DBAL\Migrations\Tools\Console\Command\AbstractCommand;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\ConsoleEvents;
use Symfony\Component\Console\Event\ConsoleCommandEvent;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Zenify\DoctrineMigrations\OutputWriter;

As per PSR2, there should be exactly one blank line after the last USE statement, 2 were found though.

final class SetConsoleOutputEventSubscriber implements EventSubscriberInterface
{

	/**
	 * @var OutputWriter
	 */
	private $outputWriter;


	public function __construct(OutputWriter $outputWriter)
	{
		$this->outputWriter = $outputWriter;
	}


	/**
	 * {@inheritdoc}
	 */
	public static function getSubscribedEvents()
	{
		return [ConsoleEvents::COMMAND => 'setOutputWriter'];
	}


	public function setOutputWriter(ConsoleCommandEvent $event)
	{
		$command = $event->getCommand();
		if ( ! $this->isMigrationCommand($command)) {
			return;
		}

		$this->outputWriter->setConsoleOutput($event->getOutput());
	}


	/**
	 * @return bool
	 */
	private function isMigrationCommand(Command $command)
	{
		return $command instanceof AbstractCommand;
	}

}

According to PSR2, the closing brace of classes should be placed on the next line directly after the body.