kaliop-uk / ezmigrationbundle

Core/Loader/Filesystem.php

Spacing +1 added lines, -1 removed lines

@@ -120,6 +120,6 @@
     {
         $rootDir = realpath($this->kernel->getRootDir() . '/..') . '/';
         // note: we handle the case of 'path = root dir', but path is expected to include a filename...
-        return $path === $rootDir ? './' : preg_replace('#^' . preg_quote($rootDir, '#'). '#', '', $path);
+        return $path === $rootDir ? './' : preg_replace('#^' . preg_quote($rootDir, '#') . '#', '', $path);
     }
 }

Core/ReferenceResolver/ExpressionResolver.php

Spacing +2 added lines, -2 removed lines

@@ -45,12 +45,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 MigrationBundleException('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

@@ -132,7 +132,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 MigrationBundleException("Could not enumerate references because of chained resolver of type: " . get_class($resolver));
                 continue;

Core/Executor/ReferenceExecutor.php

Spacing +3 added lines, -3 removed lines

@@ -163,7 +163,7 @@ 
             throw new MigrationBundleException("Invalid step definition: file '$fileName' for saving references already exists");
         }
 
-        if (! $this->referenceResolver instanceof EnumerableReferenceResolverInterface) {
+        if (!$this->referenceResolver instanceof EnumerableReferenceResolverInterface) {
             throw new MigrationBundleException("Can not save references as resolver is not enumerable");
         }
 
@@ -208,7 +208,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;
         }
@@ -230,7 +230,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/Loader/FilesystemRecursive.php

Braces +1 added lines, -2 removed lines

@@ -61,8 +61,7 @@
                         }
                     }
                 }
-            }
-            else {
+            } else {
                 throw new MigrationBundleException("Path '$path' is neither a file nor directory");
             }
         }

Core/FieldHandler/EzRichText.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function setReferenceResolver(ReferenceResolverInterface $referenceResolver)
     {
-        if (! $referenceResolver instanceof EmbeddedReferenceResolverInterface) {
+        if (!$referenceResolver instanceof EmbeddedReferenceResolverInterface) {
             throw new MigrationBundleException("Reference resolver injected into EzRichText field handler should implement EmbeddedReferenceResolverInterface");
         }
         parent::setReferenceResolver($referenceResolver);

Core/FieldHandler/EzXmlText.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 {
     public function setReferenceResolver(ReferenceResolverInterface $referenceResolver)
     {
-        if (! $referenceResolver instanceof EmbeddedReferenceResolverInterface) {
+        if (!$referenceResolver instanceof EmbeddedReferenceResolverInterface) {
             throw new MigrationBundleException("Reference resolver injected into EzXmlText field handler should implement EmbeddedReferenceResolverInterface");
         }
         parent::setReferenceResolver($referenceResolver);

Command/MassMigrateCommand.php

Spacing +8 added lines, -8 removed lines

@@ -180,14 +180,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;
     }
@@ -241,7 +241,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;
             }
@@ -314,7 +314,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));
 
@@ -332,7 +332,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 {
@@ -408,7 +408,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>');
         }

Command/ResumeCommand.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,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;