kaliop-uk / ezmigrationbundle

Core/FieldHandler/FileFieldHandler.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     protected $ioDecorator;
     protected $ioService;
 
-    public function __construct($ioRootDir, UrlDecorator $ioDecorator=null, $ioService=null)
+    public function __construct($ioRootDir, UrlDecorator $ioDecorator = null, $ioService = null)
     {
         $this->ioRootDir = $ioRootDir;
         $this->ioDecorator = $ioDecorator;

Command/MassMigrateCommand.php

Spacing +15 added lines, -15 removed lines

@@ -132,7 +132,7 @@ 
 
         $processes = array();
         /** @var MigrationDefinition $migrationDefinition */
-        foreach($paths as $path => $count) {
+        foreach ($paths as $path => $count) {
             $this->writeln("<info>Queueing processing of: $path ($count migrations)</info>", OutputInterface::VERBOSITY_VERBOSE);
 
             $process = $builder
@@ -179,14 +179,14 @@ 
         }
 
         $missed = $total - $this->migrationsDone[Migration::STATUS_DONE] - $this->migrationsDone[Migration::STATUS_FAILED] - $this->migrationsDone[Migration::STATUS_SKIPPED];
-        $this->writeln("\nExecuted ".$this->migrationsDone[Migration::STATUS_DONE].' migrations'.
-            ', failed '.$this->migrationsDone[Migration::STATUS_FAILED].
-            ', skipped '.$this->migrationsDone[Migration::STATUS_SKIPPED].
+        $this->writeln("\nExecuted " . $this->migrationsDone[Migration::STATUS_DONE] . ' migrations' .
+            ', failed ' . $this->migrationsDone[Migration::STATUS_FAILED] .
+            ', skipped ' . $this->migrationsDone[Migration::STATUS_SKIPPED] .
             ($missed ? ", missed $missed" : ''));
 
         $time = microtime(true) - $start;
         // since we use subprocesses, we can not measure max memory used
-        $this->writeln("<info>Time taken: ".sprintf('%.3f', $time)." secs</info>");
+        $this->writeln("<info>Time taken: " . sprintf('%.3f', $time) . " secs</info>");
 
         return $subprocessesFailed + $this->migrationsDone[Migration::STATUS_FAILED] + $missed;
     }
@@ -211,7 +211,7 @@ 
                 $prefix = array($php);
 
                 if ($input->getOption('child-process-php-ini-config')) {
-                    foreach($input->getOption('child-process-php-ini-config') as $iniSpec) {
+                    foreach ($input->getOption('child-process-php-ini-config') as $iniSpec) {
                         $ini = explode(':', $iniSpec, 2);
                         if (count($ini) < 2 || $ini[0] === '') {
                             throw new \Exception("Invalid php ini specification: '$iniSpec'");
@@ -240,7 +240,7 @@ 
         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) {
-                $this->writeln("<comment>Skipping migration (invalid definition?) Path: ".$migrationDefinition->path."</comment>", self::VERBOSITY_CHILD);
+                $this->writeln("<comment>Skipping migration (invalid definition?) Path: " . $migrationDefinition->path . "</comment>", self::VERBOSITY_CHILD);
                 $skipped++;
                 continue;
             }
@@ -280,7 +280,7 @@ 
                     $this->executeMigrationInProcess($migrationDefinition, $force, $migrationService, $input);
 
                     $executed++;
-                } catch(\Exception $e) {
+                } catch (\Exception $e) {
                     $failed++;
 
                     $errorMessage = $e->getMessage();
@@ -313,7 +313,7 @@ 
      * @param string $buffer
      * @param null|\Symfony\Component\Process\Process $process
      */
-    public function onChildProcessOutput($type, $buffer, $process=null)
+    public function onChildProcessOutput($type, $buffer, $process = null)
     {
         $lines = explode("\n", trim($buffer));
 
@@ -331,7 +331,7 @@ 
 
             // we tag the output with the id of the child process
             if (trim($line) !== '') {
-                $msg = '[' . ($process ? $process->getPid() : ''). '] ' . trim($line);
+                $msg = '[' . ($process ? $process->getPid() : '') . '] ' . trim($line);
                 if ($type == 'err') {
                     $this->writeErrorln($msg, OutputInterface::VERBOSITY_QUIET, OutputInterface::OUTPUT_RAW);
                 } else {
@@ -365,7 +365,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]) && in_array($migration->status, $allowedStatuses))) {
                 $toExecute[$name] = $isChild ? $migrationService->parseMigrationDefinition($migrationDefinition) : $migrationDefinition;
             }
@@ -407,7 +407,7 @@ 
     {
         $output->writeln('Found ' . count($toExecute) . ' migrations in ' . count($paths) . ' directories');
         $output->writeln('In the same directories, migrations previously executed: ' . $this->migrationsAlreadyDone[Migration::STATUS_DONE] .
-            ', failed: ' . $this->migrationsAlreadyDone[Migration::STATUS_FAILED] . ', skipped: '. $this->migrationsAlreadyDone[Migration::STATUS_SKIPPED]);
+            ', failed: ' . $this->migrationsAlreadyDone[Migration::STATUS_FAILED] . ', skipped: ' . $this->migrationsAlreadyDone[Migration::STATUS_SKIPPED]);
         if ($this->migrationsAlreadyDone[Migration::STATUS_STARTED]) {
             $output->writeln('<info>In the same directories, migrations currently executing: ' . $this->migrationsAlreadyDone[Migration::STATUS_STARTED] . '</info>');
         }
@@ -420,7 +420,7 @@ 
     protected function groupMigrationsByPath($toExecute)
     {
         $paths = array();
-        foreach($toExecute as $name => $migrationDefinition) {
+        foreach ($toExecute as $name => $migrationDefinition) {
             $path = dirname($migrationDefinition->path);
             if (!isset($paths[$path])) {
                 $paths[$path] = 1;
@@ -497,12 +497,12 @@ 
             $builderArgs[] = '--separate-process';
         }
         if ($input->getOption('set-reference')) {
-            foreach($input->getOption('set-reference') as $refSpec) {
+            foreach ($input->getOption('set-reference') as $refSpec) {
                 $builderArgs[] = '--set-reference=' . $refSpec;
             }
         }
         if ($input->getOption('child-process-php-ini-config')) {
-            foreach($input->getOption('child-process-php-ini-config') as $iniSpec) {
+            foreach ($input->getOption('child-process-php-ini-config') as $iniSpec) {
                 $builderArgs[] = '--child-process-php-ini-config=' . $iniSpec;
             }
         }

Command/StatusCommand.php

Spacing +3 added lines, -3 removed lines

@@ -93,7 +93,7 @@ 
         // which is misleading. We add the following loop to fix that, so that the status command will match more closely
         // the 'migrate' command
         if (count($paths)) {
-            foreach($index as $migrationName => $data) {
+            foreach ($index as $migrationName => $data) {
                 if (!isset($data['migration'])) {
                     $migration = $migrationsService->getMigration($migrationName);
                     if ($migration !== null) {
@@ -231,7 +231,7 @@ 
 
         if ($input->getOption('todo')) {
             foreach ($data as $migrationData) {
-                $output->writeln("$migrationData", OutputInterface::OUTPUT_RAW|OutputInterface::VERBOSITY_QUIET);
+                $output->writeln("$migrationData", OutputInterface::OUTPUT_RAW | OutputInterface::VERBOSITY_QUIET);
             }
             return 0;
         }
@@ -260,7 +260,7 @@ 
      */
     protected function sortMigrationIndex(array &$index, $sortBy)
     {
-        switch($sortBy) {
+        switch ($sortBy) {
             case 'execution':
                 uasort($index, function($m1, $m2) {
                     if (isset($m1['migration']) && $m1['migration']->executionDate !== null) {

Command/MigrateCommand.php

Spacing +6 added lines, -6 removed lines

@@ -119,7 +119,7 @@ 
                 $prefix = array($php);
 
                 if ($input->getOption('child-process-php-ini-config')) {
-                    foreach($input->getOption('child-process-php-ini-config') as $iniSpec) {
+                    foreach ($input->getOption('child-process-php-ini-config') as $iniSpec) {
                         $ini = explode(':', $iniSpec, 2);
                         if (count($ini) < 2 || $ini[0] === '') {
                             throw new \Exception("Invalid php ini specification: '$iniSpec'");
@@ -145,7 +145,7 @@ 
 
         if ($input->getOption('set-reference') && !$input->getOption('separate-process')) {
             $refResolver = $this->getContainer()->get('ez_migration_bundle.reference_resolver.customreference');
-            foreach($input->getOption('set-reference') as $refSpec) {
+            foreach ($input->getOption('set-reference') as $refSpec) {
                 $ref = explode(':', $refSpec, 2);
                 if (count($ref) < 2 || $ref[0] === '') {
                     throw new \Exception("Invalid reference specification: '$refSpec'");
@@ -246,9 +246,9 @@ 
         $time = microtime(true) - $start;
         if ($input->getOption('separate-process')) {
             // in case of using subprocesses, we can not measure max memory used
-            $this->writeln("<info>Time taken: ".sprintf('%.3f', $time)." secs</info>");
+            $this->writeln("<info>Time taken: " . sprintf('%.3f', $time) . " secs</info>");
         } else {
-            $this->writeln("<info>Time taken: ".sprintf('%.3f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB</info>');
+            $this->writeln("<info>Time taken: " . sprintf('%.3f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB</info>');
         }
 
         return $failed;
@@ -480,7 +480,7 @@ 
             $builderArgs[] = '--no-debug';
         }
         if ($input->getOption('siteaccess')) {
-            $builderArgs[] = '--siteaccess='.$input->getOption('siteaccess');
+            $builderArgs[] = '--siteaccess=' . $input->getOption('siteaccess');
         }
         switch ($this->verbosity) {
             case OutputInterface::VERBOSITY_VERBOSE:
@@ -512,7 +512,7 @@ 
             $builderArgs[] = '--no-transactions';
         }
         if ($input->getOption('set-reference')) {
-            foreach($input->getOption('set-reference') as $refSpec) {
+            foreach ($input->getOption('set-reference') as $refSpec) {
                 $builderArgs[] = '--set-reference=' . $refSpec;
             }
         }

Core/ReferenceResolver/ExpressionResolver.php

Spacing +2 added lines, -2 removed lines

@@ -44,12 +44,12 @@
         return array(
             new ExpressionFunction(
                 'resolve',
-                function ($str) {
+                function($str) {
                     /// @todo we could implement this via eg a static class var which holds a pointer to $this->referenceResolver
                     //return sprintf('(is_string(%1$s) ? FakerResolver::resolveExpressionLanguageReference(%1$s) : %1$s)', $str);
                     return "throw new \Exception('The \'resolve\' expression language operator can not be compiled, only evaluated'";
                 },
-                function ($arguments, $str) use ($resolver) {
+                function($arguments, $str) use ($resolver) {
                     if (!is_string($str)) {
                         return $str;
                     }

Core/ReferenceResolver/ChainResolver.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
         $refs = array();
 
         foreach ($this->resolvers as $resolver) {
-            if (! $resolver instanceof EnumerableReferenceResolverInterface) {
+            if (!$resolver instanceof EnumerableReferenceResolverInterface) {
                 // Disabled - we now allow chaining enumerable and non-enumerable sets
                 //throw new \Exception("Could not enumerate references because of chained resolver of type: " . get_class($resolver));
                 continue;

Core/Executor/BasePHPExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 
             $args = $dsl['arguments'];
 
-            foreach($args as &$val) {
+            foreach ($args as &$val) {
                 $val = $this->resolveReferencesRecursively($val);
             }
         } else {

Core/Executor/HTTPExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         $body = isset($dsl['body']) ? $this->resolveReferencesInText($dsl['body']) : null;
 
         if (isset($dsl['client'])) {
-            $client = $this->container->get('httplug.client.'.$this->referenceResolver->resolveReference($dsl['client']));
+            $client = $this->container->get('httplug.client.' . $this->referenceResolver->resolveReference($dsl['client']));
         } else {
             $client = $this->container->get('httplug.client');
         }

Core/Executor/LoopExecutor.php

Spacing +6 added lines, -6 removed lines

@@ -99,14 +99,14 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
-                    } catch(LoopContinueException $e) {
+                    } catch (LoopContinueException $e) {
                         // all ok, move to the next iteration
                         break;
                     }
                 }
-            } catch(LoopBreakException $e) {
+            } catch (LoopBreakException $e) {
                 // all ok, exit the loop
                 break;
             }
@@ -135,14 +135,14 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
-                    } catch(LoopContinueException $e) {
+                    } catch (LoopContinueException $e) {
                         // all ok, move to the next iteration
                         break;
                     }
                 }
-            } catch(LoopBreakException $e) {
+            } catch (LoopBreakException $e) {
                 // all ok, exit the loop
                 break;
             }