kaliop-uk / ezmigrationbundle

Core/Matcher/LocationMatcher.php

Braces +6 added lines, -2 removed lines

@@ -107,7 +107,9 @@ 
             if (!empty($sort)) {
                 $query->sortClauses = $this->getSortClauses($sort);
             }
-            if (isset($query->performCount)) $query->performCount = false;
+            if (isset($query->performCount)) {
+                $query->performCount = false;
+            }
 
             $results = $this->getSearchService()->findLocations($query);
 
@@ -254,7 +256,9 @@ 
     {
         $query = new LocationQuery();
         $query->limit = self::INT_MAX_16BIT;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ParentLocationId($parentLocationIds);
 
         $results = $this->getSearchService()->findLocations($query);

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;

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/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>');
         }