ramunasd / MigrationBundle

Twig/SchemaDumperExtension.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
      */
     protected function getColumnOption(Column $column, $optionName)
     {
-        $method = "get" . $optionName;
+        $method = "get".$optionName;
 
         return $column->$method();
     }

Command/DumpMigrationsCommand.php

Spacing +3 added lines, -3 removed lines

@@ -78,7 +78,7 @@ 
             $connection = $this->getContainer()->get('doctrine')->getConnection();
             $sqls = $schema->toSql($connection->getDatabasePlatform());
             foreach ($sqls as $sql) {
-                $output->writeln($sql . ';');
+                $output->writeln($sql.';');
             }
         } else {
             $this->dumpPhpSchema($schema, $output);
@@ -98,7 +98,7 @@ 
                 );
             }
             $this->namespace = str_replace($bundle, 'Entity', $bundles[$bundle]);
-            $this->className = $bundle . 'Installer';
+            $this->className = $bundle.'Installer';
         }
     }
 
@@ -112,7 +112,7 @@ 
         $allMetadata = $doctrine->getManager()->getMetadataFactory()->getAllMetadata();
         array_walk(
             $allMetadata,
-            function (ClassMetadata $entityMetadata) {
+            function(ClassMetadata $entityMetadata) {
                 if ($this->namespace) {
                     if ($entityMetadata->namespace == $this->namespace) {
                         $this->allowedTables[$entityMetadata->getTableName()] = true;

Command/LoadDataFixturesCommand.php

Spacing +3 added lines, -3 removed lines

@@ -92,7 +92,7 @@ 
             if (!empty($excludeBundles) && in_array($bundle->getName(), $excludeBundles)) {
                 continue;
             }
-            $path = $bundle->getPath() . $fixtureRelativePath;
+            $path = $bundle->getPath().$fixtureRelativePath;
             if (is_dir($path)) {
                 $loader->loadFromDirectory($path);
             }
@@ -139,7 +139,7 @@ 
 
         $executor = $this->getContainer()->get('rdv_migration.data_fixtures.executor');
         $executor->setLogger(
-            function ($message) use ($output) {
+            function($message) use ($output) {
                 $output->writeln(sprintf('  <comment>></comment> <info>%s</info>', $message));
             }
         );
@@ -172,6 +172,6 @@ 
                 throw new \InvalidArgumentException('Unknown fixture type');
         }
 
-        return str_replace('/', DIRECTORY_SEPARATOR, '/' . $path);
+        return str_replace('/', DIRECTORY_SEPARATOR, '/'.$path);
     }
 }

Tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
 
-$loader = require_once __DIR__ . '/../vendor/autoload.php';
+$loader = require_once __DIR__.'/../vendor/autoload.php';
 use Doctrine\Common\Annotations\AnnotationRegistry;
 AnnotationRegistry::registerLoader(array($loader, 'loadClass'));

Tests/Functional/Fixture/Kernel.php

Spacing +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
      */
     public function getCacheDir()
     {
-        $this->cacheDir = sys_get_temp_dir() . '/kernel' . mt_rand();
+        $this->cacheDir = sys_get_temp_dir().'/kernel'.mt_rand();
         mkdir($this->cacheDir, 0755, true);
         return $this->cacheDir;
     }
@@ -32,7 +32,7 @@ 
      */
     protected function getContainerClass()
     {
-        return parent::getContainerClass() . mt_rand();
+        return parent::getContainerClass().mt_rand();
     }
 
     /**
@@ -77,7 +77,7 @@ 
      */
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
-        $loader->load(__DIR__ . '/config.yml');
+        $loader->load(__DIR__.'/config.yml');
         if (is_callable($this->configCallback)) {
             $loader->load($this->configCallback);
         }
@@ -85,7 +85,7 @@ 
 
     public function __destruct()
     {
-        array_map('unlink', glob($this->cacheDir . '/*'));
+        array_map('unlink', glob($this->cacheDir.'/*'));
         rmdir($this->cacheDir);
     }
 }

Tests/Functional/DependencyInjection/ConfigurationTest.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
 {
     protected function tearDown()
     {
-        array_map('unlink', glob(__DIR__ . '/../Fixture/cache/test/*.[php|xml]*'));
+        array_map('unlink', glob(__DIR__.'/../Fixture/cache/test/*.[php|xml]*'));
     }
 
     public function testMigrationsLoadedFromDefaultPath()
@@ -43,7 +43,7 @@ 
             new TestPackageTest1Bundle(),
             new TestPackageTest2Bundle(),
         ));
-        $kernel->setConfigCallback(function ($container) {
+        $kernel->setConfigCallback(function($container) {
             $container->setParameter('rdv_migration.migration_path', 'NonExistingPath');
         });
         $kernel->boot();

Tests/Unit/Migration/Loader/MigrationsLoaderTest.php

Spacing +36 added lines, -36 removed lines

@@ -70,7 +70,7 @@ 
             ->method('dispatch')
             ->will(
                 $this->returnCallback(
-                    function ($eventName, $event) use (&$installed) {
+                    function($eventName, $event) use (&$installed) {
                         if ($eventName === MigrationEvents::PRE_UP) {
                             if (null !== $installed) {
                                 foreach ($installed as $val) {
@@ -96,7 +96,7 @@ 
     protected function getMigrationClasses(array $migrations)
     {
         return array_map(
-            function ($migration) {
+            function($migration) {
                 return get_class($migration->getMigration());
             },
             $migrations
@@ -109,20 +109,20 @@ 
     public function getMigrationsProvider()
     {
         $testPackage = 'RDV\\Bundle\\MigrationBundle\\Tests\\Unit\\Fixture\\TestPackage\\';
-        $test1Bundle = $testPackage . 'Test1Bundle\\Migrations\\Schema';
-        $test2Bundle = $testPackage . 'Test2Bundle\\Migrations\\Schema';
+        $test1Bundle = $testPackage.'Test1Bundle\\Migrations\\Schema';
+        $test2Bundle = $testPackage.'Test2Bundle\\Migrations\\Schema';
 
         return [
             [
                 [new TestPackageTest1Bundle(), new TestPackageTest2Bundle()],
                 null,
                 [
-                    $test1Bundle . '\Test1BundleInstallation',
-                    $test1Bundle . '\v1_1\Test1BundleMigration11',
-                    $test2Bundle . '\v1_0\Test2BundleMigration10',
-                    $test2Bundle . '\v1_0\Test2BundleMigration11',
-                    $test2Bundle . '\v1_1\Test2BundleMigration12',
-                    $test2Bundle . '\v1_1\Test2BundleMigration11',
+                    $test1Bundle.'\Test1BundleInstallation',
+                    $test1Bundle.'\v1_1\Test1BundleMigration11',
+                    $test2Bundle.'\v1_0\Test2BundleMigration10',
+                    $test2Bundle.'\v1_0\Test2BundleMigration11',
+                    $test2Bundle.'\v1_1\Test2BundleMigration12',
+                    $test2Bundle.'\v1_1\Test2BundleMigration11',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateBundleVersionMigration',
                 ]
             ],
@@ -130,12 +130,12 @@ 
                 [new TestPackageTest2Bundle(), new TestPackageTest1Bundle()],
                 null,
                 [
-                    $test2Bundle . '\v1_0\Test2BundleMigration10',
-                    $test2Bundle . '\v1_0\Test2BundleMigration11',
-                    $test2Bundle . '\v1_1\Test2BundleMigration12',
-                    $test2Bundle . '\v1_1\Test2BundleMigration11',
-                    $test1Bundle . '\Test1BundleInstallation',
-                    $test1Bundle . '\v1_1\Test1BundleMigration11',
+                    $test2Bundle.'\v1_0\Test2BundleMigration10',
+                    $test2Bundle.'\v1_0\Test2BundleMigration11',
+                    $test2Bundle.'\v1_1\Test2BundleMigration12',
+                    $test2Bundle.'\v1_1\Test2BundleMigration11',
+                    $test1Bundle.'\Test1BundleInstallation',
+                    $test1Bundle.'\v1_1\Test1BundleMigration11',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateBundleVersionMigration',
                 ]
             ],
@@ -143,12 +143,12 @@ 
                 [new TestPackageTest1Bundle(), new TestPackageTest2Bundle()],
                 [],
                 [
-                    $test1Bundle . '\Test1BundleInstallation',
-                    $test1Bundle . '\v1_1\Test1BundleMigration11',
-                    $test2Bundle . '\v1_0\Test2BundleMigration10',
-                    $test2Bundle . '\v1_0\Test2BundleMigration11',
-                    $test2Bundle . '\v1_1\Test2BundleMigration12',
-                    $test2Bundle . '\v1_1\Test2BundleMigration11',
+                    $test1Bundle.'\Test1BundleInstallation',
+                    $test1Bundle.'\v1_1\Test1BundleMigration11',
+                    $test2Bundle.'\v1_0\Test2BundleMigration10',
+                    $test2Bundle.'\v1_0\Test2BundleMigration11',
+                    $test2Bundle.'\v1_1\Test2BundleMigration12',
+                    $test2Bundle.'\v1_1\Test2BundleMigration11',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateBundleVersionMigration',
                 ]
             ],
@@ -158,12 +158,12 @@ 
                     ['bundle' => 'TestPackageTest1Bundle', 'version' => null],
                 ],
                 [
-                    $test1Bundle . '\Test1BundleInstallation',
-                    $test1Bundle . '\v1_1\Test1BundleMigration11',
-                    $test2Bundle . '\v1_0\Test2BundleMigration10',
-                    $test2Bundle . '\v1_0\Test2BundleMigration11',
-                    $test2Bundle . '\v1_1\Test2BundleMigration12',
-                    $test2Bundle . '\v1_1\Test2BundleMigration11',
+                    $test1Bundle.'\Test1BundleInstallation',
+                    $test1Bundle.'\v1_1\Test1BundleMigration11',
+                    $test2Bundle.'\v1_0\Test2BundleMigration10',
+                    $test2Bundle.'\v1_0\Test2BundleMigration11',
+                    $test2Bundle.'\v1_1\Test2BundleMigration12',
+                    $test2Bundle.'\v1_1\Test2BundleMigration11',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateBundleVersionMigration',
                 ]
             ],
@@ -173,11 +173,11 @@ 
                     ['bundle' => 'TestPackageTest1Bundle', 'version' => 'v1_0'],
                 ],
                 [
-                    $test1Bundle . '\v1_1\Test1BundleMigration11',
-                    $test2Bundle . '\v1_0\Test2BundleMigration10',
-                    $test2Bundle . '\v1_0\Test2BundleMigration11',
-                    $test2Bundle . '\v1_1\Test2BundleMigration12',
-                    $test2Bundle . '\v1_1\Test2BundleMigration11',
+                    $test1Bundle.'\v1_1\Test1BundleMigration11',
+                    $test2Bundle.'\v1_0\Test2BundleMigration10',
+                    $test2Bundle.'\v1_0\Test2BundleMigration11',
+                    $test2Bundle.'\v1_1\Test2BundleMigration12',
+                    $test2Bundle.'\v1_1\Test2BundleMigration11',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateBundleVersionMigration',
                 ]
             ],
@@ -188,9 +188,9 @@ 
                     ['bundle' => 'TestPackageTest2Bundle', 'version' => 'v1_0'],
                 ],
                 [
-                    $test1Bundle . '\v1_1\Test1BundleMigration11',
-                    $test2Bundle . '\v1_1\Test2BundleMigration12',
-                    $test2Bundle . '\v1_1\Test2BundleMigration11',
+                    $test1Bundle.'\v1_1\Test1BundleMigration11',
+                    $test2Bundle.'\v1_1\Test2BundleMigration12',
+                    $test2Bundle.'\v1_1\Test2BundleMigration11',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateBundleVersionMigration',
                 ]
             ],

Tests/Unit/Migration/Loader/DataFixturesLoaderTest.php

Spacing +18 added lines, -18 removed lines

@@ -44,7 +44,7 @@ 
         /** @var \Symfony\Component\HttpKernel\Bundle\Bundle $bundle */
         foreach ($bundles as $bundle) {
             $this->loader->loadFromDirectory(
-                $bundle->getPath() . '/Migrations/Data/ORM'
+                $bundle->getPath().'/Migrations/Data/ORM'
             );
         }
 
@@ -101,14 +101,14 @@ 
                 [new TestPackageTest1Bundle()],
                 [],
                 [
-                    $test1BundleNamespace . '\Migrations\Data\ORM\LoadTest1BundleData',
+                    $test1BundleNamespace.'\Migrations\Data\ORM\LoadTest1BundleData',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateDataFixturesFixture',
                 ]
             ],
             [
                 [new TestPackageTest1Bundle()],
                 [
-                    $test1BundleNamespace . '\Migrations\Data\ORM\LoadTest1BundleData' => ''
+                    $test1BundleNamespace.'\Migrations\Data\ORM\LoadTest1BundleData' => ''
                 ],
                 []
             ],
@@ -116,18 +116,18 @@ 
                 [new TestPackageTest2Bundle()],
                 [],
                 [
-                    $test1BundleNamespace . '\Migrations\Data\ORM\LoadTest1BundleData',
-                    $test2BundleNamespace . '\Migrations\Data\ORM\LoadTest2BundleData',
+                    $test1BundleNamespace.'\Migrations\Data\ORM\LoadTest1BundleData',
+                    $test2BundleNamespace.'\Migrations\Data\ORM\LoadTest2BundleData',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateDataFixturesFixture',
                 ]
             ],
             [
                 [new TestPackageTest2Bundle()],
                 [
-                    $test1BundleNamespace . '\Migrations\Data\ORM\LoadTest1BundleData' => ''
+                    $test1BundleNamespace.'\Migrations\Data\ORM\LoadTest1BundleData' => ''
                 ],
                 [
-                    $test2BundleNamespace . '\Migrations\Data\ORM\LoadTest2BundleData',
+                    $test2BundleNamespace.'\Migrations\Data\ORM\LoadTest2BundleData',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateDataFixturesFixture',
                 ]
             ],
@@ -135,18 +135,18 @@ 
                 [new TestPackageTest2Bundle(), new TestPackageTest1Bundle()],
                 [],
                 [
-                    $test1BundleNamespace . '\Migrations\Data\ORM\LoadTest1BundleData',
-                    $test2BundleNamespace . '\Migrations\Data\ORM\LoadTest2BundleData',
+                    $test1BundleNamespace.'\Migrations\Data\ORM\LoadTest1BundleData',
+                    $test2BundleNamespace.'\Migrations\Data\ORM\LoadTest2BundleData',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateDataFixturesFixture',
                 ]
             ],
             [
                 [new TestPackageTest2Bundle(), new TestPackageTest1Bundle()],
                 [
-                    $test2BundleNamespace . '\Migrations\Data\ORM\LoadTest2BundleData' => ''
+                    $test2BundleNamespace.'\Migrations\Data\ORM\LoadTest2BundleData' => ''
                 ],
                 [
-                    $test1BundleNamespace . '\Migrations\Data\ORM\LoadTest1BundleData',
+                    $test1BundleNamespace.'\Migrations\Data\ORM\LoadTest1BundleData',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateDataFixturesFixture',
                 ]
             ],
@@ -154,27 +154,27 @@ 
                 [new TestPackageTest3Bundle()],
                 [],
                 [
-                    $test3BundleNamespace . '\Migrations\Data\ORM\LoadTest3BundleData1',
-                    $test3BundleNamespace . '\Migrations\Data\ORM\LoadTest3BundleData2',
+                    $test3BundleNamespace.'\Migrations\Data\ORM\LoadTest3BundleData1',
+                    $test3BundleNamespace.'\Migrations\Data\ORM\LoadTest3BundleData2',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateDataFixturesFixture',
                 ]
             ],
             [
                 [new TestPackageTest3Bundle()],
                 [
-                    $test3BundleNamespace . '\Migrations\Data\ORM\LoadTest3BundleData1' => '1.0',
-                    $test3BundleNamespace . '\Migrations\Data\ORM\LoadTest3BundleData2' => '0.0'
+                    $test3BundleNamespace.'\Migrations\Data\ORM\LoadTest3BundleData1' => '1.0',
+                    $test3BundleNamespace.'\Migrations\Data\ORM\LoadTest3BundleData2' => '0.0'
                 ],
                 [
-                    $test3BundleNamespace . '\Migrations\Data\ORM\LoadTest3BundleData2',
+                    $test3BundleNamespace.'\Migrations\Data\ORM\LoadTest3BundleData2',
                     'RDV\Bundle\MigrationBundle\Migration\UpdateDataFixturesFixture',
                 ]
             ],
             [
                 [new TestPackageTest3Bundle()],
                 [
-                    $test3BundleNamespace . '\Migrations\Data\ORM\LoadTest3BundleData1' => '2.0',
-                    $test3BundleNamespace . '\Migrations\Data\ORM\LoadTest3BundleData2' => '1.0'
+                    $test3BundleNamespace.'\Migrations\Data\ORM\LoadTest3BundleData1' => '2.0',
+                    $test3BundleNamespace.'\Migrations\Data\ORM\LoadTest3BundleData2' => '1.0'
                 ],
                 []
             ]

Tests/Unit/Migration/MigrationExecutorWithNameGeneratorTest.php

Spacing +4 added lines, -4 removed lines

@@ -56,9 +56,9 @@ 
         $messages = $this->logger->getMessages();
         $this->assertEquals(
             [
-                '> ' . get_class($migration10),
+                '> '.get_class($migration10),
                 'CREATE TABLE TEST (id INT AUTO_INCREMENT NOT NULL)',
-                '> ' . get_class($migration11),
+                '> '.get_class($migration11),
                 'CREATE TABLE test1table (id INT NOT NULL) DEFAULT CHARACTER SET utf8 '
                 . 'COLLATE utf8_unicode_ci ENGINE = InnoDB',
                 'ALTER TABLE TEST ADD COLUMN test_column INT NOT NULL',
@@ -83,9 +83,9 @@ 
         $messages = $this->logger->getMessages();
         $this->assertEquals(
             [
-                '> ' . get_class($migration10),
+                '> '.get_class($migration10),
                 'CREATE TABLE TEST (id INT AUTO_INCREMENT NOT NULL)',
-                '> ' . get_class($migration11),
+                '> '.get_class($migration11),
                 'CREATE TABLE test1table (id INT NOT NULL) DEFAULT CHARACTER SET utf8 '
                 . 'COLLATE utf8_unicode_ci ENGINE = InnoDB',
                 'ALTER TABLE TEST ADD COLUMN test_column INT NOT NULL',