update coding style

src/Bridges/DoctrineDbal/DoctrineAdapter.php

@@ -70,5 +70,4 @@ public function escapeIdentifier($value)
 	{
 		return $this->conn->quoteIdentifier($value);
 	}
-
 }

src/Bridges/DoctrineOrm/StructureDiffGenerator.php

@@ -18,7 +18,6 @@
 
 class StructureDiffGenerator implements IDiffGenerator
 {
-
 	/** @var EntityManagerInterface */
 	private $entityManager;
 
@@ -27,10 +26,10 @@ class StructureDiffGenerator implements IDiffGenerator
 
 
 	/**
-	 * @param EntityManagerInterface $entityManager
-	 * @param string|NULL   $ignoredQueriesFile
+	 * @param  EntityManagerInterface $entityManager
+	 * @param  string|NULL            $ignoredQueriesFile
 	 */
-	public function __construct(EntityManagerInterface $entityManager, $ignoredQueriesFile = NULL)
+	public function __construct(EntityManagerInterface $entityManager, $ignoredQueriesFile = null)
 	{
 		$this->entityManager = $entityManager;
 		$this->ignoredQueriesFile = $ignoredQueriesFile;
@@ -70,7 +69,7 @@ protected function getUpdateQueries()
 
 		$schemaTool = new SchemaTool($this->entityManager);
 		$metadata = $this->entityManager->getMetadataFactory()->getAllMetadata();
-		$queries = $schemaTool->getUpdateSchemaSql($metadata, TRUE);
+		$queries = $schemaTool->getUpdateSchemaSql($metadata, true);
 
 		return $queries;
 	}
@@ -81,7 +80,7 @@ protected function getUpdateQueries()
 	 */
 	protected function getIgnoredQueries()
 	{
-		if ($this->ignoredQueriesFile === NULL) {
+		if ($this->ignoredQueriesFile === null) {
 			return [];
 		}
 
@@ -90,5 +89,4 @@ protected function getIgnoredQueries()
 
 		return $queries;
 	}
-
 }

src/Bridges/NetteDI/MigrationsExtension.php

@@ -9,6 +9,7 @@
 
 namespace Nextras\Migrations\Bridges\NetteDI;
 
+use Doctrine;
 use Nette;
 use Nette\DI\ContainerBuilder;
 use Nette\Utils\Strings;
@@ -26,15 +27,15 @@ class MigrationsExtension extends Nette\DI\CompilerExtension
 
 	/** @var array */
 	public $defaults = [
-		'dir' => NULL,
+		'dir' => null,
 		'phpParams' => [],
-		'driver' => NULL,
-		'dbal' => NULL,
+		'driver' => null,
+		'dbal' => null,
 		'printer' => 'console',
-		'groups' => NULL,        // null|array
-		'diffGenerator' => TRUE, // false|doctrine
-		'withDummyData' => FALSE,
-		'ignoredQueriesFile' => NULL,
+		'groups' => null,        // null|array
+		'diffGenerator' => true, // false|doctrine
+		'withDummyData' => false,
+		'ignoredQueriesFile' => null,
 	];
 
 	/** @var array */
@@ -59,6 +60,7 @@ class MigrationsExtension extends Nette\DI\CompilerExtension
 		'psrLog' => 'Nextras\Migrations\Bridges\PsrLog\PsrLogPrinter',
 	];
 
+
 	public function loadConfiguration()
 	{
 		$config = $this->validateConfig($this->defaults);
@@ -82,7 +84,7 @@ public function loadConfiguration()
 		}
 
 		// groups
-		if ($config['groups'] === NULL) {
+		if ($config['groups'] === null) {
 			Validators::assertField($config, 'dir', 'string|Nette\PhpGenerator\PhpLiteral');
 			Validators::assertField($config, 'withDummyData', 'bool');
 			$config['groups'] = $this->createDefaultGroupConfiguration($config['dir'], $config['withDummyData']);
@@ -126,7 +128,7 @@ public function beforeCompile()
 		}
 
 		// diff generator
-		if ($config['diffGenerator'] === TRUE) {
+		if ($config['diffGenerator'] === true) {
 			if ($builder->findByType('Doctrine\ORM\EntityManagerInterface') && $builder->hasDefinition($this->prefix('group.structures'))) {
 				Validators::assertField($config, 'ignoredQueriesFile', 'null|string');
 				$diffGenerator = $this->createDoctrineStructureDiffGeneratorDefinition($config['ignoredQueriesFile']);
@@ -166,7 +168,7 @@ private function getDbalDefinition($dbal)
 				->setClass('Nextras\Migrations\IDbal')
 				->setFactory($factory);
 
-		} elseif ($dbal === NULL) {
+		} elseif ($dbal === null) {
 			return '@Nextras\Migrations\IDbal';
 
 		} else {
@@ -187,7 +189,7 @@ private function getDbalFactory($dbal)
 			return $dbal;
 
 		} else {
-			return NULL;
+			return null;
 		}
 	}
 
@@ -202,7 +204,7 @@ private function getDriverDefinition($driver, $dbal)
 				->setClass('Nextras\Migrations\IDriver')
 				->setFactory($factory);
 
-		} elseif ($driver === NULL) {
+		} elseif ($driver === null) {
 			return '@Nextras\Migrations\IDriver';
 
 		} else {
@@ -220,7 +222,7 @@ private function getDriverFactory($driver, $dbal)
 			return new Nette\DI\Statement($this->drivers[$driver], [$dbal]);
 
 		} else {
-			return NULL;
+			return null;
 		}
 	}
 
@@ -235,7 +237,7 @@ private function getPrinterDefinition($printer)
 				->setClass('Nextras\Migrations\IPrinter')
 				->setFactory($factory);
 
-		} elseif ($printer === NULL) {
+		} elseif ($printer === null) {
 			return '@Nextras\Migrations\IPrinter';
 
 		} else {
@@ -256,7 +258,7 @@ private function getPrinterFactory($printer)
 			return $printer;
 
 		} else {
-			return NULL;
+			return null;
 		}
 	}
 
@@ -293,7 +295,7 @@ private function createDefaultGroupConfiguration($dir, $withDummyData)
 
 		foreach ($groups as $groupName => $groupConfig) {
 			$serviceName = $this->prefix("diffGenerator.$groupName");
-			$diffGenerator = $builder->hasDefinition($serviceName) ? $builder->getDefinition($serviceName) : NULL;
+			$diffGenerator = $builder->hasDefinition($serviceName) ? $builder->getDefinition($serviceName) : null;
 			$groups[$groupName]['generator'] = $diffGenerator;
 		}
 
@@ -317,7 +319,7 @@ private function createGroupDefinitions(array $groups)
 			$serviceName = lcfirst(str_replace('-', '', ucwords($groupName, '-')));
 			$groupDefinitions[] = $builder->addDefinition($this->prefix("group.$serviceName"))
 				->addTag(self::TAG_GROUP, ['for' => [$this->name]])
-				->setAutowired(FALSE)
+				->setAutowired(false)
 				->setClass('Nextras\Migrations\Entities\Group')
 				->addSetup('$name', [$groupName])
 				->addSetup('$enabled', [$enabled])
@@ -336,14 +338,14 @@ private function createExtensionHandlerDefinitions($driver, $phpParams)
 
 		$sqlHandler = $builder->addDefinition($this->prefix('extensionHandler.sql'))
 			->addTag(self::TAG_EXTENSION_HANDLER, ['for' => [$this->name], 'extension' => 'sql'])
-			->setAutowired(FALSE)
+			->setAutowired(false)
 			->setClass('Nextras\Migrations\Extensions\SqlHandler')
 			->setArguments([$driver]);
 
 		$phpHandler = $builder->addDefinition($this->prefix('extensionHandler.php'))
 			->addTag(self::TAG_EXTENSION_HANDLER, ['for' => [$this->name], 'extension' => 'php'])
 			->setClass('Nextras\Migrations\Extensions\PhpHandler')
-			->setAutowired(FALSE)
+			->setAutowired(false)
 			->setArguments([$phpParams]);
 
 		return [$sqlHandler, $phpHandler];
@@ -364,7 +366,7 @@ private function createDoctrineStructureDiffGeneratorDefinition($ignoredQueriesF
 		$builder = $this->getContainerBuilder();
 
 		return $builder->addDefinition($this->prefix('diffGenerator.structures'))
-			->setAutowired(FALSE)
+			->setAutowired(false)
 			->setClass('Nextras\Migrations\IDiffGenerator')
 			->setFactory('Nextras\Migrations\Bridges\DoctrineOrm\StructureDiffGenerator')
 			->setArguments(['@Doctrine\ORM\EntityManagerInterface', $ignoredQueriesFile]);

src/Bridges/NetteDatabase/NetteAdapter.php

@@ -70,5 +70,4 @@ public function escapeIdentifier($value)
 	{
 		return $this->conn->getSupplementalDriver()->delimite($value);
 	}
-
 }

src/Bridges/NextrasDbal/NextrasAdapter.php

@@ -108,5 +108,4 @@ public function escapeIdentifier($value)
 			return $this->conn->getDriver()->convertToSql($value, IDriver::TYPE_IDENTIFIER);
 		}
 	}
-
 }

src/Bridges/SymfonyBundle/DependencyInjection/NextrasMigrationsExtension.php

@@ -45,15 +45,15 @@ public function load(array $configs, ContainerBuilder $container)
 
 		$dbalAlias = $config['dbal'];
 		$dbalDefinition = new Definition($this->dbals[$dbalAlias]);
-		$dbalDefinition->setAutowired(TRUE);
+		$dbalDefinition->setAutowired(true);
 
 		$driverAlias = $config['driver'];
 		$driverDefinition = new Definition($this->drivers[$driverAlias]);
 		$driverDefinition->setArgument('$dbal', $dbalDefinition);
 
 		$printerAlias = $config['printer'];
 		$printerDefinition = new Definition($this->printers[$printerAlias]);
-		$printerDefinition->setAutowired(TRUE);
+		$printerDefinition->setAutowired(true);
 
 		$container->addDefinitions([
 			'nextras_migrations.dbal' => $dbalDefinition,
@@ -69,11 +69,11 @@ public function load(array $configs, ContainerBuilder $container)
 
 		if ($config['diff_generator'] === 'doctrine') {
 			$structureDiffGeneratorDefinition = new Definition('Nextras\Migrations\Bridges\DoctrineOrm\StructureDiffGenerator');
-			$structureDiffGeneratorDefinition->setAutowired(TRUE);
+			$structureDiffGeneratorDefinition->setAutowired(true);
 			$structureDiffGeneratorDefinition->setArgument('$ignoredQueriesFile', $config['ignored_queries_file']);
 
 		} else {
-			$structureDiffGeneratorDefinition = NULL;
+			$structureDiffGeneratorDefinition = null;
 		}
 
 		foreach ($config['php_params'] as $phpParamKey => $phpParamValue) {

src/Bridges/SymfonyConsole/BaseCommand.php

@@ -55,5 +55,4 @@ protected function runMigrations($mode, $config)
 
 		return 0;
 	}
-
 }

src/Bridges/SymfonyConsole/ContinueCommand.php

@@ -34,5 +34,4 @@ protected function execute(InputInterface $input, OutputInterface $output)
 	{
 		return $this->runMigrations(Runner::MODE_CONTINUE, $this->config);
 	}
-
 }

src/Bridges/SymfonyConsole/CreateCommand.php

@@ -58,9 +58,9 @@ protected function configure()
 		$this->addArgument('type', InputArgument::REQUIRED, $this->getTypeArgDescription());
 		$this->addArgument('label', InputArgument::REQUIRED, 'short description');
 
-		$this->addOption('empty', NULL, InputOption::VALUE_NONE, 'create empty file');
-		$this->addOption('diff', NULL, InputOption::VALUE_NONE, 'use schema diff as file content');
-		$this->addOption('stdin', NULL, InputOption::VALUE_NONE, 'use stdin as file content');
+		$this->addOption('empty', null, InputOption::VALUE_NONE, 'create empty file');
+		$this->addOption('diff', null, InputOption::VALUE_NONE, 'use schema diff as file content');
+		$this->addOption('stdin', null, InputOption::VALUE_NONE, 'use stdin as file content');
 	}
 
 
@@ -171,24 +171,24 @@ class_exists('Symfony\Component\String\Slugger\AsciiSlugger')
 
 
 	/**
-	 * @param  string $dir
+	 * @param  string      $dir
 	 * @param  string|NULL $found
 	 * @return bool
 	 */
-	protected function hasNumericSubdirectory($dir, & $found)
+	protected function hasNumericSubdirectory($dir, &$found)
 	{
 		$items = @scandir($dir); // directory may not exist
 
 		if ($items) {
 			foreach ($items as $item) {
 				if ($item !== '.' && $item !== '..' && is_dir($dir . '/' . $item)) {
 					$found = $dir . '/' . $item;
-					return TRUE;
+					return true;
 				}
 			}
 		}
 
-		return FALSE;
+		return false;
 	}
 
 
@@ -249,7 +249,7 @@ protected function getFileContentSource(InputInterface $input)
 	 */
 	protected function getFileContent(Group $group, $source)
 	{
-		if ($source === self::CONTENT_SOURCE_DIFF && $group->generator !== NULL) {
+		if ($source === self::CONTENT_SOURCE_DIFF && $group->generator !== null) {
 			return $group->generator->generateContent();
 
 		} elseif ($source === self::CONTENT_SOURCE_STDIN) {
@@ -269,12 +269,11 @@ protected function getFileContent(Group $group, $source)
 	 */
 	protected function createFile($path, $content, OutputInterface $output)
 	{
-		@mkdir(dirname($path), 0777, TRUE); // directory may already exist
+		@mkdir(dirname($path), 0777, true); // directory may already exist
 
 		if (file_put_contents("$path.tmp", $content) !== strlen($content) || !rename("$path.tmp", $path)) {
 			$output->writeln("Unable to write to '$path'.");
 			exit(1);
 		}
 	}
-
 }

src/Bridges/SymfonyConsole/ResetCommand.php

@@ -22,6 +22,7 @@ class ResetCommand extends BaseCommand
 	/** @var string */
 	protected static $defaultDescription = 'Drops current database and recreates it from scratch';
 
+
 	protected function configure()
 	{
 		$this->setName(self::$defaultName);
@@ -34,5 +35,4 @@ protected function execute(InputInterface $input, OutputInterface $output)
 	{
 		return $this->runMigrations(Runner::MODE_RESET, $this->config);
 	}
-
 }

src/Configurations/Configuration.php

@@ -27,8 +27,8 @@ class Configuration implements IConfiguration
 
 
 	/**
-	 * @param Group[]             $groups
-	 * @param IExtensionHandler[] $extensionHandlers (extension => IExtensionHandler)
+	 * @param  Group[]             $groups
+	 * @param  IExtensionHandler[] $extensionHandlers (extension => IExtensionHandler)
 	 */
 	public function __construct(array $groups, array $extensionHandlers)
 	{