kaliop-uk / ezmigrationbundle

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 +10 added lines, -10 removed lines

@@ -310,7 +310,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}");
         }
 
         /// @todo add support for setting in $migrationContext a userContentType, userGroupContentType ?
@@ -344,7 +344,7 @@ 
 
         try {
 
-            $i = $stepOffset+1;
+            $i = $stepOffset + 1;
             $finalStatus = Migration::STATUS_DONE;
             $finalMessage = null;
 
@@ -481,12 +481,12 @@ 
     public function resumeMigration(Migration $migration, $useTransaction = true, array $forcedReferences = array())
     {
         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();
@@ -494,24 +494,24 @@ 
 
         $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 ($forcedReferences) {
-            foreach($forcedReferences as $name => $value) {
+            foreach ($forcedReferences as $name => $value) {
                 $this->referenceResolver->addReference($name, $value, true);
             }
         }
 
         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");
+        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");
         }
 
         // update migration status
@@ -532,7 +532,7 @@ 
      * @param bool|null $forceSigchildEnabled
      * @return array
      */
-    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null )
+    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null)
     {
         $properties = array();
 

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

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;

Command/StatusCommand.php

Spacing +3 added lines, -3 removed lines

@@ -93,7 +93,7 @@ 
         // which is misleading. We add the following loop to fix that, so that the status command will match more closely
         // the 'migrate' command
         if (count($paths)) {
-            foreach($index as $migrationName => $data) {
+            foreach ($index as $migrationName => $data) {
                 if (!isset($data['migration'])) {
                     $migration = $migrationsService->getMigration($migrationName);
                     if ($migration !== null) {
@@ -231,7 +231,7 @@ 
 
         if ($input->getOption('todo')) {
             foreach ($data as $migrationData) {
-                $output->writeln("$migrationData", OutputInterface::OUTPUT_RAW|OutputInterface::VERBOSITY_QUIET);
+                $output->writeln("$migrationData", OutputInterface::OUTPUT_RAW | OutputInterface::VERBOSITY_QUIET);
             }
             return 0;
         }
@@ -260,7 +260,7 @@ 
      */
     protected function sortMigrationIndex(array &$index, $sortBy)
     {
-        switch($sortBy) {
+        switch ($sortBy) {
             case 'execution':
                 uasort($index, function($m1, $m2) {
                     if (isset($m1['migration']) && $m1['migration']->executionDate !== null) {

Core/Executor/BasePHPExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 
             $args = $dsl['arguments'];
 
-            foreach($args as &$val) {
+            foreach ($args as &$val) {
                 $val = $this->resolveReferencesRecursively($val);
             }
         } else {

Core/Executor/HTTPExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         $body = isset($dsl['body']) ? $this->resolveReferencesInText($dsl['body']) : null;
 
         if (isset($dsl['client'])) {
-            $client = $this->container->get('httplug.client.'.$this->referenceResolver->resolveReference($dsl['client']));
+            $client = $this->container->get('httplug.client.' . $this->referenceResolver->resolveReference($dsl['client']));
         } else {
             $client = $this->container->get('httplug.client');
         }

Core/Executor/LoopExecutor.php

Spacing +6 added lines, -6 removed lines

@@ -99,14 +99,14 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
-                    } catch(LoopContinueException $e) {
+                    } catch (LoopContinueException $e) {
                         // all ok, move to the next iteration
                         break;
                     }
                 }
-            } catch(LoopBreakException $e) {
+            } catch (LoopBreakException $e) {
                 // all ok, exit the loop
                 break;
             }
@@ -135,14 +135,14 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
-                    } catch(LoopContinueException $e) {
+                    } catch (LoopContinueException $e) {
                         // all ok, move to the next iteration
                         break;
                     }
                 }
-            } catch(LoopBreakException $e) {
+            } catch (LoopBreakException $e) {
                 // all ok, exit the loop
                 break;
             }

Core/Executor/ProcessExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
         $builderArgs = array($this->referenceResolver->resolveReference($dsl['command']));
 
         if (isset($dsl['arguments'])) {
-            foreach($dsl['arguments'] as $arg) {
+            foreach ($dsl['arguments'] as $arg) {
                 /// @todo should this be recursive?
                 $builderArgs[] = $this->referenceResolver->resolveReference($arg);
             }