kaliop-uk / ezmigrationbundle

Core/Executor/MigrationDefinitionExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -168,7 +168,7 @@ 
 
         foreach ($dsl['references'] as $key => $reference) {
             // BC
-            if (is_array($reference) && isset($reference['json_path']) && !isset($reference['attribute'] )) {
+            if (is_array($reference) && isset($reference['json_path']) && !isset($reference['attribute'])) {
                 $reference['attribute'] = $reference['json_path'];
             }
             $reference = $this->parseReferenceDefinition($key, $reference);
@@ -197,7 +197,7 @@ 
     {
         $migrationService = $this->migrationService;
         $executors = $migrationService->listExecutors();
-        foreach($executors as $key => $name) {
+        foreach ($executors as $key => $name) {
             $executor = $migrationService->getExecutor($name);
             if (!$executor instanceof MigrationGeneratorInterface) {
                 unset($executors[$key]);

Core/Executor/RepositoryExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -216,7 +216,7 @@ 
 
         $multivalued = ($this->expectedResultsType($step) == self::$RESULT_TYPE_MULTIPLE);
 
-        if ($item instanceof AbstractCollection  || is_array($item)) {
+        if ($item instanceof AbstractCollection || is_array($item)) {
             $items = $item;
         } else {
             $items = array($item);
@@ -261,7 +261,7 @@ 
         // allow setting *some* refs even when we have 0 or N matches
         foreach ($referencesDefinition as $key => $reference) {
             $reference = $this->parseReferenceDefinition($key, $reference);
-            switch($reference['attribute']) {
+            switch ($reference['attribute']) {
 
                 case 'count':
                     $value = count($entity);

Core/MigrationService.php

Spacing +10 added lines, -10 removed lines

@@ -283,7 +283,7 @@ 
         }
 
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition). " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
+            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition) . " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
         }
 
         /// @todo add support for setting in $migrationContext a userContentType, userGroupContentType ?
@@ -317,7 +317,7 @@ 
 
         try {
 
-            $i = $stepOffset+1;
+            $i = $stepOffset + 1;
             $finalStatus = Migration::STATUS_DONE;
             $finalMessage = null;
 
@@ -454,12 +454,12 @@ 
     public function resumeMigration(Migration $migration, $useTransaction = true, array $forcedReferences = array())
     {
         if ($migration->status != Migration::STATUS_SUSPENDED) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': it is not in suspended status");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': it is not in suspended status");
         }
 
         $migrationDefinitions = $this->getMigrationsDefinitions(array($migration->path));
         if (!count($migrationDefinitions)) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': its definition is missing");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': its definition is missing");
         }
 
         $defs = $migrationDefinitions->getArrayCopy();
@@ -467,24 +467,24 @@ 
 
         $migrationDefinition = $this->parseMigrationDefinition($migrationDefinition);
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': {$migrationDefinition->parsingError}");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': {$migrationDefinition->parsingError}");
         }
 
         // restore context
         $this->contextHandler->restoreCurrentContext($migration->name);
 
         if ($forcedReferences) {
-            foreach($forcedReferences as $name => $value) {
+            foreach ($forcedReferences as $name => $value) {
                 $this->referenceResolver->addReference($name, $value, true);
             }
         }
 
         if (!isset($this->migrationContext[$migration->name])) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is missing");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is missing");
         }
         $restoredContext = $this->migrationContext[$migration->name];
-        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'] )) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is invalid");
+        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'])) {
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is invalid");
         }
 
         // update migration status
@@ -505,7 +505,7 @@ 
      * @param bool|null $forceSigchildEnabled
      * @return array
      */
-    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null )
+    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null)
     {
         $properties = array();
 

Command/ResumeCommand.php

Spacing +3 added lines, -3 removed lines

@@ -97,7 +97,7 @@ 
         $forcedRefs = array();
         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'");
@@ -109,7 +109,7 @@ 
         $executed = 0;
         $failed = 0;
 
-        foreach($suspendedMigrations as $suspendedMigration) {
+        foreach ($suspendedMigrations as $suspendedMigration) {
             $output->writeln("<info>Resuming {$suspendedMigration->name}</info>");
 
             try {
@@ -129,7 +129,7 @@ 
 
         $time = microtime(true) - $start;
         $output->writeln("Resumed $executed migrations, failed $failed");
-        $output->writeln("Time taken: ".sprintf('%.3f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB');
+        $output->writeln("Time taken: " . sprintf('%.3f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB');
 
         if ($failed) {
             return 2;

Core/Executor/ReferenceExecutor.php

Spacing +3 added lines, -3 removed lines

@@ -160,7 +160,7 @@ 
             throw new \Exception("Invalid step definition: file '$fileName' for saving references already exists");
         }
 
-        if (! $this->referenceResolver instanceof EnumerableReferenceResolverInterface) {
+        if (!$this->referenceResolver instanceof EnumerableReferenceResolverInterface) {
             throw new \Exception("Can not save references as resolver is not enumerable");
         }
 
@@ -205,7 +205,7 @@ 
         $value = $this->dumpVar($this->referenceResolver->resolveReference($dsl['identifier']));
 
         if (isset($context['output']) && $context['output'] instanceof OutputInterface) {
-            $context['output']->write($label . $value, false, OutputInterface::OUTPUT_RAW|OutputInterface::VERBOSITY_NORMAL);
+            $context['output']->write($label . $value, false, OutputInterface::OUTPUT_RAW | OutputInterface::VERBOSITY_NORMAL);
         } else {
             echo $label . $value;
         }
@@ -227,7 +227,7 @@ 
 
         $dumper->dump(
             $cloner->cloneVar($var),
-            function ($line, $depth) use (&$output) {
+            function($line, $depth) use (&$output) {
                 // A negative depth means "end of dump"
                 if ($depth >= 0) {
                     // Adds a two spaces indentation to the line

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/AbstractCommand.php

Spacing +2 added lines, -2 removed lines

@@ -94,13 +94,13 @@
     protected function normalizePaths($paths)
     {
         $rootDir = realpath($this->getContainer()->get('kernel')->getRootDir() . '/..') . '/';
-        foreach($paths as $i => $path) {
+        foreach ($paths as $i => $path) {
             if ($path === $rootDir || $path === './') {
                 $paths[$i] = './';
             } else if (strpos($path, './') === 0) {
                 $paths[$i] = substr($path, 2);
             // q: should we also call realpath on $path? what if there are symlinks at play?
-            } elseif (strpos($path, $rootDir) === 0 ) {
+            } elseif (strpos($path, $rootDir) === 0) {
                 $paths[$i] = substr($path, strlen($rootDir));
             } elseif ($path === '') {
                 unset($paths[$i]);

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;
             }
         }