kaliop-uk / ezmigrationbundle

Core/Executor/UserManager.php

Spacing +4 added lines, -4 removed lines

@@ -151,7 +151,7 @@ 
                 if ($this->referenceResolver->isReference($userId)) {
                     $userId = $this->referenceResolver->getReferenceValue($userId);
                 }
-                array_push($usersToDelete,$userService->loadUser($userId));
+                array_push($usersToDelete, $userService->loadUser($userId));
             }
         }
 
@@ -160,7 +160,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));
             }
         }
 
@@ -169,11 +169,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);
         }
     }

Tests/phpunit/0_CollectionsTest.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-include_once(__DIR__.'/CommandTest.php');
+include_once(__DIR__ . '/CommandTest.php');
 
 use Kaliop\eZMigrationBundle\API\Collection\MigrationStepsCollection;
 use Kaliop\eZMigrationBundle\API\Value\MigrationStep;

Tests/phpunit/1_GenerateTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-include_once(__DIR__.'/CommandTest.php');
+include_once(__DIR__ . '/CommandTest.php');
 
 use Symfony\Component\Console\Input\ArrayInput;
 
@@ -70,6 +70,6 @@ 
         $exitCode = $this->app->run($input, $this->output);
         $output = $this->fetchOutput();
         $this->assertSame(0, $exitCode, 'CLI Command failed. Output: ' . $output);
-        $this->assertRegExp('?\| ' . basename($filePath) . ' +\| not executed +\|?', $output );
+        $this->assertRegExp('?\| ' . basename($filePath) . ' +\| not executed +\|?', $output);
     }
 }

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