kaliop-uk / ezmigrationbundle

Command/ResumeCommand.php

Spacing +2 added lines, -2 removed lines

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

Core/Matcher/ContentMatcher.php

Spacing +5 added lines, -5 removed lines

@@ -46,15 +46,15 @@ 
 
         foreach ($conditions as $key => $values) {
 
-            switch($key) {
+            switch ($key) {
 
                 case self::MATCH_RELATES_TO:
                     $contentService = $this->repository->getContentService();
                     $contents = array();
                     // allow to specify the objects to relate to using different ways
                     $relatedContents = $this->match($values);
-                    foreach($relatedContents as $relatedContent) {
-                        foreach($contentService->loadReverseRelations($relatedContent->contentInfo) as $relatingContent) {
+                    foreach ($relatedContents as $relatedContent) {
+                        foreach ($contentService->loadReverseRelations($relatedContent->contentInfo) as $relatingContent) {
                             $contents[$relatingContent->contentInfo->id] = $relatingContent;
                         }
                     }
@@ -65,8 +65,8 @@ 
                     $contents = array();
                     // allow to specify the objects we relate to using different ways
                     $relatingContents = $this->match($values);
-                    foreach($relatingContents as $relatingContent) {
-                        foreach($contentService->loadRelations($relatingContent->contentInfo) as $relatedContent) {
+                    foreach ($relatingContents as $relatingContent) {
+                        foreach ($contentService->loadRelations($relatingContent->contentInfo) as $relatedContent) {
                             $contents[$relatedContent->contentInfo->id] = $relatedContent;
                         }
                     }

Core/FieldHandler/EzDateAndTime.php

Spacing +7 added lines, -7 removed lines

@@ -58,26 +58,26 @@
     {
         $format = array();
         if (0 != $dateInterval->y) {
-            $format[] = $dateInterval->y.' years';
+            $format[] = $dateInterval->y . ' years';
         }
         if (0 != $dateInterval->m) {
-            $format[] = $dateInterval->m.' months';
+            $format[] = $dateInterval->m . ' months';
         }
         if (0 != $dateInterval->d) {
-            $format[] = $dateInterval->d.' days';
+            $format[] = $dateInterval->d . ' days';
         }
         //if (0 < $dateInterval->h || 0 < $dateInterval->i || 0 < $dateInterval->s) {
         //    $format .= 'T';
         //}
         if (0 != $dateInterval->h) {
-            $format[] = $dateInterval->h.' hours';
+            $format[] = $dateInterval->h . ' hours';
         }
         if (0 != $dateInterval->i) {
-            $format[] = $dateInterval->i.' minutes';
+            $format[] = $dateInterval->i . ' minutes';
         }
         if (0 != $dateInterval->s) {
-            $format[] = $dateInterval->s.' seconds';
+            $format[] = $dateInterval->s . ' seconds';
         }
-        return implode( ', ', $format);
+        return implode(', ', $format);
     }
 }

Core/Executor/ContentVersionManager.php

Spacing +3 added lines, -3 removed lines

@@ -29,18 +29,18 @@
         $versions = (array)$step->dsl['versions'];
 
         foreach ($contentCollection as $content) {
-            foreach($versions as $versionId) {
+            foreach ($versions as $versionId) {
                 try {
                     if ($versionId < 0) {
                         $contentVersions = $contentService->loadVersions($content->contentInfo);
                         // different eZ kernels apparently sort versions in different order...
                         $sortedVersions = array();
-                        foreach($contentVersions as $versionInfo) {
+                        foreach ($contentVersions as $versionInfo) {
                             $sortedVersions[$versionInfo->versionNo] = $versionInfo;
                         }
                         ksort($sortedVersions);
                         $sortedVersions = array_slice($sortedVersions, 0, $versionId);
-                        foreach($sortedVersions as $versionInfo) {
+                        foreach ($sortedVersions as $versionInfo) {
                             $contentService->deleteVersion($versionInfo);
                         }
                     } else {

Core/Matcher/ContentMatcherDirectLoad.php

Spacing +2 added lines, -2 removed lines

@@ -22,13 +22,13 @@
             $contents = array();
             switch ($key) {
                 case self::MATCH_CONTENT_ID:
-                    foreach($match as $contentId) {
+                    foreach ($match as $contentId) {
                         $content = $this->repository->getContentService()->loadContent($contentId);
                         $contents[$content->id] = $content;
                     }
                     break;
                 case self::MATCH_CONTENT_REMOTE_ID:
-                    foreach($match as $contentRemoteId) {
+                    foreach ($match as $contentRemoteId) {
                         $content = $this->repository->getContentService()->loadContentByRemoteId($contentRemoteId);
                         $contents[$content->id] = $content;
                     }

Core/Matcher/LocationMatcherDirectLoad.php

Spacing +2 added lines, -2 removed lines

@@ -24,13 +24,13 @@
             $locations = array();
             switch ($key) {
                 case self::MATCH_LOCATION_ID:
-                    foreach($match as $locationId) {
+                    foreach ($match as $locationId) {
                         $location = $this->repository->getLocationService()->loadLocation($locationId);
                         $locations[$location->id] = $location;
                     }
                     break;
                 case self::MATCH_LOCATION_REMOTE_ID:
-                    foreach($match as $locationRemoteId) {
+                    foreach ($match as $locationRemoteId) {
                         $location = $this->repository->getLocationService()->loadLocationByRemoteId($locationRemoteId);
                         $locations[$location->id] = $location;
                     }

Core/Loader/FilesystemRecursive.php

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
         if (empty($paths)) {
             $paths = array();
             /** @var $bundle \Symfony\Component\HttpKernel\Bundle\BundleInterface */
-            foreach($this->kernel->getBundles() as $bundle)
+            foreach ($this->kernel->getBundles() as $bundle)
             {
                 $path = $bundle->getPath() . "/" . $this->versionDirectory;
                 if (is_dir($path)) {
@@ -67,7 +67,7 @@ 
         }
 
         $definitions = array();
-        foreach($paths as $path) {
+        foreach ($paths as $path) {
             if (is_file($path)) {
                 $definitions[basename($path)] = $returnFilename ? $path : new MigrationDefinition(
                     basename($path),

Core/Helper/ProcessManager.php

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
         $currentProcesses = array_splice($processesQueue, 0, $maxParallel);
         // start the initial stack of processes
         foreach ($currentProcesses as $process) {
-            $process->start(function ($type, $buffer) use ($callback, $process) {
+            $process->start(function($type, $buffer) use ($callback, $process) {
                 if ($callback) {
                     $callback($type, $buffer, $process);
                 }
@@ -44,7 +44,7 @@ 
                     // directly add and start new process after the previous finished
                     if (count($processesQueue) > 0) {
                         $nextProcess = array_shift($processesQueue);
-                        $nextProcess->start(function ($type, $buffer) use ($callback, $nextProcess) {
+                        $nextProcess->start(function($type, $buffer) use ($callback, $nextProcess) {
                             if ($callback) {
                                 $callback($type, $buffer, $nextProcess);
                             }

Command/MassMigrateCommand.php

Spacing +16 added lines, -16 removed lines

@@ -89,7 +89,7 @@ 
             }
 
             $concurrency = $input->getOption('concurrency');
-            $this->writeln("Executing migrations using ".count($paths)." processes with a concurrency of $concurrency");
+            $this->writeln("Executing migrations using " . count($paths) . " processes with a concurrency of $concurrency");
 
             $kernel = $this->getContainer()->get('kernel');
             $builder = new ProcessBuilder();
@@ -110,23 +110,23 @@ 
                 $builderArgs[] = '--no-debug';
             }
             if ($input->getOption('default-language')) {
-                $builderArgs[]='--default-language='.$input->getOption('default-language');
+                $builderArgs[] = '--default-language=' . $input->getOption('default-language');
             }
             if ($input->getOption('no-transactions')) {
-                $builderArgs[]='--no-transactions';
+                $builderArgs[] = '--no-transactions';
             }
             if ($input->getOption('siteaccess')) {
-                $builderArgs[]='--siteaccess='.$input->getOption('siteaccess');
+                $builderArgs[] = '--siteaccess=' . $input->getOption('siteaccess');
             }
             if ($input->getOption('ignore-failures')) {
-                $builderArgs[]='--ignore-failures';
+                $builderArgs[] = '--ignore-failures';
             }
             if ($input->getOption('separate-process')) {
-                $builderArgs[]='--separate-process';
+                $builderArgs[] = '--separate-process';
             }
             $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
@@ -163,11 +163,11 @@ 
 
             $time = microtime(true) - $start;
 
-            $this->writeln('<info>'.$this->migrationsDone[0].' migrations executed, '.$this->migrationsDone[1].' failed, '.$this->migrationsDone[2].' skipped</info>');
+            $this->writeln('<info>' . $this->migrationsDone[0] . ' migrations executed, ' . $this->migrationsDone[1] . ' failed, ' . $this->migrationsDone[2] . ' skipped</info>');
             $this->writeln("<info>Import finished</info>\n");
 
             // since we use subprocesses, we can not measure max memory used
-            $this->writeln("Time taken: ".sprintf('%.2f', $time)." secs");
+            $this->writeln("Time taken: " . sprintf('%.2f', $time) . " secs");
 
             return $failed;
 
@@ -206,7 +206,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 @@ 
                         );
 
                         $executed++;
-                    } catch(\Exception $e) {
+                    } catch (\Exception $e) {
                         $failed++;
                         if ($input->getOption('ignore-failures')) {
                             $this->writeln("<error>Migration failed! Path: " . $migrationDefinition->path . ", Reason: " . $e->getMessage() . "</error>", self::VERBOSITY_CHILD);
@@ -306,7 +306,7 @@ 
         }
     }
 
-    public function onSubProcessOutput($type, $buffer, $process=null)
+    public function onSubProcessOutput($type, $buffer, $process = null)
     {
         $lines = explode("\n", trim($buffer));
 
@@ -324,7 +324,7 @@ 
 
             // we tag the output from the different processes
             if (trim($line) !== '') {
-                echo '[' . ($process ? $process->getPid() : ''). '] ' . trim($line) . "\n";
+                echo '[' . ($process ? $process->getPid() : '') . '] ' . trim($line) . "\n";
             }
         }
     }
@@ -346,7 +346,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] = $isChild ? $migrationService->parseMigrationDefinition($migrationDefinition) : $migrationDefinition;
             }
@@ -388,7 +388,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]);
     }
 
     protected function askForConfirmation(InputInterface $input, OutputInterface $output)
@@ -419,7 +419,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;