kaliop-uk / ezmigrationbundle

Core/Executor/LoopExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($step->context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
                     }
                 }
@@ -97,7 +97,7 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($step->context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
                     }
                 }

Core/Helper/SortConverter.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     public function sortField2Hash($value)
     {
         $ref = new \ReflectionClass('eZ\Publish\API\Repository\Values\Content\Location');
-        foreach($ref->getConstants() as $key => $val) {
+        foreach ($ref->getConstants() as $key => $val) {
             if (strpos($key, 'SORT_FIELD_') === 0 && $val == $value) {
                 $out = strtolower(substr($key, 11));
                 if ($out == 'contentobject_id') {

Core/Executor/ContentVersionManager.php

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 
     protected function versionStatusToHash($status)
     {
-        foreach(ContentVersionMatcher::STATUS_MAP as $own => $ez) {
+        foreach (ContentVersionMatcher::STATUS_MAP as $own => $ez) {
             if ($status == $ez) {
                 return $own;
             }

Core/Matcher/QueryBasedMatcher.php

Spacing +7 added lines, -7 removed lines

@@ -148,7 +148,7 @@ 
                 return new Query\Criterion\DateMetadata(Query\Criterion\DateMetadata::CREATED, self::$operatorsMap[$operator], $match);
 
             case self::MATCH_GROUP:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->groupMatcher->matchOneByKey($value)->id;
                     }
@@ -167,7 +167,7 @@ 
                 return new Query\Criterion\DateMetadata(Query\Criterion\DateMetadata::MODIFIED, self::$operatorsMap[$operator], $match);
 
             case self::MATCH_OBJECT_STATE:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->stateMatcher->matchOneByKey($value)->id;
                     }
@@ -175,7 +175,7 @@ 
                 return new Query\Criterion\ObjectStateId($values);
 
             case self::MATCH_OWNER:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->userMatcher->matchOneByKey($value)->id;
                     }
@@ -195,7 +195,7 @@ 
                 return new Query\Criterion\ParentLocationId($locationIds);
 
             case self::MATCH_SECTION:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->sectionMatcher->matchOneByKey($value)->id;
                     }
@@ -216,7 +216,7 @@ 
 
             case self::MATCH_AND:
                 $subCriteria = array();
-                foreach($values as $subCriterion) {
+                foreach ($values as $subCriterion) {
                     $value = reset($subCriterion);
                     $subCriteria[] = $this->getQueryCriterion(key($subCriterion), $value);
                 }
@@ -224,7 +224,7 @@ 
 
             case self::MATCH_OR:
                 $subCriteria = array();
-                foreach($values as $subCriterion) {
+                foreach ($values as $subCriterion) {
                     $value = reset($subCriterion);
                     $subCriteria[] = $this->getQueryCriterion(key($subCriterion), $value);
                 }
@@ -266,7 +266,7 @@ 
 
             $direction = $this->hash2SortOrder($sortItem['sort_order']);
 
-            switch($sortItem['sort_field']) {
+            switch ($sortItem['sort_field']) {
                 case self::SORT_CONTENT_ID:
                     $out[] = new SortClause\ContentId($direction);
                     break;

Core/FieldHandler/EzSelection.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         // allow user to pass in selection values by name
         $fieldSettings = null;
-        foreach($fieldValue as $key => $val) {
+        foreach ($fieldValue as $key => $val) {
 
             $val = $this->referenceResolver->resolveReference($val);
 
@@ -47,7 +47,7 @@ 
                     if ($fieldSettings === null) {
                         $fieldSettings = $this->loadFieldSettings($context['contentTypeIdentifier'], $context['fieldIdentifier']);
                     }
-                    foreach($fieldSettings['options'] as $pos => $name) {
+                    foreach ($fieldSettings['options'] as $pos => $name) {
                         if ($name === $val) {
                             $fieldValue[$key] = $pos;
                             break;

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