kaliop-uk / ezmigrationbundle

Core/MigrationService.php

Spacing +9 added lines, -9 removed lines

@@ -46,7 +46,7 @@ 
 
     public function addExecutor(ExecutorInterface $executor)
     {
-        foreach($executor->supportedTypes() as $type) {
+        foreach ($executor->supportedTypes() as $type) {
             $this->executors[$type] = $executor;
         }
     }
@@ -61,8 +61,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;
                 }
@@ -123,13 +123,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,
@@ -185,7 +185,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);
@@ -207,19 +207,19 @@ 
             if ($useTransaction) {
                 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) {
 
             if ($useTransaction) {
                 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