kaliop-uk / ezmigrationbundle

Tests/phpunit/Core/ConfigurationTest.php

Spacing +121 added lines, -121 removed lines

@@ -28,27 +28,27 @@ 
     public function setUp() {
         parent::setUp();
 
-        $this->configuration = new Configuration( $this->connection, new ConsoleOutput() );
+        $this->configuration = new Configuration($this->connection, new ConsoleOutput());
 
     }
 
     public function testGetConnection() {
         $conn = $this->configuration->getConnection();
 
-        $this->assertTrue( $conn instanceof Connection );
+        $this->assertTrue($conn instanceof Connection);
     }
 
     public function testGetVersions() {
         $version = $this->configuration->getVersions();
 
-        $this->assertTrue( is_array($version ) );
+        $this->assertTrue(is_array($version));
     }
 
     public function testcreateVersionTableIfNeeded()
     {
         $result = $this->configuration->createVersionTableIfNeeded();
 
-        $this->assertTrue( $result );
+        $this->assertTrue($result);
     }
 
     /**
@@ -61,19 +61,19 @@ 
         $conn = $this->configuration->getConnection();
 
 
-        $time = date( 'YmdHis', time() );
+        $time = date('YmdHis', time());
         $bundle = 'testBundle';
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
         /** @var $versions array */
-        $versions = $this->configuration->getMigratedVersionsByBundle( $bundle );
+        $versions = $this->configuration->getMigratedVersionsByBundle($bundle);
 
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 1, count( $versions ) );
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(1, count($versions));
 
-        $this->assertEquals( $time, $versions[0] );
+        $this->assertEquals($time, $versions[0]);
     }
 
     /**
@@ -86,23 +86,23 @@ 
 
         $conn = $this->configuration->getConnection();
 
-        $time = date( 'YmdHis', time() - 3600 );
+        $time = date('YmdHis', time() - 3600);
         $bundle = 'testBundle';
 
-        $this->configuration->setVersions( array( $bundle => array() ) );
+        $this->configuration->setVersions(array($bundle => array()));
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $time = date( 'YmdHis', time() );
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle, $time );
+        $time = date('YmdHis', time());
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $version = $this->configuration->getCurrentVersionByBundle( $bundle );
+        $version = $this->configuration->getCurrentVersionByBundle($bundle);
 
-        $this->assertEquals( (int)$time, $version );
+        $this->assertEquals((int)$time, $version);
     }
 
     /**
@@ -116,37 +116,37 @@ 
 
         $conn = $this->configuration->getConnection();
 
-        $time = date( 'YmdHis', time() - 3600 );
+        $time = date('YmdHis', time() - 3600);
         $bundle1 = 'testBundle';
         $bundle2 = 'testBundle2';
 
-        $this->configuration->setVersions( array( $bundle1 => array(), $bundle2 => array() ) );
+        $this->configuration->setVersions(array($bundle1 => array(), $bundle2 => array()));
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $time = date( 'YmdHis', time() );
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time );
+        $time = date('YmdHis', time());
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $time2 = date( 'YmdHis', time() );
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time2 );
+        $time2 = date('YmdHis', time());
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time2);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
         $versions = $this->configuration->getCurrentBundleVersions();
 
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 2, count( $versions ) );
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(2, count($versions));
 
-        $this->assertEquals( (int)$time, $versions[$bundle1] );
-        $this->assertEquals( (int)$time2, $versions[$bundle2] );
+        $this->assertEquals((int)$time, $versions[$bundle1]);
+        $this->assertEquals((int)$time2, $versions[$bundle2]);
     }
 
     /**
@@ -158,46 +158,46 @@ 
 
         $conn = $this->configuration->getConnection();
 
-        $time = date( 'YmdHis', time() - 3600 );
+        $time = date('YmdHis', time() - 3600);
         $bundle1 = 'testBundle';
         $bundle2 = 'testBundle2';
 
-        $this->configuration->setVersions( array( $bundle1 => array(), $bundle2 => array() ) );
+        $this->configuration->setVersions(array($bundle1 => array(), $bundle2 => array()));
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $time1 = date( 'YmdHis', time() );
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time1 );
+        $time1 = date('YmdHis', time());
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time1);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $time2 = date( 'YmdHis', time() );
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time2 );
+        $time2 = date('YmdHis', time());
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time2);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
         $versions = $this->configuration->getMigratedVersions();
 
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 2, count( $versions ) );
-        $this->assertArrayHasKey( $bundle1, $versions );
-        $this->assertArrayHasKey( $bundle2, $versions );
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(2, count($versions));
+        $this->assertArrayHasKey($bundle1, $versions);
+        $this->assertArrayHasKey($bundle2, $versions);
 
-        $this->assertTrue( is_array( $versions[$bundle1] ) );
-        $this->assertTrue( is_array( $versions[$bundle2] ) );
-        $this->assertEquals( 2, count( $versions[$bundle1] ) );
-        $this->assertEquals( 2, count( $versions[$bundle2] ) );
+        $this->assertTrue(is_array($versions[$bundle1]));
+        $this->assertTrue(is_array($versions[$bundle2]));
+        $this->assertEquals(2, count($versions[$bundle1]));
+        $this->assertEquals(2, count($versions[$bundle2]));
 
-        $this->assertEquals( $time, $versions[$bundle1][0] );
-        $this->assertEquals( $time, $versions[$bundle2][0] );
-        $this->assertEquals( $time1, $versions[$bundle1][1] );
-        $this->assertEquals( $time2, $versions[$bundle2][1] );
+        $this->assertEquals($time, $versions[$bundle1][0]);
+        $this->assertEquals($time, $versions[$bundle2][0]);
+        $this->assertEquals($time1, $versions[$bundle1][1]);
+        $this->assertEquals($time2, $versions[$bundle2][1]);
     }
 
     /**
@@ -209,10 +209,10 @@ 
 
         $conn = $this->configuration->getConnection();
 
-        $time = date( 'YmdHis', time() - 3600 );
-        $time1 = date( 'YmdHis', time() );
-        $time2 = date( 'YmdHis', time() + 3600 );
-        $time3 = date( 'YmdHis', time() + 7200 );
+        $time = date('YmdHis', time() - 3600);
+        $time1 = date('YmdHis', time());
+        $time2 = date('YmdHis', time() + 3600);
+        $time3 = date('YmdHis', time() + 7200);
         $bundle1 = 'testBundle';
         $bundle2 = 'testBundle2';
 
@@ -229,35 +229,35 @@ 
             )
         );
 
-        $this->configuration->setVersions( $versions );
+        $this->configuration->setVersions($versions);
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time1 );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle1, $time1);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
-        $sql = sprintf( "INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time2 );
+        $sql = sprintf("INSERT INTO %s ( 'bundle', 'version' ) VALUES( '%s', '%s' )", $this->configuration->versionTableName, $bundle2, $time2);
 
-        $conn->executeQuery( $sql );
+        $conn->executeQuery($sql);
 
         $results = $this->configuration->migrationsToExecute();
 
-        $this->assertTrue( is_array( $results ) );
-        $this->assertEquals( 2, count( $results ) );
-        $this->assertArrayHasKey( $bundle1, $results );
-        $this->assertArrayHasKey( $bundle2, $results );
+        $this->assertTrue(is_array($results));
+        $this->assertEquals(2, count($results));
+        $this->assertArrayHasKey($bundle1, $results);
+        $this->assertArrayHasKey($bundle2, $results);
 
-        $this->assertEquals( 1, count( $results[$bundle1] ) );
-        $this->assertEquals( $time2, key( $results[$bundle1] ) );
-        $this->assertEquals( 1, count( $results[$bundle2] ) );
-        $this->assertEquals( $time3, key( $results[$bundle2] ) );
+        $this->assertEquals(1, count($results[$bundle1]));
+        $this->assertEquals($time2, key($results[$bundle1]));
+        $this->assertEquals(1, count($results[$bundle2]));
+        $this->assertEquals($time3, key($results[$bundle2]));
     }
 
     /**
@@ -270,16 +270,16 @@ 
         $bundle = 'testBundle';
         $version = '1';
 
-        $versions = $this->configuration->getMigratedVersionsByBundle( $bundle );
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 0, count( $versions ) );
+        $versions = $this->configuration->getMigratedVersionsByBundle($bundle);
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(0, count($versions));
 
-        $this->configuration->markVersionMigrated( $bundle, $version );
-        $versions = $this->configuration->getMigratedVersionsByBundle( $bundle );
+        $this->configuration->markVersionMigrated($bundle, $version);
+        $versions = $this->configuration->getMigratedVersionsByBundle($bundle);
 
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 1, count( $versions ) );
-        $this->assertEquals( $version, $versions[0] );
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(1, count($versions));
+        $this->assertEquals($version, $versions[0]);
     }
 
     /**
@@ -294,25 +294,25 @@ 
         $version1 = '1';
         $version2 = '2';
 
-        $versions = $this->configuration->getMigratedVersionsByBundle( $bundle );
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 0, count( $versions ) );
+        $versions = $this->configuration->getMigratedVersionsByBundle($bundle);
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(0, count($versions));
 
-        $this->configuration->markVersionMigrated( $bundle, $version1 );
-        $versions = $this->configuration->getMigratedVersionsByBundle( $bundle );
+        $this->configuration->markVersionMigrated($bundle, $version1);
+        $versions = $this->configuration->getMigratedVersionsByBundle($bundle);
 
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 1, count( $versions ) );
-        $this->assertEquals( $version1, $versions[0] );
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(1, count($versions));
+        $this->assertEquals($version1, $versions[0]);
 
-        $this->configuration->markVersionMigrated( $bundle, $version2 );
-        $this->configuration->markVersionNotMigrated( $bundle, $version1 );
+        $this->configuration->markVersionMigrated($bundle, $version2);
+        $this->configuration->markVersionNotMigrated($bundle, $version1);
 
-        $versions = $this->configuration->getMigratedVersionsByBundle( $bundle );
+        $versions = $this->configuration->getMigratedVersionsByBundle($bundle);
 
-        $this->assertTrue( is_array( $versions ) );
-        $this->assertEquals( 1, count( $versions ) );
-        $this->assertEquals( $version2, $versions[0] );
+        $this->assertTrue(is_array($versions));
+        $this->assertEquals(1, count($versions));
+        $this->assertEquals($version2, $versions[0]);
     }
 
     /**
@@ -322,21 +322,21 @@ 
     public function testRegisterVersionFromDirectories() {
         try {
             $this->setupVfsStructure();
-        } catch ( \Exception $e ) {
-            var_dump( $e->getMessage() );
-            $this->markTestSkipped( "The VfsStream bundle is not installed." );
+        } catch (\Exception $e) {
+            var_dump($e->getMessage());
+            $this->markTestSkipped("The VfsStream bundle is not installed.");
         }
 
         $paths = array(
-            'bundle1' => array( vfsStream::url( 'bundles/bundle1/MigrationVersions' ) ),
-            'bundle2' => array( vfsStream::url( 'bundles/bundle2/MigrationVersions' ) )
+            'bundle1' => array(vfsStream::url('bundles/bundle1/MigrationVersions')),
+            'bundle2' => array(vfsStream::url('bundles/bundle2/MigrationVersions'))
         );
 
         $this->configuration->createVersionTableIfNeeded();
 
-        $versions = $this->configuration->registerVersionFromDirectories( $paths );
+        $versions = $this->configuration->registerVersionFromDirectories($paths);
 
-        $this->markTestSkipped( "FIXME: figure out a way to test this" );
+        $this->markTestSkipped("FIXME: figure out a way to test this");
     }
 
     /**
@@ -344,9 +344,9 @@ 
      */
     public function testCheckDuplicateVersion()
     {
-        $this->configuration->setVersions( array( 'testBundle' => array( '1' ) ) );
+        $this->configuration->setVersions(array('testBundle' => array('1')));
 
-        $this->configuration->checkDuplicateVersion( 'testBundle', '1' );
+        $this->configuration->checkDuplicateVersion('testBundle', '1');
     }
 
     /**
@@ -356,11 +356,11 @@ 
 
         try {
         $this->setupVfsStructure();
-        } catch( \Exception $e ) {
-            $this->markTestSkipped( 'Problem with VfsStream.' );
+        } catch (\Exception $e) {
+            $this->markTestSkipped('Problem with VfsStream.');
         }
 
-        $path = vfsStream::url( 'bundles/bundle1/MigrationVersions/Version20010101010101.php' );
+        $path = vfsStream::url('bundles/bundle1/MigrationVersions/Version20010101010101.php');
     }
 
     /**
@@ -374,21 +374,21 @@ 
             $structure = array(
                 'bundle1' => array(
                     'MigrationVersions' => array(
-                        'Version20010101010101.php' => '<?php namespace MigrationVersions\bundle1;'."\n" . $this->dummyVersionContent . "\n?>",
-                        'Version20010101010102.php' => '<?php namespace MigrationVersions\bundle1;'."\n" . $this->dummyVersionContent . "\n?>",
+                        'Version20010101010101.php' => '<?php namespace MigrationVersions\bundle1;' . "\n" . $this->dummyVersionContent . "\n?>",
+                        'Version20010101010102.php' => '<?php namespace MigrationVersions\bundle1;' . "\n" . $this->dummyVersionContent . "\n?>",
                     )
                 ),
                 'bundle2' => array(
                     'MigrationVersions' => array(
-                        'Version20010101010103.php' => '<?php namespace MigrationVersions\bundle2;'."\n" . $this->dummyVersionContent . "\n?>",
+                        'Version20010101010103.php' => '<?php namespace MigrationVersions\bundle2;' . "\n" . $this->dummyVersionContent . "\n?>",
                     )
                 )
             );
-        } catch( Exception $e ) {
-            throw new \Exception( $e->getMessage() );
+        } catch (Exception $e) {
+            throw new \Exception($e->getMessage());
         }
 
-        $this->root = vfsStream::setup( 'bundles', null, $structure );
+        $this->root = vfsStream::setup('bundles', null, $structure);
     }
 
     /**

Tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-if (!file_exists($file = __DIR__.'/../vendor/autoload.php')) {
+if (!file_exists($file = __DIR__ . '/../vendor/autoload.php')) {
     throw new \RuntimeException('Install the dependencies to run the test suite.');
 }
 

Command/MigrateCommand.php

Spacing +7 added lines, -7 removed lines

@@ -61,7 +61,7 @@ 
     {
         $migrationsService = $this->getMigrationService();
 
-        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path')) ;
+        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path'));
         $migrations = $migrationsService->getMigrations();
 
         if (!count($migrationDefinitions)) {
@@ -71,7 +71,7 @@ 
 
         // filter away all migrations except 'to do' ones
         $toExecute = array();
-        foreach($migrationDefinitions as $name => $migrationDefinition) {
+        foreach ($migrationDefinitions as $name => $migrationDefinition) {
             if (!isset($migrations[$name]) || (($migration = $migrations[$name]) && $migration->status == Migration::STATUS_TODO)) {
                 $toExecute[$name] = $migrationsService->parseMigrationDefinition($migrationDefinition);
             }
@@ -88,7 +88,7 @@ 
 
         $data = array();
         $i = 1;
-        foreach($toExecute as $name => $migrationDefinition) {
+        foreach ($toExecute as $name => $migrationDefinition) {
             $notes = '';
             if ($migrationDefinition->status != MigrationDefinition::STATUS_PARSED) {
                 $notes = '<error>' . $migrationDefinition->parsingError . '</error>';
@@ -123,7 +123,7 @@ 
             $output->writeln("=============================================\n");
         }
 
-        foreach($toExecute as $name => $migrationDefinition) {
+        foreach ($toExecute as $name => $migrationDefinition) {
 
             // let's skip migrations that we know are invalid - user was warned and he decide to proceed anyway
             if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
@@ -135,12 +135,12 @@ 
 
             try {
                 $migrationsService->executeMigration($migrationDefinition);
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 if ($input->getOption('ignore-failures')) {
-                    $output->writeln("\n<error>Migration failed!".$e->getMessage()."</error>\n");
+                    $output->writeln("\n<error>Migration failed!" . $e->getMessage() . "</error>\n");
                     continue;
                 }
-                $output->writeln("\n<error>Migration aborted!".$e->getMessage()."</error>");
+                $output->writeln("\n<error>Migration aborted!" . $e->getMessage() . "</error>");
                 return 1;
             }
 

Command/StatusCommand.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
     {
         $migrationsService = $this->getMigrationService();
 
-        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path')) ;
+        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path'));
         $migrations = $migrationsService->getMigrations();
 
         if (!count($migrationDefinitions) && !count($migrations)) {
@@ -51,10 +51,10 @@ 
 
         // create a unique ist of all migrations (coming from db) and definitions (coming from disk)
         $index = array();
-        foreach($migrationDefinitions as $migrationDefinition) {
+        foreach ($migrationDefinitions as $migrationDefinition) {
             $index[$migrationDefinition->name] = array('definition' => $migrationDefinition);
         }
-        foreach($migrations as $migration) {
+        foreach ($migrations as $migration) {
             if (isset($index[$migration->name])) {
                 $index[$migration->name]['migration'] = $migration;
             } else {
@@ -68,7 +68,7 @@ 
                         {
                             $index[$migration->name]['definition'] = reset($migrationDefinitionCollection);
                         }
-                    } catch(\Exception $e) {
+                    } catch (\Exception $e) {
                         /// @todo one day we should be able to limit the kind of exceptions we have to catch here...
                     }
                 }
@@ -80,7 +80,7 @@ 
 
         $data = array();
         $i = 1;
-        foreach($index as $name => $value) {
+        foreach ($index as $name => $value) {
             if (!isset($value['migration'])) {
                 $migrationDefinition = $migrationsService->parseMigrationDefinition($value['definition']);
                 $notes = '';

Command/GenerateCommand.php

Spacing +4 added lines, -4 removed lines

@@ -80,7 +80,7 @@ 
         }
 
         $activeBundles = array();
-        foreach($this->getApplication()->getKernel()->getBundles() as $bundle)
+        foreach ($this->getApplication()->getKernel()->getBundles() as $bundle)
         {
             $activeBundles[] = $bundle->getName();
         }
@@ -155,7 +155,7 @@ 
                     $className = 'Migration';
                 }
                 // Make sure that php class names are unique, not only migration definition file names
-                $existingMigrations = count(glob($migrationDirectory.'/*_' . $className .'*.php'));
+                $existingMigrations = count(glob($migrationDirectory . '/*_' . $className . '*.php'));
                 if ($existingMigrations) {
                     $className = $className . sprintf('%03d', $existingMigrations + 1);
                 }
@@ -201,7 +201,7 @@ 
         if ($migrationType == 'role') {
             $code = $this->generateRoleTemplate($parameters['role']);
         } else {
-            $code = $this->getContainer()->get('twig')->render($this->thisBundle . ':MigrationTemplate:'.$template, $parameters);
+            $code = $this->getContainer()->get('twig')->render($this->thisBundle . ':MigrationTemplate:' . $template, $parameters);
         }
 
         file_put_contents($path, $code);
@@ -267,7 +267,7 @@ 
         /** @var \eZ\Publish\API\Repository\SectionService $sectionService */
         $sectionService = $repository->getSectionService();
 
-        foreach($limitation->limitationValues as $value)
+        foreach ($limitation->limitationValues as $value)
         {
 
         }

Command/MigrationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        if (/*! $input->getOption('add') &&*/ ! $input->getOption('delete')) {
+        if (/*! $input->getOption('add') &&*/ !$input->getOption('delete')) {
             throw new \InvalidArgumentException('You must specify whether you want to --delete the specified migration.');
         }
 

Core/Executor/UserManager.php

Spacing +4 added lines, -4 removed lines

@@ -132,7 +132,7 @@ 
                 $this->dsl['user_id'] = array($this->dsl['user_id']);
             }
             foreach ($this->dsl['user_id'] as $id) {
-                array_push($usersToDelete,$userService->loadUser($id));
+                array_push($usersToDelete, $userService->loadUser($id));
             }
         }
         if (isset($this->dsl['email'])) {
@@ -140,7 +140,7 @@ 
                 $this->dsl['email'] = array($this->dsl['email']);
             }
             foreach ($this->dsl['email'] as $email) {
-                $usersToDelete = array_merge($usersToDelete,$userService->loadUsersByEmail($email));
+                $usersToDelete = array_merge($usersToDelete, $userService->loadUsersByEmail($email));
             }
         }
         if (isset($this->dsl['username'])) {
@@ -148,11 +148,11 @@ 
                 $this->dsl['username'] = array($this->dsl['username']);
             }
             foreach ($this->dsl['username'] as $username) {
-                array_push($usersToDelete,$userService->loadUserByLogin($username));
+                array_push($usersToDelete, $userService->loadUserByLogin($username));
             }
         }
 
-        foreach($usersToDelete as $user){
+        foreach ($usersToDelete as $user) {
             $userService->deleteUser($user);
         }
     }

Core/Executor/ContentManager.php

Spacing +4 added lines, -4 removed lines

@@ -125,7 +125,7 @@ 
         }
 
         $draft = $contentService->createContentDraft($contentInfo);
-        $contentService->updateContent($draft->versionInfo,$contentUpdateStruct);
+        $contentService->updateContent($draft->versionInfo, $contentUpdateStruct);
         $content = $contentService->publishVersion($draft->versionInfo);
 
         if (array_key_exists('new_remote_id', $this->dsl)) {
@@ -158,7 +158,7 @@ 
 
         $contentService = $this->repository->getContentService();
 
-        if (isset($this->dsl['object_id'])){
+        if (isset($this->dsl['object_id'])) {
             if (!is_array($this->dsl['object_id'])) {
                 $this->dsl['object_id'] = array($this->dsl['object_id']);
             }
@@ -186,7 +186,7 @@ 
      * @param ContentCreateStruct $createStruct
      * @param array $fields
      */
-    protected function setFields(ContentCreateStruct &$createStruct, array $fields)
+    protected function setFields(ContentCreateStruct & $createStruct, array $fields)
     {
         foreach ($fields as $field) {
             // each $field is one key value pair
@@ -214,7 +214,7 @@ 
      * @param ContentUpdateStruct $updateStruct
      * @param array $fields
      */
-    protected function setFieldsToUpdate(ContentUpdateStruct &$updateStruct, array $fields, ContentType $contentType)
+    protected function setFieldsToUpdate(ContentUpdateStruct & $updateStruct, array $fields, ContentType $contentType)
     {
         foreach ($fields as $field) {
             // each $field is one key value pair

Core/Executor/UserGroupManager.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
             $groupIds = array($groupIds);
         }
 
-        foreach($groupIds as $groupId) {
+        foreach ($groupIds as $groupId) {
             $userGroup = $userService->loadUserGroup($groupId);
             $userService->deleteUserGroup($userGroup);
         }