kaliop-uk / ezmigrationbundle

Command/MassMigrateCommand.php

Spacing +12 added lines, -12 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('%.2f', $time)." secs</info>");
+        $this->writeln("<info>Time taken: " . sprintf('%.2f', $time) . " secs</info>");
 
         return $subprocessesFailed + $this->migrationsDone[Migration::STATUS_FAILED] + $missed;
     }
@@ -228,7 +228,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;
             }
@@ -268,7 +268,7 @@ 
                     $this->executeMigrationInProcess($migrationDefinition, $force, $migrationService, $input);
 
                     $executed++;
-                } catch(\Exception $e) {
+                } catch (\Exception $e) {
                     $failed++;
 
                     $errorMessage = $e->getMessage();
@@ -301,7 +301,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));
 
@@ -319,7 +319,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 {
@@ -353,7 +353,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;
             }
@@ -395,7 +395,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>');
         }
@@ -408,7 +408,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;

Command/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -213,9 +213,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('%.2f', $time)." secs</info>");
+            $this->writeln("<info>Time taken: " . sprintf('%.2f', $time) . " secs</info>");
         } else {
-            $this->writeln("<info>Time taken: ".sprintf('%.2f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB</info>');
+            $this->writeln("<info>Time taken: " . sprintf('%.2f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB</info>');
         }
 
         return $failed;
@@ -442,7 +442,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:

Core/Executor/TagManager.php

Spacing +4 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
 
         // sort tags by depth so that there will be no errors in case we are deleting parent and child
         $tagsCollection = $tagsCollection->getArrayCopy();
-        uasort($tagsCollection, function ($t1, $t2) {
+        uasort($tagsCollection, function($t1, $t2) {
             if ($t1->depth == $t2->depth) return 0;
             return ($t1->depth > $t2->depth) ? -1 : 1;
         });
@@ -266,7 +266,7 @@ 
             case 'create':
                 // sort top to bottom
                 $tagCollection = $tagCollection->getArrayCopy();
-                uasort($tagCollection, function ($t1, $t2) {
+                uasort($tagCollection, function($t1, $t2) {
                     if ($t1->depth == $t2->depth) return 0;
                     return ($t1->depth > $t2->depth) ? 1 : -1;
                 });
@@ -274,7 +274,7 @@ 
             case 'delete':
                 // sort bottom to top
                 $tagCollection = $tagCollection->getArrayCopy();
-                uasort($tagCollection, function ($t1, $t2) {
+                uasort($tagCollection, function($t1, $t2) {
                     if ($t1->depth == $t2->depth) return 0;
                     return ($t1->depth > $t2->depth) ? -1 : 1;
                 });
@@ -352,7 +352,7 @@ 
     protected function getTagKeywords($tag)
     {
         $out = array();
-        foreach($tag->languageCodes as $languageCode) {
+        foreach ($tag->languageCodes as $languageCode) {
             $out[$languageCode] = $tag->getKeyword($languageCode);
         }
         return $out;

Braces +9 added lines, -3 removed lines

@@ -162,7 +162,9 @@ 
         // sort tags by depth so that there will be no errors in case we are deleting parent and child
         $tagsCollection = $tagsCollection->getArrayCopy();
         uasort($tagsCollection, function ($t1, $t2) {
-            if ($t1->depth == $t2->depth) return 0;
+            if ($t1->depth == $t2->depth) {
+                return 0;
+            }
             return ($t1->depth > $t2->depth) ? -1 : 1;
         });
 
@@ -267,7 +269,9 @@ 
                 // sort top to bottom
                 $tagCollection = $tagCollection->getArrayCopy();
                 uasort($tagCollection, function ($t1, $t2) {
-                    if ($t1->depth == $t2->depth) return 0;
+                    if ($t1->depth == $t2->depth) {
+                        return 0;
+                    }
                     return ($t1->depth > $t2->depth) ? 1 : -1;
                 });
                 break;
@@ -275,7 +279,9 @@ 
                 // sort bottom to top
                 $tagCollection = $tagCollection->getArrayCopy();
                 uasort($tagCollection, function ($t1, $t2) {
-                    if ($t1->depth == $t2->depth) return 0;
+                    if ($t1->depth == $t2->depth) {
+                        return 0;
+                    }
                     return ($t1->depth > $t2->depth) ? -1 : 1;
                 });
                 break;

Core/Matcher/TagMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -156,7 +156,7 @@
             $childTags = $this->findTagsByParentTagIds(array_keys($parentTags));
             $tags = array_merge($tags, $childTags);
             $parentTags = $childTags;
-        } while (count( $childTags ) > 0);
+        } while (count($childTags) > 0);
 
         return $tags;
     }

Core/Matcher/TrashMatcher.php

Braces +3 added lines, -1 removed lines

@@ -56,7 +56,9 @@
 
             $query = new Query();
             $query->limit = $this->queryLimit;
-            if (isset($query->performCount)) $query->performCount = false;
+            if (isset($query->performCount)) {
+                $query->performCount = false;
+            }
             $query->filter = $this->getQueryCriterion($key, $values);
             $results = $this->repository->getTrashService()->findTrashItems($query);