kaliop-uk / ezmigrationbundle

Command/MigrationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 
             $output->writeln('<info>Migration: ' . $migration->name . '</info>');
             $output->writeln('Status: ' . $status);
-            $output->writeln('Executed on: <info>' . ($migration->executionDate != null ? date("Y-m-d H:i:s", $migration->executionDate) : '--'). '</info>');
+            $output->writeln('Executed on: <info>' . ($migration->executionDate != null ? date("Y-m-d H:i:s", $migration->executionDate) : '--') . '</info>');
             $output->writeln('Execution notes: <info>' . $migration->executionError . '</info>');
 
             if ($migration->status == Migration::STATUS_SUSPENDED) {

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

Core/ReferenceResolver/EmbeddedRegexpReferenceResolverTrait.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
     public function hasEmbeddedReferences($string)
     {
         $regexp = $this->getEmbeddedRegexp();
-        return (bool) preg_match_all($regexp, $string, $matches);
+        return (bool)preg_match_all($regexp, $string, $matches);
     }
     
     /**
@@ -52,7 +52,7 @@ 
     {
         // we need to alter the regexp we usr for std ref resolving, as it will be used to match parts of text, not the whole string
         $regexp = substr($this->getRegexp(), 1, -1);
-        return '/' . preg_quote($this->beginToken). preg_replace(array('/^\^/'), array(''), $regexp) . '[^' . $this->endToken . ']+' . preg_quote($this->endToken) . '/';
+        return '/' . preg_quote($this->beginToken) . preg_replace(array('/^\^/'), array(''), $regexp) . '[^' . $this->endToken . ']+' . preg_quote($this->endToken) . '/';
     }
 
     /**

Core/Process/Process.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
     public static function forceSigchildEnabled($force)
     {
-        self::$forceSigchildEnabled = (bool) $force;
+        self::$forceSigchildEnabled = (bool)$force;
     }
 
     protected function isSigchildEnabled()

Core/Process/ProcessBuilder.php

Spacing +9 added lines, -9 removed lines

@@ -22,23 +22,23 @@
     public function getProcess()
     {
         $parentClass = get_parent_class($this);
-        $this_prefix = \Closure::bind(function(ProcessBuilder $builder){return $builder->prefix;}, null, $parentClass);
+        $this_prefix = \Closure::bind(function(ProcessBuilder $builder) {return $builder->prefix; }, null, $parentClass);
         $this_prefix = $this_prefix($this);
-        $this_arguments = \Closure::bind(function(ProcessBuilder $builder){return $builder->arguments;}, null, $parentClass);
+        $this_arguments = \Closure::bind(function(ProcessBuilder $builder) {return $builder->arguments; }, null, $parentClass);
         $this_arguments = $this_arguments($this);
-        $this_options = \Closure::bind(function(ProcessBuilder $builder){return $builder->options;}, null, $parentClass);
+        $this_options = \Closure::bind(function(ProcessBuilder $builder) {return $builder->options; }, null, $parentClass);
         $this_options = $this_options($this);
-        $this_inheritEnv = \Closure::bind(function(ProcessBuilder $builder){return $builder->inheritEnv;}, null, $parentClass);
+        $this_inheritEnv = \Closure::bind(function(ProcessBuilder $builder) {return $builder->inheritEnv; }, null, $parentClass);
         $this_inheritEnv = $this_inheritEnv($this);
-        $this_env = \Closure::bind(function(ProcessBuilder $builder){return $builder->env;}, null, $parentClass);
+        $this_env = \Closure::bind(function(ProcessBuilder $builder) {return $builder->env; }, null, $parentClass);
         $this_env = $this_env($this);
-        $this_cwd = \Closure::bind(function(ProcessBuilder $builder){return $builder->cwd;}, null, $parentClass);
+        $this_cwd = \Closure::bind(function(ProcessBuilder $builder) {return $builder->cwd; }, null, $parentClass);
         $this_cwd = $this_cwd($this);
-        $this_input = \Closure::bind(function(ProcessBuilder $builder){return $builder->input;}, null, $parentClass);
+        $this_input = \Closure::bind(function(ProcessBuilder $builder) {return $builder->input; }, null, $parentClass);
         $this_input = $this_input($this);
-        $this_timeout = \Closure::bind(function(ProcessBuilder $builder){return $builder->timeout;}, null, $parentClass);
+        $this_timeout = \Closure::bind(function(ProcessBuilder $builder) {return $builder->timeout; }, null, $parentClass);
         $this_timeout = $this_timeout($this);
-        $this_outputDisabled = \Closure::bind(function(ProcessBuilder $builder){return $builder->outputDisabled;}, null, $parentClass);
+        $this_outputDisabled = \Closure::bind(function(ProcessBuilder $builder) {return $builder->outputDisabled; }, null, $parentClass);
         $this_outputDisabled = $this_outputDisabled($this);
 
         if (0 === count($this_prefix) && 0 === count($this_arguments)) {

Command/AbstractCommand.php

Braces +1 added lines, -2 removed lines

@@ -78,8 +78,7 @@
                 $this->errOutput->setVerbosity(OutputInterface::VERBOSITY_NORMAL);
                 $this->errOutput->writeln($message, $type);
                 $this->errOutput->setVerbosity(OutputInterface::VERBOSITY_QUIET);
-            }
-            else
+            } else
             {
                 $this->errOutput->writeln($message, $type);
             }

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

Core/MigrationService.php

Braces +3 added lines, -2 removed lines

@@ -599,8 +599,9 @@
      */
     public function getCurrentContext($migrationName)
     {
-        if (!isset($this->migrationContext[$migrationName]))
-            return null;
+        if (!isset($this->migrationContext[$migrationName])) {
+                    return null;
+        }
         $context = $this->migrationContext[$migrationName];
         // avoid attempting to store the current outputInterface when saving the context
         if (isset($context['output'])) {

Spacing +8 added lines, -8 removed lines

@@ -314,7 +314,7 @@ 
         }
 
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new MigrationBundleException("Can not execute " . $this->getEntityName($migrationDefinition). " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
+            throw new MigrationBundleException("Can not execute " . $this->getEntityName($migrationDefinition) . " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
         }
 
         // BC: handling of legacy method call signature
@@ -375,7 +375,7 @@ 
         $this->migrationContext[$migration->name] = array('context' => $migrationContext);
 
         $steps = array_slice($migrationDefinition->steps->getArrayCopy(), $stepOffset);
-        $i = $stepOffset+1;
+        $i = $stepOffset + 1;
         $finalStatus = Migration::STATUS_DONE;
         $finalMessage = '';
         $error = null;
@@ -583,7 +583,7 @@ 
 
             // save migration status
 
-            $finalMessage = ($finalMessage != '' && $messageSuffix != '') ? $finalMessage . '. '. $messageSuffix : $finalMessage . $messageSuffix;
+            $finalMessage = ($finalMessage != '' && $messageSuffix != '') ? $finalMessage . '. ' . $messageSuffix : $finalMessage . $messageSuffix;
 
             try {
                 $this->storageHandler->endMigration(new Migration(
@@ -639,12 +639,12 @@ 
         }
 
         if ($migration->status != Migration::STATUS_SUSPENDED) {
-            throw new MigrationBundleException("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': it is not in suspended status");
+            throw new MigrationBundleException("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 MigrationBundleException("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': its definition is missing");
+            throw new MigrationBundleException("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': its definition is missing");
         }
 
         $defs = $migrationDefinitions->getArrayCopy();
@@ -652,17 +652,17 @@ 
 
         $migrationDefinition = $this->parseMigrationDefinition($migrationDefinition);
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new MigrationBundleException("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': {$migrationDefinition->parsingError}");
+            throw new MigrationBundleException("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': {$migrationDefinition->parsingError}");
         }
 
         // restore context
         $this->contextHandler->restoreCurrentContext($migration->name);
         if (!isset($this->migrationContext[$migration->name])) {
-            throw new MigrationBundleException("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is missing");
+            throw new MigrationBundleException("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 MigrationBundleException("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is invalid");
+            throw new MigrationBundleException("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is invalid");
         }
 
         // update migration status

Core/Matcher/ContentMatcher.php

Braces +12 added lines, -4 removed lines

@@ -145,7 +145,9 @@ 
                     // q: when getting a query via QueryType, should we always inject offset/limit?
                     $query->limit = $limit != 0 ? $limit : $this->queryLimit;
                     $query->offset = $offset;
-                    if (isset($query->performCount)) $query->performCount = false;
+                    if (isset($query->performCount)) {
+                        $query->performCount = false;
+                    }
                     if (!empty($sort)) {
                         $query->sortClauses = $this->getSortClauses($sort);
                     } else {
@@ -271,7 +273,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ParentLocationId($parentLocationIds);
         $results = $this->getSearchService()->findContent($query);
 
@@ -311,7 +315,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ContentTypeIdentifier($contentTypeIdentifiers);
         // sort objects by depth, lower to higher, so that deleting them has less chances of failure
         // NB: we only do this in eZP versions that allow depth sorting on content queries
@@ -339,7 +345,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ContentTypeId($contentTypeIds);
         // sort objects by depth, lower to higher, so that deleting them has less chances of failure
         // NB: we only do this in eZP versions that allow depth sorting on content queries