kaliop-uk / ezmigrationbundle

Core/MigrationService.php

Spacing +9 added lines, -9 removed lines

@@ -45,7 +45,7 @@ 
 
     public function addExecutor(ExecutorInterface $executor)
     {
-        foreach($executor->supportedTypes() as $type) {
+        foreach ($executor->supportedTypes() as $type) {
             $this->executors[$type] = $executor;
         }
     }
@@ -60,8 +60,8 @@ 
     {
         // we try to be flexible in file types we support, and the same time avoid loading all files in a directory
         $handledDefinitions = array();
-        foreach($this->loader->listAvailableDefinitions($paths) as $migrationName => $definitionPath) {
-            foreach($this->DefinitionParsers as $definitionParser) {
+        foreach ($this->loader->listAvailableDefinitions($paths) as $migrationName => $definitionPath) {
+            foreach ($this->DefinitionParsers as $definitionParser) {
                 if ($definitionParser->supports($migrationName)) {
                     $handledDefinitions[] = $definitionPath;
                 }
@@ -122,13 +122,13 @@ 
      */
     public function parseMigrationDefinition(MigrationDefinition $migrationDefinition)
     {
-        foreach($this->DefinitionParsers as $definitionParser) {
+        foreach ($this->DefinitionParsers as $definitionParser) {
             if ($definitionParser->supports($migrationDefinition->name)) {
                 // parse the source file
                 $migrationDefinition = $definitionParser->parseMigrationDefinition($migrationDefinition);
 
                 // and make sure we know how to handle all steps
-                foreach($migrationDefinition->steps as $step) {
+                foreach ($migrationDefinition->steps as $step) {
                     if (!isset($this->executors[$step->type])) {
                         return new MigrationDefinition(
                             $migrationDefinition->name,
@@ -172,7 +172,7 @@ 
 
             $i = 1;
 
-            foreach($migrationDefinition->steps as $step) {
+            foreach ($migrationDefinition->steps as $step) {
                 // we validated the fact that we have a good executor at parsing time
                 $executor = $this->executors[$step->type];
                 $executor->execute($step);
@@ -193,16 +193,16 @@ 
 
             try {
                 $this->repository->commit();
-            } catch(\RuntimeException $e) {
+            } catch (\RuntimeException $e) {
                 // at present time, the ez5 repo does not support nested commits. So if some migration step has committed
                 // already, we get an exception a this point. Extremely poor design, but what can we do ?
                 /// @todo log warning
             }
 
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
             try {
                 $this->repository->rollBack();
-            } catch(\RuntimeException $e2) {
+            } catch (\RuntimeException $e2) {
                 // at present time, the ez5 repo does not support nested commits. So if some migration step has committed
                 // already, we get an exception a this point. Extremely poor design, but what can we do ?
                 /// @todo log error

Command/MigrateCommand.php

Spacing +4 added lines, -4 removed lines

@@ -67,7 +67,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);
             }
@@ -100,7 +100,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>';
@@ -135,7 +135,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 decided to proceed anyway
             if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
@@ -147,7 +147,7 @@ 
 
             try {
                 $migrationsService->executeMigration($migrationDefinition);
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 if ($input->getOption('ignore-failures')) {
                     $output->writeln("\n<error>Migration failed! Reason: " . $e->getMessage() . "</error>\n");
                     continue;