doctrine / migrations

lib/Doctrine/DBAL/Migrations/Configuration/Configuration.php

Spacing +13 added lines, -13 removed lines

@@ -196,10 +196,10 @@ 
      */
     public function validate()
     {
-        if ( ! $this->migrationsNamespace) {
+        if (!$this->migrationsNamespace) {
             throw MigrationException::migrationsNamespaceRequired();
         }
-        if ( ! $this->migrationsDirectory) {
+        if (!$this->migrationsDirectory) {
             throw MigrationException::migrationsDirectoryRequired();
         }
     }
@@ -374,7 +374,7 @@ 
     public function setMigrationsFinder(MigrationFinderInterface $finder)
     {
         if (($this->migrationsAreOrganizedByYear || $this->migrationsAreOrganizedByYearAndMonth)
-            && ! ($finder instanceof MigrationDeepFinderInterface)) {
+            && !($finder instanceof MigrationDeepFinderInterface)) {
             throw MigrationException::configurationIncompatibleWithFinder(
                 'organize-migrations',
                 $finder
@@ -471,7 +471,7 @@ 
             $this->registerMigrationsFromDirectory($this->getMigrationsDirectory());
         }
 
-        if ( ! isset($this->migrations[$version])) {
+        if (!isset($this->migrations[$version])) {
             throw MigrationException::unknownMigrationVersion($version);
         }
 
@@ -507,7 +507,7 @@ 
         $this->createMigrationTable();
 
         $version = $this->connection->fetchColumn(
-            "SELECT " . $this->migrationsColumnName . " FROM " . $this->migrationsTableName . " WHERE " . $this->migrationsColumnName . " = ?",
+            "SELECT ".$this->migrationsColumnName." FROM ".$this->migrationsTableName." WHERE ".$this->migrationsColumnName." = ?",
             [$version->getVersion()]
         );
 
@@ -524,7 +524,7 @@ 
         $this->connect();
         $this->createMigrationTable();
 
-        $ret = $this->connection->fetchAll("SELECT " . $this->migrationsColumnName . " FROM " . $this->migrationsTableName);
+        $ret = $this->connection->fetchAll("SELECT ".$this->migrationsColumnName." FROM ".$this->migrationsTableName);
 
         return array_map('current', $ret);
     }
@@ -564,12 +564,12 @@ 
         }
 
         $where = null;
-        if ( ! empty($this->migrations)) {
+        if (!empty($this->migrations)) {
             $migratedVersions = [];
             foreach ($this->migrations as $migration) {
                 $migratedVersions[] = sprintf("'%s'", $migration->getVersion());
             }
-            $where = " WHERE " . $this->migrationsColumnName . " IN (" . implode(', ', $migratedVersions) . ")";
+            $where = " WHERE ".$this->migrationsColumnName." IN (".implode(', ', $migratedVersions).")";
         }
 
         $sql = sprintf(
@@ -624,7 +624,7 @@ 
         $versions = array_map('strval', array_keys($this->migrations));
         array_unshift($versions, '0');
         $offset = array_search((string) $version, $versions);
-        if ($offset === false || ! isset($versions[$offset + $delta])) {
+        if ($offset === false || !isset($versions[$offset + $delta])) {
             // Unknown version or delta out of bounds.
             return null;
         }
@@ -705,7 +705,7 @@ 
         $this->connect();
         $this->createMigrationTable();
 
-        $result = $this->connection->fetchColumn("SELECT COUNT(" . $this->migrationsColumnName . ") FROM " . $this->migrationsTableName);
+        $result = $this->connection->fetchColumn("SELECT COUNT(".$this->migrationsColumnName.") FROM ".$this->migrationsTableName);
 
         return $result !== false ? $result : 0;
     }
@@ -891,7 +891,7 @@ 
      */
     private function ensureOrganizeMigrationsIsCompatibleWithFinder()
     {
-        if ( ! ($this->migrationFinder instanceof MigrationDeepFinderInterface)) {
+        if (!($this->migrationFinder instanceof MigrationDeepFinderInterface)) {
             throw MigrationException::configurationIncompatibleWithFinder(
                 'organize-migrations',
                 $this->migrationFinder
@@ -913,7 +913,7 @@ 
     private function shouldExecuteMigration($direction, Version $version, $to, $migrated)
     {
         if ($direction === Version::DIRECTION_DOWN) {
-            if ( ! in_array($version->getVersion(), $migrated)) {
+            if (!in_array($version->getVersion(), $migrated)) {
                 return false;
             }
 
@@ -934,7 +934,7 @@ 
      */
     private function ensureMigrationClassExists($class)
     {
-        if ( ! class_exists($class)) {
+        if (!class_exists($class)) {
             throw MigrationException::migrationClassNotFound($class, $this->getMigrationsNamespace());
         }
     }

lib/Doctrine/DBAL/Migrations/Configuration/XmlConfiguration.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         libxml_use_internal_errors(true);
         $xml = new \DOMDocument();
         $xml->load($file);
-        if ( ! $xml->schemaValidate(__DIR__ . DIRECTORY_SEPARATOR . "XML" . DIRECTORY_SEPARATOR . "configuration.xsd")) {
+        if (!$xml->schemaValidate(__DIR__.DIRECTORY_SEPARATOR."XML".DIRECTORY_SEPARATOR."configuration.xsd")) {
             libxml_clear_errors();
             throw MigrationException::configurationNotValid('XML configuration did not pass the validation test.');
         }

lib/Doctrine/DBAL/Migrations/Configuration/YamlConfiguration.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     {
         $config = Yaml::parse(file_get_contents($file));
 
-        if ( ! is_array($config)) {
+        if (!is_array($config)) {
             throw new \InvalidArgumentException('Not valid configuration.');
         }
 

lib/Doctrine/DBAL/Migrations/Event/Listeners/AutoCommitListener.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function onMigrationsMigrated(MigrationsEventArgs $args)
     {
         $conn = $args->getConnection();
-        if ( ! $args->isDryRun() && ! $conn->isAutoCommit()) {
+        if (!$args->isDryRun() && !$conn->isAutoCommit()) {
             $conn->commit();
         }
     }

lib/Doctrine/DBAL/Migrations/OutputWriter.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function __construct(\Closure $closure = null)
     {
         if ($closure === null) {
-            $closure = function ($message) {
+            $closure = function($message) {
             };
         }
         $this->closure = $closure;

lib/Doctrine/DBAL/Migrations/Tools/Console/Command/GenerateCommand.php

Doc Comments +5 added lines

@@ -106,6 +106,11 @@
         return $this->instanceTemplate;
     }
 
+    /**
+     * @param string $version
+     * @param string $up
+     * @param string $down
+     */
     protected function generateMigration(Configuration $configuration, InputInterface $input, $version, $up = null, $down = null)
     {
         $placeHolders = [

Spacing +8 added lines, -8 removed lines

@@ -117,8 +117,8 @@ 
         $replacements = [
             $configuration->getMigrationsNamespace(),
             $version,
-            $up ? "        " . implode("\n        ", explode("\n", $up)) : null,
-            $down ? "        " . implode("\n        ", explode("\n", $down)) : null,
+            $up ? "        ".implode("\n        ", explode("\n", $up)) : null,
+            $down ? "        ".implode("\n        ", explode("\n", $down)) : null,
         ];
 
         $code = str_replace($placeHolders, $replacements, $this->getTemplate());
@@ -126,12 +126,12 @@ 
 
         $directoryHelper = new MigrationDirectoryHelper($configuration);
         $dir             = $directoryHelper->getMigrationDirectory();
-        $path            = $dir . '/Version' . $version . '.php';
+        $path            = $dir.'/Version'.$version.'.php';
 
         file_put_contents($path, $code);
 
         if ($editorCmd = $input->getOption('editor-cmd')) {
-            proc_open($editorCmd . ' ' . escapeshellarg($path), [], $pipes);
+            proc_open($editorCmd.' '.escapeshellarg($path), [], $pipes);
         }
 
         return $path;
@@ -151,16 +151,16 @@ 
 
     private function loadTemplateFile(string $customTemplate, OutputInterface $output) : void
     {
-        $customTemplate = getcwd() . '/' . $customTemplate;
+        $customTemplate = getcwd().'/'.$customTemplate;
 
-        if ( ! is_file($customTemplate)) {
-            throw new \InvalidArgumentException('The specified template « ' . $customTemplate . ' » cannot be found.');
+        if (!is_file($customTemplate)) {
+            throw new \InvalidArgumentException('The specified template « '.$customTemplate.' » cannot be found.');
         }
 
         $templateContent = file_get_contents($customTemplate);
 
         if ($templateContent === false) {
-            throw new \InvalidArgumentException('Cannot read file contents of template « ' . $customTemplate . ' ».');
+            throw new \InvalidArgumentException('Cannot read file contents of template « '.$customTemplate.' ».');
         }
 
         $output->writeln(sprintf('Using custom migration template "<info>%s</info>"', $customTemplate));